[PATCH 1/1] efi_loader: simplify PE consistency check

2020-05-29 Thread Heinrich Schuchardt
Knowing that at least one section header follows the optional header we only need to check for the length of the 64bit header which is longer than the 32bit header. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_image_loader.c | 15 ++- 1 file changed, 6 insertions(+), 9

[PATCH 1/1] efi_loader: function description cmp_pe_section()

2020-05-29 Thread Heinrich Schuchardt
Rework the description of function cmp_pe_section(). Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_image_loader.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/lib/efi_loader/efi_image_loader.c b/lib/efi_loader/efi_image_loader.c index

[PATCH 1/1] efi_loader: function descriptions efi_image_loader.c

2020-05-29 Thread Heinrich Schuchardt
We want to follow the Linux kernel style for function descriptions. Add missing parentheses after function names. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_image_loader.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

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

2020-05-29 Thread Heiko Schocher
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 -0400) are available in the Git repository at:

Re: [PATCH 1/1] i2c: observe scl_count in i2c_deblock_gpio_loop()

2020-05-29 Thread Heiko Schocher
Hello Heinrich, Am 09.05.2020 um 18:20 schrieb Heinrich Schuchardt: When compiling with -Wtype-limits we see this error: drivers/i2c/i2c-uclass.c: In function ‘i2c_deblock_gpio_loop’: drivers/i2c/i2c-uclass.c:517:21: error: comparison of unsigned expression >= 0 is always true

Re: [PATCH v2] misc: i2c_eeprom: implement different probe test eeprom offset

2020-05-29 Thread Heiko Schocher
Hello Eugen, Am 07.05.2020 um 10:53 schrieb Eugen Hristev: Because of this commit : 5ae84860b0 ("misc: i2c_eeprom: verify that the chip is functional at probe()") at probe time, each eeprom is tested for read at offset 0. The Atmel AT24MAC402 eeprom has different mapping. One i2c slave address

Re: [PATCH 2/2] i2c: designware_i2c: Check if the device is powered

2020-05-29 Thread Heiko Schocher
Hello Simon, Am 22.04.2020 um 18:13 schrieb Simon Glass: From: Raul E Rangel If the device doesn't return a version that means the device is non-functional. The dw_i2c_regs had invalid offsets for the version field. I got the correct value from the DesignWare databook. It also matches what

Re: [PATCH 1/2] i2c: designware_i2c: Tidy up use of NULL priv

2020-05-29 Thread Heiko Schocher
Hello Simon, Am 22.04.2020 um 18:13 schrieb Simon Glass: At present we still have pre-driver-model code in this driver and it makes things a bit confusing. In particular calc_bus_speed() is called with priv as NULL if not using driver model. This results in spk_cnt and comp_param1 being read

[PATCHv5][ 3/7] board: tbs2910: move CONFIG_BOOTCOMMAND from header to defconfig

2020-05-29 Thread Denis 'GNUtoo' Carikli
This doesn't affect the size of the image: with arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola GNU/Linux distribution, the text, data, bss and total sizes remain unchanged. Signed-off-by: Denis 'GNUtoo' Carikli Reviewed-by: Soeren Moch --- Changelog since v4: --- - Added the

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

2020-05-29 Thread Denis 'GNUtoo' Carikli
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 requested. - Fixed the switch positions as suggested. ---

[PATCHv5][ 2/7] board: tbs2910: disable CONFIG_GZIP

2020-05-29 Thread Denis 'GNUtoo' Carikli
As that the resulting image is already very close to the size limit, and that CONFIG_GZIP is not strictly required, removing it shouldn't hurt. With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola GNU/Linux distribution we have the following size reduction: - text: 9752 - data: 0 - bss: 16 -

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

2020-05-29 Thread Denis 'GNUtoo' Carikli
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 already very close to the size limit and PXE isn't strictly

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

2020-05-29 Thread Denis 'GNUtoo' Carikli
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. With arm-linux-gnueabi-gcc 9.3.0-1 from the Parabola GNU/Linux distribution, we have the following size differences: - text: -1708 bytes - data: -28

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

2020-05-29 Thread Denis 'GNUtoo' Carikli
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. With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola GNU/Linux

[PATCHv5][ 1/7] board: tbs2910: disable loadb and loads commands

2020-05-29 Thread Denis 'GNUtoo' Carikli
The loadb and loads commands are not needed for booting. There are also more reliable and faster alternatives to loadb and loads that can be used with the current configuration. As that the resulting image is already very close to the size limit, removing the loadb and loads commands shouldn't

[PATCH] ARM: imx: ddr: Add missing PHY reset

2020-05-29 Thread Marek Vasut
The iMX7D RM 9.2.4.9.3 Power removal flow Table 9-11. Re-enabling power explicitly says both the DDR controller and the PHY must be reset in the correct sequence. Currently the code only resets the controller. This leads to a misbehavior where the system brings the DRAM up after reboot, but the

[RFC PATCH 0/1] gpio: Handle NULL pointers gracefully

2020-05-29 Thread Pratyush Yadav
Hi, Patch [0] added devm_gpiod_get_index_optional() which would return NULL when when no GPIO was assigned to the requested function. This is convenient for drivers that need to handle optional GPIOs. We need to take a stance on who is responsible for the NULL check: the driver or the GPIO core?

[RFC PATCH 1/1] gpio: Handle NULL pointers gracefully

2020-05-29 Thread Pratyush Yadav
Prepare the way for a managed GPIO API by handling NULL pointers without crashing or failing. validate_desc() comes from Linux with the prints removed to reduce code size. Signed-off-by: Jean-Jacques Hiblot Signed-off-by: Pratyush Yadav --- drivers/gpio/Kconfig | 9

[PATCH] rockchip: enable RNG on RockPro64

2020-05-29 Thread Marcin Juszkiewicz
RNG module works fine on RockPro64 so let's enable it by default. => rng : 77 08 dd 04 2c 4d b8 cf 25 07 29 2d c0 ce 28 3b w...,M..%.)-..(; 0010: d4 5a 38 68 b1 40 97 59 b8 a5 7c 42 f8 4c 38 28 .Z8h.@.Y..|B.L8( 0020: 25 20 0b 4b f0 1a d5 c1 e6 a2 c2 34 5a 5e 64 26 %

[PATCH v2 0/2] gpio: Add a managed API

2020-05-29 Thread Pratyush Yadav
Hi, This is a re-submission of Jean-Jacques' earlier work in October last year. It can be found at [0]. The goal is to facilitate porting drivers from the linux kernel. Most of the series will be about adding managed API to existing infrastructure (GPIO, reset, regmap (already submitted)). This

[PATCH v2 1/2] drivers: gpio: Add a managed API to get a GPIO from the device-tree

2020-05-29 Thread Pratyush Yadav
From: Jean-Jacques Hiblot Add managed functions to get a gpio from the devce-tree, based on a property name (minus the '-gpios' suffix) and optionally an index. When the device is unbound, the GPIO is automatically released and the data structure is freed. Signed-off-by: Jean-Jacques Hiblot

[PATCH v2 2/2] test: gpio: Add tests for the managed API

2020-05-29 Thread Pratyush Yadav
From: Jean-Jacques Hiblot Add a test to verify that GPIOs can be acquired/released using the managed API. Also check that the GPIOs are released when the consumer device is removed. Signed-off-by: Jean-Jacques Hiblot --- arch/sandbox/dts/test.dts | 10 test/dm/gpio.c| 102

Re: [PULL u-boot] Please pull u-boot-amlogic-20200529

2020-05-29 Thread Tom Rini
at: > > https://gitlab.denx.de/u-boot/custodians/u-boot-amlogic.git > tags/u-boot-amlogic-20200529 > > for you to fetch changes up to 3678a5f0597d1cd491e3bded19ea72e570b5c8b1: > > arm: dts: meson-gxl: fix USB gadget by adding missing nodes for U-Boot > (2020-05

Re: [PATCH 05/10] core: extend struct driver_info to point to device

2020-05-29 Thread Simon Glass
Hi Walter, On Fri, 29 May 2020 at 13:21, Walter Lozano wrote: > > Hi Simon, > > On 29/5/20 16:00, Simon Glass wrote: > > Hi Walter, > > > > On Fri, 29 May 2020 at 12:56, Walter Lozano > > wrote: > >> > >> On 29/5/20 15:15, Walter Lozano wrote: > >>> Currently when creating an U_BOOT_DEVICE

Re: [PATCH 05/10] core: extend struct driver_info to point to device

2020-05-29 Thread Walter Lozano
Hi Simon, On 29/5/20 16:00, Simon Glass wrote: Hi Walter, On Fri, 29 May 2020 at 12:56, Walter Lozano wrote: On 29/5/20 15:15, Walter Lozano wrote: Currently when creating an U_BOOT_DEVICE entry a struct driver_info is declared, which contains the data needed to instantiate the device.

Re: [PATCH 00/10] improve OF_PLATDATA support

2020-05-29 Thread Walter Lozano
Hi Jagan On 29/5/20 15:25, Jagan Teki wrote: Hi Walter, On Fri, May 29, 2020 at 11:45 PM Walter Lozano wrote: When using OF_PLATDATA dtbs are converted to C structs in order to save space as we can remove both dtbs and libraries from TPL/SPL binaries. This patchset tries to improve its

Re: [PATCH] rockchip: rockpro64: Set cooling levels for pwm-fan

2020-05-29 Thread Simon Glass
Hi Kurt, On Fri, 29 May 2020 at 06:42, Kurt Miller wrote: > > On Fri, 2020-05-29 at 09:27 +0100, Peter Robinson wrote: > > On Thu, May 28, 2020 at 8:32 PM Kurt Miller > > wrote: > > > > > > > > > The cooling levels are tuned to the fan that comes with the rockpro64 NAS > > > case. A

Re: [PATCH 05/10] core: extend struct driver_info to point to device

2020-05-29 Thread Simon Glass
Hi Walter, On Fri, 29 May 2020 at 12:56, Walter Lozano wrote: > > > On 29/5/20 15:15, Walter Lozano wrote: > > Currently when creating an U_BOOT_DEVICE entry a struct driver_info > > is declared, which contains the data needed to instantiate the device. > > However, the actual device is created

Re: [PATCH 05/10] core: extend struct driver_info to point to device

2020-05-29 Thread Walter Lozano
On 29/5/20 15:15, Walter Lozano wrote: Currently when creating an U_BOOT_DEVICE entry a struct driver_info is declared, which contains the data needed to instantiate the device. However, the actual device is created at runtime and there is no proper way to get the device based on its struct

Re: [PATCH 00/10] improve OF_PLATDATA support

2020-05-29 Thread Jagan Teki
Hi Walter, On Fri, May 29, 2020 at 11:45 PM Walter Lozano wrote: > > When using OF_PLATDATA dtbs are converted to C structs in order to save > space as we can remove both dtbs and libraries from TPL/SPL binaries. > > This patchset tries to improve its support by overcoming some limitations > in

[PATCH 10/10] dtoc add test for cd-gpios

2020-05-29 Thread Walter Lozano
Add a test for dtoc taking into account the cd-gpios property. Signed-off-by: Walter Lozano --- tools/dtoc/dtoc_test_phandle_cd_gpios.dts | 42 ++ tools/dtoc/test_dtoc.py | 67 +++ 2 files changed, 109 insertions(+) create mode 100644

[PATCH 08/10] dtoc: update tests to match new platdata

2020-05-29 Thread Walter Lozano
After using a new approach to link nodes when OF_PLATDATA is enabled the test cases need to be update. This patch updates the tests based on this new implementation. Signed-off-by: Walter Lozano --- tools/dtoc/test_dtoc.py | 95 +++-- 1 file changed, 54

[PATCH 09/10] dtoc: update dtb_platdata to support cd-gpios

2020-05-29 Thread Walter Lozano
Currently dtoc does not support the property cd-gpios used to declare the gpios for card detect in mmc. This patch adds support to cd-gpios property. Signed-off-by: Walter Lozano --- tools/dtoc/dtb_platdata.py | 13 - tools/dtoc/test_dtoc.py| 2 +- 2 files changed, 9

[PATCH 05/10] core: extend struct driver_info to point to device

2020-05-29 Thread Walter Lozano
Currently when creating an U_BOOT_DEVICE entry a struct driver_info is declared, which contains the data needed to instantiate the device. However, the actual device is created at runtime and there is no proper way to get the device based on its struct driver_info. This patch extends struct

[PATCH 06/10] dtoc: extend dtoc to use struct driver_info when linking nodes

2020-05-29 Thread Walter Lozano
In the current implementation, when dtoc parses a dtb to generate a struct platdata it converts the information related to linked nodes as pointers to struct platdata of destination nodes. By doing this, it makes difficult to get pointer to udevices created based on these information. This patch

[PATCH 07/10] dm: doc: update of-plat with new phandle support

2020-05-29 Thread Walter Lozano
Update documentation to reflect the new phandle support when OF_PLATDATA is used. Now phandles are implemented as pointers to U_BOOT_DEVICE, which makes it possible to get a pointer to the actual device. Signed-off-by: Walter Lozano --- doc/driver-model/of-plat.rst | 24

[PATCH 04/10] core: drop const for struct driver_info

2020-05-29 Thread Walter Lozano
In order to prepare for a new support of phandle when OF_PLATDATA is used drop the const for struct driver_info as this struct will need to be updated on runtime. Signed-off-by: Walter Lozano --- drivers/core/device.c| 2 +- drivers/core/root.c | 2 +-

[PATCH 02/10] dtoc: add option to disable warnings

2020-05-29 Thread Walter Lozano
As dtoc now performs checks for valid driver names, when running dtoc tests several warnings arise as these tests don't use valid driver names. This patch adds an option to disable those warning, which is only intended for running tests. Signed-off-by: Walter Lozano ---

[PATCH 03/10] dm: doc: update of-plat with the suppor for driver aliases

2020-05-29 Thread Walter Lozano
Update the documentation with the support for driver aliases using U_BOOT_DRIVER_ALIAS. Signed-off-by: Walter Lozano --- doc/driver-model/of-plat.rst | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/doc/driver-model/of-plat.rst b/doc/driver-model/of-plat.rst

[PATCH 01/10] dtoc: add support to scan drivers

2020-05-29 Thread Walter Lozano
Currently dtoc scans dtbs to convert them to struct platdata and to generate U_BOOT_DEVICE entries. These entries need to be filled with the driver name, but at this moment the information used is the compatible name present in the dtb. This causes that only nodes with a compatible name that

[PATCH 00/10] improve OF_PLATDATA support

2020-05-29 Thread Walter Lozano
When using OF_PLATDATA dtbs are converted to C structs in order to save space as we can remove both dtbs and libraries from TPL/SPL binaries. This patchset tries to improve its support by overcoming some limitations in the current implementation First, the support for scan and check for valid

[PATCH] env: Add support for explicit write access list

2020-05-29 Thread Marek Vasut
This option marks any U-Boot variable which does not have explicit 'w' writeable flag set as read-only. This way the environment can be locked down and only variables explicitly configured to be writeable can ever be changed by either 'env import', 'env set' or loading user environment from

[PATCH] env: Add option to only ever append environment

2020-05-29 Thread Marek Vasut
Add configuration option which prevents the environment hash table to be ever cleared and reloaded with different content. This is useful in case the first environment loaded into the hash table contains e.g. sensitive content which must not be dropped or reloaded. Signed-off-by: Marek Vasut ---

[PATCH] env: Fix invalid env handling in env_init()

2020-05-29 Thread Marek Vasut
In case the env storage driver marks environment as ENV_INVALID, we must reset the $ret return value to -ENOENT to let the env init code reset the environment to the default one a bit further down. Signed-off-by: Marek Vasut --- env/env.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

[PATCH] env: nowhere: Implement .load callback

2020-05-29 Thread Marek Vasut
Add .load callback for the 'nowhere' environment driver. This is useful for when the 'nowhere' driver is used in combination with other drivers and should be used to load the default environment. Signed-off-by: Marek Vasut --- env/nowhere.c | 7 +++ 1 file changed, 7 insertions(+) diff

[PATCH] env: Add H_DEFAULT flag

2020-05-29 Thread Marek Vasut
Add another internal environment flag which indicates that the operation is resetting the environment to the default one. This allows the env code to discern between import of external environment and reset to default. Signed-off-by: Marek Vasut --- env/common.c | 3 ++- include/search.h |

[PATCH 3/4] sunxi: pmic_bus: Add DM I2C support

2020-05-29 Thread Benedikt Spranger
Add support for U-Boot DM. Signed-off-by: Benedikt Spranger Reviewed-by: Kurt Kanzenbach --- arch/arm/mach-sunxi/pmic_bus.c | 20 1 file changed, 20 insertions(+) diff --git a/arch/arm/mach-sunxi/pmic_bus.c b/arch/arm/mach-sunxi/pmic_bus.c index 091c59331b..8ecd037528

[PATCH 2/4] i2c: mvtwsi: Add Allwinner sun4i compatible

2020-05-29 Thread Benedikt Spranger
Add the compatible string for the Allwinner A10 i2c controller. Signed-off-by: Benedikt Spranger Reviewed-by: Kurt Kanzenbach --- drivers/i2c/mvtwsi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/i2c/mvtwsi.c b/drivers/i2c/mvtwsi.c index d3cc9b9d83..57cd442eee 100644 ---

[PATCH 4/4] common: Add LX TestBox checks

2020-05-29 Thread Benedikt Spranger
The TestBox board is an open hardware enhancement for the Lamobo R1 router board. The Testbox board is used in the CI-RT project to manage devices under test (https://ci-rt.linutronix.de). The hardware project is located at https://github.com/ci-rt/testbox-shield Check if the hardware is present

[PATCH 0/4] Add support for LX TestBox

2020-05-29 Thread Benedikt Spranger
The Testbox board is an open hardware enhancement for the Lamobo R1 router board. The Testbox board is used in the CI-RT project to manage devices under test (https://ci-rt.linutronix.de). Add support for the LX TestBox. Benedikt Spranger (4): sunxi: pmic_bus: Refactor pmic_bus_read() and

[PATCH 1/4] sunxi: pmic_bus: Refactor pmic_bus_read() and pmic_bus_write()

2020-05-29 Thread Benedikt Spranger
Unify I2C calls to ease the driver model conversion. Signed-off-by: Benedikt Spranger Reviewed-by: Kurt Kanzenbach --- arch/arm/mach-sunxi/pmic_bus.c | 28 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/arch/arm/mach-sunxi/pmic_bus.c

[PATCH] power: pmic: Add SPL Kconfig entry for PFUZE100

2020-05-29 Thread Marek Vasut
Add Kconfig entry for the PFUZE PMIC, SPL variant. Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Jaehoon Chung Cc: Peng Fan Cc: Stefano Babic --- drivers/power/pmic/Kconfig | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig

[PATCH] board: dh_stm32mp1: update the gpio hog support

2020-05-29 Thread Patrick Delaunay
This patch update the current gpio hog implementation with the new API gpio_hog_probe_all() actviated with CONFIG_GPIO_HOG. Signed-off-by: Patrick Delaunay --- board/dhelectronics/dh_stm32mp1/board.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git

[PATCH] image: android: fix abootimg support

2020-05-29 Thread Christian Gmeiner
abootimg creates images where all load addresses are 0. Android Boot Image Info: * file name = artifacts/fastboot.img * image size = 31381504 bytes (29.93 MB) page size = 2048 bytes * Boot Name = "" * kernel size = 9397406 bytes (8.96 MB) ramdisk size = 21981144 bytes (20.96 MB)

[PATCH] configs: rpi: set NR_DRAM_BANKS to four

2020-05-29 Thread matthias . bgg
From: Matthias Brugger With the new RPi4 which has 8 GB of RAM, we can have up to four DRAM banks. Bump up the configuration files to detect all the memory in U-Boot. Signed-off-by: Matthias Brugger --- configs/rpi_4_32b_defconfig | 2 +- configs/rpi_4_defconfig | 2 +-

Re: Cannot boot firefly-rk3399: mmc block read error

2020-05-29 Thread Simon Glass
Hi, OK thank you. I got it booting once but I'm actually not sure how. It looks like that patch is in rk/master, but this is what I am getting: U-Boot TPL 2020.07-rc3-00062-gfb14231483 (May 29 2020 - 08:27:15) Channel 0: DDR3, 800MHz BW=32 Col=10 Bk=8 CS0 Row=15 CS1 Row=15 CS=2 Die BW=16

Re: [PATCH 08/24] arm: Remove configs/P1020MBG-PC_36BIT_SDCARD_defconfig board

2020-05-29 Thread Tom Rini
On Fri, May 29, 2020 at 06:06:16PM +0530, Jagan Teki wrote: > On Fri, May 29, 2020 at 2:14 AM Tom Rini wrote: > > > > On Thu, May 28, 2020 at 02:32:39PM -0600, Simon Glass wrote: > > > Hi, > > > > > > On Thu, 28 May 2020 at 14:15, Tom Rini wrote: > > > > > > > > On Thu, May 28, 2020 at

Re: [PATCH v2] spi: Kirkwood: Drop non-DM support and enable DM_SPI on all boards

2020-05-29 Thread Jagan Teki
Thanks for the work. On Fri, May 29, 2020 at 4:01 PM Bhargav Shah wrote: > > Dropped DM supprot for Kirkwood > Enable DM_SPI on all boards using Kirkwood > > Signed-off-by: Bhargav Shah > Reviewed-by: Stefan Roese > > --- > Changes for v2: > - Added more meaningful commit message > -

Re: [PATCH v13 12/19] riscv: sifive: dts: fu540: set ethernet clock rate

2020-05-29 Thread Bin Meng
On Fri, May 29, 2020 at 2:05 PM Pragnesh Patel wrote: > > Set ethernet clock rate to 125 Mhz so that it will work with 1000Mbps, > Earlier this is done by FSBL. With this change We can remove the > ethernet clock rate code from FSBL. > > Signed-off-by: Pragnesh Patel > --- >

Re: [PATCH 1/2] riscv: sifive: fu540: enable all cache ways from U-Boot proper

2020-05-29 Thread Bin Meng
On Fri, May 29, 2020 at 2:45 PM Pragnesh Patel wrote: > > Add L2 cache node to enable all cache ways from U-Boot proper. > > Signed-off-by: Pragnesh Patel > Reviewed-by: Bin Meng > --- > arch/riscv/cpu/fu540/Makefile | 1 + > arch/riscv/cpu/fu540/cache.c | 53

Re: [PATCH] rockchip: rockpro64: Set cooling levels for pwm-fan

2020-05-29 Thread Kurt Miller
On Fri, 2020-05-29 at 09:27 +0100, Peter Robinson wrote: > On Thu, May 28, 2020 at 8:32 PM Kurt Miller > wrote: > > > > > > The cooling levels are tuned to the fan that comes with the rockpro64 NAS > > case. A gpu_thermal zone was not added because having two active cooling > > maps control

Re: [PATCH v2 20/23] phytec: Drop pcm051

2020-05-29 Thread Jagan Teki
On Wed, May 27, 2020 at 7:40 PM Lars Poeschel wrote: > > On Wed, May 27, 2020 at 06:26:34PM +0530, Jagan Teki wrote: > > OF_CONTROL, DM_SPI and other driver model migration deadlines > > are expired for this board. > > > > Drop it. > > Acked-by: Lars Poeschel Slite changes on the commit message

Re: [PATCH v2 11/23] pandora: Drop omap3 pandora

2020-05-29 Thread Jagan Teki
On Thu, May 28, 2020 at 3:12 PM Gražvydas Ignotas wrote: > > I'm not aware of any remaining mainline u-boot users of this device (also > asked around a bit on IRC), so: > Acked-by: Grazvydas Ignotas Slite changes on the commit message and Applied to u-boot-spi/master

Re: [PATCH 08/24] arm: Remove configs/P1020MBG-PC_36BIT_SDCARD_defconfig board

2020-05-29 Thread Jagan Teki
On Fri, May 29, 2020 at 2:14 AM Tom Rini wrote: > > On Thu, May 28, 2020 at 02:32:39PM -0600, Simon Glass wrote: > > Hi, > > > > On Thu, 28 May 2020 at 14:15, Tom Rini wrote: > > > > > > On Thu, May 28, 2020 at 07:07:14AM +, Priyanka Jain wrote: > > > > >-Original Message- > > > >

RE: [PATCH v4 0/6] Introduce DSA Ethernet switch class and Felix driver

2020-05-29 Thread Claudiu Manoil
>-Original Message- >From: U-Boot On Behalf Of Claudiu Manoil >Sent: Tuesday, May 5, 2020 8:43 PM >To: u-boot@lists.denx.de >Cc: joe.hershber...@ni.com; Alexandru Marginean >; Vladimir Oltean > >Subject: [PATCH v4 0/6] Introduce DSA Ethernet switch class and Felix driver > Hi Joe, How

[PULL u-boot] Please pull u-boot-amlogic-20200529

2020-05-29 Thread Neil Armstrong
mmit 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: https://gitlab.denx.de/u-boot/custodians/u-boot-amlogic.git tags/u-boot-amlogic-20200529 for you to fetch change

Re: [PATCH] arm: dts: meson-gxl: fix USB gadget by adding missing nodes for U-Boot

2020-05-29 Thread Neil Armstrong
On 28/05/2020 15:47, Neil Armstrong wrote: > The khadas-vim, khadas-vim2, libretech-ac & libretech-*-pc boards were missing > DT tweak to enable USB gadget. > Add them to their -u-boot.dtsi files and include the right gxl-u-boot.dtsi. > > Fixes: a19e8a0f03 ("arm: dts: meson-gxl: Add USB Gadget

Re: [PATCH] board: xilinx/zynqmp: Remove 1-second delay at boot

2020-05-29 Thread Michal Simek
On 26. 05. 20 10:20, Mike Looijmans wrote: > SPL boot is slow on the ZynqMP because of this delay. The initialization done > in psu_init no longer requires an external delay, so this can be removed to > speed up the SPL boot flow considerably. > > Signed-off-by: Mike Looijmans > --- >

Re: [PATCH 0/5] Enable I2C in SPL, support runtime detection of add-on board

2020-05-29 Thread Jagan Teki
Hi Kever, On Fri, May 29, 2020 at 3:25 PM Kever Yang wrote: > > Hi Suniel, > > Any idea bout ChenYu's comment? Yes, ChenYu's point is valid, right now we are in communication with Libretech about the I/O differences to differentiate baseboard vs expansion. We will come back with proper

Re: [PATCH 02/13] efi_loader: image_loader: add a check against certificate type of authenticode

2020-05-29 Thread Heinrich Schuchardt
On 5/29/20 8:41 AM, AKASHI Takahiro wrote: > UEFI specification requires that we shall support three type of > certificates of authenticode in PE image: > WIN_CERT_TYPE_EFI_GUID with the guid, EFI_CERT_TYPE_PCKS7_GUID > WIN_CERT_TYPE_PKCS_SIGNED_DATA > WIN_CERT_TYPE_EFI_PKCS1_15 > > As EDK2

[PATCH v2] spi: Kirkwood: Drop non-DM support and enable DM_SPI on all boards

2020-05-29 Thread Bhargav Shah
Dropped DM supprot for Kirkwood Enable DM_SPI on all boards using Kirkwood Signed-off-by: Bhargav Shah Reviewed-by: Stefan Roese --- Changes for v2: - Added more meaningful commit message - Corrected typo from Krikwood to Kirkwood in commit message --- diff --git

Re: [PATCH 01/13] efi_loader: signature: move efi_guid_cert_type_pkcs7 to efi_signature.c

2020-05-29 Thread Heinrich Schuchardt
On 5/29/20 8:41 AM, AKASHI Takahiro wrote: > The global variable, efi_guid_cert_type_pkcs7, will also be used in > efi_image_loader.c in a succeeding patch so as to correctly handle > a signature type of authenticode in signed image. > > Meanwhile, it is currently defined in efi_variable.c. Once

Re: [PATCH v6 04/16] arm: mach-rockchip: bind sub-nodes for rk3399_syscon

2020-05-29 Thread Kever Yang
On 2020/5/26 上午11:32, Frank Wang wrote: There are some sub-nodes under the grf DT, so add bind callback function in rk3399 syscon driver to scan them recursively. Signed-off-by: Frank Wang Reviewed-by: Kever Yang Thanks, - Kever --- arch/arm/mach-rockchip/rk3399/syscon_rk3399.c | 3

Re: [PATCH] rockchip: rockpro64: enable DM_KEYBOARD

2020-05-29 Thread Kever Yang
On 2020/5/27 上午12:18, Marcin Juszkiewicz wrote: USB stack uses DM so DM_KEYBOARD is needed to get USB keyboard working. Signed-off-by: Marcin Juszkiewicz Reviewed-by: Kever Yang Thanks, - Kever --- configs/rockpro64-rk3399_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [PATCH] rockchip: enable USB OHCI host for RockPro64

2020-05-29 Thread Kever Yang
On 2020/5/25 下午10:44, Marcin Juszkiewicz wrote: U-Boot has video output enabled so time to get keyboard working. => usb reset;usb tree resetting USB... Bus usb@fe38: USB EHCI 1.00 Bus usb@fe3a: USB OHCI 1.0 Bus usb@fe3c: USB EHCI 1.00 Bus usb@fe3e: USB OHCI 1.0 Bus dwc3: usb

Re: [PATCH] rockchip: enable RNG on RockPro64

2020-05-29 Thread Kever Yang
Hi Marcin,     The commit message is acceptable, but you will need to send a new V2 patch for this instead of reply this by the comment. Thanks, - Kever On 2020/5/25 下午4:52, Marcin Juszkiewicz wrote: RNG module works fine on RockPro64 so let's enable it by default. => rng : 77

Re: Failsafe booting to alternate linux image/fs

2020-05-29 Thread Michal Simek
Hi, On 29. 05. 20 1:25, Brendan Simon (eTRIX) wrote: > I came across the "Boot Count Limit" feature in the U-Boot wiki, and it > looks like it will do exactly what I want. > > https://www.denx.de/wiki/view/DULG/UBootBootCountLimit > > > However, it also states that the "feature is

Re: [PATCH 0/5] Enable I2C in SPL, support runtime detection of add-on board

2020-05-29 Thread Kever Yang
Hi Suniel,     Any idea bout ChenYu's comment? Thanks, - Kever On 2020/5/23 上午10:07, Chen-Yu Tsai wrote: Hi, On Sat, May 23, 2020 at 2:04 AM wrote: From: Suniel Mahesh This patch series adds runtime detection of add-on board(ROC-RK3399-PC Mezzanine) by enabling I2C in SPL. This add on

Re: [PATCH 3/3] doc: rockchip: Remove list of supported boards

2020-05-29 Thread Kever Yang
On 2020/5/22 下午10:14, Walter Lozano wrote: As documentation is being moved to doc/boards/rockchip create a warning message and remove the redundant list of supported boards. Signed-off-by: Walter Lozano Reviewed-by: Kever Yang Thanks, - Kever --- doc/README.rockchip | 72

Re: [PATCH 1/3] doc: board: rockchip: Improve supported board list format

2020-05-29 Thread Kever Yang
On 2020/5/22 下午10:14, Walter Lozano wrote: As an additional step to move documentation to doc/boards/rockchip improve format of the supported board list to make it more readable. Additionally, add the configuration files used to build them based on doc/README.rockchip. Signed-off-by: Walter

Re: [PATCH 2/3] doc: board: rockchip: Add missing supported boards

2020-05-29 Thread Kever Yang
On 2020/5/22 下午10:14, Walter Lozano wrote: Update the list of supported boards with the information available on doc/README.rockchip. Signed-off-by: Walter Lozano Reviewed-by: Kever Yang Thanks, - Kever --- doc/board/rockchip/rockchip.rst | 16 1 file changed, 16

Re: [PATCH] rockchip: rockpro64: Set cooling levels for pwm-fan

2020-05-29 Thread Peter Robinson
On Thu, May 28, 2020 at 8:32 PM Kurt Miller wrote: > > The cooling levels are tuned to the fan that comes with the rockpro64 NAS > case. A gpu_thermal zone was not added because having two active cooling > maps control one physical fan causes them to compete for the fan speed > which results in

RE: [PATCH v12 01/18] misc: add driver for the SiFive otp controller

2020-05-29 Thread Pragnesh Patel
Hi Simon, >-Original Message- >From: Simon Glass >Sent: 28 May 2020 08:38 >To: Pragnesh Patel >Cc: U-Boot Mailing List ; Atish Patra >; Palmer Dabbelt ; Bin >Meng ; Paul Walmsley ; >Jagan Teki ; Anup Patel >; Sagar Kadam ; rick >; Tero Kristo ; Heiko Stuebner >; Finley Xiao chips.com>;

[PATCH 1/2] riscv: sifive: fu540: enable all cache ways from U-Boot proper

2020-05-29 Thread Pragnesh Patel
Add L2 cache node to enable all cache ways from U-Boot proper. Signed-off-by: Pragnesh Patel Reviewed-by: Bin Meng --- arch/riscv/cpu/fu540/Makefile | 1 + arch/riscv/cpu/fu540/cache.c | 53 +++ arch/riscv/dts/fu540-c000-u-boot.dtsi | 4 ++

[PATCH 0/2] Enable all cache ways and Enable SiFive PWM driver

2020-05-29 Thread Pragnesh Patel
Patch 1: enable all cache ways from U-Boot proper This patch enables all cache ways from U-Boot proper, earlier this was done by FSBL. (https://github.com/sifive/freedom-u540-c000-bootloader/blob/master/fsbl/main.c#L428) L2 cache is of 2 MB (16 cache ways) and 1 cache way is of 128 KB. FSBL is

[PATCH 2/2] riscv: sifive: fu540: Enable SiFive PWM driver

2020-05-29 Thread Pragnesh Patel
This patch enables SiFive PWM driver for the SiFive Unleashed board. Signed-off-by: Pragnesh Patel --- board/sifive/fu540/Kconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/board/sifive/fu540/Kconfig b/board/sifive/fu540/Kconfig index 86193d7668..683668d059 100644 ---

[PATCH 13/13] test/py: efi_secboot: add a test for verifying with digest of signed image

2020-05-29 Thread AKASHI Takahiro
Signature database (db or dbx) may have not only certificates that contain a public key for RSA decryption, but also digests of signed images. In this test case, if database has an image's digest (EFI_CERT_SHA256_GUID) and if the value matches to a hash value calculated from image's binary,

[PATCH 12/13] test/py: efi_secboot: add a test for multiple signatures

2020-05-29 Thread AKASHI Takahiro
In this test case, an image is signed multiple times with different keys. If any of signatures contained is not verified, the whole authentication check should fail. Signed-off-by: AKASHI Takahiro --- test/py/tests/test_efi_secboot/conftest.py| 3 ++

[PATCH 11/13] test/py: efi_secboot: add a test against certificate revocation

2020-05-29 Thread AKASHI Takahiro
Revocation database (dbx) may have not only certificates, but also message digests of certificates with revocation time (EFI_CERT_X509_SHA256_GUILD). In this test case, if the database has such a digest and if the value matches to a certificate that created a given image's signature,

[PATCH 09/13] test/py: efi_secboot: fix test case 1g of test_authvar

2020-05-29 Thread AKASHI Takahiro
In the test case (1g) of test_authvar, "db" is mistakenly used, and it ends up being the exact same as (1f). So correct it as "dbx" test case. Signed-off-by: AKASHI Takahiro --- test/py/tests/test_efi_secboot/test_authvar.py | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-)

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

2020-05-29 Thread AKASHI Takahiro
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 --- .../py/tests/test_efi_secboot/test_authvar.py | 73 +-- test/py/tests/test_efi_secboot/test_signed.py | 34

[PATCH 10/13] test/py: efi_secboot: split "signed image" test case-1 into two cases

2020-05-29 Thread AKASHI Takahiro
Split the existing test case-1 into case1 and a new case-2: case-1 for non-SecureBoot mode; case-2 for SecureBoot mode. In addition, one corner case is added to case-2; a image is signed but a corresponding certificate is not yet installed in "db." Signed-off-by: AKASHI Takahiro ---

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

2020-05-29 Thread AKASHI Takahiro
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 verification check for multiple signatures case is a bit

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

2020-05-29 Thread AKASHI Takahiro
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 a given binary image, it is granted for loading.

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

2020-05-29 Thread AKASHI Takahiro
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 insertions(+), 28 deletions(-) diff --git

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

2020-05-29 Thread AKASHI Takahiro
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. Signed-off-by: AKASHI Takahiro --- lib/efi_loader/efi_signature.c | 5 +++-- 1 file changed, 3

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

2020-05-29 Thread AKASHI Takahiro
Since certificate table, which is indexed by IMAGE_DIRECTORY_ENTRY_SECURITY and contains authenticode in PE image, doesn't always exist, we should make sure that we will retrieve its pointer only if it exists. Signed-off-by: AKASHI Takahiro --- lib/efi_loader/efi_image_loader.c | 29

[PATCH 01/13] efi_loader: signature: move efi_guid_cert_type_pkcs7 to efi_signature.c

2020-05-29 Thread AKASHI Takahiro
The global variable, efi_guid_cert_type_pkcs7, will also be used in efi_image_loader.c in a succeeding patch so as to correctly handle a signature type of authenticode in signed image. Meanwhile, it is currently defined in efi_variable.c. Once some secure storage solution for UEFI variables is

[PATCH 02/13] efi_loader: image_loader: add a check against certificate type of authenticode

2020-05-29 Thread AKASHI Takahiro
UEFI specification requires that we shall support three type of certificates of authenticode in PE image: WIN_CERT_TYPE_EFI_GUID with the guid, EFI_CERT_TYPE_PCKS7_GUID WIN_CERT_TYPE_PKCS_SIGNED_DATA WIN_CERT_TYPE_EFI_PKCS1_15 As EDK2 does, we will support the first two that are pkcs7

[PATCH 00/13] efi_loader: rework/improve UEFI secure boot code

2020-05-29 Thread AKASHI Takahiro
Summary === I'm currently working on reworking UEFI secure boot, aiming to add "intermediate certificates" support. In this effort, I found a couple of issues that should immediately be fixed or useful improvements even without intermediate certificates support. Each commit in this patch

  1   2   >