Re: [U-Boot] [PATCH 01/16] clk: introduce clk_dev_binded

2019-07-29 Thread Lukasz Majewski
Hi Peng, > Hi Peng, > > > Hi Lukasz, > > > > > Subject: [PATCH 01/16] clk: introduce clk_dev_binded > > > > Do you have time to review the patchset? Patches were delegated to > > you. > > The i.MX related PR (with base CCF) has been pulled to mainline just > yesteday, so I'm going to r

Re: [U-Boot] [PATCH] usb: ehci-omap: Add Support DM_USB and OF_CONTROL

2019-07-29 Thread Heiko Schocher
Hello Adam, Am 29.07.2019 um 18:05 schrieb Adam Ford: On Mon, Jul 29, 2019 at 9:38 AM Marek Vasut wrote: On 7/29/19 3:19 PM, Adam Ford wrote: On Mon, Jul 29, 2019 at 7:30 AM Marek Vasut wrote: On 7/29/19 2:16 PM, Adam Ford wrote: On Sun, Jun 30, 2019 at 5:21 PM Adam Ford wrote: The om

Re: [U-Boot] [PATCH 1/2] configs: am65x_evm_a53: Disable K3_SYSTEM_CONTROLLER

2019-07-29 Thread Lokesh Vutla
On 29/07/19 10:18 PM, Suman Anna wrote: > The K3 System Controller driver is used for loading and starting > the System Firmware, and is used only on R5 SPL. It need not be > enabled and built for the A53 U-Boot and SPL, so disable it from > both the GP and HS AM65x A53 defconfigs. > > While at

Re: [U-Boot] [PATCH] arm: dts: k3-am654-base-board: Fix cpsw_nuss power-domains property

2019-07-29 Thread Lokesh Vutla
On 29/07/19 9:43 PM, Suman Anna wrote: > The commit 355be915ed08 ("arm: dts: k3-am654: Update power-domains > property for each node") has updated the power-domain cells value > and updated power-domains property in various existing dts nodes but > missed updating the cpsw_nuss node. This results

Re: [U-Boot] [PATCH v2 05/11] spl: support booting via RISC-V OpenSBI

2019-07-29 Thread Anup Patel
On Mon, Jul 29, 2019 at 9:21 PM Auer, Lukas wrote: > > Hi Anup, > > On Mon, 2019-07-29 at 14:02 +0530, Anup Patel wrote: > > On Sun, Jul 28, 2019 at 9:55 PM Lukas Auer > > wrote: > > > RISC-V OpenSBI is an open-source implementation of the RISC-V Supervisor > > > Binary Interface (SBI) specificat

Re: [U-Boot] [PATCH] sifive: riscv: update Hifive Unleashed configuration infrastructure

2019-07-29 Thread Anup Patel
On Mon, Jul 29, 2019 at 6:13 PM Sagar Shrikant Kadam wrote: > > This patch aligns the current implementation of HiFive Unleashed > board configuration framework with the one described in doc/README.kconfig. > > Signed-off-by: Sagar Shrikant Kadam > --- > arch/riscv/Kconfig

Re: [U-Boot] [PATCH] spl_mmc: always use find_mmc_device() to get mmc handler

2019-07-29 Thread Kever Yang
Hi Peng, On 2019/7/30 上午9:30, Peng Fan wrote: Subject: Re: [U-Boot] [PATCH] spl_mmc: always use find_mmc_device() to get mmc handler +Peng Hi Kever, On 25/07/19 3:10 PM, Kever Yang wrote: Like cmd/mmc.c, the spl_mmc.c are using block driver interface like blk_dread() to access mmc devices, w

Re: [U-Boot] [PATCH v1 1/3] rtc: move RTC_RX8025 to Kconfig

2019-07-29 Thread Tom Rini
On Tue, Jul 16, 2019 at 05:31:33AM +0200, Heiko Schocher wrote: > move RTC_RX8025 to Kconfig and fixup board configs. > > Signed-off-by: Heiko Schocher Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot ma

[U-Boot] [ANN] U-Boot v2019.10-rc1 released

2019-07-29 Thread Tom Rini
Hey all, It's release day, and here is v2019.10-rc1. Once again, there's a few things that I still want to see come in, from my own queue, and I know there's also a few PRs to be made still too. So I expect this week to still be relatively busy. But after that, things should settle down. In te

Re: [U-Boot] [PATCH] spl_mmc: always use find_mmc_device() to get mmc handler

2019-07-29 Thread Peng Fan
> Subject: Re: [U-Boot] [PATCH] spl_mmc: always use find_mmc_device() to get > mmc handler > > +Peng > > Hi Kever, > > On 25/07/19 3:10 PM, Kever Yang wrote: > > Like cmd/mmc.c, the spl_mmc.c are using block driver interface like > > blk_dread() to access mmc devices, we need to get the mmc devi

Re: [U-Boot] [PATCH] gitlab-ci: Switch to a Docker image that contains a QEMU we build

2019-07-29 Thread Tom Rini
On Wed, Jul 24, 2019 at 03:53:28PM -0400, Tom Rini wrote: > Rather than buiding QEMU for each test.py build it once in our > Dockerfile and re-use it as needed. > > Signed-off-by: Tom Rini Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [U-Boot] [PATCH] doc: arch: sandbox: Replace all the instances of README.sandbox

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 01:52:04PM +0530, Keerthy wrote: > commit 49116e6d236d ("doc: arch: Convert README.sandbox to reST") > Moves README.sandbox to doc/arch. > Replace all the existing instances to point to the right documentation > file. > > Signed-off-by: Keerthy Applied to u-boot/master,

Re: [U-Boot] [PATCH v6 1/3] common: Move bootm_decomp_image() to image.c (as image_decomp())

2019-07-29 Thread Tom Rini
On Wed, Jul 24, 2019 at 07:37:54PM -0700, Julius Werner wrote: > Upcoming patches want to add decompression to use cases that are no > longer directly related to booting. It makes sense to retain a single > decompression routine, but it should no longer be in bootm.c (which is > not compiled for a

Re: [U-Boot] [RESEND PATCH v2 03/13] i2c: remove i2c driver-model compatibility layer

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:00AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > There are no more users of the compatibility layer for i2c. Remove the > driver and all references to it. > > Signed-off-by: Bartosz Golaszewski > Acked-by: Heiko Schocher Applied to u-boot/m

Re: [U-Boot] [PATCH] doc: board: Add missing board index.rst files

2019-07-29 Thread Tom Rini
On Thu, Jul 25, 2019 at 07:04:04PM -0700, Bin Meng wrote: > These board index.rst files are missing in previous html doc patch > series, hence it causes some build warnings. Add those files. > > Reported-by: Heinrich Schuchardt > Signed-off-by: Bin Meng > Reviewed-by: Heinrich Schuchardt Appl

Re: [U-Boot] [PATCH v1] gpio: fixes for gpio-hog support

2019-07-29 Thread Tom Rini
On Wed, Jul 17, 2019 at 06:59:51AM +0200, Heiko Schocher wrote: > recently added gpio hog patch was "in discussion" > state with Simon Glass. This patch now adds most > of comments from Simon Glass. > > Signed-off-by: Heiko Schocher Applied to u-boot/master, thanks! -- Tom signature.asc Des

Re: [U-Boot] [PATCH v4 2/2] board: mediatek: Add pumpkin board support

2019-07-29 Thread Tom Rini
On Thu, Jul 18, 2019 at 07:08:09PM +0200, Fabien Parent wrote: > The pumpkin board is made by Gossamer Engineering and is using > a MediaTek SoC. The board currently comes in two available version: > MT8516 SoC and MT8167 SoC. > The board provides the following IOs: eMMC, NAND, SD card, USB type-A

Re: [U-Boot] [PATCH v4 1/2] pinctrl: mediatek: fix warning

2019-07-29 Thread Tom Rini
On Thu, Jul 18, 2019 at 07:08:08PM +0200, Fabien Parent wrote: > Fix the following warning when CONFIG_PINCONF=n: > > drivers/pinctrl/mediatek/pinctrl-mtk-common.c:35:36: > warning: ‘mtk_drive’ defined but not used [-Wunused-const-variable=] > static const struct mtk_drive_desc mtk_drive[] = { >

Re: [U-Boot] [RESEND PATCH v2 09/13] davinci: omapl138-lcdk: enable SPL_SYS_MALLOC_SIMPLE

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:06AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > We're working towards enabling the driver model in SPL for da850-lcdk. > Enable SPL_SYS_MALLOC_SIMPLE in defconfig to reduce SPL size. > > Signed-off-by: Bartosz Golaszewski Applied to u-boot/

Re: [U-Boot] [PATCH 1/1] tools/logos: remove black background of U-Boot logo

2019-07-29 Thread Tom Rini
On Thu, Jul 25, 2019 at 08:46:15PM +0200, Heinrich Schuchardt wrote: > When generating the Sphinx documentation with command 'make htmldocs' the > U-Boot logo is displayed with a black box as a background. > > Remove the black box. > > Signed-off-by: Heinrich Schuchardt Applied to u-boot/maste

Re: [U-Boot] [RESEND PATCH v2 01/13] davinci: remove CONFIG_DM_I2C_COMPAT from defconfigs

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:57:58AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > This option is no longer used on any davinci board but still selected > in defconfigs which causes the following warning: > > = WARNING == > This board us

Re: [U-Boot] [RESEND PATCH v2 12/13] davinci: omapl138-lcdk: enable driver-model in SPL

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:09AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > Enable CONFIG_SPL_DM and enable the driver model for serial by defining > an appropriate device in the board file for da850-lcdk. > > Signed-off-by: Bartosz Golaszewski Applied to u-boot/maste

Re: [U-Boot] [PATCH] Add validation for icache/dcache arguments - arguments different from off/on/flush are currently silently ignored.

2019-07-29 Thread Tom Rini
On Sat, Jul 13, 2019 at 02:54:58PM -0400, eric.pe...@yahoo.com wrote: > From: Eric Perie > > Signed-off-by: Eric Perie Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH v1 2/3] rtc, rx8025: fix Coding Style

2019-07-29 Thread Tom Rini
On Tue, Jul 16, 2019 at 05:31:34AM +0200, Heiko Schocher wrote: > fix Coding Style for this driver. > > Signed-off-by: Heiko Schocher Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list U-Boot@

Re: [U-Boot] [RESEND PATCH v2 08/13] davinci: da850-evm: enable driver model for NAND

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:05AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > Enable the driver-model on da850-evm. We need to add a dummy nand node > to the device tree, as the real nand node is a sub-node of the aemif > device. > > On linux the aemif driver populates al

Re: [U-Boot] [PATCH 1/1] lib: uuid: alignment error in gen_rand_uuid()

2019-07-29 Thread Tom Rini
On Sun, Jul 14, 2019 at 11:31:50PM +0200, Heinrich Schuchardt wrote: > Packed structures like struct uuid are not aligned. GCC 9.1 therefore > throws an error when trying to compile gen_rand_uuid(). > > lib/uuid.c: In function ‘gen_rand_uuid’: > lib/uuid.c:244:2: error: converting a packed ‘struc

Re: [U-Boot] [PATCH v1 3/3] rtc, rx8025: add DM support

2019-07-29 Thread Tom Rini
On Tue, Jul 16, 2019 at 05:31:35AM +0200, Heiko Schocher wrote: > add DM support for this RTC driver. > > Signed-off-by: Heiko Schocher Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list U-Boo

Re: [U-Boot] [PATCH v6 3/3] fit: Support compat string property in configuration node

2019-07-29 Thread Tom Rini
On Wed, Jul 24, 2019 at 07:37:56PM -0700, Julius Werner wrote: > This patch adds support for an optional optimization to compatible > string matching where the compatible string property from the root node > of the kernel FDT can be copied into the configuration node of the FIT > image. This is mo

Re: [U-Boot] [RESEND PATCH v2 11/13] davinci: omapl138-lcdk: remove unneeded includes

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:08AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > There's no SPI on da850-lcdk. Remove unnecessary includes. > > Signed-off-by: Bartosz Golaszewski Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [U-Boot] [RESEND PATCH v2 13/13] configs: omapl138_lcdk: enable NAND self-init in SPL

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:10AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > Now that we have enabled the driver-model in SPL, we can remove the > code disabling NAND self-init in SPL from the config include for > omapl138-lcdk. > > Signed-off-by: Bartosz Golaszewski A

Re: [U-Boot] [RESEND PATCH v2 10/13] davinci: omapl138-lcdk: enable CONFIG_USE_TINY_PRINTF

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:07AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > We're working towards enabling the driver-model in SPL for da850-lcdk. > Enable CONFIG_USE_TINY_PRINTF in defconfig to reduce SPL size. > > Signed-off-by: Bartosz Golaszewski Applied to u-boot

Re: [U-Boot] [PATCH] Drop linker-generated array creation when CONFIG_CMDLINE is disabled

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 09:24:53AM +0200, andrej.ros...@f-secure.com wrote: > From: Andrej Rosano > > Linker generated array entry is not needed when the command line is > disabled. Remove this code in that case. > > This is required as the commit 80a48dd47e3bf3ede676fae5a630cb6c80de3e69 > brea

Re: [U-Boot] [PATCH v6 2/3] fit: Support compression for non-kernel components (e.g. FDT)

2019-07-29 Thread Tom Rini
On Wed, Jul 24, 2019 at 07:37:55PM -0700, Julius Werner wrote: > This patch adds support for compressing non-kernel image nodes in a FIT > image (kernel nodes could already be compressed previously). This can > reduce the size of FIT images and therefore improve boot times > (especially when an im

Re: [U-Boot] [RESEND PATCH v2 05/13] nand: davinci: make davinci_nand_init() static

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:02AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > This function is only used within the driver itself. No need > to export it. > > Signed-off-by: Bartosz Golaszewski Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP

Re: [U-Boot] [RESEND PATCH v2 02/13] sandbox: remove CONFIG_DM_I2C_COMPAT from defconfigs

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:57:59AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > There are no more "real" users of CONFIG_DM_I2C_COMPAT and we'll soon > remove it altogether. Stop building it in sandbox mode. > > Signed-off-by: Bartosz Golaszewski > Acked-by: Heiko Schocher

Re: [U-Boot] [RESEND PATCH v2 04/13] omapl138_lcdk_defconfig: don't build support for SPI

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:01AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > The following warning is emited when building u-boot for da850-lcdk: > > = WARNING == > This board does not use CONFIG_DM_SPI. Please update > the board b

Re: [U-Boot] [RESEND PATCH v2 06/13] nand: davinci: add support for driver model

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:03AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > Extend the davinci NAND driver to support the driver model. For now this > doesn't add any device-tree parsing due to the fact that we can't access > the actual nand node on the device-tree - it'

Re: [U-Boot] [RESEND PATCH v2 07/13] davinci: omapl138-lcdk: enable driver model for NAND

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:58:04AM +0200, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > Enable the driver-model on da850-lcdk. We need to add a dummy nand node > to the device tree, as the real nand node is a sub-node of the aemif > device. > > On linux the aemif driver populates a

Re: [U-Boot] [PATCH v2] arm: mach-keystone: Use appended original image size in image processing

2019-07-29 Thread Tom Rini
On Tue, Jul 16, 2019 at 09:49:38AM -0400, Andrew F. Davis wrote: > When an image is signed/encrypted on K2 devices the image may get padded > to meet alignment requirements for the signature/encryption type. The > original size is appended as 4 bytes little-endian to the end of the > final image.

Re: [U-Boot] Please pull u-boot-video

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 11:09:00AM +0200, Anatolij Gustschin wrote: > Hi Tom, > > please pull video patches for v2019.10. > > Travis CI: https://travis-ci.org/vdsao/u-boot-video/builds/564743256 > > Thanks, > Anatolij > > The following changes since commit 75551c8bfc9545e31ec2ce238cac385790400

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

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 03:27:59PM -0600, Simon Glass wrote: > Hi Tom, > > Gitlab is here: > https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/356 > > The following changes since commit 75551c8bfc9545e31ec2ce238cac3857904007b8: > > Merge branch '2019-07-26-ti-imports' (2019-07-27 1

Re: [U-Boot] [PULL] u-boot-socfpga/master

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 02:34:43PM +0200, Marek Vasut wrote: > The following changes since commit 0de815356474912ef5bef9a69f0327a5a93bb2c2: > > Merge branch '2019-07-17-master-imports' (2019-07-18 11:31:37 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-socfp

Re: [U-Boot] [PATCH] ARM: da850_am18xxevm: Remove config and references

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:28:39AM -0500, Adam Ford wrote: > This board has been abandonded for a while. I do not have this hardware > and nobody has appeared to notice and/or care that it's orphaned. > Since much of this code is inserted into the da850evm code base, removing > it from support wi

Re: [U-Boot] Pull request: u-boot-rockchip u-boot-rockchip-20190729

2019-07-29 Thread Tom Rini
t repository at: > > https://gitlab.denx.de/u-boot/custodians/u-boot-rockchip.git > tags/u-boot-rockchip-20190729 > > for you to fetch changes up to 26008cd42b590dc71ee9c1ca667a218542aab342: > > rockchip: rv1108: Migrate to use common board file (2019-07-29 10:27:48 > +

Re: [U-Boot] [PATCH v3] dw_mmc: turn on the IO supply

2019-07-29 Thread Peng Fan
Hi Kever, You could pick it up, not depends on mmc branch. Thanks, Peng. From: Kever Yang [mailto:kever.y...@rock-chips.com] Sent: 2019年7月26日 15:06 To: Peng Fan Cc: Urja Rannikko ; u-boot@lists.denx.de Subject: Re: [U-Boot] [PATCH v3] dw_mmc: turn on the IO supply Hi Peng, Simon, This patch i

Re: [U-Boot] [PATCH] i2c: imx_lpi2c: add ipg clk

2019-07-29 Thread Peng Fan
> Subject: Re: [PATCH] i2c: imx_lpi2c: add ipg clk > > Hi Peng, > > On Wed, 24 Jul 2019 08:54:16 + > Peng Fan peng@nxp.com wrote: > ... > > +++ b/drivers/i2c/imx_lpi2c.c > > @@ -471,6 +471,17 @@ static int imx_lpi2c_probe(struct udevice *bus) > > dev_err(bus, "Failed t

Re: [U-Boot] [PATCH v3 1/6] net: phy: mv88e61xx: rework to enable detection of 88E6071 devices

2019-07-29 Thread Joe Hershberger
On Mon, Jul 29, 2019 at 2:01 PM Anatolij Gustschin wrote: > > Extend the driver to init switch register offsets from variables > instead of compile time macros and enable 88E6071 detection. > Ethernet transfer (e.g. tftp) does not work yet, so enable the > registration of the 'indirect mii' bus fo

Re: [U-Boot] [PATCH v3 5/6] net: phy: mv88e61xx: register phy_driver struct for 88E6071

2019-07-29 Thread Joe Hershberger
On Mon, Jul 29, 2019 at 2:02 PM Anatolij Gustschin wrote: > > Support probing and init for 88E6071 switch. > > Signed-off-by: Anatolij Gustschin > Reviewed-by: Chris Packham > Tested-by: Chris Packham Acked-by: Joe Hershberger ___ U-Boot mailing lis

Re: [U-Boot] [PATCH v3 3/6] net: phy: mv88E61xx: fix ENERGY_DET init for mv88E6071

2019-07-29 Thread Joe Hershberger
On Mon, Jul 29, 2019 at 2:01 PM Anatolij Gustschin wrote: > > On mv88E6071 the 'EDet' field offset, width and sense control > bits are different, adjust the driver to init the PHY control > register as needed. This fixes not working link detection and > tftp transfers. > > Signed-off-by: Anatolij

Re: [U-Boot] [PATCH v3 2/6] net: phy: mv88e61xx: add CPU port parameter init for 88E6071

2019-07-29 Thread Joe Hershberger
On Mon, Jul 29, 2019 at 2:02 PM Anatolij Gustschin wrote: > > On 88E6071 chip the port status register bit field offsets > for duplex and link bits differ. Extend the driver to use > 88E6071 specific offset values. The width of bit fields for > speed status differ, too. Adapt for proper port speed

Re: [U-Boot] [PATCH v3 6/6] net: phy: fix switch vendor name

2019-07-29 Thread Joe Hershberger
On Mon, Jul 29, 2019 at 2:02 PM Anatolij Gustschin wrote: > > Fix vendor name in MV88E61xx option description. > > Signed-off-by: Anatolij Gustschin > Reviewed-by: Chris Packham > Tested-by: Chris Packham Acked-by: Joe Hershberger ___ U-Boot mailing

Re: [U-Boot] [PATCH v2 4/6] net: phy: mv88E61xx: add config option for mv88E6071 support

2019-07-29 Thread Joe Hershberger
On Thu, Jul 25, 2019 at 4:42 PM Anatolij Gustschin wrote: > > On Tue, 23 Jul 2019 04:26:17 + > Joe Hershberger joe.hershber...@ni.com wrote: > ... > > > +config MV88E61XX_88E6020_FAMILY > > > + bool "Marvell MV88E6020 family support." > > > + help > > > + The driver support

Re: [U-Boot] [PATCH 1/1] Dockerfile: build GRUB UEFI targets

2019-07-29 Thread Tom Rini
On Mon, Jul 29, 2019 at 08:04:15PM +0200, Heinrich Schuchardt wrote: > On 7/29/19 3:07 PM, Tom Rini wrote: > >On Sat, Jul 27, 2019 at 10:43:37AM +0200, Heinrich Schuchardt wrote: > > > >>Build GRUB UEFI targets grubarm.efi and grubaa64.efi. These are needed for > >>running test_efi_grub_net(). > >>

Re: [U-Boot] [PATCH v2 00/11] SPL support for RISC-V

2019-07-29 Thread Anup Patel
On Sun, Jul 28, 2019 at 9:27 PM Lukas Auer wrote: > > This series adds support for SPL to RISC-V U-Boot. Images can be booted > via OpenSBI (FW_DYNAMIC firmware) or by directly jumping to them. In the > former case, OpenSBI and U-Boot proper are bundled as a FIT image and > made available to U-Boo

Re: [U-Boot] [PATCH 40/53] binman: Add a few more features to the wishlist

2019-07-29 Thread sjg
Add mention of a few other desirable features that may be implemented in the future. Signed-off-by: Simon Glass --- tools/binman/README | 2 ++ 1 file changed, 2 insertions(+) Applied to u-boot-dm, thanks! ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH 30/53] binman: Update documentation for image creation

2019-07-29 Thread sjg
There are a few more steps in the process now. Update the documentation to reflect this. Signed-off-by: Simon Glass --- tools/binman/README | 23 ++- 1 file changed, 18 insertions(+), 5 deletions(-) Applied to u-boot-dm, thanks! _

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

2019-07-29 Thread Simon Glass
Hi Tom, Gitlab is here: https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/356 The following changes since commit 75551c8bfc9545e31ec2ce238cac3857904007b8: Merge branch '2019-07-26-ti-imports' (2019-07-27 19:50:52 -0400) are available in the Git repository at: https://gitlab.denx

Re: [U-Boot] [PATCH 27/53] binman: Tidy up _SetupDtb() to use its own temporary file

2019-07-29 Thread sjg
At present EnsureCompiled() uses an file from the 'output' directory (in the tools module) when compiling the device tree. This is fine in most cases, allowing useful inspection of the output files from binman. However in functional tests, _SetupDtb() creates an output directory and immediately re

Re: [U-Boot] [PATCH 29/53] binman: Add info to allow safely repacking an image later

2019-07-29 Thread sjg
At present it is not possible to discover the contraints to repacking an image (e.g. maximum section size) since this information is not preserved from the original image description. Add new 'orig-offset' and 'orig-size' properties to hold this. Add them to the main device tree in the image. Sig

Re: [U-Boot] [PATCH 16/53] dtoc: Update Fdt.FromData() to allow a name

2019-07-29 Thread sjg
It is confusing when something goes wrong with a device tree which was created from data rather than a file, since there is no identifying filename. Add an option to provide this. Use the filename as the name, where available Signed-off-by: Simon Glass --- tools/dtoc/fdt.py | 8 +++- 1 file

Re: [U-Boot] [PATCH 33/53] binman: Add more tests for image header position

2019-07-29 Thread sjg
The positioning does not currently work correctly if at the end of an image with no fixed size. Also if the header is in the middle of an image it can cause a gap in the image since the header position is normally at the image end, so entries after it are placed after the end of the image. Fix the

Re: [U-Boot] [PATCH 38/53] binman: Adjust fmap to ignore CBFS files

2019-07-29 Thread sjg
The FMAP is not intended to show the files inside a CBFS. The FMAP can be used to locate the CBFS itself, but then the CBFS must be read to find out what is in it. Update the FMAP to work this way and add some debugging while we are here. Signed-off-by: Simon Glass --- tools/binman/README.entr

Re: [U-Boot] [PATCH 24/53] binman: Add a function to obtain the image for an Entry

2019-07-29 Thread sjg
At present we have an 'image' property in the entry for this purpose, but this is not necessary and seems error-prone in the presence of inheritance. Add a function instead. The Entry_section class overrides this with a special version, since top-level sections are in fact images, since Image inher

Re: [U-Boot] [PATCH 21/53] binman: Store the entry in output_fdt_files

2019-07-29 Thread sjg
In some cases we want to access the Entry object for a particular device tree. This allows us to read its contents or update it. Add this information to output_fdt_files and provide a function to read it. Also rename output_fdt_files since its name is no-longer descriptive. Signed-off-by: Simon G

Re: [U-Boot] [PATCH 46/53] binman: Update state when replacing device-tree entries

2019-07-29 Thread sjg
Since the state module holds references to all the device trees used by binman, it must be updated when the device trees are updated. Add support for this. Signed-off-by: Simon Glass --- tools/binman/etype/blob_dtb.py | 9 + tools/binman/state.py | 16 tools/d

Re: [U-Boot] [PATCH 06/53] binman: Convert GetFdtSet() to use a dict

2019-07-29 Thread sjg
At present this function returns a set of device-tree filenames. It has no way of returning the actual device-tree object. Change it to a dictionary so that we can add this feature in a future patch. Also drop fdt_set since it is no-longer used. Signed-off-by: Simon Glass --- tools/binman/entr

Re: [U-Boot] [PATCH 36/53] binman: Support shrinking a entry after packing

2019-07-29 Thread sjg
Sometimes an entry may shrink after it has already been packed. In that case we must repack the items. Of course it is always possible to just leave the entry at its original size and waste space at the end. This is what binman does by default, since there is the possibility of the entry changing s

Re: [U-Boot] [PATCH 34/53] binman: Allow updating entries that change size

2019-07-29 Thread sjg
So far we don't allow entries to change size when repacking. But this is not very useful since it is common for entries to change size after an updated binary is built, etc. Add support for this, respecting the original offset/size/alignment constraints of the image layout. For this to work the or

Re: [U-Boot] [PATCH 31/53] binman: Write the original input fdtmap to a file

2019-07-29 Thread sjg
When reading an image in, write its fdtmap to a file in the output directory. This is useful for debugging. Update the 'ls' command to set up the output directory; otherwise it will fail. Signed-off-by: Simon Glass --- tools/binman/control.py | 6 +- tools/binman/image.py | 5 - 2 fil

Re: [U-Boot] [PATCH 32/53] binman: Move Image.BuildImage() into a single function

2019-07-29 Thread sjg
Now that an Image is an Entry_section, there is no need for the separate BuildSection() function. Drop it and add a bit of logging. Signed-off-by: Simon Glass --- tools/binman/image.py | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) Applied to u-boot-dm, thanks! __

Re: [U-Boot] [PATCH 35/53] binman: Update the _testing entry to support shrinkage

2019-07-29 Thread sjg
Sometimes entries shrink after packing. As a start towards supporting this, update the _testing entry to handle the test case. Signed-off-by: Simon Glass --- tools/binman/etype/_testing.py | 25 +++-- tools/binman/ftest.py | 16 2 files changed, 27

Re: [U-Boot] [PATCH 07/53] binman: Rename state.GetFdts()

2019-07-29 Thread sjg
This function name conflicts with Entry.GetFdts() which has a different purpose. Rename it to avoid confusion. Also update a stale comment relating to this function. Signed-off-by: Simon Glass --- tools/binman/control.py | 6 +++--- tools/binman/state.py | 6 +++--- 2 files changed, 6 inserti

Re: [U-Boot] [PATCH 01/53] dtoc: Return a non-zero exit code when tests fail

2019-07-29 Thread sjg
At present 'dtoc -t' return a success code even if some of the tests fail. Fix this by checking the test result and setting the exit code. This allows 'make qcheck' to function as expected. Signed-off-by: Simon Glass --- tools/dtoc/dtoc.py | 7 ++- 1 file changed, 6 insertions(+), 1 deletio

Re: [U-Boot] [PATCH 20/53] binman: Allow state functions to fail to return data

2019-07-29 Thread sjg
At present these state functions raise an exception if they cannot find what is requested. But in some cases the information is optional (e.g. an fdtmap in a coming patch) so it is better to return gracefully. Update these two functions to return None when the data cannot be found. Signed-off-by:

Re: [U-Boot] [PATCH 15/53] binman: Show a helpful error when a DT property is missing

2019-07-29 Thread sjg
At present a Python exception is raised which does not show the node information. Add a more helpful exception in this case. Signed-off-by: Simon Glass --- tools/dtoc/fdt.py | 23 --- tools/dtoc/test_fdt.py | 21 + 2 files changed, 41 insertions(+),

Re: [U-Boot] [PATCH 19/53] binman: Support loading entry data from a file

2019-07-29 Thread sjg
When modifying an image it is convenient to load the data from the file into each entry so that it can be reprocessed. Add a new LoadData() method to handle this. Signed-off-by: Simon Glass --- tools/binman/entry.py | 5 + tools/binman/etype/section.py | 5 + 2 files changed, 10

Re: [U-Boot] [PATCH 28/53] binman: Support updating entries in an existing image

2019-07-29 Thread sjg
While it is useful and efficient to build images in a single pass from a unified description, it is sometimes desirable to update the image later. Add support for replace an existing file with one of the same size. This avoids needing to repack the file. Support for more advanced updates will come

Re: [U-Boot] [PATCH 10/53] binman: Adjust state.fdt_files to be keyed by entry type

2019-07-29 Thread sjg
It makes more sense to use entry type as the key for this dictionary, since the filename can in principle be anything. Make this change and also rename fdt_files and add a comment to explain it better. Signed-off-by: Simon Glass --- tools/binman/etype/blob_dtb.py | 4 +- tools/bin

Re: [U-Boot] [PATCH 09/53] binman: Adjust GetFdt() to be keyed by etype

2019-07-29 Thread sjg
At present the FDTs are keyed by their default filename (not their actual filename). It seems easier to key by the entry type, since this is always the same for each FDT type. To do this, add a new Entry method called GetFdtEtype(). This is necessary since some entry types contain a device tree wh

Re: [U-Boot] [PATCH 26/53] binman: Allow the fdtmap to remain unchanged

2019-07-29 Thread sjg
When updating an existing image where the size of all entries remains the same, we should not need to regenerate the fdtmap. Update the entry to return the same fdtmap as was read from the image. Signed-off-by: Simon Glass --- tools/binman/etype/fdtmap.py | 55 --

Re: [U-Boot] [PATCH] Makefile: fix implementation of BINMAN_DEBUG

2019-07-29 Thread sjg
On Fri, 19 Jul 2019 at 11:21, Stephen Warren wrote: > > From: Stephen Warren > > binman only accepts the -D argument early on the command-line, yet the > Makefile currently passes it near the end. This causes the build to fail > if this feature is used. Re-order the command-line to fix this. > >

Re: [U-Boot] [PATCH 25/53] binman: Add a constant for common entry properties

2019-07-29 Thread sjg
We use this same combination of properties several times in tests. Add a constant for it to avoid typos, etc. Signed-off-by: Simon Glass --- tools/binman/ftest.py | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) Applied to u-boot-dm, thanks! ___

Re: [U-Boot] [PATCH 41/53] binman: Add a prefix before CBFS hex offsets

2019-07-29 Thread sjg
Add a 0x prefix to these errors to avoid confusion. Signed-off-by: Simon Glass --- tools/binman/cbfs_util.py | 4 ++-- tools/binman/cbfs_util_test.py | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) Applied to u-boot-dm, thanks! ___ U-

Re: [U-Boot] [PATCH 02/53] binman: Move image-processing code into a function

2019-07-29 Thread sjg
The Binman() function is very long. Split out the image code to make it more manageable. Signed-off-by: Simon Glass --- tools/binman/control.py | 103 ++-- 1 file changed, 57 insertions(+), 46 deletions(-) Applied to u-boot-dm, thanks! __

Re: [U-Boot] [PATCH 11/53] binman: Simplify state.fdt_subset

2019-07-29 Thread sjg
At present this excludes the device tree passed in to binman since it is always returned first by GetAllFdts(). However, this is easy to ensure by adding a check in that function. Change this dict to includes all device trees, and rename it to fdt_set. Signed-off-by: Simon Glass --- tools/binma

Re: [U-Boot] [PATCH 17/53] dtoc: Update Fdt.GetNode() to handle the root node

2019-07-29 Thread sjg
This function currently fails if the root node is requested. Requesting the root node is sometimes useful, so fix the bug. Signed-off-by: Simon Glass --- tools/dtoc/fdt.py | 2 ++ tools/dtoc/test_fdt.py | 5 + 2 files changed, 7 insertions(+) Applied to u-boot-dm, thanks!

Re: [U-Boot] [PATCH 52/53] binman: Correct the error message for invalid path

2019-07-29 Thread sjg
At present this message references -o for output file. But binman uses -f now. Fix it. Signed-off-by: Simon Glass --- tools/binman/control.py | 4 ++-- tools/binman/ftest.py | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) Applied to u-boot-dm, thanks! _

Re: [U-Boot] [PATCH 04/53] binman: Use print() to print output

2019-07-29 Thread sjg
At present tout writes directly to stdout. This is not necessary and it prevents tests from redirecting output. Change it to use print() for the non-progress output. Signed-off-by: Simon Glass --- tools/patman/tout.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) Applied to u-boot-d

Re: [U-Boot] [PATCH 18/53] binman: Store image fdtmap when loading from a file

2019-07-29 Thread sjg
This data provides all the information about the position and size of each entry. Store it for later use when loading an image. It can be reused as is if the image is modified without changing offsets/sizes. Signed-off-by: Simon Glass --- tools/binman/image.py | 11 ++- 1 file changed,

Re: [U-Boot] [PATCH 39/53] binman: Place Intel descriptor at image start

2019-07-29 Thread sjg
The Intel descriptor must always appear at the start of an (x86) image, so it is supposed to position itself there always. However there is no explicit test for this. Add one and fix a bug introduced by the recent change to adjust Entry to read the node in a separate call. Signed-off-by: Simon Gla

Re: [U-Boot] [PATCH 08/53] binman: Rename state.GetFdt()

2019-07-29 Thread sjg
This function name conflicts with Fdt.Node.GetFdt() which has a different purpose. Rename it to avoid confusion. The new name suggests it is indexed by entry type rather than filename. This will be tidied up in a future commit. Signed-off-by: Simon Glass --- tools/binman/etype/u_boot_dtb_with_

Re: [U-Boot] [PATCH 37/53] libfdt: Copy the struct region in fdt_resize()

2019-07-29 Thread sjg
At present this function appears to copy only the data before the struct region and the data in the string region. It does not seem to copy the struct region itself. From the arguments of this function it seems that it should support fdt and buf being different. This patch attempts to fix this pro

Re: [U-Boot] [PATCH 05/53] binman: Move image/fdt code into PrepareImagesAndDtbs()

2019-07-29 Thread sjg
Further reduce the size of the main Binman() function by moving this setup code into its own function. Note that the 'images' value is accessed from other modules so must be made a global. Signed-off-by: Simon Glass --- tools/binman/control.py | 125 +++- 1

Re: [U-Boot] [PATCH 44/53] binman: Support replacing data in a cbfs

2019-07-29 Thread sjg
At present binman cannot replace data within a CBFS since it does not allow rewriting of the files in that CBFS. Implement this by using the new WriteData() method to handle the case. Add a header to compressed data so that the amount of compressed data can be determined without reference to the s

Re: [U-Boot] [PATCH 43/53] binman: Update Entry.WriteData() to handle special sections

2019-07-29 Thread sjg
At present this method assumes that the parent section does not need to recalculate its position or adjust any metadata it may contain. But when the entry changes size this may not be true. Also if the parent section is more than just a container (e.g. it is a CBFS) then the section may need to reg

Re: [U-Boot] [PATCH 49/53] binman: Move control.WriteEntry further down the file

2019-07-29 Thread sjg
Move this function after the extraction logic so we can keep the writing logic in one place. Signed-off-by: Simon Glass --- tools/binman/control.py | 81 - 1 file changed, 40 insertions(+), 41 deletions(-) Applied to u-boot-dm, thanks! __

Re: [U-Boot] [PATCH 23/53] binman: Adjust Entry to read the node in a separate call

2019-07-29 Thread sjg
At present the Entry constructor sets up the object and then immediately reads its device-tree node to obtain its properties. This breaks a convention that constructors should not do any processing. A consequence is that we must pass all arguments to the constructor and cannot have the node-readin

Re: [U-Boot] [PATCH 14/53] binman: Add a bit of logging in entries when packing

2019-07-29 Thread sjg
Use the new logging feature to log information about progress with packing. This is useful to see how binman is figuring things out. Also update elf.py to use the same feature. Signed-off-by: Simon Glass --- tools/binman/elf.py | 9 +++-- tools/binman/elf_test.py | 19 +

Re: [U-Boot] [PATCH 03/53] binman: Move GetFdtSet() into blob_dtb

2019-07-29 Thread sjg
At present we check the filename to see if an entry holds a device-tree file. It is easier to use the base class designed for this purpose. Move this method implementation into Entry_blob_dtb and update the default one to return an empty set. Signed-off-by: Simon Glass --- tools/binman/entry.p

Re: [U-Boot] [PATCH 13/53] patman: Update tout to avoid open-coding the debug levels

2019-07-29 Thread sjg
Use the debug level constants instead of open-coding them in the file. Signed-off-by: Simon Glass --- tools/patman/tout.py | 18 +++--- 1 file changed, 7 insertions(+), 11 deletions(-) Applied to u-boot-dm, thanks! ___ U-Boot mailing list

  1   2   3   >