Re: [PATCH 05/13] efi_loader: signature: make efi_hash_regions more generic

2020-05-30 Thread Heinrich Schuchardt
On 5/29/20 8:41 AM, AKASHI Takahiro wrote: > There are a couple of occurrences of hash calculations in which a new > efi_hash_regions will be commonly used. > > Signed-off-by: AKASHI Takahiro > --- > lib/efi_loader/efi_signature.c | 44 +- > 1 file changed, 16

Re: [PATCH 06/13] efi_loader: image_loader: verification for all signatures should pass

2020-05-30 Thread Heinrich Schuchardt
On 5/29/20 8:41 AM, AKASHI Takahiro wrote: > A signed image may have multiple signatures in > - each WIN_CERTIFICATE in authenticode, and/or > - each SignerInfo in pkcs7 SignedData (of WIN_CERTIFICATE) > > In the initial implementation of efi_image_authenticate(), the criteria > of

[PATCH 1/3] pinctrl: imx7: move soc info to data section

2020-05-30 Thread Peng Fan
The soc info without initialization value should be put into data section. The driver could be used before relocation, with it in BSS section could cause issue, since BSS section is not initializated and it might overwrite other areas that used by others, such as dtb. Signed-off-by: Peng Fan ---

[PATCH 3/3] pinctrl: imx5: move soc info to data section

2020-05-30 Thread Peng Fan
The soc info without initialization value should be put into data section. The driver could be used before relocation, with it in BSS section could cause issue, since BSS section is not initializated and it might overwrite other areas that used by others, such as dtb. Signed-off-by: Peng Fan ---

[PATCH 2/3] pinctrl: imx8m: move soc info to data section

2020-05-30 Thread Peng Fan
The soc info without initialization value should be put into data section. The driver could be used before relocation, with it in BSS section could cause issue, since BSS section is not initializated and it might overwrite other areas that used by others, such as dtb. Signed-off-by: Peng Fan ---

[PATCH] usb: xhci-dwc3: Fix usage of bulk PHY API

2020-05-30 Thread Samuel Holland
The PHY consumer is responsible for allocating the struct phy_bulk. During conversion to the bulk API, the contents of the struct were replaced by a pointer instead of the struct itself. This leads to a null pointer dereference in generic_phy_get_bulk() when setting bulk->phys. Fixes:

Re: [PATCH 08/13] test/py: efi_secboot: remove all "re.search"

2020-05-30 Thread Heinrich Schuchardt
On 5/29/20 8:41 AM, AKASHI Takahiro wrote: > Currently, we don't use any regular expression in matching outputs from > U-Boot. Since its use is just redundant, we can remove all. > > Signed-off-by: AKASHI Takahiro Applying: test/py: efi_secboot: remove all "re.search" error: patch failed:

[PATCH V2 RESEND 0/7] imx: drivers: ddr: ddr driver update

2020-05-30 Thread Peng Fan
Seems the previous V2 not show all patches in patchwork https://patchwork.ozlabs.org/project/uboot/list/?series=176250=%2A=both V2: Drop patch to print dram rate remove board ddr ecc part from patch enabling inline ecc feature This is to upstream NXP vendor tree ddr driver related fix and

[PATCH V2 3/7] drivers: ddr: imx8mp: Add inline ECC feature support

2020-05-30 Thread Peng Fan
From: Sherry Sun the DRAM Controller in i.MX8MP will support a feature called "Inline ECC". This is supported for all 3 supported DRAM technologies (LPDDR4, DDR4 and DDR3L). When this feature is enabled by software, the DRAM Controller reserves 12.5% of DRAM capacity for ECC information, and

[PATCH V2 2/7] driver: ddr: imx: correct the pwrctl setting of selfref_en on imx8m

2020-05-30 Thread Peng Fan
From: Jacky Bai The 'selfref_en' should be bit'0', so correct the setting to enable the auto self-refresh. Reviewed-by: Jian Li Reviewed-by: Ye Li Signed-off-by: Jacky Bai Signed-off-by: Peng Fan --- drivers/ddr/imx/imx8m/ddr_init.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH V2 1/7] driver: ddr: imx: skip ddr_ss_gpr config on imx8mn

2020-05-30 Thread Peng Fan
From: Jacky Bai There is no DDR_SS_GPR0 exits on i.MX8MN, so skip setting this register on i.MX8MN. Signed-off-by: Jacky Bai Signed-off-by: Peng Fan --- drivers/ddr/imx/imx8m/ddr_init.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/ddr/imx/imx8m/ddr_init.c

Re: [PATCH 03/13] efi_loader: image_loader: retrieve authenticode only if it exists

2020-05-30 Thread Heinrich Schuchardt
On 5/29/20 8:41 AM, AKASHI Takahiro wrote: > Since certificate table, which is indexed by IMAGE_DIRECTORY_ENTRY_SECURITY %s/Since certificate/Since the certificate/ > and contains authenticode in PE image, doesn't always exist, we should make > sure that we will retrieve its pointer only if it

Re: [PATCH 04/13] efi_loader: signature: fix a size check against revocation list

2020-05-30 Thread Heinrich Schuchardt
On 5/29/20 8:41 AM, AKASHI Takahiro wrote: > Since the size check against an entry in efi_search_siglist() is > incorrect, this function will never find out a to-be-matched certificate > and its associated revocation time in signature list. %s/in signature/in the signature/ > > Signed-off-by:

Re: [PATCH 07/13] efi_loader: image_loader: add digest-based verification for signed image

2020-05-30 Thread Heinrich Schuchardt
On 5/29/20 8:41 AM, AKASHI Takahiro wrote: > In case that a type of certificate in "db" or "dbx" is > EFI_CERT_X509_SHA256_GUID, it is actually not a certificate which contains > a public key for RSA decryption, but a digest of image to be loaded. > If the value matches to a value calculated from

Aw: [PATCH] usb: xhci-dwc3: Fix usage of bulk PHY API

2020-05-30 Thread Frank Wunderlich
Hi, my mistake is already fixed and merged by commit "usb: dwc3: fix NULL pointer issue" by original author of xhci/bulk-api https://gitlab.denx.de/u-boot/u-boot/-/commit/58221d7e876c2f75ae7696aad853d81ccd76fb7a regards Frank

Re: [PATCH 2/2] dma-mapping: Add header file for ARCH_DMA_MINALIGN

2020-05-30 Thread Tom Rini
On Sat, May 30, 2020 at 01:20:10PM -0400, Tom Rini wrote: > On Sat, May 30, 2020 at 10:29:04AM -0600, Simon Glass wrote: > > > This is defined in the asm/cache.h header file. Update this header file to > > include it. > > > > Signed-off-by: Simon Glass > > Reviewed-by: Tom Rini > > There's

Please pull u-boot-dm

2020-05-30 Thread Simon Glass
Hi Tom, The following changes since commit 606cced58bf44bdf640760711be40d15a169aa71: Merge tag 'u-boot-amlogic-20200529' of https://gitlab.denx.de/u-boot/custodians/u-boot-amlogic (2020-05-29 08:57:04 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git

Re: [PATCH 04/18] common: Drop net.h from common header

2020-05-30 Thread Adam Ford
On Mon, Apr 6, 2020 at 10:04 PM Simon Glass wrote: > > Move this header out of the common header. Network support is used in > quite a few places but it still does not warrant blanket inclusion. > > Note that this net.h header itself has quite a lot in it. It could be > split into the driver-mode

[PATCH 2/2] dma-mapping: Add header file for ARCH_DMA_MINALIGN

2020-05-30 Thread Simon Glass
This is defined in the asm/cache.h header file. Update this header file to include it. Signed-off-by: Simon Glass --- include/linux/dma-mapping.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/dma-mapping.h b/include/linux/dma-mapping.h index 20b6d60dd8..6a107d52e0 100644

[PATCH 1/2] usb: ohci: Add header file for ARCH_DMA_MINALIGN

2020-05-30 Thread Simon Glass
This is defined in the asm/cache.h header file. Update this header file to include it so it gets the same value consistently across U-Boot. This fixes 'usb host' on omapl138_lcdk. Fixes: 90526e9fbac ("common: Drop net.h from common header") Reported-by: Adam Ford Signed-off-by: Simon Glass ---

Re: [PATCH 04/18] common: Drop net.h from common header

2020-05-30 Thread Adam Ford
On Sat, May 30, 2020 at 11:29 AM Simon Glass wrote: > > Hi Adam, > > On Sat, 30 May 2020 at 08:59, Adam Ford wrote: > > > > On Mon, Apr 6, 2020 at 10:04 PM Simon Glass wrote: > > > > > > Move this header out of the common header. Network support is used in > > > quite a few places but it still

Re: [PATCH 04/18] common: Drop net.h from common header

2020-05-30 Thread Simon Glass
Hi Adam, On Sat, 30 May 2020 at 08:59, Adam Ford wrote: > > On Mon, Apr 6, 2020 at 10:04 PM Simon Glass wrote: > > > > Move this header out of the common header. Network support is used in > > quite a few places but it still does not warrant blanket inclusion. > > > > Note that this net.h

Re: [PATCH 1/2] usb: ohci: Add header file for ARCH_DMA_MINALIGN

2020-05-30 Thread Tom Rini
On Sat, May 30, 2020 at 10:29:03AM -0600, Simon Glass wrote: > This is defined in the asm/cache.h header file. Update this header file to > include it so it gets the same value consistently across U-Boot. > > This fixes 'usb host' on omapl138_lcdk. > > Fixes: 90526e9fbac ("common: Drop net.h

Re: [PATCH 1/2] usb: ohci: Add header file for ARCH_DMA_MINALIGN

2020-05-30 Thread Adam Ford
On Sat, May 30, 2020 at 11:29 AM Simon Glass wrote: > > This is defined in the asm/cache.h header file. Update this header file to > include it so it gets the same value consistently across U-Boot. > > This fixes 'usb host' on omapl138_lcdk. Thank you. This series also fixes the da850-evm. > >

Re: [PATCH 2/2] dma-mapping: Add header file for ARCH_DMA_MINALIGN

2020-05-30 Thread Adam Ford
On Sat, May 30, 2020 at 12:27 PM Tom Rini wrote: > > On Sat, May 30, 2020 at 01:20:10PM -0400, Tom Rini wrote: > > On Sat, May 30, 2020 at 10:29:04AM -0600, Simon Glass wrote: > > > > > This is defined in the asm/cache.h header file. Update this header file to > > > include it. > > > > > >

Re: test_efi_selftest_test_input_ex test failure

2020-05-30 Thread Heinrich Schuchardt
jAm May 30, 2020 2:44:04 PM UTC schrieb Simon Glass : >Hi Heinrich, > >I got a failure on this commit: > >https://gitlab.denx.de/u-boot/custodians/u-boot-dm/-/jobs/102827 > >then a retry succeeded: > >https://gitlab.denx.de/u-boot/custodians/u-boot-dm/-/jobs/102828 > >Any ideas on this? > >Some

Re: [U-Boot] Please pull from u-boot-i2c

2020-05-30 Thread Tom Rini
On Sat, May 30, 2020 at 05:51:18AM +0200, Heiko Schocher wrote: > Hello Tom, > > please pull from u-boot-i2c master > > The following changes since commit ab80137cc436e977ef91a154372ae5aeae3f4fb0: > > Merge https://gitlab.denx.de/u-boot/custodians/u-boot-marvell (2020-05-27 > 10:56:25

Re: [PATCH] tbs2910: migrate to DM_VIDEO

2020-05-30 Thread Soeren Moch
On 28.05.20 10:07, Anatolij Gustschin wrote: > On Thu, 28 May 2020 09:54:42 +0200 > Soeren Moch sm...@web.de wrote: > ... >>> I'll merge this via u-boot-video tree. v2 patch depends on a few more >>> video patches which are under review/built-test currently. >> OK, I will try to test this new

Re: [PATCHv5][ 4/7] board: tbs2910_defconfig: Add support for booting general purpose distributions

2020-05-30 Thread Soeren Moch
On 30.05.20 05:24, Denis 'GNUtoo' Carikli wrote: > This is needed to enable distro_boot support later on. > > Adding the necessary configuration to enable booting general > purpose distributions is typically done by enabling > CONFIG_DISTRO_DEFAULTS. > > However the resulting image size is is

Re: [PATCHv5][ 6/7] board: tbs2910: Enable distro_boot support.

2020-05-30 Thread Soeren Moch
On 30.05.20 05:24, Denis 'GNUtoo' Carikli wrote: > This keeps the compatibility with the old bootcmd. > > The fdtfile environment variable also needed to be set to > imx6q-tbs2910.dtb to enable booting mainline kernels > otherwise with extlinux.conf it tries to load > mx6-tbs2910.dtb instead. > >

Re: [PATCHv5][ 5/7] board: tbs2910: remove CONFIG_DHCP

2020-05-30 Thread Soeren Moch
On 30.05.20 05:24, Denis 'GNUtoo' Carikli wrote: > As that the resulting image is already very close to the > size limit, and that CONFIG_DHCP is not strictly required, > removing it shouldn't hurt. You removed CONFIG_CMD_DHCP.  This is used to check the ethernet connection of the board. And since

Re: [PATCHv5][ 7/7] board: tbs2910: add documentation

2020-05-30 Thread Soeren Moch
On 30.05.20 05:24, Denis 'GNUtoo' Carikli wrote: > This documents the u-boot installation procedure and the > hardware in order to get started. > > Signed-off-by: Denis 'GNUtoo' Carikli > --- > Changelog since v4: > --- > - Added dded information about SYSBOOT boot support as

Re: [PATCH 2/2] dma-mapping: Add header file for ARCH_DMA_MINALIGN

2020-05-30 Thread Tom Rini
On Sat, May 30, 2020 at 10:29:04AM -0600, Simon Glass wrote: > This is defined in the asm/cache.h header file. Update this header file to > include it. > > Signed-off-by: Simon Glass Reviewed-by: Tom Rini There's also drivers/usb/host/ohci.h and lib/elf.c which also reference

[PATCH] ARM: imx: hab: panic on authentication failure

2020-05-30 Thread Marek Vasut
Instead of hang()ing the system and thus disallowing any automated recovery possibility from a HAB authentication failure, panic() . The panic() function can be configured to hang() the system after printing an error message, however the default is to reset the system instead. This allows

Re: [PATCH] ARM: imx: hab: panic on authentication failure

2020-05-30 Thread Fabio Estevam
Hi Marek, [Adding Breno] On Sat, May 30, 2020 at 3:29 PM Marek Vasut wrote: > > Instead of hang()ing the system and thus disallowing any automated > recovery possibility from a HAB authentication failure, panic() . > The panic() function can be configured to hang() the system after > printing

test_efi_selftest_test_input_ex test failure

2020-05-30 Thread Simon Glass
Hi Heinrich, I got a failure on this commit: https://gitlab.denx.de/u-boot/custodians/u-boot-dm/-/jobs/102827 then a retry succeeded: https://gitlab.denx.de/u-boot/custodians/u-boot-dm/-/jobs/102828 Any ideas on this? Some time ago I had a problem with buildman where the 4k pipe buffer for

Re: test_efi_selftest_test_input_ex test failure

2020-05-30 Thread Heinrich Schuchardt
On 5/30/20 5:24 PM, Heinrich Schuchardt wrote: > > jAm May 30, 2020 2:44:04 PM UTC schrieb Simon Glass : >> Hi Heinrich, >> >> I got a failure on this commit: >> >> https://gitlab.denx.de/u-boot/custodians/u-boot-dm/-/jobs/102827 >> >> then a retry succeeded: >> >>

Re: test_efi_selftest_test_input_ex test failure

2020-05-30 Thread Simon Glass
Hi Heinrich, On Sat, 30 May 2020 at 09:51, Heinrich Schuchardt wrote: > > On 5/30/20 5:24 PM, Heinrich Schuchardt wrote: > > > > jAm May 30, 2020 2:44:04 PM UTC schrieb Simon Glass : > >> Hi Heinrich, > >> > >> I got a failure on this commit: > >> > >>

Re: [PATCH] ARM: imx: hab: panic on authentication failure

2020-05-30 Thread Marek Vasut
On 5/30/20 10:14 PM, Patrick Wildt wrote: > On Sat, May 30, 2020 at 03:31:29PM -0300, Fabio Estevam wrote: >> Hi Marek, >> >> [Adding Breno] >> >> On Sat, May 30, 2020 at 3:29 PM Marek Vasut wrote: >>> >>> Instead of hang()ing the system and thus disallowing any automated >>> recovery possibility

[PATCH 1/1] Dockerfile: add libguestfs-tools package

2020-05-30 Thread Heinrich Schuchardt
In some of our Python tests we build disk images. Currently this involves using the sudo command for losetup and mount. For some tests sudo is not necessary if the command guestmount of package libguestfs-tools is available. Another useful tool in libguestfs-tools is virt-make-fs. Generally we

[PATCH 1/2] ARM: imx: m53menlo: Do not fail boot on invalid splash screen

2020-05-30 Thread Marek Vasut
None of these splash screen loading errors are so critical as to justify complete failure to boot, so just print error message as needed and return 0, the boot can very likely continue without the splash. Fix a couple of missing free(dst) instances as well. Signed-off-by: Marek Vasut Cc: Fabio

[PATCH 2/2] ARM: dts: imx: m53menlo: Convert to DM_ETH

2020-05-30 Thread Marek Vasut
Convert the board to DM_ETH instead of legacy networking. This requires a minor addition to the DT to satisfy the requirement for specifying a PHY node. No functional change from board user perspective. Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: NXP i.MX U-Boot Team Cc: Peng Fan Cc:

[PATCH 1/1] test/py: use actual core count for parallel builds

2020-05-30 Thread Heinrich Schuchardt
When building U-Boot we should not blindly use make -j8 but consider the actual core count given by os.cpu_count(). Signed-off-by: Heinrich Schuchardt --- test/py/conftest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/py/conftest.py b/test/py/conftest.py index

Pull request: u-boot-rockchip-20200530

2020-05-30 Thread Kever Yang
-rockchip.git tags/u-boot-rockchip-20200530 for you to fetch changes up to 63a3c89855cabc173c76a9958053408d34d33dc2: rockchip: rockpro64: enable DM_KEYBOARD (2020-05-30 07:38:50 +0800) Frank Wang (8): arm: mach-rockchip: bind

Re: [PATCH] ARM: imx: hab: panic on authentication failure

2020-05-30 Thread Patrick Wildt
On Sat, May 30, 2020 at 03:31:29PM -0300, Fabio Estevam wrote: > Hi Marek, > > [Adding Breno] > > On Sat, May 30, 2020 at 3:29 PM Marek Vasut wrote: > > > > Instead of hang()ing the system and thus disallowing any automated > > recovery possibility from a HAB authentication failure, panic() . >

[PULL] u-boot-sh/master

2020-05-30 Thread Marek Vasut
The following changes since commit ab80137cc436e977ef91a154372ae5aeae3f4fb0: Merge https://gitlab.denx.de/u-boot/custodians/u-boot-marvell (2020-05-27 10:56:25 -0400) are available in the Git repository at: git://git.denx.de/u-boot-sh.git master for you to fetch changes up to

[PULL] u-boot-usb/master

2020-05-30 Thread Marek Vasut
The following changes since commit ab80137cc436e977ef91a154372ae5aeae3f4fb0: Merge https://gitlab.denx.de/u-boot/custodians/u-boot-marvell (2020-05-27 10:56:25 -0400) are available in the Git repository at: git://git.denx.de/u-boot-usb.git master for you to fetch changes up to

Re: [PATCH] ARM: imx: hab: panic on authentication failure

2020-05-30 Thread Patrick Wildt
On Sat, May 30, 2020 at 10:29:19PM +0200, Marek Vasut wrote: > On 5/30/20 10:14 PM, Patrick Wildt wrote: > > On Sat, May 30, 2020 at 03:31:29PM -0300, Fabio Estevam wrote: > >> Hi Marek, > >> > >> [Adding Breno] > >> > >> On Sat, May 30, 2020 at 3:29 PM Marek Vasut wrote: > >>> > >>> Instead of

Re: Please pull u-boot-dm

2020-05-30 Thread Tom Rini
On Sat, May 30, 2020 at 09:16:10AM -0600, Simon Glass wrote: > Hi Tom, > > The following changes since commit 606cced58bf44bdf640760711be40d15a169aa71: > > Merge tag 'u-boot-amlogic-20200529' of > https://gitlab.denx.de/u-boot/custodians/u-boot-amlogic (2020-05-29 > 08:57:04 -0400) > > are