Re: [RFC PATCH] binman: bintool: etype: Add support for ti-secure entry

2023-03-09 Thread Neha Malcom Francis
Hi Andrew, Simon On 01/03/23 22:41, Andrew Davis wrote: On 2/28/23 9:10 AM, Simon Glass wrote: Hi Neha, On Tue, 28 Feb 2023 at 02:50, Neha Malcom Francis wrote: Hi Simon, On 28/02/23 06:05, Simon Glass wrote: Hi Neha, On Fri, 24 Feb 2023 at 05:03, Neha Malcom Francis wrote: core-sec

[GIT PULL] Please pull u-boot-mmc master

2023-03-09 Thread Jaehoon Chung
Dear Tom, Please pull u-boot-mmc master into u-boot master branch. If there is any problem, let me know, plz. Best Regards, Jaehoon Chung CI: https://source.denx.de/u-boot/custodians/u-boot-mmc/-/pipelines/15512 The following changes since commit b08ffdffdce95e267e782366f4a77bf6b5537b28:

Re: [PATCH v2 28/32] test: cmd: fdt: Test fdt memory

2023-03-09 Thread Marek Vasut
On 3/9/23 23:09, Simon Glass wrote: Hi Marek, On Wed, 8 Mar 2023 at 15:16, Marek Vasut wrote: On 3/8/23 23:23, Simon Glass wrote: Hi Marek, On Wed, 1 Mar 2023 at 19:09, Marek Vasut wrote: Add 'fdt memory' test which works as follows: - Create custom FDT with /memory node, with select #*c

[PATCH v2 1/2] test: Wrap assert macros in ({ ... }) and fix missing semicolons

2023-03-09 Thread Marek Vasut
Wrap the assert macros in ({ ... }) so they can be safely used both as right side argument as well as in conditionals without curly brackets around them. In the process, find a bunch of missing semicolons, fix them. Reviewed-by: Simon Glass Signed-off-by: Marek Vasut --- Cc: Heinrich Schuchardt

[PATCH v2 2/2] test: cmd: fdt: Drop new unneeded curly brackets

2023-03-09 Thread Marek Vasut
Drop no longer needed { } around ut_assert*() functions in FDT test. No functional change. Reviewed-by: Simon Glass Signed-off-by: Marek Vasut --- Cc: Heinrich Schuchardt Cc: Simon Glass Cc: Tom Rini --- V2: Add RB from Simon --- test/cmd/fdt.c | 14 ++ 1 file changed, 6 insertio

Re: [PATCH 3/3] mmc: mv_sdhci: Depend on DM_MMC

2023-03-09 Thread Jaehoon Chung
On 2/10/23 21:23, Stefan Roese wrote: > All build targets using this driver already use DM_MMC. So let's depend > this driver on this Kconfig symbol and remove the non-DM driver part. > > Signed-off-by: Stefan Roese > Cc: Tom Rini > Cc: Simon Glass > Cc: Peng Fan > Cc: Jaehoon Chung > Reviewe

Re: [PATCH 2/3] mmc: mv_sdhci: Remove CONFIG_MMC_SDHCI_IO_ACCESSORS support

2023-03-09 Thread Jaehoon Chung
On 2/10/23 21:23, Stefan Roese wrote: > CONFIG_MMC_SDHCI_IO_ACCESSORS is not supported and/or used by this > driver so let's remove these unused parts completely. > > Signed-off-by: Stefan Roese > Cc: Tom Rini > Cc: Simon Glass > Cc: Peng Fan > Cc: Jaehoon Chung > Reviewed-by: Jaehoon Chung

Re: [PATCH 1/3] mmc: mv_sdhci: Simplify call to sdhci_mvebu_mbus_config()

2023-03-09 Thread Jaehoon Chung
On 2/10/23 21:23, Stefan Roese wrote: > This driver already depends on CONFIG_ARCH_MVEBU, so there is no need > to have some checks for this Kconfig symbol in the driver itself. Let's > remove these superfluous checks. > > Signed-off-by: Stefan Roese > Cc: Tom Rini > Cc: Simon Glass > Cc: Peng

Re: [PATCH v2 3/3] test: dm: mmc: Check block erasing boundaries

2023-03-09 Thread Jaehoon Chung
On 1/26/23 18:24, Loic Poulain wrote: > Verify that erasing blocks does not impact adjacent ones. > - Write four blocks [0 1 2 3] > - Erase two blocks [ 1 2 ] > - Verify [0 1 2 3 ] > > Signed-off-by: Loic Poulain Applied to u-boot-mmc/master. Best Regards, Jaehoon Chung > --- > v2: Add this c

Re: [PATCH v2 2/3] mmc: erase: Use TRIM erase when available

2023-03-09 Thread Jaehoon Chung
On 1/26/23 18:24, Loic Poulain wrote: > The default erase command applies on erase group unit, and > simply round down to erase group size. When the start block > is not aligned to erase group size (e.g. erasing partition) > it causes unwanted erasing of the previous blocks, part of > the same eras

Re: [PATCH v2 1/3] mmc: Check support for TRIM operations

2023-03-09 Thread Jaehoon Chung
On 1/26/23 18:24, Loic Poulain wrote: > When secure/insecure TRIM operations are supported. > When used as erase command argument it applies the > erase operation to write blocks instead of erase > groups. > > Signed-off-by: Loic Poulain > Reviewed-by: Simon Glass > Reviewed-by: Jaehoon Chung

Re: [PATCH 2/2] spl: mmc: Pass eMMC HW partition 7 through

2023-03-09 Thread Jaehoon Chung
On 1/5/23 23:28, Marek Vasut wrote: > The eMMC HW partition 0 and 7 both mean USER HW partition. Use this as > a mean of propagating A/B copy selection within USER HW partition. The > spl_mmc_get_uboot_raw_sector() can detect that a USER HW partition is > in use and based on whether it is 0 or 7, s

Re: [PATCH 1/2] spl: mmc: Pass eMMC HW partition and SW partition to spl_mmc_get_uboot_raw_sector()

2023-03-09 Thread Jaehoon Chung
On 1/5/23 23:28, Marek Vasut wrote: > Pass both eMMC HW partition and software partition numbers to > spl_mmc_get_uboot_raw_sector() so the function can better decide > which offset within the partition to load payload from. > > Signed-off-by: Fedor Ross > Signed-off-by: Marek Vasut Applied to

Re: [PATCH v2 28/32] test: cmd: fdt: Test fdt memory

2023-03-09 Thread Simon Glass
Hi Marek, On Wed, 8 Mar 2023 at 15:16, Marek Vasut wrote: > > On 3/8/23 23:23, Simon Glass wrote: > > Hi Marek, > > > > On Wed, 1 Mar 2023 at 19:09, Marek Vasut > > wrote: > >> > >> Add 'fdt memory' test which works as follows: > >> - Create custom FDT with /memory node, with select #*cells, map

Re: [PATCH v2 02/26] cmd: nvedit: check for ENV_SUPPORT

2023-03-09 Thread Troy Kisky
On Thu, Mar 9, 2023 at 11:36 AM Tom Rini wrote: > On Thu, Mar 09, 2023 at 11:20:33AM -0800, Troy Kisky wrote: > > On Wed, Mar 1, 2023 at 7:33 AM Tom Rini wrote: > > > > > On Fri, Feb 24, 2023 at 10:10:23AM -0800, Troy Kisky wrote: > > > > Avoid error messages when SPL,TPL,VPL build don't > > > >

Re: [PATCH] README.mpc85xx-sd-spi-boot: Suggest the NXP boot format github repo

2023-03-09 Thread Pali Rohár
On Thursday 09 March 2023 08:30:29 Fabio Estevam wrote: > Hi Leo, > > On Wed, Mar 8, 2023 at 6:01 PM Leo Li wrote: > > > It mimics the branches on CodeAurora. I will ask them to update the > > default branch to master. > > The default branch is master now, thanks. > > Pali, > > Is the patch

Re: [PATCH 4/6] compulab: imx8mm-cl-iot-gate: Fix some function declarations in eeprom_spl.c

2023-03-09 Thread Tom Rini
On Fri, Mar 10, 2023 at 03:34:14AM +0800, Paul Liu wrote: > Hi Tom, > > Maybe we should fix the header by changing it to u8? > Because the eeprom only returns one byte for the DDRSUBIND value. As the board maintainer, yes, please feel free to submit a new patch to better correct the issue. I hi

Re: [PATCH v2 02/26] cmd: nvedit: check for ENV_SUPPORT

2023-03-09 Thread Tom Rini
On Thu, Mar 09, 2023 at 11:20:33AM -0800, Troy Kisky wrote: > On Wed, Mar 1, 2023 at 7:33 AM Tom Rini wrote: > > > On Fri, Feb 24, 2023 at 10:10:23AM -0800, Troy Kisky wrote: > > > Avoid error messages when SPL,TPL,VPL build don't > > > have the environment options of the main build. > > > This i

Re: [PATCH 2/2] rockchip: ringneck-px30: enforce ENV_IS_NOWHERE with Kconfig select

2023-03-09 Thread Troy Kisky
On Wed, Mar 1, 2023 at 9:39 AM Quentin Schulz wrote: > From: Quentin Schulz > > Instead of letting the compiler error out if CONFIG_ENV_IS_NOWHERE is > not selected by the user, let's just enforce it when the user builds for > Ringneck PX30 so that no check needs to be performed by the compiler

Re: [PATCH 4/6] compulab: imx8mm-cl-iot-gate: Fix some function declarations in eeprom_spl.c

2023-03-09 Thread Paul Liu
Hi Tom, Maybe we should fix the header by changing it to u8? Because the eeprom only returns one byte for the DDRSUBIND value. Yours, Paul On Fri, 10 Mar 2023 at 00:22, Tom Rini wrote: > We have a few places here we the function declarations do not match > their prototypes in another header,

Re: [PATCH v2 02/26] cmd: nvedit: check for ENV_SUPPORT

2023-03-09 Thread Troy Kisky
On Wed, Mar 1, 2023 at 7:33 AM Tom Rini wrote: > On Fri, Feb 24, 2023 at 10:10:23AM -0800, Troy Kisky wrote: > > Avoid error messages when SPL,TPL,VPL build don't > > have the environment options of the main build. > > This is needed when defined(CONFIG_ENV_IS_IN_xxx) is changed > > to CONFIG_IS_

Re: Please pull u-boot-dm into -next

2023-03-09 Thread Tom Rini
On Wed, Mar 08, 2023 at 02:26:34PM -0800, Simon Glass wrote: > Hi Tom, > > This is for the -next branch > > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15499 > > > The following changes since commit 70ed05ce6c051e55e126e67e72ab20409134c953: > > Merge tag 'next-20230307

[PATCH 6/6] RFC: arm: Enable LTO by default

2023-03-09 Thread Tom Rini
At this point, we have had wide enough testing of LTO on ARM platforms that we hope to have covered all of the issues that linking in manner exposes in terms of run-time failures. Note that on 32bit ARM we must use our private libgcc in order to ensure that we do not have a size mismatch over wchar

[PATCH 5/6] purism: librem5: Fix a function declaration in spl.c

2023-03-09 Thread Tom Rini
Here we implement usb_gadget_handle_interrupts() but did not include so did not have the declaration correct. Fix this and add the missing include. Signed-off-by: Tom Rini --- Cc: Angus Ainslie Cc: ker...@puri.sm --- board/purism/librem5/spl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deleti

[PATCH 4/6] compulab: imx8mm-cl-iot-gate: Fix some function declarations in eeprom_spl.c

2023-03-09 Thread Tom Rini
We have a few places here we the function declarations do not match their prototypes in another header, correct them. Signed-off-by: Tom Rini --- Cc: "Ying-Chun Liu (PaulLiu)" --- board/compulab/imx8mm-cl-iot-gate/eeprom_spl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --gi

[PATCH 3/6] examples: Don't use LTO for hello_world

2023-03-09 Thread Tom Rini
If we're building U-Boot with LTO, we don't want to use that for examples as it's more work than required. Signed-off-by: Tom Rini --- examples/standalone/Makefile | 4 1 file changed, 4 insertions(+) diff --git a/examples/standalone/Makefile b/examples/standalone/Makefile index 5b48a9d43c

[PATCH 2/6] spl: Add function prototype for spl_mmc_get_uboot_raw_sector

2023-03-09 Thread Tom Rini
We did not add a prototype for spl_mmc_get_uboot_raw_sector to include/spl.h before, so add and document one now. Correct the incorrect prototype in board/advantech/imx8mp_rsb3720a1/imx8mp_rsb3720a1.c and ensure that we have spl.h where we define a non-weak spl_mmc_get_uboot_raw_sector as well. Si

[PATCH 1/6] arm: Correct cpu_reset function prototype on some platforms

2023-03-09 Thread Tom Rini
Some platforms were not including which sets the prototype for reset_cpu, and in turn had it set wrong. Correct these cases. Signed-off-by: Tom Rini --- arch/arm/mach-hpe/gxp/reset.c| 3 ++- arch/arm/mach-mediatek/mt7981/init.c | 3 ++- arch/arm/mach-mediatek/mt7986/init.c |

Re: [PATCH 2/4] nvmem: core: allow nvmem_cell_post_process_t callbacks to adjust buffer

2023-03-09 Thread Rafał Miłecki
On 2023-03-09 14:10, Miquel Raynal wrote: Hello, ra...@milecki.pl wrote on Thu, 09 Mar 2023 12:52:37 +0100: On 2023-03-09 12:44, Srinivas Kandagatla wrote: > On 09/03/2023 11:23, Miquel Raynal wrote: >> Hi Srinivas, >> >> srinivas.kandaga...@linaro.org wrote on Thu, 9 Mar 2023 10:53:07 >> +000

Re: [PATCH 2/4] nvmem: core: allow nvmem_cell_post_process_t callbacks to adjust buffer

2023-03-09 Thread Rafał Miłecki
On 2023-03-09 12:44, Srinivas Kandagatla wrote: On 09/03/2023 11:23, Miquel Raynal wrote: Hi Srinivas, srinivas.kandaga...@linaro.org wrote on Thu, 9 Mar 2023 10:53:07 +: On 09/03/2023 10:32, Miquel Raynal wrote: Hi Srinivas, srinivas.kandaga...@linaro.org wrote on Thu, 9 Mar 2023 10:

Re: [PATCH 2/4] nvmem: core: allow nvmem_cell_post_process_t callbacks to adjust buffer

2023-03-09 Thread Miquel Raynal
Hello, ra...@milecki.pl wrote on Thu, 09 Mar 2023 12:52:37 +0100: > On 2023-03-09 12:44, Srinivas Kandagatla wrote: > > On 09/03/2023 11:23, Miquel Raynal wrote: > >> Hi Srinivas, > >> >> srinivas.kandaga...@linaro.org wrote on Thu, 9 Mar 2023 10:53:07 >> > >> >> +: > >> >>> On 09/03/2

Re: [PATCH v2 0/5] Support the required clocks to enable USB on SAM9X60

2023-03-09 Thread Eugen Hristev
On 3/8/23 16:39, Claudiu Beznea wrote: This patch series originates from a bigger patch series: https://lists.denx.de/pipermail/u-boot/2022-December/502865.html A driver for clock operations on SAM9X60's USB clock has been added as well as its registration on CCF. In order for USB to work proper

[PATCH v2 3/3] configs: rockchip: rock5b-rk3588: enable USB and regulators

2023-03-09 Thread Eugen Hristev
Enable USB command, USB drivers, PHY and regulators, for USB host operations. Signed-off-by: Eugen Hristev --- configs/rock5b-rk3588_defconfig | 18 +- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/configs/rock5b-rk3588_defconfig b/configs/rock5b-rk3588_defconfig

[PATCH v2 2/3] phy: rockchip-inno-usb2: add initial support for rk3588 PHY

2023-03-09 Thread Eugen Hristev
Add initial support for the rk3588 PHY variant. The driver now looks for phy-supply and enables/disables the vbus accordingly. The lookup for the host-port reg inside the struct now does a do {} while() instead of a while() {} in order to allow a first check for reg == 0. Co-developed-by: Frank Wa

[PATCH v2 1/3] ARM: dts: rockchip: rk3588-rock-5b-u-boot: add USB 2.0 host

2023-03-09 Thread Eugen Hristev
Add USB 2.0 host nodes and PHYs. Co-developed-by: William Wu Signed-off-by: William Wu Signed-off-by: Eugen Hristev --- arch/arm/dts/rk3588-rock-5b-u-boot.dtsi | 169 1 file changed, 169 insertions(+) diff --git a/arch/arm/dts/rk3588-rock-5b-u-boot.dtsi b/arch/arm/dt

Re: [PATCH 2/2] efi: remove error in efi_disk_remove

2023-03-09 Thread Heinrich Schuchardt
On 3/9/23 11:59, Patrick DELAUNAY wrote: On 3/9/23 09:54, Heinrich Schuchardt wrote: On 3/8/23 14:26, Patrick Delaunay wrote: EFI has no reason to block the driver remove when the associated EFI resources failed to be released. This patch avoids DM issue when an EFI resource can't be released

Re: [PATCH 2/4] nvmem: core: allow nvmem_cell_post_process_t callbacks to adjust buffer

2023-03-09 Thread Srinivas Kandagatla
On 09/03/2023 11:23, Miquel Raynal wrote: Hi Srinivas, srinivas.kandaga...@linaro.org wrote on Thu, 9 Mar 2023 10:53:07 +: On 09/03/2023 10:32, Miquel Raynal wrote: Hi Srinivas, srinivas.kandaga...@linaro.org wrote on Thu, 9 Mar 2023 10:12:24 +: On 22/02/2023 17:22, Rafał Mił

Re: [PATCH] README.mpc85xx-sd-spi-boot: Suggest the NXP boot format github repo

2023-03-09 Thread Fabio Estevam
Hi Leo, On Wed, Mar 8, 2023 at 6:01 PM Leo Li wrote: > It mimics the branches on CodeAurora. I will ask them to update the default > branch to master. The default branch is master now, thanks. Pali, Is the patch good now?

Re: [PATCH 2/4] nvmem: core: allow nvmem_cell_post_process_t callbacks to adjust buffer

2023-03-09 Thread Miquel Raynal
Hi Srinivas, srinivas.kandaga...@linaro.org wrote on Thu, 9 Mar 2023 10:53:07 +: > On 09/03/2023 10:32, Miquel Raynal wrote: > > Hi Srinivas, > > > > srinivas.kandaga...@linaro.org wrote on Thu, 9 Mar 2023 10:12:24 +: > > > >> On 22/02/2023 17:22, Rafał Miłecki wrote: > >>> @@ -1791

Re: [PATCH 2/2] efi: remove error in efi_disk_remove

2023-03-09 Thread Patrick DELAUNAY
On 3/9/23 09:54, Heinrich Schuchardt wrote: On 3/8/23 14:26, Patrick Delaunay wrote: EFI has no reason to block the driver remove when the associated EFI resources failed to be released. This patch avoids DM issue when an EFI resource can't be released, for example if this resource wasn't cre

Re: [PATCH 1/2] efi: remove error in efi_disk_probe

2023-03-09 Thread Patrick DELAUNAY
Hi, On 3/9/23 09:57, Heinrich Schuchardt wrote: On 3/8/23 14:26, Patrick Delaunay wrote: EFI has no reason to block the dm core device_probe() in the callback efi_disk_probe() registered with EVT_DM_POST_PROBE. This patch avoids to have error in DM core on device_probe()    ret = device_notif

Re: [PATCH 2/4] nvmem: core: allow nvmem_cell_post_process_t callbacks to adjust buffer

2023-03-09 Thread Srinivas Kandagatla
On 09/03/2023 10:32, Miquel Raynal wrote: Hi Srinivas, srinivas.kandaga...@linaro.org wrote on Thu, 9 Mar 2023 10:12:24 +: On 22/02/2023 17:22, Rafał Miłecki wrote: @@ -1791,11 +1792,15 @@ ssize_t nvmem_device_cell_read(struct nvmem_device *nvmem, if (!nvmem) r

Re: [PATCH 2/3] phy: rockchip-inno-usb2: add initial support for rk3588 PHY

2023-03-09 Thread Eugen Hristev
On 3/9/23 11:11, Xavier Drudis Ferran wrote: El Wed, Mar 08, 2023 at 01:59:54PM +0200, Eugen Hristev deia: On 3/8/23 13:30, Xavier Drudis Ferran wrote: El Fri, Mar 03, 2023 at 09:31:33AM +0200, Eugen Hristev deia: @@ -105,6 +130,17 @@ static int rockchip_usb2phy_power_off(struct phy *phy)

Re: [PATCH 2/4] nvmem: core: allow nvmem_cell_post_process_t callbacks to adjust buffer

2023-03-09 Thread Miquel Raynal
Hi Srinivas, srinivas.kandaga...@linaro.org wrote on Thu, 9 Mar 2023 10:12:24 +: > On 22/02/2023 17:22, Rafał Miłecki wrote: > > @@ -1791,11 +1792,15 @@ ssize_t nvmem_device_cell_read(struct nvmem_device > > *nvmem, > > if (!nvmem) > > return -EINVAL; > > > + /* Cel

Re: [PATCH 2/4] nvmem: core: allow nvmem_cell_post_process_t callbacks to adjust buffer

2023-03-09 Thread Srinivas Kandagatla
On 22/02/2023 17:22, Rafał Miłecki wrote: @@ -1791,11 +1792,15 @@ ssize_t nvmem_device_cell_read(struct nvmem_device *nvmem, if (!nvmem) return -EINVAL; + /* Cells with read_post_process hook may realloc buffer we can't allow here */ + if (info->read_post_pro

Re: [PATCH 1/1] lib/vsprintf.c: fix integer overflow in vsprintf

2023-03-09 Thread Rasmus Villemoes
On 09/03/2023 03.12, Ying-Chun Liu (PaulLiu) wrote: > From: Tom Cherry > > vsnprintf_internal() adds 'size' to 'buf' and vsprintf() sets 'size' > to 'INT_MAX' which can overflow. Yes, and? vsprintf_internal then detects that by looking at whether "end" is now before "buf", and if so corrects it

Re: [PATCH 2/3] phy: rockchip-inno-usb2: add initial support for rk3588 PHY

2023-03-09 Thread Xavier Drudis Ferran
El Wed, Mar 08, 2023 at 01:59:54PM +0200, Eugen Hristev deia: > On 3/8/23 13:30, Xavier Drudis Ferran wrote: > > El Fri, Mar 03, 2023 at 09:31:33AM +0200, Eugen Hristev deia: > > > @@ -105,6 +130,17 @@ static int rockchip_usb2phy_power_off(struct phy > > > *phy) > > > struct udevice *par

Re: [PATCH 1/2] efi: remove error in efi_disk_probe

2023-03-09 Thread Heinrich Schuchardt
On 3/8/23 14:26, Patrick Delaunay wrote: EFI has no reason to block the dm core device_probe() in the callback efi_disk_probe() registered with EVT_DM_POST_PROBE. This patch avoids to have error in DM core on device_probe() ret = device_notify(dev, EVT_DM_POST_PROBE); only because EFI is no

Re: [PATCH 2/2] efi: remove error in efi_disk_remove

2023-03-09 Thread Heinrich Schuchardt
On 3/8/23 14:26, Patrick Delaunay wrote: EFI has no reason to block the driver remove when the associated EFI resources failed to be released. This patch avoids DM issue when an EFI resource can't be released, for example if this resource wasn't created, for duplicated device name (error EFI_ALR