[U-Boot] [PATCH 2/5] arm: fsl-layerscape: move forward the non-secure access permission setup

2016-07-03 Thread Zhiqiang Hou
From: Hou Zhiqiang Move forward the basic non-secure access enable operation, so the subsequent individual device access permission can override it. And collect the dispersed callers in board level, and then move them to SoC level. Signed-off-by: Hou Zhiqiang --- Tested on LS1043ARDB, LS1021AQD

[U-Boot] [PATCH 3/5] fsl: csu: add an API to set individual device access permission

2016-07-03 Thread Zhiqiang Hou
From: Hou Zhiqiang Add this API to make the individual device is able to be set to the specified permission. Signed-off-by: Hou Zhiqiang --- board/freescale/common/ns_access.c | 34 -- include/fsl_csu.h | 1 + 2 files changed, 21 insertions(+),

[U-Boot] [PATCH 5/5] fsl-layerscape: Add workaround for PCIe erratum A010315

2016-07-03 Thread Zhiqiang Hou
From: Hou Zhiqiang As the access to serders protocol unselected PCIe controller will hang. So disable the R/W permission to unselected PCIe controller including its CCSR, IO space and memory space according to the serders protocol field of RCW. Signed-off-by: Hou Zhiqiang --- Tested on LS1043AR

[U-Boot] [PATCH 4/5] fsl: csu: add an API to disable all R/W permission to PCIe

2016-07-03 Thread Zhiqiang Hou
From: Hou Zhiqiang Signed-off-by: Hou Zhiqiang --- .../include/asm/arch-fsl-layerscape/ns_access.h| 1 + board/freescale/common/ns_access.c | 28 ++ include/fsl_csu.h | 1 + 3 files changed, 30 insertions(+) diff --git

Re: [U-Boot] [PATCH 2/4] usb: dwc2-otg: redefine fifo-size for rk3288

2016-07-03 Thread Ziyuan Xu
Hi Simon, On 2016年07月01日 23:15, Simon Glass wrote: Hi, On 30 June 2016 at 23:00, Ziyuan Xu wrote: Hi Simon, On 2016年06月30日 23:23, Simon Glass wrote: Hi Ziyuan, On 30 June 2016 at 00:21, Ziyuan Xu wrote: Redefine RX FIFO size & TX FIFO size for rk3288. Signed-off-by: Ziyuan Xu --- d

[U-Boot] [PATCH] driver: fsl_qspi: disable AHB buffer prefetch

2016-07-03 Thread Yunhui Cui
From: Yunhui Cui A-009282: QuadSPI: QuadSPI data pre-fetch can result in incorrect data Affects: QuadSPI Description: With AHB buffer prefetch enabled, the QuadSPI may return incorrect data on the AHB interface. The buffer pre-fetch is enabled if the fetch size as configured either in the LUT or

Re: [U-Boot] [PATCH 24/27] dm: zynq: usb: Convert to CONFIG_DM_USB

2016-07-03 Thread Siva Durga Prasad Paladugu
Hi Simon, > -Original Message- > From: Siva Durga Prasad Paladugu > Sent: Tuesday, June 28, 2016 11:38 AM > To: 'Simon Glass' ; U-Boot Mailing List b...@lists.denx.de> > Cc: Marek Vasut ; 'Hans de Goede' > Subject: RE: [U-Boot] [PATCH 24/27] dm: zynq: usb: Convert to > CONFIG_DM_USB > >

Re: [U-Boot] [Patch v2 2/4] armv8/fsl_lsch2: Add LS1046A SoC support

2016-07-03 Thread Prabhakar Kushwaha
> -Original Message- > From: Gong Qianyu [mailto:qianyu.g...@nxp.com] > Sent: Friday, July 01, 2016 4:19 PM > To: york sun ; Prabhakar Kushwaha > ; u-boot@lists.denx.de > Cc: Mingkai Hu ; Zhiqiang Hou > ; Shaohui Xie ; Wenbin > Song ; Zhiqiang Hou ; > Mihai Bantea ; Qianyu Gong > > Subje

[U-Boot] Logging SPL output with test/py

2016-07-03 Thread Simon Glass
Hi Stephen, I'd like to get access to SPL output from a test. It seems that only the output from U-Boot proper is logged, although I cannot see why. For background, I have a new sandbox_spl target which includes SPL code. I will likely merge it into the normal sandbox target... It is definitely

Re: [U-Boot] [PATCH v4 1/2] armv8: Support loading 32-bit OS in AArch32 execution state

2016-07-03 Thread Huan Wang
> On 30.06.16 09:16, Huan Wang wrote: > > Hi, Alex, > > > >>> Am 15.06.2016 um 05:04 schrieb Huan Wang : > >>> > >>> Hi, Alex, > >>> > > On 06/08/2016 07:14 AM, Alison Wang wrote: > > To support loading a 32-bit OS, the execution state will change > > from > > AArch64 to AArch32 whe

Re: [U-Boot] UBIFS status in the mainline

2016-07-03 Thread Max Filippov
On Sun, Jul 3, 2016 at 9:53 PM, Stefan Agner wrote: > On 2016-07-02 19:27, Max Filippov wrote: >> I'm getting the following build errors when I'm trying to build U-Boot >> for a board with UBIFS in its config from the current U-Boot mainline: >> >> fs/built-in.o: In function `do_fs_type': >> (.tex

Re: [U-Boot] [PATCH] armv8: Enable CPUECTLR.SMPEN for data coherency

2016-07-03 Thread Mingkai Hu
> -Original Message- > From: Edward L Swarthout > Sent: Saturday, July 02, 2016 5:44 AM > To: Prabhakar Kushwaha; york sun; Qianyu Gong; albert.u.b...@aribaud.net; > u-boot@lists.denx.de; s.temerkha...@gmail.com; > yamada.masah...@socionext.com > Cc: Mingkai Hu > Subject: RE: [U-Boot] [PA

Re: [U-Boot] [PATCH] MMC: Do not write to mmc_uclass_priv if it was not allocated

2016-07-03 Thread Jaehoon Chung
On 07/02/2016 03:17 AM, Simon Glass wrote: > Hi, > > On 1 July 2016 at 04:37, Jaehoon Chung wrote: >> On 07/01/2016 04:28 AM, Simon Glass wrote: >>> Hi Meteusz, >>> >>> On 30 June 2016 at 12:18, Mateusz Kulikowski >>> wrote: On 29.06.2016 05:27, Simon Glass wrote: > Hi Matt, > >

[U-Boot] [RFC PATCH] sunxi: SPL: place BSS in SRAM if possible

2016-07-03 Thread Andre Przywara
The SPL BSS is placed in DRAM for all sunxi boards at the moment, because the space in the A1 SRAM space, where the SPL is normally loaded, is limited. But this is problematic, because any BSS variable could be used before DRAM is initialised, eventually hanging the board. Many SoCs however have ei

[U-Boot] [PATCH] SPL: tiny-printf: avoid any BSS usage

2016-07-03 Thread Andre Przywara
As printf calls may be executed quite early, we should avoid using any BSS stored variables, since some boards put BSS in DRAM, which may not have been initialised yet. Explicitly mark those "static global" variables as belonging to the .data section, to keep tiny-printf clear of any BSS usage. Ple

Re: [U-Boot] [PATCH v2 4/5] tools: patman: Handle missing blank line for 'Series-changes'

2016-07-03 Thread Simon Glass
On 27 June 2016 at 00:24, Bin Meng wrote: > 'Series-changes' uses blank line to indicate its end. If that is > missing, series internal state variable 'in_change' may be wrong. > Correct its state. > > Signed-off-by: Bin Meng > Acked-by: Simon Glass > > --- > > Changes in v2: > - Geneate warning

Re: [U-Boot] [PATCH v2 5/5] tools: patman: Handle missing 'END' in non-last commit of a series

2016-07-03 Thread Simon Glass
On 28 June 2016 at 21:27, Simon Glass wrote: > On 26 June 2016 at 23:24, Bin Meng wrote: >> The following python error: >> >> Traceback (most recent call last): >> File "./tools/patman/patman", line 144, in >> series = patchstream.FixPatches(series, args) >> File "./tools/patman/patchstr

Re: [U-Boot] [PATCH v2 2/5] tools: patman: Handle tag sections without an 'END'

2016-07-03 Thread Simon Glass
On 27 June 2016 at 00:24, Bin Meng wrote: > 'Cover-letter', 'Series-notes' and 'Commit-notes' tags require an > 'END' to be put at the end of its section. If we forget to put an > 'END' in those sections, and these sections are followed by another > patman tag, patman generates incorrect patches.

Re: [U-Boot] [PATCH v2 3/5] tools: patman: Generate cover letter correctly when 'END' is missing

2016-07-03 Thread Simon Glass
On 28 June 2016 at 21:27, Simon Glass wrote: > On 26 June 2016 at 23:24, Bin Meng wrote: >> If 'END' is missing in a 'Cover-letter' section, and that section >> happens to show up at the very end of the commit message, and the >> commit is the last commit of the series, patman fails to generate >

Re: [U-Boot] [PATCH 14/14] test: Convert the vboot test to test/py

2016-07-03 Thread Simon Glass
Hi Teddy, On 3 July 2016 at 15:38, Teddy Reed wrote: > Hi Simon, > > On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: >> Now that we have a suitable test framework we should move all tests into it. >> The vboot test is a suitable candidate. Rewrite it in Python and move the >> data files into

Re: [U-Boot] [PATCH v2 1/5] tools: patman: Use cover_match for 'Cover-letter'

2016-07-03 Thread Simon Glass
On 28 June 2016 at 21:27, Simon Glass wrote: > On 26 June 2016 at 23:24, Bin Meng wrote: >> Like other patman tags, use a new variable cover_match to indicate >> a match for 'Cover-letter'. >> >> Signed-off-by: Bin Meng >> >> --- >> >> Changes in v2: >> - New patch to use cover_match for 'Cover-

Re: [U-Boot] [PATCH v3 5/5] sandbox: Find keyboard driver using driver model

2016-07-03 Thread Simon Glass
On 19 June 2016 at 17:33, Simon Glass wrote: > The cros-ec keyboard is always a child of the cros-ec node. Rather than > searching the device tree, looking at the children. Remove the compat string > which is now unused. > > Signed-off-by: Simon Glass > --- > > Changes in v3: > - Add new patch to

Re: [U-Boot] [PATCH] dm: Sort the uclass id in alphabetical order

2016-07-03 Thread Simon Glass
On 25 June 2016 at 20:53, Simon Glass wrote: > On 22 June 2016 at 03:29, Bin Meng wrote: >> Some uclass ids are out of order. Per the comments, sort them >> in alphabetical order. >> >> Signed-off-by: Bin Meng >> --- >> >> include/dm/uclass-id.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 d

Re: [U-Boot] [PATCH v3 3/5] fdt: Add a note to avoid adding new compatible strings

2016-07-03 Thread Simon Glass
On 19 June 2016 at 19:48, Bin Meng wrote: > On Mon, Jun 20, 2016 at 7:33 AM, Simon Glass wrote: >> The list is shrinking and we should avoid adding new things. Instead, a >> proper driver should be created with driver model. >> >> Signed-off-by: Simon Glass >> --- >> >> Changes in v3: >> - Add n

Re: [U-Boot] [PATCH v3 4/5] fdt: x86: Tidy up a few COMPAT string definitions

2016-07-03 Thread Simon Glass
On 19 June 2016 at 19:48, Bin Meng wrote: > On Mon, Jun 20, 2016 at 7:33 AM, Simon Glass wrote: >> The 'COMPAT_' part should appear only once so drop the duplicate part. It is >> ignored anyway, but let's keep things consistent. >> >> Signed-off-by: Simon Glass >> --- >> >> Changes in v3: >> - A

Re: [U-Boot] [PATCH v3 2/5] fdt: Drop unused exynos compatible strings

2016-07-03 Thread Simon Glass
On 29 June 2016 at 06:22, Jaehoon Chung wrote: > Hi Simon, > > On 06/20/2016 08:33 AM, Simon Glass wrote: >> A few drivers have moved to driver model, so we can drop these strings. >> >> Signed-off-by: Simon Glass > > CC'd Minkyu. > > Acked-by: Jaehoon Chung > > Best Regards, > Jaehoon Chung Ap

Re: [U-Boot] [PATCH v3 1/5] x86: fdt: Drop the unused compatible strings in fdtdec

2016-07-03 Thread Simon Glass
On 19 June 2016 at 17:33, Simon Glass wrote: > We have drivers for several more devices now, so drop the strings which are > no-longer used. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > > --- > > Changes in v3: > - Rebase to master > > arch/x86/cpu/ivybridge/lpc.c | 6 -- > incl

Re: [U-Boot] [PATCH v2] dm: gpio: MPC85XX GPIO platform data support

2016-07-03 Thread Simon Glass
On 16 June 2016 at 21:50, Simon Glass wrote: > On 13 June 2016 at 16:17, Hamish Martin > wrote: >> Define a platform data structure for the MPC85XX GPIO driver to allow >> use of the driver without device tree. Users should define the GPIO >> blocks for their platform like this: >> struct mpc85

Re: [U-Boot] [RESEND] common: block: fix compiler error with CONFIG_FASTBOOT_FLASH_MMC_DEV

2016-07-03 Thread Simon Glass
On 16 June 2016 at 21:52, Simon Glass wrote: > On 15 June 2016 at 02:56, Ziyuan Xu wrote: >> This fixes the following compiler error: >> >> common/fb_mmc.c: In function ‘fb_mmc_erase’: >> common/fb_mmc.c:209:17: error: ‘struct blk_desc’ has no member named >> ‘block_erase’ >> >> Signed-off-by: Zi

Re: [U-Boot] [PATCH 20/27] dm: spl: mmc: Support CONFIG_BLK in SPL MMC

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Update the method of accessing the block device so that it works with > CONFIG_BLK enabled. > > Signed-off-by: Simon Glass > --- > > common/spl/spl_mmc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Applied to u-boot-dm/next. _

Re: [U-Boot] [PATCH 19/27] dm: mmc: msmsdhic: Drop old MMC code

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Now that we have fully moved to driver model, drop the old code. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/Kconfig | 2 +- > drivers/mmc/msm_sdhci.c | 15 --- > 2 files changed, 1 insertion(+), 16 deletions(-) Applied to

Re: [U-Boot] [PATCH 1/9] dm: imx: serial: Support DTE mode when using driver model

2016-07-03 Thread Simon Glass
On 3 July 2016 at 13:33, Stefan Agner wrote: > From: Stefan Agner > > The MXC UART IP can be run in DTE or DCE mode. This depends on the > board wiring and the pinmux used and hence is board specific. This > extends platform data with a new field to choose wheather DTE > mode shall be used. > > S

Re: [U-Boot] [PATCH 18/27] dm: mmc: Move dragonboard410c to use CONFIG_BLK and CONFIG_DM_MMC_OPS

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Update this board to use driver model for block devices and MMC operations. > > Signed-off-by: Simon Glass > --- > > configs/dragonboard410c_defconfig | 2 ++ > 1 file changed, 2 insertions(+) Applied to u-boot-dm/next.

Re: [U-Boot] [PATCH 21/27] dm: dfu: mmc: Support CONFIG_BLK in DFU for MMC

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Update the method of accessing the block device so that it works with > CONFIG_BLK enabled. > > Signed-off-by: Simon Glass > --- > > drivers/dfu/dfu_mmc.c | 11 +-- > 1 file changed, 5 insertions(+), 6 deletions(-) Applied to u-boot-dm/next

Re: [U-Boot] [PATCH 13/27] rockchip: Add MAINTAINER files for kylin_rk3036, evb_rk3036

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > These boards should have maintainer entries. Add them. > > Signed-off-by: Simon Glass > --- > > board/evb_rk3036/evb_rk3036/MAINTAINERS | 6 ++ > board/kylin/kylin_rk3036/MAINTAINERS| 6 ++ > 2 files changed, 12 insertions(+) Applied to

Re: [U-Boot] [PATCH 16/27] dm: mmc: sdhci: Support CONFIG_BLK and CONFIG_DM_MMC_OPS

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Add support for using driver model for block devices and MMC operations in > this driver. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/sdhci.c | 40 ++- > include/sdhci.h | 80 > ++

Re: [U-Boot] [PATCH 15/27] dm: mmc: sdhci: Refactor configuration setup to support DM

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Move the configuration setting into a separate function which can be used by > the driver-model code. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/sdhci.c | 107 > +--- > 1 file changed, 59 in

Re: [U-Boot] [PATCH 17/27] dm: mmc: msm_sdhci: Support CONFIG_BLK and CONFIG_DM_MMC_OPS

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Add support for using driver model for block devices and MMC operations in > this driver. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/msm_sdhci.c | 51 > +++-- > 1 file changed, 49 insertions(+),

Re: [U-Boot] [PATCH 14/27] dm: sandbox: Convert to use CONFIG_CMD_MMC_OPS

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Update the sandbox MMC emulation to use driver model for MMC operations. > > Signed-off-by: Simon Glass > --- > > arch/Kconfig | 1 + > configs/sandbox_defconfig | 4 ++-- > drivers/mmc/sandbox_mmc.c | 17 ++--- > 3 files

Re: [U-Boot] [PATCH 11/27] dm: mmc: dwmmc: Support CONFIG_DM_MMC_OPS

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Add support to dwmmc for using driver model for MMC operations. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/dw_mmc.c | 33 + > include/dwmmc.h | 9 + > 2 files changed, 42 insertions(+) Applied

Re: [U-Boot] [PATCH 12/27] dm: mmc: rockchip: Enable CONFIG_DM_MMC_OPS for all boards

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Enable this option to move rockchip over to use driver model for MMC > operations. > > Signed-off-by: Simon Glass > --- > > arch/arm/Kconfig | 1 + > drivers/mmc/rockchip_dw_mmc.c | 3 ++- > 2 files changed, 3 insertions(+), 1 deletion(

Re: [U-Boot] [PATCH 10/27] dm: mmc: Add a way to use driver model for MMC operations

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > The driver model conversion for MMC has moved in small steps. The first step > was to have an MMC device (CONFIG_DM_MMC). The second was to use a child > block device (CONFIG_BLK). The final one is to use driver model for MMC > operations (CONFIG_DM_M

Re: [U-Boot] [PATCH 09/27] rockchip: Disable CONFIG_SDHCI

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > This option is not actually needed for rockchip boards. Drop it, since it > will not support driver-model MMC operation support. > > Signed-off-by: Simon Glass > --- > > include/configs/rk3036_common.h | 1 - > include/configs/rk3288_common.h | 1 -

Re: [U-Boot] [PATCH 03/27] mmc: Add function declarations for mmc_bread() and mmc_switch_part()

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > These private functions are used both in the driver-model implementation and > in the legacy code. Add them to the header. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/mmc.c | 9 - > drivers/mmc/mmc_private.h | 17 ++

Re: [U-Boot] [PATCH 06/27] mmc: Move MMC boot code into its own file

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Rather than having an #ifdef in the main mmc.c file, control this feature > from the Makefile by moving the code into its own file. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/Makefile | 3 ++ > drivers/mmc/mmc.c | 126 +-

Re: [U-Boot] [PATCH 08/27] mmc: Move tracing code into separate functions

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Move this code into separate functions so that it can be used from the uclass > also. Add static inline versions for when the option is disabled. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/mmc.c | 41 ++-

Re: [U-Boot] [PATCH 04/27] dm: mmc: Move CONFIG_BLK code into the mmc uclass

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Rather than having #ifdef in mmc.c, move this code into the uclass file. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/mmc-uclass.c | 80 + > drivers/mmc/mmc.c| 85 > ++-

Re: [U-Boot] [PATCH 07/27] dm: mmc: rockchip: Support only CONFIG_BLK

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Since all Rockchip boards use CONFIG_BLK, we can remove this old code. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/rockchip_dw_mmc.c | 11 --- > 1 file changed, 11 deletions(-) Applied to u-boot-dm/next. ___

Re: [U-Boot] [PATCH 05/27] dm: mmc: Move non-CONFIG_BLK code into mmc_legacy.c

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Rather than having #ifdef in mmc.c, move this code into the legacy file. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/mmc.c| 95 > > drivers/mmc/mmc_legacy.c | 91

Re: [U-Boot] [PATCH 4/9] mx7: set soc environment according to exact SoC type

2016-07-03 Thread Simon Glass
On 3 July 2016 at 13:33, Stefan Agner wrote: > From: Stefan Agner > > This can be useful if the same U-Boot binary is used for boards > available with a i.MX 7Solo and i.MX 7Dual. > > Signed-off-by: Stefan Agner > --- > > arch/arm/cpu/armv7/mx7/soc.c | 12 > include/configs/mx7_com

Re: [U-Boot] [PATCH 01/27] dm: mmc: dwmmc: Add comments to the dwmmc setup functions

2016-07-03 Thread Simon Glass
On 3 July 2016 at 16:28, Simon Glass wrote: > Hi Jaehoon, > > On 27 June 2016 at 04:54, Jaehoon Chung wrote: >> Hi Simon, >> >> >> On 06/13/2016 02:30 PM, Simon Glass wrote: >>> These comments were missed when the original code was written. Add them to >>> help people port their drivers over. >>>

Re: [U-Boot] [PATCH 8/9] power: pmic: add Ricoh RN5T567 PMIC support

2016-07-03 Thread Simon Glass
Hi, On 3 July 2016 at 13:33, Stefan Agner wrote: > From: Stefan Agner > > Add power driver for Ricoh PMIC used on Colibri iMX7. > > Signed-off-by: Stefan Agner > Acked-by: Marcel Ziswiler > --- > > drivers/power/pmic/Makefile | 1 + > drivers/power/pmic/pmic_rn5t567.c | 32 ++

Re: [U-Boot] [PATCH 02/27] rockchip: Use 'select' instead of defaults in Kconfig

2016-07-03 Thread Simon Glass
On 12 June 2016 at 23:30, Simon Glass wrote: > Rockchip uses driver model for all subsystems. Specify this in the arm > Kconfig rather than as defaults in the Rockchip Kconfig. This means that > boards cannot turn these options off, which seems correct. > > Signed-off-by: Simon Glass > --- > > a

Re: [U-Boot] [PATCH 1/2] usb: dm: Add a usb_for_each_root_dev() helper function

2016-07-03 Thread Simon Glass
On 3 July 2016 at 12:22, Hans de Goede wrote: > Iterating over usb-root devs and doing something for all of them is > a bit tricky with dm, factor out the proven usb_show_tree() for this > into a helper function. > > Signed-off-by: Hans de Goede > --- > cmd/usb.c | 15 --- > 1 file c

Re: [U-Boot] [PATCH 2/2] usb: dm: Make "usb info" use usb_for_each_root_dev()

2016-07-03 Thread Simon Glass
On 3 July 2016 at 12:22, Hans de Goede wrote: > The old dm "usb info" implementation has several issues: > > 1) NULL pointer deref when a bus has no children > 2) Not showing usb devices on busses without an emulated root-hub (otg host) > 3) Attempting to show devices on inactive busses > 4) "usb

Re: [U-Boot] [PATCH 2/9] usb: ehci-mx6: configure power polarity in usb_power_config

2016-07-03 Thread Marek Vasut
On 07/03/2016 09:33 PM, Stefan Agner wrote: From: Stefan Agner USBNC_n_CTRL1 bit 9 actually controls the power pin polarity. Rename UCTRL_PM to align reference manual and set the bit in the appropriate callback usb_power_config. Signed-off-by: Stefan Agner Just for the extra safety, can you

Re: [U-Boot] dm: usb: Prevent NULL hub in usb_device_info()

2016-07-03 Thread Marek Vasut
On 07/03/2016 10:10 PM, Bernhard Nortmann wrote: Hi Hans! Hi, Am 03.07.2016 um 20:21 schrieb Hans de Goede: Hi, On 02-07-16 23:13, Hans de Goede wrote: Hi, [...] I've prepared 2 patches which re-use the "usb tree" code rather then re-inventing the wheel again: https://github.com/jwrdegoe

Re: [U-Boot] [PATCH 3/9] usb: ehci-mx6: introduce config for high active power pin

2016-07-03 Thread Marek Vasut
On 07/03/2016 09:33 PM, Stefan Agner wrote: From: Stefan Agner Add a new config CONFIG_MXC_USB_OTG_HACTIVE which configures the OTG Power Pin to be high active. Low active is the reset value of the affected configuration register, hence the config option is named by the non-reset configuration.

Re: [U-Boot] [PATCH 01/27] dm: mmc: dwmmc: Add comments to the dwmmc setup functions

2016-07-03 Thread Simon Glass
Hi Jaehoon, On 27 June 2016 at 04:54, Jaehoon Chung wrote: > Hi Simon, > > > On 06/13/2016 02:30 PM, Simon Glass wrote: >> These comments were missed when the original code was written. Add them to >> help people port their drivers over. >> >> Signed-off-by: Simon Glass >> --- >> >> include/dwm

Re: [U-Boot] [PATCH] SPL: sunxi: don't force .BSS into DRAM

2016-07-03 Thread André Przywara
On 02/07/16 12:49, Hans de Goede wrote: Hi Hans, > On 30-06-16 17:24, Simon Glass wrote: >> Hi, >> >> On 30 June 2016 at 03:00, Hans de Goede wrote: >>> Hi Andre, >>> >>> On 30-06-16 02:25, Andre Przywara wrote: Probably due to some (ill-founded) fear of a large BSS all sunxi boards >>

Re: [U-Boot] [PATCH 14/14] test: Convert the vboot test to test/py

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > Now that we have a suitable test framework we should move all tests into it. > The vboot test is a suitable candidate. Rewrite it in Python and move the > data files into an appropriate directory. > > Signed-off-by: Simon Glass > ---

Re: [U-Boot] [PATCH 12/14] tools: Correct error handling in fit_image_process_hash()

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > We should not be returning -1 as an error code. This can mask a situation > where we run out of space adding things to the FIT. By returning the correct > error in this case (-ENOSPC) it can be handled by the higher-level code. > > Th

[U-Boot] [PATCH 09/11] buildman: Add a quick-start note

2016-07-03 Thread Simon Glass
For those who just want to build a board, it is useful to see a quick hint right at the start of the documentation. Add a few commands showing how to download toolchains and build a board. Signed-off-by: Simon Glass --- tools/buildman/README | 14 ++ 1 file changed, 14 insertions(+)

Re: [U-Boot] [PATCH] SPL: sunxi: don't force .BSS into DRAM

2016-07-03 Thread Simon Glass
Hi Hans, On 2 July 2016 at 04:49, Hans de Goede wrote: > Hi, > > > On 30-06-16 17:24, Simon Glass wrote: >> >> Hi, >> >> On 30 June 2016 at 03:00, Hans de Goede wrote: >>> >>> Hi Andre, >>> >>> On 30-06-16 02:25, Andre Przywara wrote: Probably due to some (ill-founded) fear of a l

Re: [U-Boot] [PATCH 01/14] test: Add a README

2016-07-03 Thread Simon Glass
+Roger, for Travis-CI Hi Teddy, On 3 July 2016 at 13:17, Teddy Reed wrote: > Hi Simon, > > On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: >> Add a few notes about how testing works in U-Boot. >> >> Signed-off-by: Simon Glass > > Reviewed-by: Teddy Reed Thanks for the comments. Will tidy

[U-Boot] [PATCH 05/11] buildman: Improve the toolchain progress/error output

2016-07-03 Thread Simon Glass
Use colour to make it easier to see what is going on. Also print a message before downloading a new toolchain. Mention --fetch-arch in the message that is shown when there are no available toolchains, since this is the quickest way to resolve the problem. Signed-off-by: Simon Glass --- tools/bu

[U-Boot] [PATCH 10/11] Drop the MAKEALL tool

2016-07-03 Thread Simon Glass
Buildman has been around for 3 years now. It has had a lot of use and testing. Perhaps it is time to remove MAKEALL. Signed-off-by: Simon Glass --- MAKEALL | 850 1 file changed, 850 deletions(-) delete mode 100755 MAKEALL diff

Re: [U-Boot] [PATCH] splash: Accommodate DM_USB in splash_init_usb()

2016-07-03 Thread Simon Glass
On 1 July 2016 at 12:47, Alexey Brodkin wrote: > Current implementation of splash_init_usb() requires usb_stor_scan() > which doesn't exist in case of DM_USB simply because real probing > happens right in usb_init(). > > So disable usage of usb_stor_scan() in case of DM_USB. > > Signed-off-by: Ale

[U-Boot] [PATCH 07/11] buildman: Drop the toolchain error when downloading toolchains

2016-07-03 Thread Simon Glass
It doesn't make sense to complain about missing toolchains when the --fetch-arch option is being used. The user is presumably aware that there is a toolchain problem and is actively correcting it by running with this option. Refactor the code to avoid printing this confusing message. Signed-off-b

[U-Boot] [PATCH 11/11] Drop references to MAKEALL in the documentation

2016-07-03 Thread Simon Glass
It is confusing to mention MAKEALL when it is not the normal way of building U-Boot anymore. Update the documentation to suit. Signed-off-by: Simon Glass --- README | 35 ++- board/Barix/ipam390/README.ipam390 | 12 ++-- doc/RE

[U-Boot] [PATCH 03/11] buildman: Fix the 'help' test to use the correct path

2016-07-03 Thread Simon Glass
When buildman is run via a symlink, this test fails. Fix it to work the same way as buildman itself. Signed-off-by: Simon Glass --- tools/buildman/func_test.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/buildman/func_test.py b/tools/buildman/func_test.py index a0b

[U-Boot] [PATCH 08/11] buildman: Avoid overwriting existing toolchain entries

2016-07-03 Thread Simon Glass
The current code for setting up the toolchain config always writes the new paths to an item called 'toolchain'. This means that it will overwrite any existing toolchain item with the same name. In practice, this means that: buildman --fetch-arch all will fetch all toolchains, but only the path

[U-Boot] [PATCH 06/11] buildman: Fix a typo in TestSettingsHasPath()

2016-07-03 Thread Simon Glass
The function comment should say 'buildman'. Fix it. Signed-off-by: Simon Glass --- tools/buildman/toolchain.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py index 6f39a2b..584c9f2 100644 --- a/tools/buildman/toolcha

[U-Boot] [PATCH 04/11] buildman: Allow the toolchain error to be suppressed

2016-07-03 Thread Simon Glass
When there are no toolchains a warning is printed. But in some cases this is confusing, such as when the user is fetching new toolchains. Adjust the function to supress the warning in this case. Signed-off-by: Simon Glass --- tools/buildman/toolchain.py | 18 +- 1 file changed,

[U-Boot] [PATCH 02/11] buildman: Automatically create a config file if needed

2016-07-03 Thread Simon Glass
If there is no ~/.buildman file, buildman currently complains and exists. To make things a little more friendly, create an empty one automatically. This will not allow things to be built, but --fetch-arch can be used to handle that. Signed-off-by: Simon Glass --- tools/buildman/bsettings.py | 4

[U-Boot] [PATCH 01/11] buildman: Tidy up the README a little

2016-07-03 Thread Simon Glass
Tidy up some problems found by a recent review. Signed-off-by: Simon Glass --- tools/buildman/README | 53 +-- 1 file changed, 30 insertions(+), 23 deletions(-) diff --git a/tools/buildman/README b/tools/buildman/README index 26755c5..cd672cf 100

[U-Boot] [PATCH 00/11] buildman: Make the tool friendlier for first-time users

2016-07-03 Thread Simon Glass
This makes a few minor improvements to buildman to make it work more easiler for first-time users: - Improve progress and warning messages when fetching toolchains - Fix a bug where toolchain paths can be overwritten when fetching - Note at the top of the help how to get started Also this series

Re: [U-Boot] [PATCH 13/14] test/py: Fix up after the rename of CONFIG_SYS_HUSH_PARSER

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > At present all the hush tests are skipped on sandbox because the test thinks > that this option is disabled. In fact it has just been renamed. > > It might be better to use the full CONFIG_xxx name in tests with > @pytest.mark.buildco

Re: [U-Boot] [PATCH 10/14] test/py: Add a helper to run a list of U-Boot commands

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > Some tests want to execute a sequence of commands. Add a helper for this. > > Signed-off-by: Simon Glass Reviewed-by: Teddy Reed > --- > > test/py/u_boot_console_base.py | 16 > 1 file changed, 16 insertions(+) >

Re: [U-Boot] [PATCH 07/14] test/py: Return output from run_and_log()

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > It is useful to be able to obtain the output from a command. Return it from > this function. > > Signed-off-by: Simon Glass Reviewed-by: Teddy Reed > --- > > test/py/u_boot_utils.py | 5 +++-- > 1 file changed, 3 insertions(+), 2

Re: [U-Boot] [PATCH 09/14] test/py: Provide a way to check that a command fails

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > Sometimes we want to run a command and check that it fails. Add a function > to handle this. It can check the return code and also make sure that the > output contains a given error message. > > Signed-off-by: Simon Glass > --- > >

Re: [U-Boot] [PATCH 06/14] test/py: Provide output from exceptions with RunAndLog()

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > Tests may want to look at the output from running a command, even if it > fails (e.g. with a non-zero return code). Provide a means to obtain this. > > Another approach would be to return a class object containing both the > output an

Re: [U-Boot] [PATCH 05/14] test/py: Allow RunAndLog() to return the output

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > Tests may want to look at the output from running a command. Return it so > that this is possible. > > Signed-off-by: Simon Glass Reviewed-by: Teddy Reed > --- > > test/py/multiplexed_log.py | 3 ++- > 1 file changed, 2 insertion

Re: [U-Boot] [PATCH 02/14] test: Add a simple script to run tests on sandbox

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > A common check before sending patches is to run all available tests on > sandbox. But everytime I do this I have to look up the README. This presents > quite a barrier to actually doing this. > > Add a shell script to help. To run the

Re: [U-Boot] [PATCH 03/14] sandbox: Don't exit when bootm completes

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > At present sandbox exits when the 'bootm' command completes, since it is not > actually able to run the OS that is loaded. Normally 'bootm' failure is > considered a fatal error in U-Boot. > > However this is annoying for tests, which

Re: [U-Boot] [PATCH 08/14] test/py: Add an option to execute a string containing a command

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > It is sometimes inconvenient to convert a string into a list for execution > with run_and_log(). Provide a helper function to do this. > > Signed-off-by: Simon Glass Reviewed-by: Teddy Reed > --- > > test/py/u_boot_utils.py | 12

Re: [U-Boot] [PATCH 04/14] test/py: Allow tests to control the sandbox device-tree file

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > Normally tests will run with the test.dtb file designed for this purpose. > However, the verified boot tests need to run with their own device-tree > file, containing a public key. > > Make the device-tree file a config option so that

Re: [U-Boot] [PATCH 01/14] test: Add a README

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > Add a few notes about how testing works in U-Boot. > > Signed-off-by: Simon Glass Reviewed-by: Teddy Reed > --- > > test/README | 82 > + > 1 file changed, 82 insertion

Re: [U-Boot] dm: usb: Prevent NULL hub in usb_device_info()

2016-07-03 Thread Bernhard Nortmann
Hi Hans! Am 03.07.2016 um 20:21 schrieb Hans de Goede: Hi, On 02-07-16 23:13, Hans de Goede wrote: Hi, [...] I've prepared 2 patches which re-use the "usb tree" code rather then re-inventing the wheel again: https://github.com/jwrdegoede/u-boot-sunxi/commit/fecf9576247c9423a20570057a1de59f1d

Re: [U-Boot] [PATCH 11/14] tools: Add an error code when fit_handle_file() fails

2016-07-03 Thread Teddy Reed
Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass wrote: > The error code may provide useful information for debugging. Add it to the > error string. > > Signed-off-by: Simon Glass Reviewed-by: Teddy Reed > --- > > tools/fit_image.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletion

[U-Boot] [PATCH 0/9] mx7: add Colibri iMX7S/iMX7D support

2016-07-03 Thread Stefan Agner
From: Stefan Agner This patchset adds support for the Toradex i.MX 7Solo and 7Dual based computer on modules Colibri iMX7S/iMX7D. Beside the board support it adds support for the Ricoh RN5T567 PMIC which is used on this board. It also brings several smaller i.MX 7 related fixes/cleanups. Stef

[U-Boot] [PATCH 9/9] colibri_imx7: add Colibri iMX7S/iMX7D module support

2016-07-03 Thread Stefan Agner
From: Stefan Agner This commit adds support for the Toradex Computer on Modules Colibri iMX7S/iMX7D. The two modules/SoC's are very similar hence can be easily supported by one board. The board code detects RAM size at runtime which is one of the differences between the two boards. The board also

[U-Boot] [PATCH 5/9] mx7_common: Put display board info config into board file

2016-07-03 Thread Stefan Agner
CONFIG_DISPLAY_BOARDINFO should not be placed in mx7_common because some boards might need a different config such as CONFIG_DISPLAY_BOARDINFO_LATE. Move it to the board file instead. Signed-off-by: Stefan Agner --- include/configs/mx7_common.h | 1 - include/configs/mx7dsabresd.h | 2 ++ incl

[U-Boot] [PATCH 8/9] power: pmic: add Ricoh RN5T567 PMIC support

2016-07-03 Thread Stefan Agner
From: Stefan Agner Add power driver for Ricoh PMIC used on Colibri iMX7. Signed-off-by: Stefan Agner Acked-by: Marcel Ziswiler --- drivers/power/pmic/Makefile | 1 + drivers/power/pmic/pmic_rn5t567.c | 32 +++ include/power/rn5t567_pmic.h | 115 +

[U-Boot] [PATCH 6/9] mx7_common: use Kconfig for ARMv7 non-secure mode

2016-07-03 Thread Stefan Agner
From: Stefan Agner Use existing Kconfig symbols to let the user configure whether to build a U-Boot with non-secure mode support or not. This also allows to enable virtualization extension easily. Signed-off-by: Stefan Agner --- arch/arm/cpu/armv7/mx7/Kconfig | 2 ++ configs/mx7dsabresd_defco

[U-Boot] [PATCH 2/9] usb: ehci-mx6: configure power polarity in usb_power_config

2016-07-03 Thread Stefan Agner
From: Stefan Agner USBNC_n_CTRL1 bit 9 actually controls the power pin polarity. Rename UCTRL_PM to align reference manual and set the bit in the appropriate callback usb_power_config. Signed-off-by: Stefan Agner --- drivers/usb/host/ehci-mx6.c | 10 +- 1 file changed, 5 insertions(+)

[U-Boot] [PATCH 7/9] imx_watchdog: add weak attribute to reset_cpu function

2016-07-03 Thread Stefan Agner
This allows to overwrite reset_cpu function in case a board level reset is preferred (e.g. through PMIC). Signed-off-by: Stefan Agner --- drivers/watchdog/imx_watchdog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/watchdog/imx_watchdog.c b/drivers/watchdog/imx_wa

[U-Boot] [PATCH 3/9] usb: ehci-mx6: introduce config for high active power pin

2016-07-03 Thread Stefan Agner
From: Stefan Agner Add a new config CONFIG_MXC_USB_OTG_HACTIVE which configures the OTG Power Pin to be high active. Low active is the reset value of the affected configuration register, hence the config option is named by the non-reset configuration. Signed-off-by: Stefan Agner --- drivers/u

[U-Boot] [PATCH 4/9] mx7: set soc environment according to exact SoC type

2016-07-03 Thread Stefan Agner
From: Stefan Agner This can be useful if the same U-Boot binary is used for boards available with a i.MX 7Solo and i.MX 7Dual. Signed-off-by: Stefan Agner --- arch/arm/cpu/armv7/mx7/soc.c | 12 include/configs/mx7_common.h | 2 ++ 2 files changed, 14 insertions(+) diff --git a/

[U-Boot] [PATCH 1/9] dm: imx: serial: Support DTE mode when using driver model

2016-07-03 Thread Stefan Agner
From: Stefan Agner The MXC UART IP can be run in DTE or DCE mode. This depends on the board wiring and the pinmux used and hence is board specific. This extends platform data with a new field to choose wheather DTE mode shall be used. Signed-off-by: Stefan Agner --- drivers/serial/serial_mxc.

  1   2   >