Re: EFI from usb HDD

2021-08-18 Thread Michal Simek
On 8/19/21 6:14 AM, AKASHI Takahiro wrote: > On Wed, Aug 18, 2021 at 11:07:09AM +0200, Michal Simek wrote: >> >> >> On 8/18/21 7:13 AM, AKASHI Takahiro wrote: >>> On Tue, Aug 17, 2021 at 09:20:31AM +0200, Michal Simek wrote: On 8/12/21 11:43 AM, AKASHI Takahiro wrote: > On

Re: [PATCH v2] imx: spl: fix imx8m secure boot

2021-08-18 Thread Heiko Schocher
Hello Tim, On 18.08.21 23:02, Tim Harvey wrote: > On Mon, Aug 16, 2021 at 11:17 PM Heiko Schocher wrote: >> >> cherry-picked from NXP code: >> 719d665a87c6: ("MLK-20467 imx8m: Fix issue for booting signed image through >> uuu") >> >> which fixes secure boot on imx8m based boards. Problem was >>

Re: EFI from usb HDD

2021-08-18 Thread AKASHI Takahiro
On Wed, Aug 18, 2021 at 11:07:09AM +0200, Michal Simek wrote: > > > On 8/18/21 7:13 AM, AKASHI Takahiro wrote: > > On Tue, Aug 17, 2021 at 09:20:31AM +0200, Michal Simek wrote: > >> > >> > >> On 8/12/21 11:43 AM, AKASHI Takahiro wrote: > >>> On Fri, Jul 30, 2021 at 08:22:18AM +0200, Michal Simek

[PATCH 28/28] RFC: rpi: Switch over to use bootflow

2021-08-18 Thread Simon Glass
Drop the distro boot scripts and use bootflow instead. Note: This is just a demonstration. It does not support EFI at present. Signed-off-by: Simon Glass --- boot/Kconfig | 2 +- doc/develop/distro.rst | 3 +++ include/configs/rpi.h | 37 + 3

[PATCH 24/28] bootmethod: doc: Add documentation

2021-08-18 Thread Simon Glass
Add documentation for this feature, including the commands. Signed-off-by: Simon Glass --- MAINTAINERS | 3 + doc/develop/bootflow.rst| 323 ++ doc/develop/index.rst | 1 + doc/device-tree-bindings/bootmethod.txt

[PATCH 23/28] bootmethod: Add tests for bootmethod and bootflow

2021-08-18 Thread Simon Glass
Add a set of combined tests for these two commands. Expand the sandbox console-recording limit so that these can work. These tests rely on a filesystem script which is not yet added to the Python tests. It is included here as a shell script. Signed-off-by: Simon Glass --- MAINTAINERS

[PATCH 27/28] ethernet: Add a bootmethod

2021-08-18 Thread Simon Glass
Add a bootmethod for Ethernet. It uses the distro boot mechanism to locate a file. Signed-off-by: Simon Glass --- MAINTAINERS | 1 + configs/sandbox_defconfig | 1 + net/Kconfig | 9 + net/Makefile | 1 + net/eth-uclass.c | 8 +

[PATCH 25/28] mmc: Allow for children other than the block device

2021-08-18 Thread Simon Glass
At present the MMC uclass assumes that the only child it can have is a block device. Update this so we can add a bootmethod too. Signed-off-by: Simon Glass --- drivers/mmc/mmc-uclass.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/mmc-uclass.c

[PATCH 26/28] mmc: Add a bootmethod

2021-08-18 Thread Simon Glass
Add a bootmethod for MMC. It mostly just calls the bootmethod helper function. Add a function to obtain the block device for an MMC controller. Fix up the comment for mmc_get_blk_desc() while we are here. Signed-off-by: Simon Glass --- MAINTAINERS | 1 +

[PATCH 21/28] bootmethod: Add a command

2021-08-18 Thread Simon Glass
Add a 'bootmethod' command to handle listing and selection of bootmethods. Signed-off-by: Simon Glass --- MAINTAINERS | 1 + cmd/Kconfig | 8 +++ cmd/bootmethod.c | 130 +++ 3 files changed, 139 insertions(+) create mode 100644

[PATCH 22/28] bootflow: Add a command

2021-08-18 Thread Simon Glass
Add a 'bootflow' command to handle listing and selection of bootflow. Signed-off-by: Simon Glass --- MAINTAINERS | 1 + boot/bootmethod.c | 26 +++ cmd/Makefile | 1 + cmd/bootflow.c| 399 ++ 4 files changed, 427 insertions(+)

[PATCH 20/28] bootmethod: Add an implementation of distro boot

2021-08-18 Thread Simon Glass
Add a bootmethod type which handles distro boot, so we can boot a bootflow using this commonly used mechanism. In effect, this provides the same functionality as the 'pxe' and 'sysboot' commands and shares the same code. But the interface into it is via a bootmethod. For now this requires the

[PATCH 19/28] bootmethod: Add the uclass and core implementation

2021-08-18 Thread Simon Glass
Add a uclass for bootmethod and the various helpers needed to make it work. Signed-off-by: Simon Glass --- MAINTAINERS| 6 + boot/Kconfig | 9 + boot/Makefile | 1 + boot/bootmethod.c | 445 +

[PATCH 16/28] pxe: Refactor sysboot to have one helper

2021-08-18 Thread Simon Glass
The only difference between the three helpers is the filesystem type. Factor this out and call the filesystem functions directly, instead of through the command-line interpreter. This allows the file size to be obtained directly, instead of via an environment variable. We cannot do the same thing

[PATCH 18/28] pxe: Allow calling the pxe_get logic directly

2021-08-18 Thread Simon Glass
Refactor this code so that we can call the 'pxe get' command without going through the command-line interpreter. This makes it easier to get the information we need, without going through environment variables. Signed-off-by: Simon Glass --- cmd/pxe.c | 92

[PATCH 17/28] doc: Move distro boot doc to rST

2021-08-18 Thread Simon Glass
Move this over to the new rST format. Signed-off-by: Simon Glass --- doc/{README.distro => develop/distro.rst} | 177 ++ doc/develop/index.rst | 1 + 2 files changed, 80 insertions(+), 98 deletions(-) rename doc/{README.distro => develop/distro.rst}

[PATCH 15/28] pxe: Return the file size from the getfile() function

2021-08-18 Thread Simon Glass
It is pretty strange that the pxe code uses the 'filesize' environment variable find the size of a file it has just read. Partly this is because it uses the command-line interpreter to parse its request to load the file. As a first step towards unwinding this, return it directly from the

[PATCH 14/28] lib: Add a function to convert a string to a hex value

2021-08-18 Thread Simon Glass
Add an xtoa() function, similar to itoa() but for hex instead. Signed-off-by: Simon Glass --- include/vsprintf.h | 16 ++-- lib/vsprintf.c | 20 ++-- test/print_ut.c| 16 3 files changed, 48 insertions(+), 4 deletions(-) diff --git

[PATCH 12/28] pxe: Drop get_bootfile_path()

2021-08-18 Thread Simon Glass
This function no longer makes sense, since it is pretty easy to prepend the boot directory to the filename. Drop it and update its only caller. Signed-off-by: Simon Glass --- boot/pxe_utils.c| 53 + include/pxe_utils.h | 3 ++- 2 files changed,

[PATCH 13/28] lib: Add tests for simple_itoa()

2021-08-18 Thread Simon Glass
Add test and a comment for this function. Signed-off-by: Simon Glass --- include/vsprintf.h | 13 - test/print_ut.c| 17 + 2 files changed, 29 insertions(+), 1 deletion(-) diff --git a/include/vsprintf.h b/include/vsprintf.h index 83d187e53d4..4479df0af3f

[PATCH 11/28] pxe: Clean up the use of bootfile

2021-08-18 Thread Simon Glass
The 'bootfile' environment variable is read in the bowels of pxe_util to provide a directory to which all loaded files are relative. This is not obvious from the API to PXE and it is strange to make the caller set an environment variable rather than pass this as a parameter. The code is also

[PATCH 09/28] pxe: Tidy up code style a little in pxe_utils

2021-08-18 Thread Simon Glass
There are a few more blank lines than makes sense for readability. Also free() handles a NULL pointer so drop the pointless checks. Signed-off-by: Simon Glass --- boot/pxe_utils.c | 66 ++-- 1 file changed, 13 insertions(+), 53 deletions(-) diff

[PATCH 07/28] pxe: Move pxe_utils files

2021-08-18 Thread Simon Glass
Move the header file into the main include/ directory so we can use it from the bootmethod code. Move the C file into boot/ since it relates to booting. Signed-off-by: Simon Glass --- boot/Makefile| 3 +++ {cmd => boot}/pxe_utils.c| 0 cmd/Makefile | 4 ++--

[PATCH 10/28] pxe: Move common parsing coding into pxe_util

2021-08-18 Thread Simon Glass
Both the syslinux and pxe commands use essentially the same code to parse and run extlinux.conf files. Move this into a common function. Signed-off-by: Simon Glass --- boot/pxe_utils.c| 20 cmd/pxe.c | 15 --- cmd/sysboot.c | 18

[PATCH 08/28] pxe: Tidy up some comments in pxe_utils

2021-08-18 Thread Simon Glass
Some of these functions are a big vague in the comments. Tidy them up a bit. Signed-off-by: Simon Glass --- boot/pxe_utils.c | 189 ++- 1 file changed, 138 insertions(+), 51 deletions(-) diff --git a/boot/pxe_utils.c b/boot/pxe_utils.c index

[PATCH 06/28] pxe: Tidy up the is_pxe global

2021-08-18 Thread Simon Glass
Move this into the context to avoid a global variable. Also rename it since the current name does not explain what it actually affects. Signed-off-by: Simon Glass --- cmd/pxe.c | 6 ++ cmd/pxe_utils.c | 13 +++-- cmd/pxe_utils.h | 8 +--- cmd/sysboot.c | 4 +--- 4

[PATCH 05/28] pxe: Add a userdata field to the context

2021-08-18 Thread Simon Glass
Allow the caller to provide some info which is passed back to the readfile() method. Signed-off-by: Simon Glass --- cmd/pxe.c | 4 ++-- cmd/pxe_utils.c | 3 ++- cmd/pxe_utils.h | 6 +- cmd/sysboot.c | 2 +- 4 files changed, 10 insertions(+), 5 deletions(-) diff --git a/cmd/pxe.c

[PATCH 04/28] pxe: Move do_getfile() into the context

2021-08-18 Thread Simon Glass
Rather than having a global variable, pass the function as part of the context. Signed-off-by: Simon Glass --- cmd/pxe.c | 10 -- cmd/pxe_utils.c | 9 - cmd/pxe_utils.h | 20 +--- cmd/sysboot.c | 20 ++-- 4 files changed, 35

[PATCH 03/28] pxe: Use a context pointer

2021-08-18 Thread Simon Glass
At present the PXE functions pass around a pointer to command-table entry which is very strange. It is only needed in a few places and it is odd to pass around a data structure from another module in this way. For bootmethod we will need to provide some context information when reading files.

[PATCH 02/28] pxe: Move API comments to the header files

2021-08-18 Thread Simon Glass
Put the function comments in the header file so that the full API can we examined in one place. Expand the comments to cover parameters and return values. Signed-off-by: Simon Glass --- cmd/pxe_utils.c | 45 - cmd/pxe_utils.h | 77

[PATCH 01/28] Create a new boot/ directory

2021-08-18 Thread Simon Glass
Quite a lot of the code in common/relates to booting and images. Before adding more it seems like a good time to move the code into its own directory. Most files with 'boot' or 'image' in them are moved, except: - autoboot.c which relates to U-Boot automatically running a script - bootstage.c

[PATCH 00/28] Initial implementation of bootmethod/bootflow

2021-08-18 Thread Simon Glass
Bootmethod and bootflow provide a built-in way for U-Boot to automatically boot an Operating System without custom scripting and other customisation: - bootmethod - a method to scan a device to find bootflows (owned by U-Boot) - bootflow - a description of how to boot (owned by the distro)

[PATCH 11/11] test: Add a way to skip console checking until a string matches

2021-08-18 Thread Simon Glass
Some tests produce a lot of output that does not need to be individually checked by an assertion. Add a macro to handle this. Signed-off-by: Simon Glass --- include/test/ut.h | 24 test/ut.c | 26 ++ 2 files changed, 50 insertions(+)

[PATCH 09/11] sandbox: Add a way to map a file into memory

2021-08-18 Thread Simon Glass
It is useful to map a file into memory so that it can be accessed using simple pointers. Add a function to support this. Signed-off-by: Simon Glass --- arch/sandbox/cpu/os.c | 29 + include/os.h | 13 + 2 files changed, 42 insertions(+) diff

[PATCH 06/11] test: Tidy a comment in the bloblist test

2021-08-18 Thread Simon Glass
Fix up a copy error. Signed-off-by: Simon Glass --- test/bloblist.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/bloblist.c b/test/bloblist.c index 4104e6a92f6..b48be38dc3e 100644 --- a/test/bloblist.c +++ b/test/bloblist.c @@ -14,7 +14,7 @@

[PATCH 08/11] sandbox: Add a way to find the size of a file

2021-08-18 Thread Simon Glass
Add a function to return the size of a file. This is useful in situations where we need to allocate memory for it before reading it. Signed-off-by: Simon Glass --- arch/sandbox/cpu/os.c | 22 -- include/os.h | 8 2 files changed, 24 insertions(+), 6

[PATCH 10/11] sandbox: mmc: Support a backing file

2021-08-18 Thread Simon Glass
Provide a way for sandbox MMC to present data from a backing file. This allows a filesystem to be created on the host and easily served via an emulated mmc device. Signed-off-by: Simon Glass --- doc/device-tree-bindings/mmc/sandbox,mmc.txt | 18 drivers/mmc/sandbox_mmc.c

[PATCH 07/11] dm: core: Fix a few incorrect comments on first/next functions

2021-08-18 Thread Simon Glass
Tighten up these comments to make the behaviour clearer. Signed-off-by: Simon Glass --- include/dm/device.h | 4 ++-- include/dm/uclass.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/include/dm/device.h b/include/dm/device.h index 0a9718a5b81..ef6241bca8b 100644 ---

[PATCH 05/11] sqfs: Suppress the message about missing filesystem

2021-08-18 Thread Simon Glass
This message comes up a lot when scanning filesystems. It suggests to the user that there is some sort of error, but in fact there is no reason to expect that a particular partition has a sqfs filesystem. Other filesystems don't print this error. Turn it into a debug message. Signed-off-by:

[PATCH 03/11] sandbox: Add license headers to the dts files

2021-08-18 Thread Simon Glass
Many of these files are missing a header. Fix this. Signed-off-by: Simon Glass --- arch/sandbox/dts/overlay0.dts | 5 + arch/sandbox/dts/overlay1.dts | 5 + arch/sandbox/dts/sandbox.dts | 5 + arch/sandbox/dts/sandbox.dtsi | 1 + arch/sandbox/dts/sandbox64.dts | 4

[PATCH 04/11] btrfs: Suppress the message about missing filesystem

2021-08-18 Thread Simon Glass
This message comes up a lot when scanning filesystems. It suggests to the user that there is some sort of error, but in fact there is no reason to expect that a particular partition has a btrfs filesystem. Other filesystems don't print this error. Turn it into a debug message. Signed-off-by:

[PATCH 02/11] sandbox: Correct handling of --rm_memory

2021-08-18 Thread Simon Glass
This option has no argument so we should not trip to skip one. Fix it. Signed-off-by: Simon Glass --- arch/sandbox/cpu/os.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/sandbox/cpu/os.c b/arch/sandbox/cpu/os.c index 1103530941b..151f42a5d6c 100644 --- a/arch/sandbox/cpu/os.c +++

[PATCH 01/11] dtoc: Further improve documentation about warnings

2021-08-18 Thread Simon Glass
Split this information into subsections and expand it. Signed-off-by: Simon Glass --- doc/develop/driver-model/of-plat.rst | 92 ++-- 1 file changed, 88 insertions(+), 4 deletions(-) diff --git a/doc/develop/driver-model/of-plat.rst

[PATCH 00/11] sandbox: Minor fixes and improvements

2021-08-18 Thread Simon Glass
This series collects a few patches that correct and improve sandbox and sandbox_spl: - Fix a bug in --rm-memory works - Support an MMC backing file for MMC emulator, to allow filesystems to be used without needing the 'host' filesystem - Allow skipping console output when checking verbose

[PATCH 14/16] Convert CONFIG_I2C_SET_DEFAULT_BUS_NUM et al to Kconfig

2021-08-18 Thread Tom Rini
This converts the following to Kconfig: CONFIG_I2C_SET_DEFAULT_BUS_NUM CONFIG_I2C_DEFAULT_BUS_NUMBER Signed-off-by: Tom Rini --- configs/MPC8548CDS_36BIT_defconfig | 1 + configs/MPC8548CDS_defconfig| 1 + configs/MPC8548CDS_legacy_defconfig

[PATCH 12/16] i2c: fsl_i2c: Migrate to Kconfig

2021-08-18 Thread Tom Rini
- As there are no boards that use different values for speed / slave on different buses, use a single option. - Switch to using the common SYS_I2C_SPEED / SYS_I2C_SLAVE options. - Introduce _HAS_ options for additional buses as only the first one is common to all users. - Convert all remaining

[PATCH 09/16] Convert CONFIG_SYS_I2C_SPEED et al to Kconfig

2021-08-18 Thread Tom Rini
This converts the following to Kconfig: CONFIG_SYS_I2C_SPEED CONFIG_SYS_I2C_SLAVE Signed-off-by: Tom Rini --- arch/arm/mach-kirkwood/include/mach/config.h | 8 - arch/arm/mach-mvebu/include/mach/config.h| 2 -- board/compulab/imx8mm-cl-iot-gate/spl.c | 2 +- cmd/eeprom.c

[PATCH 16/16] varisys: Remove dead code

2021-08-18 Thread Tom Rini
The platforms this code was for have been removed, drop this support code as well. Signed-off-by: Tom Rini --- board/varisys/common/Makefile | 21 -- board/varisys/common/eeprom.h | 6 - board/varisys/common/sys_eeprom.c | 500 -- 3 files changed, 527

[PATCH 15/16] README: Remove some old i2c related text

2021-08-18 Thread Tom Rini
These sections are no longer relevant, remove. Signed-off-by: Tom Rini --- README | 13 - 1 file changed, 13 deletions(-) diff --git a/README b/README index fea0d3c0d0f9..495c1ae3549e 100644 --- a/README +++ b/README @@ -1462,19 +1462,6 @@ The following options need to be

[PATCH 13/16] tegra: Cleanup I2C support

2021-08-18 Thread Tom Rini
Remove the irrelevant section from the README and remove the non-functional SPL-undef section. Signed-off-by: Tom Rini --- README | 5 - include/configs/tegra-common-post.h | 5 - 2 files changed, 10 deletions(-) diff --git a/README b/README index

[PATCH 11/16] rcar: i2c: Migrate SYS_I2C_SH to Kconfig

2021-08-18 Thread Tom Rini
- Migrate SYS_I2C_SH and related defines to Kconfig - Remove currently unused SYS_I2C_SH related defines - Cleanup related README section. Cc: Marek Vasut Signed-off-by: Tom Rini --- README| 20 arch/arm/mach-rmobile/include/mach/r8a7790.h | 4

[PATCH 08/16] cmd/i2c.c: Remove unused legacy code

2021-08-18 Thread Tom Rini
There are no longer any cases where we are neither SYS_I2C_LEGACY nor DM_I2C. Remove these code paths. Signed-off-by: Tom Rini --- cmd/i2c.c | 50 -- 1 file changed, 50 deletions(-) diff --git a/cmd/i2c.c b/cmd/i2c.c index

[PATCH 06/16] ti: i2c: Convert CONFIG_SYS_OMAP24_I2C to CONFIG_SYS_I2C namespace

2021-08-18 Thread Tom Rini
The omap24xx I2C driver uses its own CONFIG namespace for common I2C variables. Rather than convert more of them to Kconfig, rename these to the common I2C ones and remove the entirely unused functionality. As part of this, we make the am335x_shc platforms consistent with their intended speed

Re: [PATCH 3/3] rockchip: px30: Change sfc node to match upstream linux proposed

2021-08-18 Thread Chris Morgan
On Tue, Aug 17, 2021 at 09:41:01AM +0800, Jon Lin wrote: > TX single line is more compatible for corresponding board > > Signed-off-by: Jon Lin > --- > There are still some bugs with this. Note that if you want for now you can just abandon this patch (or fix it, either way is fine). If you

Re: [PATCH 1/3] spi: rockchip_sfc: Impletment set_speed logic

2021-08-18 Thread Chris Morgan
On Tue, Aug 17, 2021 at 09:40:59AM +0800, Jon Lin wrote: > Set clock related processing into set_speed logic. And Optimize > printing format. > > Signed-off-by: Jon Lin > --- I tested this one and it tests out okay. I figured out the issue that was causing the performance regression. For the

[PATCH 07/16] i2c: Remove non-DM code from ihs_i2c.c

2021-08-18 Thread Tom Rini
This driver and it's only user are converted to DM_I2C, remove legacy code. Signed-off-by: Tom Rini --- README| 24 - drivers/i2c/ihs_i2c.c | 228 -- 2 files changed, 252 deletions(-) diff --git a/README b/README index

[PATCH 04/16] am335x: Drop non-DM_I2C code

2021-08-18 Thread Tom Rini
On this platform, we have DM_I2C and SPL_DM_I2C always enabled. Remove legacy options. Signed-off-by: Tom Rini --- board/ti/am335x/board.c | 18 -- board/ti/am335x/mux.c | 9 - 2 files changed, 27 deletions(-) diff --git a/board/ti/am335x/board.c

[PATCH 05/16] am43xx: Drop non-DM_I2C code

2021-08-18 Thread Tom Rini
On this platform, we have DM_I2C and SPL_DM_I2C always enabled. Remove legacy options. Cc: Lokesh Vutla Signed-off-by: Tom Rini --- board/ti/am43xx/board.c | 27 --- 1 file changed, 27 deletions(-) diff --git a/board/ti/am43xx/board.c b/board/ti/am43xx/board.c index

[PATCH 02/16] Convert CONFIG_SYS_I2C_EARLY_INIT to Kconfig

2021-08-18 Thread Tom Rini
Convert SYS_I2C_EARLY_INIT to Kconfig, and make it depend on SPL_SYS_I2C_LEGACY. Remove the weak implementation as it's either something that needs to exist for real, or shouldn't be called. Signed-off-by: Tom Rini --- arch/arm/include/asm/arch-fsl-layerscape/config.h | 3 ---

[PATCH 03/16] i2c: designware: Remove unused non-DM functionality

2021-08-18 Thread Tom Rini
There are no users of more than 1 i2c bus in the non-DM case currently. Remove the additional defines for this. Signed-off-by: Tom Rini --- drivers/i2c/designware_i2c.c | 18 -- 1 file changed, 18 deletions(-) diff --git a/drivers/i2c/designware_i2c.c

[PATCH 00/16] Further I2C Kconfig migrations

2021-08-18 Thread Tom Rini
Continuing on top of the last series I posted, this migrates still more symbols and updates the README a bit. There's still more that could be done, but we're close to the end of what's only slightly painful to migrate. I think a small 3rd round, on top of the forthcoming sunxi related updates

[PATCH v2 2/2] spi: rockchip_sfc: Using read_poll

2021-08-18 Thread Jon Lin
Using read_poll logic. Tested-by: Chris Morgan Signed-off-by: Jon Lin --- Changes in v2: - Fix assigned but never used return error codes drivers/spi/rockchip_sfc.c | 67 -- 1 file changed, 35 insertions(+), 32 deletions(-) diff --git

[PATCH v2 1/2] spi: rockchip_sfc: Impletment set_speed logic

2021-08-18 Thread Jon Lin
Set clock related processing into set_speed logic. And Optimize printing format. Tested-by: Chris Morgan Signed-off-by: Jon Lin --- (no changes since v1) drivers/spi/rockchip_sfc.c | 83 ++ 1 file changed, 40 insertions(+), 43 deletions(-) diff --git

Re: [PATCH 3/3] rockchip: px30: Change sfc node to match upstream linux proposed

2021-08-18 Thread Jon Lin
On 2021/8/19 1:13, Chris Morgan wrote: On Tue, Aug 17, 2021 at 09:41:01AM +0800, Jon Lin wrote: TX single line is more compatible for corresponding board Signed-off-by: Jon Lin --- There are still some bugs with this. Note that if you want for now you can just abandon this patch (or fix

[PATCH 2/2] spi: rockchip_sfc: Using read_poll

2021-08-18 Thread Jon Lin
Using read_poll logic. Tested-by: Chris Morgan Signed-off-by: Jon Lin --- drivers/spi/rockchip_sfc.c | 67 -- 1 file changed, 35 insertions(+), 32 deletions(-) diff --git a/drivers/spi/rockchip_sfc.c b/drivers/spi/rockchip_sfc.c index

[PATCH 1/2] spi: rockchip_sfc: Impletment set_speed logic

2021-08-18 Thread Jon Lin
Set clock related processing into set_speed logic. And Optimize printing format. Tested-by: Chris Morgan Signed-off-by: Jon Lin --- drivers/spi/rockchip_sfc.c | 83 ++ 1 file changed, 40 insertions(+), 43 deletions(-) diff --git

RISC-V MC at Plumbers

2021-08-18 Thread Palmer Dabbelt
Looks like our first attempt at sending a message bounced, so this never got properly announced. We've got Microconference at Plumbers again this year, the CFP is still open.

Re: [PATCH 1/2] imx8mm-cl-iot-gate: Do not build fip.bin by default

2021-08-18 Thread Fabio Estevam
[Adding Marek] On Wed, Aug 18, 2021 at 6:39 PM Fabio Estevam wrote: > > Hi Paul, > > On Wed, Aug 18, 2021 at 6:32 PM Paul Liu wrote: > > > > Hi Fabio, > > > > I got several boards. With all different PN. But all of them are 2GB > > memory. And the recent master doesn't boot on one of my board.

Re: [PATCH 2/5] board: gateworks: venice: do not overwrite serial#

2021-08-18 Thread Tim Harvey
On Wed, Aug 18, 2021 at 3:07 PM Fabio Estevam wrote: > > Hi Tim, > > On Wed, Aug 18, 2021 at 7:03 PM Tim Harvey wrote: > > > Ooops... the error starts with '#' so was not put in the commit. > > Should I send just that one patch as a v2 and leave the rest alone? > > It is preferred if you could

[PATCH] image: rsa: Move padding_algos to linker lists

2021-08-18 Thread Alexandru Gagniuc
We are not guaranteed to have the padding_pkcs_15_verify symbol since commit 92c960bc1d ("lib: rsa: Remove #ifdefs from rsa.h"), and commit 61416fe9df ("Kconfig: FIT_SIGNATURE should not select RSA_VERIFY") The padding_algos only make sense with RSA verification, which can now be disabled in lieu

[PATCH v2 3/5] arm: dts: imx8mm-venice-gw700x: fix mp5416 pmic config

2021-08-18 Thread Tim Harvey
Fix various MP5416 PMIC configurations: - Update regulator names per dt-bindings - ensure values fit among valid register values - add required regulator-max-microamp property - add regulator-always-on prop Signed-off-by: Tim Harvey --- v2: no changes ---

[PATCH v2 4/5] board: gateworks: venice: update thermal temp thresholds per cpu grade

2021-08-18 Thread Tim Harvey
Update the passive/critical thermal zone dt config per CPU temperature grade. Signed-off-by: Tim Harvey --- v2: no changes --- board/gateworks/venice/imx8mm_venice.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/board/gateworks/venice/imx8mm_venice.c

[PATCH v2 5/5] arm: dts: imx8mm-venice*: remove thermal zone overrides

2021-08-18 Thread Tim Harvey
Remove the unnecessary thermal zone overrides. Signed-off-by: Tim Harvey --- v2: no changes --- arch/arm/dts/imx8mm-venice-gw7901.dts | 12 arch/arm/dts/imx8mm-venice-gw7902.dts | 12 2 files changed, 24 deletions(-) diff --git a/arch/arm/dts/imx8mm-venice-gw7901.dts

[PATCH v2 2/5] board: gateworks: venice: do not overwrite serial#

2021-08-18 Thread Tim Harvey
Do not overwrite existing serial# env to avoid: ## Error: Can't overwrite "serial#" ## Error inserting "serial#" variable, errno=1 Signed-off-by: Tim Harvey --- v2: added missing error to commit log --- board/gateworks/venice/imx8mm_venice.c | 3 ++- 1 file changed, 2 insertions(+), 1

[PATCH v2 1/5] board: gateworks: venice: display hwmon details by default

2021-08-18 Thread Tim Harvey
Display hwmon values by default when using the 'gsc' command. Signed-off-by: Tim Harvey --- v2: no changes --- board/gateworks/venice/gsc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/board/gateworks/venice/gsc.c b/board/gateworks/venice/gsc.c index 7d6acd7b4a..065d1fb8cc 100644 ---

Re: [PATCH 2/5] board: gateworks: venice: do not overwrite serial#

2021-08-18 Thread Fabio Estevam
Hi Tim, On Wed, Aug 18, 2021 at 7:03 PM Tim Harvey wrote: > Ooops... the error starts with '#' so was not put in the commit. > Should I send just that one patch as a v2 and leave the rest alone? It is preferred if you could just re-send the whole series as v2. It makes Stefano's life easier

Re: [PATCH 2/5] board: gateworks: venice: do not overwrite serial#

2021-08-18 Thread Tim Harvey
On Wed, Aug 18, 2021 at 2:54 PM Fabio Estevam wrote: > > Hi Tim, > > On Wed, Aug 18, 2021 at 6:51 PM Tim Harvey wrote: > > > > Do not overwrite existing serial# env to avoid: > > Missed a sentence here? Ooops... the error starts with '#' so was not put in the commit. Should I send just that one

Re: [PATCH 2/5] board: gateworks: venice: do not overwrite serial#

2021-08-18 Thread Fabio Estevam
Hi Tim, On Wed, Aug 18, 2021 at 6:51 PM Tim Harvey wrote: > > Do not overwrite existing serial# env to avoid: Missed a sentence here?

[PATCH 4/5] board: gateworks: venice: update thermal temp thresholds per cpu grade

2021-08-18 Thread Tim Harvey
Update the passive/critical thermal zone dt config per CPU temperature grade. Signed-off-by: Tim Harvey --- board/gateworks/venice/imx8mm_venice.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/board/gateworks/venice/imx8mm_venice.c

[PATCH 5/5] arm: dts: imx8mm-venice*: remove thermal zone overrides

2021-08-18 Thread Tim Harvey
Remove the unnecessary thermal zone overrides. Signed-off-by: Tim Harvey --- arch/arm/dts/imx8mm-venice-gw7901.dts | 12 arch/arm/dts/imx8mm-venice-gw7902.dts | 12 2 files changed, 24 deletions(-) diff --git a/arch/arm/dts/imx8mm-venice-gw7901.dts

[PATCH 2/5] board: gateworks: venice: do not overwrite serial#

2021-08-18 Thread Tim Harvey
Do not overwrite existing serial# env to avoid: Signed-off-by: Tim Harvey --- board/gateworks/venice/imx8mm_venice.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/board/gateworks/venice/imx8mm_venice.c b/board/gateworks/venice/imx8mm_venice.c index

[PATCH 3/5] arm: dts: imx8mm-venice-gw700x: fix mp5416 pmic config

2021-08-18 Thread Tim Harvey
Fix various MP5416 PMIC configurations: - Update regulator names per dt-bindings - ensure values fit among valid register values - add required regulator-max-microamp property - add regulator-always-on prop Signed-off-by: Tim Harvey --- arch/arm/dts/imx8mm-venice-gw700x.dtsi | 56

[PATCH 1/5] board: gateworks: venice: display hwmon details by default

2021-08-18 Thread Tim Harvey
Display hwmon values by default when using the 'gsc' command. Signed-off-by: Tim Harvey --- board/gateworks/venice/gsc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/board/gateworks/venice/gsc.c b/board/gateworks/venice/gsc.c index 7d6acd7b4a..065d1fb8cc 100644 ---

Re: [PATCH 1/2] imx8mm-cl-iot-gate: Do not build fip.bin by default

2021-08-18 Thread Fabio Estevam
Hi Paul, On Wed, Aug 18, 2021 at 6:32 PM Paul Liu wrote: > > Hi Fabio, > > I got several boards. With all different PN. But all of them are 2GB memory. > And the recent master doesn't boot on one of my board. I haven't tried all of > the combinations. With the U-Boot from Compulab, it reports

Re: [PATCH 1/2] imx8mm-cl-iot-gate: Do not build fip.bin by default

2021-08-18 Thread Paul Liu
Hi Fabio, I got several boards. With all different PN. But all of them are 2GB memory. And the recent master doesn't boot on one of my board. I haven't tried all of the combinations. After bisect, I found commit e27bddff breaks the boot. It just hang there. But if I change PHYS_SDRAM_SIZE to 1GB

Re: [PATCH v2] imx: spl: fix imx8m secure boot

2021-08-18 Thread Tim Harvey
On Mon, Aug 16, 2021 at 11:17 PM Heiko Schocher wrote: > > cherry-picked from NXP code: > 719d665a87c6: ("MLK-20467 imx8m: Fix issue for booting signed image through > uuu") > > which fixes secure boot on imx8m based boards. Problem was > that FIT header and so IVT header too, was loaded to >

A mea culpa undefined reference in v2021.10-rc2, padding_algos, linker lists

2021-08-18 Thread Alex G.
Hi Simon, I'm seeing an undefined reference to padding_pkcs_15_verify with v2021.10-rc2. It happens when enabling FIT_SIGNATURE. I've tracked it down to the following two commits: commit 92c960bc1d ("lib: rsa: Remove #ifdefs from rsa.h") commit 61416fe9df ("Kconfig: FIT_SIGNATURE should not

[PATCH v2] imx8mm-evk: Generate a single bootable flash.bin again

2021-08-18 Thread Fabio Estevam
After the conversion to binman in commit 8996e6b7c6a1 ("imx8mm_evk: switch to use binman to pack images"), it is necessary to flash both flash.bin and u-boot.itb to get a bootable system. Prior to this commit, only flash.bin was needed. Such new requirement breaks existing distro mechanisms to

Re: u-boot & ZFS

2021-08-18 Thread ChrisO
On Wed, Aug 18, 2021 at 09:53:06AM -0400, Tom Rini wrote: > On Wed, Aug 18, 2021 at 03:25:58PM +0200, ChrisO wrote: > > On Wed, Aug 18, 2021 at 12:26:11PM +0200, Lukasz Majewski wrote: > > > Hi ChrisO, > > > > > > > On Tue, Aug 17, 2021 at 08:35:14PM -0400, Tom Rini wrote: > > > > > On Tue, Aug

Re: [PATCH] imx8mm-evk: Generate a single bootable flash.bin again

2021-08-18 Thread Tim Harvey
On Wed, Aug 18, 2021 at 5:19 AM Fabio Estevam wrote: > > After the conversion to binman in commit 8996e6b7c6a1 ("imx8mm_evk: switch > to use binman to pack images"), it is necessary to flash both flash.bin and > u-boot.itb to get a bootable system. Prior to this commit, only flash.bin > was

Re: u-boot & ZFS

2021-08-18 Thread ChrisO
On Wed, Aug 18, 2021 at 03:12:04PM +0100, Dimitri John Ledkov wrote: > On Wed, Aug 18, 2021 at 2:26 PM ChrisO wrote: > > > > On Wed, Aug 18, 2021 at 12:26:11PM +0200, Lukasz Majewski wrote: > > > Hi ChrisO, > > > > > > > On Tue, Aug 17, 2021 at 08:35:14PM -0400, Tom Rini wrote: > > > > > On Tue,

Re: [PATCH] mkimage: use environment variable MKIMAGE_SIGN_PIN to set pin for OpenSSL Engine

2021-08-18 Thread Alex G.
Hi Marc, On 7/23/21 3:17 PM, Marc Kleine-Budde wrote: This patch adds the possibility to pass the PIN the OpenSSL Engine used during signing via the environment variable MKIMAGE_SIGN_PIN. This follows the approach used during kernel module signing ("KBUILD_SIGN_PIN") or UBIFS image signing

[PATCH v2] dm: usb: add hook for low level usb init

2021-08-18 Thread Simon-Pierre Allaire
This patch adds a hook to enable us to call a custom function when booting from USB Signed-off-by: Simon-Pierre Allaire --- Changes for v2: - Added description of the patch drivers/usb/host/usb-uclass.c | 7 ++- drivers/usb/musb-new/musb_uboot.c | 13 + 2 files

Re: [PATCH] imx8mm-evk: Generate a single bootable flash.bin again

2021-08-18 Thread Heiko Schocher
Hello Fabio, On 18.08.21 14:19, Fabio Estevam wrote: > After the conversion to binman in commit 8996e6b7c6a1 ("imx8mm_evk: switch > to use binman to pack images"), it is necessary to flash both flash.bin and > u-boot.itb to get a bootable system. Prior to this commit, only flash.bin > was needed.

RE: [EXT] Re: [PATCH 01/15] crypto/fsl: Add support for CAAM Job ring driver model

2021-08-18 Thread Gaurav Jain
Hi Simon > -Original Message- > From: Simon Glass > Sent: Tuesday, August 17, 2021 9:39 PM > To: Gaurav Jain > Cc: U-Boot Mailing List ; Franck Lenormand > ; Stefano Babic ; Fabio > Estevam ; Peng Fan ; Priyanka > Jain ; Ye Li ; Horia Geanta > ; Ji Luo ; Silvano Di Ninno > ; Sahil

Re: [PATCH 0/4] clk: at91: fix system clocks display for sam9x60

2021-08-18 Thread Eugen.Hristev
On 7/16/21 8:43 AM, Claudiu Beznea wrote: > Hi, > > After added CCF support for SAM9X60 the CPU, crystal and master clocks > were not displayed correctly. These clocks are now displayed by at91_cpu > driver. > Adapt master clock driver by adding control logic for its prescaler and > divider such

Re: [PATCH] ARM: at91: Add chip ID of SAMA5D29

2021-08-18 Thread Eugen.Hristev
On 7/14/21 4:43 PM, Hari Prasath wrote: > Add SAMA5D29 SoC for identification during the boot up. > > Signed-off-by: Hari Prasath > --- Applied to u-boot-atmel/next, thanks !

Re: [PATCH] arm: at91: gardena-smart-gateway-at91sam: Adjust to production values

2021-08-18 Thread Eugen.Hristev
On 6/17/21 7:57 PM, Reto Schneider wrote: > From: Reto Schneider > > This commit updates the default config with the values that will be used > soon on the Atmel / Microchip AT91SAM9G25 based GARDENA smart gateway. > > Signed-off-by: Reto Schneider > --- > This patch applies to current

Re: u-boot & ZFS

2021-08-18 Thread Dimitri John Ledkov
On Wed, Aug 18, 2021 at 2:26 PM ChrisO wrote: > > On Wed, Aug 18, 2021 at 12:26:11PM +0200, Lukasz Majewski wrote: > > Hi ChrisO, > > > > > On Tue, Aug 17, 2021 at 08:35:14PM -0400, Tom Rini wrote: > > > > On Tue, Aug 17, 2021 at 06:03:10PM +0200, ChrisO wrote: > > > > > > > > > Hi > > > > > > >

Re: [PATCH v2 15/16] clk: Detect failure to set defaults

2021-08-18 Thread Harm Berntsen
On Thu, 2021-05-13 at 19:39 -0600, Simon Glass wrote: > When the default clocks cannot be set, the clock is silently probed and > the error is ignored. This is incorrect, since having the clocks at the > correct speed may be important for operation of the system. > > Fix it by checking the return

  1   2   >