Re: [U-Boot] [PATCH v2 5/7] i2c: at91_i2c: Change error return -ENODEV to -EINVAL

2016-09-18 Thread Heiko Schocher
Hello Wenyou Yang, Am 18.09.2016 um 08:53 schrieb Wenyou Yang: Change the error return value -ENODEV from to -EINVAL for more reasonable. Signed-off-by: Wenyou Yang --- Changes in v2: None drivers/i2c/at91_i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [U-Boot] [PATCH v2 4/7] i2c: at91_i2c: Remove unneccessary clock calling

2016-09-18 Thread Heiko Schocher
Hello Wenyou Yang, Am 18.09.2016 um 08:53 schrieb Wenyou Yang: Due to the peripheral clock driver improvement, remove the unneccessary clock calling. Signed-off-by: Wenyou Yang --- Changes in v2: None drivers/i2c/at91_i2c.c | 16 1 file changed, 16

Re: [U-Boot] [PATCH v2 03/17] thermal: Kconfig: Add IMX_THERMAL entry

2016-09-18 Thread Peng Fan
Hi Jagan, On Sat, Sep 17, 2016 at 02:18:35AM +0530, Jagan Teki wrote: >Added kconfig for IMX_THERMAL driver. > >Cc: Simon Glass >Cc: Fabio Estevam >Cc: Stefano Babic >Cc: Peng Fan >Cc: Matteo Lisi

Re: [U-Boot] [PATCH v2 02/17] serial: Kconfig: Add MXC_UART entry

2016-09-18 Thread Peng Fan
Hi Jagan, On Sat, Sep 17, 2016 at 02:18:34AM +0530, Jagan Teki wrote: >Added kconfig for MXC_UART driver. > >Cc: Simon Glass >Cc: Fabio Estevam >Cc: Stefano Babic >Cc: Peng Fan >Cc: Matteo Lisi

Re: [U-Boot] [PATCH v2 3/5] power: regulator: palmas: Add regulator support

2016-09-18 Thread Keerthy
On Monday 19 September 2016 09:21 AM, Keerthy wrote: The driver provides regulator set/get voltage enable/disable functions for palmas family of PMICs. Signed-off-by: Keerthy Reviewed-by: Simon Glass --- Changes in v2: * Converted all dm_i2c_reg to

[U-Boot] [PATCH 5/5] imx: mx6ullevk: correct boot device macro

2016-09-18 Thread van . freenix
From: Peng Fan Correct boot device macro according to kconfig entry in common/Kconfig Signed-off-by: Peng Fan Cc: Stefano Babic --- board/freescale/mx6ullevk/imximage.cfg | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[U-Boot] [PATCH 2/5] imx: mx6: Add plugin support

2016-09-18 Thread van . freenix
From: Peng Fan Add mx6_plugin.S which calls boot rom setup function, generate the second ivt, and jump back to boot rom. Signed-off-by: Peng Fan Signed-off-by: Ye Li Signed-off-by: Utkarsh Gupta ---

[U-Boot] [PATCH 4/5] imx: mx6ullevk: support plugin

2016-09-18 Thread van . freenix
From: Peng Fan Add plugin code for mx6ullevk. Define CONFIG_USE_PLUGIN in include/configs/mx6ullevk.h to use plugin code. Signed-off-by: Peng Fan Cc: Stefano Babic --- board/freescale/mx6ullevk/Makefile | 4 ++

[U-Boot] [PATCH 1/5] tools: imximage: add plugin support

2016-09-18 Thread van . freenix
From: Peng Fan Add plugin support for imximage. Define CONFIG_USE_PLUGIN to enable using plugin. Signed-off-by: Peng Fan Cc: Stefano Babic Cc: Eric Nelson Cc: Ye Li --- tools/imximage.c | 260

[U-Boot] [PATCH 3/5] imx: mx7: Add plugin support

2016-09-18 Thread van . freenix
From: Peng Fan Add mx7_plugin.S which calls boot rom setup function, generate the second ivt, and jump back to boot rom. Signed-off-by: Peng Fan Signed-off-by: Ye Li Cc: Stefano Babic ---

[U-Boot] [PATCH v2 4/5] configs: am57xx_evm_defconfig: Enable PALMAS options

2016-09-18 Thread Keerthy
Enable palmas PMIC config options. Signed-off-by: Keerthy --- configs/am57xx_evm_defconfig | 3 +++ 1 file changed, 3 insertions(+) diff --git a/configs/am57xx_evm_defconfig b/configs/am57xx_evm_defconfig index d49129d..b277783 100644 --- a/configs/am57xx_evm_defconfig +++

[U-Boot] [PATCH v2 5/5] configs: dra7xx_evm_defconfig: Enable PALMAS options

2016-09-18 Thread Keerthy
Enable palmas PMIC config options. Signed-off-by: Keerthy --- configs/dra7xx_evm_defconfig | 3 +++ 1 file changed, 3 insertions(+) diff --git a/configs/dra7xx_evm_defconfig b/configs/dra7xx_evm_defconfig index 64184de..882e615 100644 --- a/configs/dra7xx_evm_defconfig +++

[U-Boot] [PATCH v2 2/5] power: pmic: Palmas: Add the base pmic support

2016-09-18 Thread Keerthy
Add support to bind the regulators/child nodes with the pmic. Signed-off-by: Keerthy Reviewed-by: Simon Glass --- Changes in v2: * Added Simon's Reviewed-by. drivers/power/pmic/Kconfig | 7 + drivers/power/pmic/Makefile | 1 +

[U-Boot] [PATCH v2 3/5] power: regulator: palmas: Add regulator support

2016-09-18 Thread Keerthy
The driver provides regulator set/get voltage enable/disable functions for palmas family of PMICs. Signed-off-by: Keerthy Reviewed-by: Simon Glass --- Changes in v2: * Converted all dm_i2c_reg to dm_i2c_read_reg * Removed an instance of double blank

[U-Boot] [PATCH v2 1/5] power: regulator: Add ctrl_reg and volt_reg fields for pmic

2016-09-18 Thread Keerthy
The ctrl reg contains bit fields to enable and disable regulators, and volt_reg has the bit fields to configure the voltage values. The registers are frequently accessed hence make them part of dm_regulator_uclass_platdata structure. Signed-off-by: Keerthy ---

[U-Boot] [PATCH v2 0/5] power: pmic: Add support for Palmas family of PMICs

2016-09-18 Thread Keerthy
The series adds support for Palmas family of PMICs. Implements functions to configure regulators. Enable/Disable Get/Set voltages of regulators. Supports TPS659038, TPS65917, Palmas. Tested on TPS659038, TPS65917 using DRA7XX-EVM and AM57XX-EVM. Keerthy (5): power: regulator: Add ctrl_reg

Re: [U-Boot] [PATCH 3/5] power: regulator: palmas: Add regulator support

2016-09-18 Thread Keerthy
On Monday 19 September 2016 06:28 AM, Simon Glass wrote: On 13 September 2016 at 22:28, Keerthy wrote: The driver provides regulator set/get voltage enable/disable functions for palmas family of PMICs. Signed-off-by: Keerthy ---

Re: [U-Boot] [PATCH 08/27] spl: Add a way to declare an SPL image loader

2016-09-18 Thread Simon Glass
Hi Tom, On 18 September 2016 at 16:15, Tom Rini wrote: > On Sun, Sep 18, 2016 at 01:44:57PM -0600, Simon Glass wrote: >> Add a linker list macro which can be used to declare an SPL image loader. >> Update spl_load_image() to search available loaders for the correct one. >> >>

[U-Boot] Please pull u-boot-rockchip

2016-09-18 Thread Simon Glass
Hi Tom, Here is some more RK3399 support and a few other changes. The following changes since commit 9a6535e05f17acf03e891266a650cb6029124743: Merge branch 'master' of git://git.denx.de/u-boot-uniphier (2016-09-18 14:05:30 -0400) are available in the git repository at:

[U-Boot] Please pull u-boot-dm

2016-09-18 Thread Simon Glass
Hi Tom, This includes some refactoring to improve the dtoc code. The following changes since commit 9a6535e05f17acf03e891266a650cb6029124743: Merge branch 'master' of git://git.denx.de/u-boot-uniphier (2016-09-18 14:05:30 -0400) are available in the git repository at:

Re: [U-Boot] [PATCH 02/21] mmc: tegra: use correct alias for SDHCI/MMC nodes

2016-09-18 Thread Jaehoon Chung
On 09/14/2016 01:45 AM, Stephen Warren wrote: > From: Stephen Warren > > The Tegra MMC driver currently honors "sdhci" entries in /aliases. The > MMC core however uses "mmc" entries in /aliases. This difference will be > relevant once the Tegra MMC driver is converted to DM,

[U-Boot] [PATCH] mmc: dw_mmc: push/pop all FIFO data if any data request

2016-09-18 Thread Ziyuan Xu
From: "jacob2.chen" When DTO interrupt occurred, there are any remaining data still in FIFO due to RX FIFO threshold is larger than remaining data. It also causes that dwmmc didn't trigger RXDR interrupt, so is TX. It's responsibility of driver to read remaining

Re: [U-Boot] [PATCH v1 2/3] arm: mvebu: add DB-88F6820-AMC board

2016-09-18 Thread Chris Packham
On Mon, Sep 19, 2016 at 12:58 PM, Simon Glass wrote: > Hi Chris, > > On 13 September 2016 at 19:42, Chris Packham wrote: >> This board is a plug in card for Marvell's switch system development >> kits. Form-factor aside it is similar to the

Re: [U-Boot] [RFC PATCH] kconfig: introduce kconfig for UBI

2016-09-18 Thread Simon Glass
On 13 September 2016 at 23:03, Heiko Schocher wrote: > move the UBI config options into Kconfig. > > Signed-off-by: Heiko Schocher > --- > Tested with tbot: > http://lists.denx.de/pipermail/u-boot/2016-June/258119.html > > result: > Boards : 1196 > compile err :

Re: [U-Boot] [PATCH 00/27] spl: Use linker list and parameters for SPL image loading

2016-09-18 Thread Simon Glass
Hi Tom, On 18 September 2016 at 16:14, Tom Rini wrote: > On Sun, Sep 18, 2016 at 01:44:49PM -0600, Simon Glass wrote: > >> At present the SPL code uses a global spl_image variable which is shared >> amongst lots of files, some in common/spl and some elsewhere. There is no >>

Re: [U-Boot] [PATCH v2] power: regulator: Add support for gpio regulators

2016-09-18 Thread Simon Glass
On 15 September 2016 at 05:34, Keerthy wrote: > Add support for gpio regulators. As of now this driver caters > to gpio regulators with one gpio. Supports setting voltage values to gpio > regulators and retrieving the values. > > Signed-off-by: Keerthy > --- >

Re: [U-Boot] [PATCH 3/3] ARM: tegra: fix ULPI PHY on Ventana and Seaboard

2016-09-18 Thread Simon Glass
On 15 September 2016 at 12:19, Stephen Warren wrote: > From: Stephen Warren > > Commit ce02a71c2374 "tegra: dts: Sync tegra20 device tree files with > Linux" enabled the ULPI USB port on Ventana, but made no attempt to ensure > that U-Boot code could

Re: [U-Boot] [PATCH] image: Fix expected return value

2016-09-18 Thread Simon Glass
On 16 September 2016 at 06:34, Mario Six wrote: > Commit bac17b7 ("image-fit: switch ENOLINK to ENOENT") changed the > return value of fit_get_node_from_config from ENOLINK to ENOENT. > > This causes a problem in boot_get_ramdisk, which treats the return value > -ENOLINK

Re: [U-Boot] [PATCH 3/5] power: regulator: palmas: Add regulator support

2016-09-18 Thread Simon Glass
On 13 September 2016 at 22:28, Keerthy wrote: > The driver provides regulator set/get voltage > enable/disable functions for palmas family of PMICs. > > Signed-off-by: Keerthy > --- > drivers/power/regulator/Kconfig| 8 + >

Re: [U-Boot] [PATCH 2/3] ARM: tegra: fix USB controller aliases

2016-09-18 Thread Simon Glass
On 15 September 2016 at 12:19, Stephen Warren wrote: > From: Stephen Warren > > Some boards have a different set of USB controllers enabled in DT than > the set referenced by /alias entries. This patch fixes that. For > example, this avoids the

Re: [U-Boot] [PATCH 1/3] ARM: tegra: fix USB ULPI PHY reset signal inversion confusion

2016-09-18 Thread Simon Glass
On 15 September 2016 at 12:19, Stephen Warren wrote: > From: Stephen Warren > > USB ULPI PHY reset signals are typically active low. Consequently, they > should be marked as GPIO_ACTIVE_LOW in device tree, and indeed they are in > the Linux kernel DTs,

Re: [U-Boot] [PATCH] Revert "Increase default of CONFIG_SYS_MALLOC_F_LEN for SPL_OF_CONTROL"

2016-09-18 Thread Simon Glass
On 15 September 2016 at 05:12, Masahiro Yamada wrote: > This reverts commit 90c08d9e08c7a108ab904f3bbdeb558081757892. > > I took a closer look at this and found CONFIG_SYS_MALLOC_F_LEN=0x2000 > was too much to enable SPL_OF_CONTROL. Perhaps 0x800 is enough, but >

Re: [U-Boot] [PATCH 2/5] power: pmic: Palmas: Add the base pmic support

2016-09-18 Thread Simon Glass
On 13 September 2016 at 22:28, Keerthy wrote: > Add support to bind the regulators/child nodes with the pmic. > > Signed-off-by: Keerthy > --- > drivers/power/pmic/Kconfig | 7 + > drivers/power/pmic/Makefile | 1 + > drivers/power/pmic/palmas.c | 77

Re: [U-Boot] [PATCH v1 2/3] arm: mvebu: add DB-88F6820-AMC board

2016-09-18 Thread Simon Glass
Hi Chris, On 13 September 2016 at 19:42, Chris Packham wrote: > This board is a plug in card for Marvell's switch system development > kits. Form-factor aside it is similar to the DB-88F6820-GP with the > following differences. > - TCLK is 200MHz > - SPI1 is used > - No

Re: [U-Boot] [PATCH 01/21] ARM: tegra: fdt: Add 'non-removable' property to all eMMC nodes

2016-09-18 Thread Simon Glass
On 13 September 2016 at 10:45, Stephen Warren wrote: > From: Tom Warren > > During debug of the DM_MMC changes to the Tegra MMC driver, I > noticed that the 'removable' property wasn't being set correctly > for the eMMC parts on most Tegra boards. Since

Re: [U-Boot] [PATCH 02/21] mmc: tegra: use correct alias for SDHCI/MMC nodes

2016-09-18 Thread Simon Glass
On 13 September 2016 at 10:45, Stephen Warren wrote: > From: Stephen Warren > > The Tegra MMC driver currently honors "sdhci" entries in /aliases. The > MMC core however uses "mmc" entries in /aliases. This difference will be > relevant once the Tegra

Re: [U-Boot] [PATCH 2/2] cros_ec: Add crosec flashinfo command

2016-09-18 Thread Simon Glass
Hi Moritz, On 13 September 2016 at 15:44, Moritz Fischer wrote: > Add command to print out the flash info as reported by the > ec. The data read back includes size, write block size, > erase block size. > > Signed-off-by: Moritz Fischer > Cc:

Re: [U-Boot] [PATCH 1/2] cros_ec: Add function to read back flash parameters

2016-09-18 Thread Simon Glass
On 13 September 2016 at 15:44, Moritz Fischer wrote: > Add support for reading back flash parameters as reported by > the ec. > > Signed-off-by: Moritz Fischer > Cc: Simon Glass > Cc: u-boot@lists.denx.de > --- >

Re: [U-Boot] [PATCH 2/4] ARM: tegra: configure Ethernet address on Tegra186

2016-09-18 Thread Simon Glass
On 12 September 2016 at 11:51, Stephen Warren wrote: > From: Stephen Warren > > On Tegra186, the bootloader which runs before U-Boot passes the Ethernet > MAC address to U-Boot using device tree. Extract this value and write it > to the environment, so

Re: [U-Boot] [PATCH 3/4] ARM: tegra: add DWC EQoS (ethernet) to Tegra186 DT

2016-09-18 Thread Simon Glass
On 12 September 2016 at 11:51, Stephen Warren wrote: > From: Stephen Warren > > Tegra186 includes a Synopsys DWC EQoS (Ethernet) device. Add this to the > Tegra186 SoC DT so that boards can make use of it. > > Signed-off-by: Stephen Warren

Re: [U-Boot] [PATCH 4/4] ARM: tegra: enable Ethernet on p2771-0000

2016-09-18 Thread Simon Glass
On 12 September 2016 at 11:51, Stephen Warren wrote: > From: Stephen Warren > > Enable the Ethernet device in DT, provide board-specific configuration, > and enable the driver in Kconfig. > > Signed-off-by: Stephen Warren > --- >

Re: [U-Boot] [PATCH 2/2] net: add driver for Synopsys Ethernet QoS device

2016-09-18 Thread Simon Glass
On 12 September 2016 at 11:48, Stephen Warren wrote: > From: Stephen Warren > > This driver supports the Synopsys Designware Ethernet QoS (Quality of > Service) a/k/a eqos IP block, which is a different design than the HW > supported by the existing

Re: [U-Boot] [PATCH 1/2] dt: net: add DWC EQoS binding

2016-09-18 Thread Simon Glass
On 12 September 2016 at 11:48, Stephen Warren wrote: > From: Stephen Warren > > The Synopsys DWC EQoS is a configurable Ethernet MAC/DMA IP block which > supports multiple options for bus type, clocking and reset structure, and > feature list. > > This

Re: [U-Boot] [PATCH 1/4] ARM: tegra: add SoC-level hook for board_late_init()

2016-09-18 Thread Simon Glass
On 12 September 2016 at 11:51, Stephen Warren wrote: > From: Stephen Warren > > Extend the Tegra186 implementation of board_late_init() to call a per-SoC > "hook" function. This will allow SoC-specific (rather than Tegra-wide) > functionality to be

Re: [U-Boot] [PATCH 4/4] cmd/fat: Do not crash on write when is not specified

2016-09-18 Thread Simon Glass
On 11 September 2016 at 14:51, Stefan Brüns wrote: > argc is checked, but is off by one. In case is not specified, > create an empty file, which is identical to the ext4write behaviour. > > Signed-off-by: Stefan Brüns > --- >

Re: [U-Boot] [PATCH 1/3] treewide: rename CONFIG_SPL_SERIAL_SUPPORT to CONFIG_SPL_SERIAL

2016-09-18 Thread Simon Glass
Hi, On 11 September 2016 at 08:21, Masahiro Yamada wrote: > We conventionally named SPL options in CONFIG_SPL_*_SUPPORT format, > but the suffix _SUPPORT did not add any value. Besides, it prevents > us from exploiting the CONFIG_IS_ENABLED() macro. > > If we move

Re: [U-Boot] [PATCH] rockchip: miniarm: remove eMMC support

2016-09-18 Thread Simon Glass
On 9 September 2016 at 07:44, Ziyuan Xu wrote: > The latest rk3288-miniarm board doesn't have eMMC device, so remove it. > > Signed-off-by: Ziyuan Xu > --- > > arch/arm/dts/rk3288-miniarm.dtsi | 12 >

Re: [U-Boot] [PATCH v2 4/4] fastboot: move FASTBOOT_FLASH options into Kconfig

2016-09-18 Thread Simon Glass
On 9 September 2016 at 02:27, Petr Kulhavy wrote: > Move FASTBOOT_MBR_NAME and FASTBOOT_GPT_NAME into Kconfig. > Add dependency on the FASTBOOT_FLASH setting (also for FASTBOOT_MBR_NAME). > Remove the now redundant GPT_ENTRY_NAME. > > Signed-off-by: Petr Kulhavy >

Re: [U-Boot] [RFC v2] bootm: fix passing argc to standalone apps

2016-09-18 Thread Simon Glass
On 9 September 2016 at 02:18, Zubair Lutfullah Kakakhel wrote: > This bug appears in b6396403 which makes u-boot unable to pass > arguments via bootm to a standalone application without this patch. > > Steps to reproduce. > > Compile a u-boot. Use mkimage to package

Re: [U-Boot] [PATCH v2 2/4] fastboot: add support for writing MBR

2016-09-18 Thread Simon Glass
On 9 September 2016 at 02:27, Petr Kulhavy wrote: > Add special target "mbr" (otherwise configurable via CONFIG_FASTBOOT_MBR_NAME) > to write MBR partition table. > Partitions are now searched using the generic function which finds any > partiiton by name. For MBR the partition

Re: [U-Boot] [PATCH v2 1/4] disk: part: implement generic function part_get_info_by_name()

2016-09-18 Thread Simon Glass
On 9 September 2016 at 02:27, Petr Kulhavy wrote: > So far partition search by name has been supported only on the EFI partition > table. This patch extends the search to all partition tables. > > Rename part_get_info_efi_by_name() to part_get_info_by_name(), move it from >

Re: [U-Boot] [PATCH v2 01/10] board_f: Add a mach_cpu_init callback

2016-09-18 Thread Simon Glass
On 9 September 2016 at 07:44, Paul Burton wrote: > Currently we have a mismash of architectures which use arch_cpu_init > from architecture-wide code (arc, avr32, blackfin, mips, nios2, xtensa) > and architectures which use arch_cpu_init from machine/SoC level code > (arm,

Re: [U-Boot] [PATCH v2 3/4] disk: part: refactor generic name creation for DOS and ISO

2016-09-18 Thread Simon Glass
On 9 September 2016 at 02:27, Petr Kulhavy wrote: > In both DOS and ISO partition tables the same code to create partition name > like "hda1" was repeated. > > Code moved to into a new function part_set_generic_name() in part.c and > optimized. > Added recognition of MMC and SD

Re: [U-Boot] [PATCH 1/2] sandbox, x86: select DM_KEYBOARD instead of default y entry

2016-09-18 Thread Simon Glass
On 8 September 2016 at 03:47, Masahiro Yamada wrote: > Once we migrate to DM-based drivers, we cannot go back to legacy > ones, i.e. config options like DM_* are not user-configurable. > > Make SANDBOX and X86 select DM_KEYBOARD like other platforms do. > >

Re: [U-Boot] [PATCH 2/2] input: specify the default of I8042_KEYB in more correct manner

2016-09-18 Thread Simon Glass
Hi Masahiro, On 8 September 2016 at 03:47, Masahiro Yamada wrote: > Creating multiple entries of "config FOO" often gives us bad > experiences. In this case, we should specify "default X86" > as platforms that want this keyboard by default. > > Signed-off-by:

Re: [U-Boot] [PATCH] test/py/tests/test_vboot.py: Add check that we boot the image

2016-09-18 Thread Simon Glass
On 18 September 2016 at 07:46, Tom Rini wrote: > > Make sure that when we're telling bootm to boot an image, and we expect > the image to boot we get the output from sandbox that we attempted to > run Linux and that U-Boot completed its job. > > Cc: Simon Glass

Re: [U-Boot] [PATCH v3 00/45] Kconfig: Move CONFIG_SPL_..._SUPPORT to Kconfig

2016-09-18 Thread Simon Glass
Hi Tom, On 18 September 2016 at 09:58, Tom Rini wrote: > On Mon, Sep 12, 2016 at 11:18:18PM -0600, Simon Glass wrote: > >> This series moves all the CONFIG_SPL_..._SUPPORT options to Kconfig and >> fixes up existing boards to continue to build. >> >> It also adds a few small

Re: [U-Boot] [PATCH] ARM: Respect CONFIG_SPL_STACK define in lowlevel_init.S

2016-09-18 Thread Simon Glass
On 4 September 2016 at 21:36, Siarhei Siamashka wrote: > The SPL and U-Boot proper may use different initial stack > locations, which are configured via CONFIG_SPL_STACK and > CONFIG_SYS_INIT_SP_ADDR defines. The lowlevel_init.S > code needs to handle this in the same

Re: [U-Boot] [PATCH] cmd/fdt: add possibilty to have 'extrasize' on fdt resize

2016-09-18 Thread Simon Glass
Hi Hannes, On 5 September 2016 at 02:00, Hannes Schmelzer wrote: > Sometimes devicetree nodes and or properties are added out of the u-boot > console, maybe through some script or manual interaction. > > The devicetree as loaded or embedded is quite small, so

Re: [U-Boot] Please pull ARC changes

2016-09-18 Thread Tom Rini
On Fri, Sep 16, 2016 at 11:30:34AM +, Alexey Brodkin wrote: > Hi Tom, > > That's a minor change that accommodates changes in upcoming ARC GNU tools. > > The following changes since commit 8cbb389bb3da80cbf8911f8386cbff92c6a78afe: > >   Prepare v2016.09 (2016-09-12 10:05:51 -0400) > > are

Re: [U-Boot] [PATCH 26/27] spl: Update spl_load_simple_fit() to take an spl_image param

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:45:15PM -0600, Simon Glass wrote: > Upda the SPL FIT code to use the spl_image parameter. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature

Re: [U-Boot] Please pull u-boot-sunxi master

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 04:56:19PM +0200, Hans de Goede wrote: > Hi Tom, > > Here is the first sunxi pull-req for v2016.11. > > This consists of a couple of small bug-fixes / > defconfig improvements + 1 new board. > > The following changes since commit

Re: [U-Boot] [PATCH 27/27] spl: Make spl_boot_list a local variable

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:45:16PM -0600, Simon Glass wrote: > There is no need for this to be in the BSS region. By moving it we can delay > use of BSS in SPL. This is useful for machines where the BSS region is not > in writeable space. On 64-bit x86, SPL runs from SPI flash and it is easier >

Re: [U-Boot] pull request: u-boot-uniphier/master (2nd round)

2016-09-18 Thread Tom Rini
On Mon, Sep 19, 2016 at 12:35:55AM +0900, Masahiro Yamada wrote: > Hi Tom, > > Here is the revised version of my pull request > for the v2016.11 development cycle. > > The following UniPhier updates are included: > > - DM migration > * remove legacy xHCI driver > * convert MMC

Re: [U-Boot] [PATCH 25/27] spl: Update fat functions to take an spl_image parameter

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:45:14PM -0600, Simon Glass wrote: > Update the fat loader to avoid using the spl_image global variable. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature

Re: [U-Boot] [PATCH 24/27] spl: Update ext functions to take an spl_image parameter

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:45:13PM -0600, Simon Glass wrote: > Update the ext loader to avoid using the spl_image global variable. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature

Re: [U-Boot] [PATCH 18/27] spl: spi: Move the generic SPI loader into common/spl

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:45:07PM -0600, Simon Glass wrote: > All the other SPL loaders are in this directory, so move the SPI one in > there too. > > There are two board-specific SPI loaders (fsl and sunxi). These remain in > the drivers/mtd/spi directory, since they do not contain generic

Re: [U-Boot] [PATCH 23/27] spl: Pass spl_image as a parameter to load_image() methods

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:45:12PM -0600, Simon Glass wrote: > Rather than having a global variable, pass the spl_image as a parameter. > This avoids BSS use, and makes it clearer what the function is actually > doing. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini

Re: [U-Boot] [v2, 1/5] mmc: fsl_esdhc: don't set XFERTYP_RSPTYP_48_BUSY for CMD with busy response

2016-09-18 Thread Jaehoon Chung
On 09/13/2016 04:30 PM, Y.B. Lu wrote: > Hi York, > > I found the MMC maintainer had been changed to Jeahoon. > It seems Panto haven't commented on upstream for a very long time. > > MMC > M: Jaehoon Chung > S: Maintained > T: git

Re: [U-Boot] [v2, 4/5] mmc: add workaround for eSDHC erratum A009620

2016-09-18 Thread Jaehoon Chung
On 08/02/2016 06:20 PM, Yangbo Lu wrote: > Erratum Title: > Data timeout error not getting set in case of command with busy > response (R1b) as well as for busy period after last write block > transfer. > > Description: > In the event that a busy timeout occurs for a command with a busy >

Re: [U-Boot] [v2, 2/5] mmc: send STOP command when the READ/WRITE commands fail

2016-09-18 Thread Jaehoon Chung
Hi Yangbo, On 08/02/2016 06:20 PM, Yangbo Lu wrote: > The STOP command should be sent to stop data transfer when the > READ/WRITE commands fail. Otherwise, any subsequent command will > fail to be sent. > > Signed-off-by: Yangbo Lu > --- > Changes for v2: > - None > ---

Re: [U-Boot] [v2, 1/5] mmc: fsl_esdhc: don't set XFERTYP_RSPTYP_48_BUSY for CMD with busy response

2016-09-18 Thread Jaehoon Chung
Hi Yangbo, On 08/02/2016 06:20 PM, Yangbo Lu wrote: > For CMD with busy response, the eSDHC driver would poll DAT0 until > CMD completion rather than polling IRQSTAT. So, don't set > XFERTYP_RSPTYP_48_BUSY to avoid interrupts (DTOE or TC) in IRQSTAT. Sorry for late.. I missed your patchset. >

Re: [U-Boot] [PATCH] ML: SPL: NOR: Add CONFIG_SPL_NOR_COPY_ENTIRE_IMAGE define to enable whole image copy from NOR

2016-09-18 Thread Lukasz Majewski
Hi Tom, > On Sun, Sep 18, 2016 at 09:10:34PM +0200, Lukasz Majewski wrote: > > Hi Tom, > > > > > On Fri, Sep 16, 2016 at 10:43:51AM +0200, Lukasz Majewski wrote: > > > > > > > This define gives the possibility to copy entire image > > > > (including header) from NOR parallel memory to e.g.

Re: [U-Boot] [PATCH] Revert "image-fit: switch ENOLINK to ENOENT"

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 03:14:30PM +0200, Marek Vasut wrote: > On 09/17/2016 05:42 AM, Jonathan Gray wrote: > > On Fri, Sep 16, 2016 at 10:21:43PM +0100, Paul Burton wrote: > >> Commit bac17b78dace ("image-fit: switch ENOLINK to ENOENT") changed > >> fit_get_node_from_config to return -ENOENT when

[U-Boot] [PATCH 11/13] buildman: Drop the 'active' flag in the builder

2016-09-18 Thread Simon Glass
This serves no real purpose, since when we are not active, we exit. Drop it. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 7 --- tools/buildman/builderthread.py | 10 +- 2 files changed, 1 insertion(+), 16 deletions(-) diff --git

[U-Boot] [PATCH 10/13] buildman: Allow builds to terminate cleanly

2016-09-18 Thread Simon Glass
It is annoying that buildman does not respond cleanly to Ctrl-C or SIGINT, particularly on machines with lots of CPUS. Unfortunately queue.join() blocks the main thread and does not allow it to see the signal. Use a separate thread instead, Signed-off-by: Simon Glass ---

[U-Boot] [PATCH 07/13] buildman: Print a message when removing old directories

2016-09-18 Thread Simon Glass
When buildman starts, it prepares its output directory by removing any old build directories which will not be used this time. This can happen if a previous build left directories around for commit hashes which are no-longer part of the branch. This can take quite a while, so print a message to

[U-Boot] [PATCH 13/13] buildman: Drop the 'alive' flag in BuilderThread

2016-09-18 Thread Simon Glass
This is not used, so drop it. Signed-off-by: Simon Glass --- tools/buildman/builderthread.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/tools/buildman/builderthread.py b/tools/buildman/builderthread.py index 06be5d9..a02aa19 100644 ---

[U-Boot] [PATCH 06/13] buildman: Tidy up the 'cloning' message

2016-09-18 Thread Simon Glass
On a machine with a lot of CPUs this prints a lot of useless lines of the form: Cloning repo for thread Adjust the output so that these all appear on one line, and disappear when the cloning is complete. Note: This cloning is actually unnecessary and very wasteful on disk space (about 3.5GB

[U-Boot] [PATCH 05/13] patman: Flush output when there is no newline

2016-09-18 Thread Simon Glass
Output which does not include a newline will not be displayed unless flushed. Add a flush to ensure that it becomes visible. Signed-off-by: Simon Glass --- tools/patman/terminal.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/patman/terminal.py

[U-Boot] [PATCH 09/13] buildman: Put our local libraries first in the path

2016-09-18 Thread Simon Glass
If patman is installed on the machine (e.g. in the standard dist-packages directory), it will find libraries from there in preference to our local libraries. Adjust the order of the path to ensure that local libraries are found first. Signed-off-by: Simon Glass ---

[U-Boot] [PATCH 12/13] buildman: Don't show a stacktrace on Ctrl-C

2016-09-18 Thread Simon Glass
When Ctrl-C is pressed, just exited quietly. There is no sense in displaying a stack trace since buildman will always be in the same place: waiting for threads to complete building all the jobs on the queue. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 7

[U-Boot] [PATCH 08/13] buildman: Print a message indicating the build is starting

2016-09-18 Thread Simon Glass
Make it clear when buildman actually starts building. This happens when it has prepared the threads, working directory and output directories. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 1 + tools/buildman/test.py| 4 ++-- 2 files changed, 3

[U-Boot] [PATCH 01/13] Makefile: Add a target to create the .cfg files

2016-09-18 Thread Simon Glass
A common requirement when converting CONFIG options to Kconfig is to check that the effective configuration has not changed due to the conversion. Add a target which creates this configuration (in the form of u-boot.cfg) but does not build U-Boot. This speeds up the checking. Signed-off-by: Simon

[U-Boot] [PATCH 04/13] buildman: Squash useless output from -K

2016-09-18 Thread Simon Glass
When using #define CONFIG_SOME_OPTION, the value it set to '1'. When using defconfig (i.e. CONFIG_SOME_OPTION=y) the value is set to 'y'. This results in differences showing up with -K. These differences are seldom useful. Adjust buildman to suppress these differences by default. Signed-off-by:

[U-Boot] [PATCH 00/13] buildman: Threading and output improvements

2016-09-18 Thread Simon Glass
This series includes a few minor improvements to buildman: - Cleaner output and updates on what buildman is doing at the start - a way to generate only the CONFIG options in a build, for speed (-D) - a fix for pressing Ctrl-C during a build (so that it exits cleanly). Simon Glass (13):

[U-Boot] [PATCH 03/13] buildman: Add documentation for CONFIG checking

2016-09-18 Thread Simon Glass
The -K option is not mentioned in the README at present. Add some notes to describe how this is used. Signed-off-by: Simon Glass --- tools/buildman/README | 37 + 1 file changed, 37 insertions(+) diff --git a/tools/buildman/README

[U-Boot] [PATCH 02/13] buildman: Add an option to just create the config

2016-09-18 Thread Simon Glass
Normally buildman does a full build of a board. This includes creating the u-boot.cfg file which contains all the configuration options. Buildman uses this file with the -K option, to show differences in effective configuration for each commit. Doing a full build of U-Boot just to create the

Re: [U-Boot] [PATCH] Revert "image-fit: switch ENOLINK to ENOENT"

2016-09-18 Thread Jonathan Gray
On Sun, Sep 18, 2016 at 03:14:30PM +0200, Marek Vasut wrote: > On 09/17/2016 05:42 AM, Jonathan Gray wrote: > > On Fri, Sep 16, 2016 at 10:21:43PM +0100, Paul Burton wrote: > >> Commit bac17b78dace ("image-fit: switch ENOLINK to ENOENT") changed > >> fit_get_node_from_config to return -ENOENT when

Re: [U-Boot] [PATCH 07/27] spl: Convert boot_device into a struct

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:44:56PM -0600, Simon Glass wrote: > At present some spl_xxx_load_image() functions take a parameter and some > don't. Of those that do, most take an integer but one takes a string. > > Convert this parameter into a struct so that we can pass all functions the > same

Re: [U-Boot] [PATCH 00/27] spl: Use linker list and parameters for SPL image loading

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:44:49PM -0600, Simon Glass wrote: > At present the SPL code uses a global spl_image variable which is shared > amongst lots of files, some in common/spl and some elsewhere. There is no > need for this to be global, and in fact a parameter makes it easier to > understand

Re: [U-Boot] [PATCH 08/27] spl: Add a way to declare an SPL image loader

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:44:57PM -0600, Simon Glass wrote: > Add a linker list macro which can be used to declare an SPL image loader. > Update spl_load_image() to search available loaders for the correct one. > > Signed-off-by: Simon Glass > --- > > common/spl/spl.c | 20

Re: [U-Boot] [PATCH 06/27] spl: Kconfig: Move SPL_DISPLAY_PRINT to Kconfig

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:44:55PM -0600, Simon Glass wrote: > Move this option to Kconfig and tidy up existing uses. Also add a function > comment to the header file. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini -- Tom signature.asc

Re: [U-Boot] [PATCH 05/27] spl: Add function comments to spl_start_uboot()

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:44:54PM -0600, Simon Glass wrote: > Add some comments to describe this function. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature

Re: [U-Boot] [PATCH 04/27] spl: Add a parameter to jump_to_image_linux()

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:44:53PM -0600, Simon Glass wrote: > Instead of using the global spl_image variable, pass the required struct in > as an argument. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini -- Tom signature.asc Description:

Re: [U-Boot] [PATCH 03/27] spl: Add a parameter to spl_parse_image_header()

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:44:52PM -0600, Simon Glass wrote: > Instead of using the global spl_image variable, pass the required struct in > as an argument. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini -- Tom signature.asc Description:

Re: [U-Boot] [PATCH 01/27] spl: Move spl_board_load_image() into a generic header

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:44:50PM -0600, Simon Glass wrote: > At present this is only used on ARM and sandbox, but it is just as > applicable to other architectures. Move the function prototype into the > generic SPL header. > > Signed-off-by: Simon Glass Reviewed-by: Tom

Re: [U-Boot] [PATCH 02/27] spl: Add a parameter to spl_set_header_raw_uboot()

2016-09-18 Thread Tom Rini
On Sun, Sep 18, 2016 at 01:44:51PM -0600, Simon Glass wrote: > Rather than act on the global variable, pass the required struct in as a > parameter. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital

[U-Boot] [PATCH 19/27] spl: Convert spl_spi_load_image() to use linker list

2016-09-18 Thread Simon Glass
Add a linker list declaration for this method and remove the explicit switch() code. Also set up the sunxi function. Signed-off-by: Simon Glass --- common/spl/spl.c| 4 common/spl/spl_spi.c| 4 +++- drivers/mtd/spi/sunxi_spi_spl.c | 4 +++-

[U-Boot] [PATCH 24/27] spl: Update ext functions to take an spl_image parameter

2016-09-18 Thread Simon Glass
Update the ext loader to avoid using the spl_image global variable. Signed-off-by: Simon Glass --- common/spl/spl_ext.c| 21 - common/spl/spl_mmc.c| 4 ++-- drivers/mtd/spi/sunxi_spi_spl.c | 9 + include/spl.h

  1   2   3   >