[PATCH V4 resend] spl: imx8mm: enlarge SPL_MAX_SIZE

2022-07-11 Thread Peng Fan (OSS)
From: Peng Fan The CONFIG_SPL_MAX_SIZE could be 0x27000 for i.MX8MM when SPL_TEXT_BASE set to 0x7E1000. The DDR firmware max uses 96KB, there is a 4KB padding header before SPL_TEXT_BASE, so the SPL MAX SIZE is `256KB - 96KB - 4KB`. Signed-off-by: Peng Fan --- V4 resend: Hi Stefano, this

[PATCH 0/4] test: uncover NULL dereferences in logging

2022-07-11 Thread Heinrich Schuchardt
Setting CONFIG_LOG_MAX_LEVEL=9 in sandbox_defconfig exposed to NULL dereferences and bugs in the log tests. log_debug() statements are only executed for CONFIG_LOG_MAX_LEVEL > 6. We should have a defconfig that exposes problems in these statements. Heinrich Schuchardt (4): dm: avoid NULL

[PATCH 1/4] dm: avoid NULL dereference in add_item()

2022-07-11 Thread Heinrich Schuchardt
acpi_add_other_item() passes dev = NULL. Instead of dev->name write the string "other" to the debug log: ACPI: Writing ACPI tables at 1fd3000 0base: writing table '' * other: Added type 3, 11fd4000, size 240 1facs: writing table 'FACS' * other: Added type 3,

[PATCH 2/4] test: fix log tests

2022-07-11 Thread Heinrich Schuchardt
Consider CONFIG_LOG_MAX_LEVEL and gd->default_log_level in * do_log_test_helpers() * log_test_dropped() * log_test_level_deny() Signed-off-by: Heinrich Schuchardt --- test/log/log_test.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/test/log/log_test.c

[PATCH 3/4] dm: avoid NULL dereference in lists_bind_fdt()

2022-07-11 Thread Heinrich Schuchardt
If parameter drv of lists_bind_fdt() is specified, we want to bind to this specific driver even if its field of_match is NULL. If entry->of_match is NULL, we should not dereference it in a debug statement. Fixes: d3e773613b6d ("dm: core: Use U-Boot logging instead of pr_debug()") Signed-off-by:

[PATCH 4/4] configs: sandbox_defconfig: CONFIG_LOG_MAX_LEVEL=9

2022-07-11 Thread Heinrich Schuchardt
Without setting CONFIG_LOG_MAX_LEVEL to a value above 6 we will not detect NULL dereferences and other errors in log_debug() calls. Signed-off-by: Heinrich Schuchardt --- Resent --- configs/sandbox_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/sandbox_defconfig

[PATCH 1/1] dm: fix logic of lists_bind_fdt()

2022-07-11 Thread Heinrich Schuchardt
If parameter drv of lists_bind_fdt() is specified, we shall bind only to this very driver and to no other. If the driver drv has an of_match property, we shall only bind to the driver if it matches the compatible string of the device. Signed-off-by: Heinrich Schuchardt --- drivers/core/lists.c

Re: [PATCH] arm: riscv: Remove additional ifdef from code guarded by CONFIG_IS_ENABLED

2022-07-11 Thread Rick Chen
> From: Michal Simek > Sent: Thursday, July 07, 2022 4:47 PM > To: Simon Glass ; u-boot@lists.denx.de; g...@xilinx.com > Cc: Alexandru Gagniuc ; Jan Kiszka > ; Leo Yu-Chi Liang(梁育齊) ; > Marek Vasut ; Peter Hoyes ; Rick Jian-Zhi > Chen(陳建志) ; Tom Rini > Subject: [PATCH] arm: riscv: Remove

RE: [v4 03/12] spi: aspeed: Add ASPEED SPI controller driver

2022-07-11 Thread Chin-Ting Kuo
Hi Cédric, > -Original Message- > From: Cédric Le Goater > Sent: Friday, July 8, 2022 4:52 PM > To: Chin-Ting Kuo ; Joel Stanley > > Subject: Re: [v4 03/12] spi: aspeed: Add ASPEED SPI controller driver > > On 7/8/22 07:42, Chin-Ting Kuo wrote: > > Hi Joel, > > > >> -Original

Re: [PATCH 3/3] doc: environment: Further expand on Image locations and provide example

2022-07-11 Thread Heinrich Schuchardt
On 7/10/22 18:17, Tom Rini wrote: On Sun, Jul 10, 2022 at 02:26:04PM +0200, Heinrich Schuchardt wrote: On 6/30/22 12:06, Simon Glass wrote: On Mon, 20 Jun 2022 at 08:32, Tom Rini wrote: Start by elaborating on what some of our constraints tend to be with image location values, and document

Re: [PATCH 1/2] designprinciples.rst: Perform minor cleanups

2022-07-11 Thread Claudius Heine
On 2022-07-08 20:38, Tom Rini wrote: - Remove some missed wiki markup, and escape a "\n" correctly. - Use gender-neutral language to refer to the user, consistently. Cc: Claudius Heine Signed-off-by: Tom Rini --- doc/develop/designprinciples.rst | 28 ++-- 1 file

Re: [PATCH 2/2] process.rst: Perform minor cleanups

2022-07-11 Thread Claudius Heine
Hi Tom, On 2022-07-08 20:38, Tom Rini wrote: - Use gender-neutral language to refer to the user, consistently. - Reword a few places so that they read more naturally. - Make the long standing practice around "Twilight Time" more clear, hopefully. - Replace a reference to MAKEALL with a

Re: [PATCH 2/2] misc: usb251xb: Support 8/16 bit device tree values

2022-07-11 Thread Marek Vasut
On 6/14/22 15:21, Stefan Herbrechtsmeier wrote: From: Stefan Herbrechtsmeier The device tree binding [1] specify the vendor-id, product-id, device-id and language-id as 16 bit values and the linux driver reads the boost-up value as 8 bit value. [1]

Re: [PATCH 1/2] dm: core: Add functions to read 8/16-bit integers

2022-07-11 Thread Marek Vasut
On 6/14/22 15:21, Stefan Herbrechtsmeier wrote: From: Stefan Herbrechtsmeier Add functions to read 8/16-bit integers like the existing functions for 32/64-bit to simplify read of 8/16-bit integers from device tree properties. Signed-off-by: Stefan Herbrechtsmeier Reviewed-by: Marek Vasut

Re: [PATCH v8 1/9] efi_loader: expose END device path node

2022-07-11 Thread Ilias Apalodimas
On Sun, 10 Jul 2022 at 12:10, Heinrich Schuchardt wrote: > > On 6/19/22 06:55, Masahisa Kojima wrote: > > This commit exposes the END device path node. > > > > Signed-off-by: Masahisa Kojima > > Reviewed-by: Heinrich Schuchardt > > > --- > > No change in v8 > > > > Newly created in v7 > > > >

Re: [PATCH] misc: Port USB251xB/xBi Hi-Speed Hub Controller Driver from Linux

2022-07-11 Thread Stefan Herbrechtsmeier
Hi Marek, Am 14.06.2022 um 09:38 schrieb Marek Vasut: On 6/14/22 09:08, Stefan Herbrechtsmeier wrote: Am 13.06.2022 um 22:26 schrieb Marek Vasut: On 6/13/22 19:23, Stefan Herbrechtsmeier wrote: [snip] +    if (dev_read_u32(dev, "vendor-id", >vendor_id)) +    hub->vendor_id =

Re: [PATCH] spl: mmc: Use correct MMC device when loading image

2022-07-11 Thread Quentin Schulz
Hi Harald, On 7/6/22 12:58, Harald Seiler wrote: When attempting to load images from multiple MMC devices in sequence, spl_mmc_load() chooses the wrong device from the second attempt onwards. The reason is that MMC initialization is only done on its first call and spl_mmc_load() will then

Re: [PATCH 3/3] doc: environment: Further expand on Image locations and provide example

2022-07-11 Thread Tom Rini
On Mon, Jul 11, 2022 at 03:10:36PM +0200, Heinrich Schuchardt wrote: > On 7/11/22 14:41, Tom Rini wrote: > > On Mon, Jul 11, 2022 at 08:42:08AM +0200, Heinrich Schuchardt wrote: > > > On 7/10/22 18:17, Tom Rini wrote: > > > > On Sun, Jul 10, 2022 at 02:26:04PM +0200, Heinrich Schuchardt wrote: > >

Re: [PATCH v1] arm: relocate: Replace ADR instruction with non-pseudo-instruction

2022-07-11 Thread Andre Przywara
On Mon, 11 Jul 2022 15:11:13 +0100 Andre Przywara wrote: > On Mon, 11 Jul 2022 13:57:40 +0100 > Andre Przywara wrote: > > Hi, > > > On Sun, 10 Jul 2022 03:09:53 -0400 > > Jesse Taube wrote: > > > > Hi Jesse, > > > > > In Binutils 2.37 the ADR instruction has changed > > > use alternate

Re: [PATCH v2 8/9] clocks: qcom: Add clock driver for QCS404 SoC

2022-07-11 Thread Ramon Fried
On Fri, Jul 8, 2022 at 4:14 PM Sumit Garg wrote: > > Currently its a dummy clock driver as clocks for UART and eMMC have been > already enabled by ABL. Along with this import "qcom,gcc-qcs404.h" header > from Linux mainline to support DT bindings. I think it's better to initialize the clocks also

Re: [PATCH sunxi/next] spi: sunxi: use XCH status to detect in-progress transfer

2022-07-11 Thread Icenowy Zheng
在 2022-07-11星期一的 00:03 +0100,Andre Przywara写道: > On Tue, 28 Jun 2022 14:49:24 +0800 > Icenowy Zheng wrote: > > Hi Icenowy, > > > The current detection of RX FIFO depth seems to be not reliable, > > and > > XCH will self-clear when a transfer is done. > > many thanks for sending this, indeed

Re: [RFC PATCH 1/2] efi_loader: Add SPI I/O protocol support

2022-07-11 Thread Ilias Apalodimas
Hi Paul On Fri, 8 Jul 2022 at 12:46, Paul Barker wrote: > > This addition allows UEFI applications running under u-boot to access > peripherals on SPI busses. It is based on the UEFI Platform > Initialization (PI) Specification, Version 1.7 Errata A (April 2020). > Only the core functionality

Re: [RFC PATCH 1/3] eficonfig: add UEFI Secure Boot Key enrollment interface

2022-07-11 Thread Masahisa Kojima
Hi Heinrich, On Sun, 10 Jul 2022 at 18:37, Heinrich Schuchardt wrote: > > On 7/8/22 11:14, Ilias Apalodimas wrote: > > On Sun, Jun 19, 2022 at 02:20:20PM +0900, Masahisa Kojima wrote: > >> This commit adds the menu-driven UEFI Secure Boot Key > >> enrollment interface. User can enroll the PK,

[ANN] U-Boot v2022.07 released

2022-07-11 Thread Tom Rini
Hey all, It's a week after the initially scheduled release day, but the last week was as quiet as I expected, so now we're releasing v2022.07. At this point, the DM_ETH migration is 2 years past the deadline, and I have sent some off-list reminders about migration earlier in the year. One of my

Re: [PATCH v2 7/9] pinctrl: qcom: Add pinctrl driver for QCS404 SoC

2022-07-11 Thread Ramon Fried
On Fri, Jul 8, 2022 at 4:14 PM Sumit Garg wrote: > > Currently this pinctrl driver only supports BLSP UART2 specific pin > configuration. > > Signed-off-by: Sumit Garg > --- > arch/arm/mach-snapdragon/Makefile | 1 + > arch/arm/mach-snapdragon/pinctrl-qcs404.c | 55

Re: [PATCH v2 6/9] mmc: msm_sdhci: Add SDCC version 5.0.0 support

2022-07-11 Thread Ramon Fried
On Fri, Jul 8, 2022 at 4:14 PM Sumit Garg wrote: > > For SDCC version 5.0.0, MCI registers are removed from SDCC interface > and some registers are moved to HC. So add support to use the new > compatible string "qcom,sdhci-msm-v5". Based on this new msm variant, > pick the relevant variant data

Re: [PATCH] rockchip: rk3399: boot_devices: fix eMMC node name

2022-07-11 Thread Art Nikpal
Yes ! need to apply this patch dts was changed grep @fe33 arch/arm/dts/*.dtsi arch/arm/dts/rk3399.dtsi:sdhci: mmc@fe33 { same need to change boot_devices in rk3399.c to /mmc@fe32 Reviewed-by: Artem Lapkin Tested-by: Artem Lapkin On Tue, Jul 12, 2022 at 2:22 AM Xavier

Re: [PATCH v2 0/9] New boards support: db845c and qcs404-evb

2022-07-11 Thread Sumit Garg
Hi Peng, On Tue, 12 Jul 2022 at 06:27, Peng Fan wrote: > > > > On 7/8/2022 9:13 PM, Sumit Garg wrote: > > Add support for two new boards db845c and qcs404-evb: > > - db845c is a 96boards compliant platform aka RB3 based on Qualcomm > >SDM845 SoC. > > - qcs404-evb is an evaluation board from

Re: [TF-A] [RFC] Proposed location to host the firmware handoff specification.

2022-07-11 Thread Julius Werner
> That draft is the DEN0135 document [2]. > I realise that I haven’t made it sufficiently explicit in this thread that > the DEN0135 document [2] is still at draft quality (see ALP maturity called > out in the title page and footers). > Please do not consider this a finished document . We’ve

Re: [PATCH v2 8/9] clocks: qcom: Add clock driver for QCS404 SoC

2022-07-11 Thread Sumit Garg
Hi Ramon, Thanks for your review. On Mon, 11 Jul 2022 at 20:08, Ramon Fried wrote: > > On Fri, Jul 8, 2022 at 4:14 PM Sumit Garg wrote: > > > > Currently its a dummy clock driver as clocks for UART and eMMC have been > > already enabled by ABL. Along with this import "qcom,gcc-qcs404.h" header

[PATCH 1/1] net: phy: possible NULL dereference in fixed_phy_create()

2022-07-11 Thread Heinrich Schuchardt
We check if phydev is NULL. Only but if it is non-NULL we set one component of phydev. But even if it is NULL we set another. We should not dereference NULL in either case. Fixes: e24b58f5ed4f ("net: phy: don't require PHY interface mode during PHY creation") Signed-off-by: Heinrich Schuchardt

[PATCH 1/1] cmd: undefined return value of do_extension_apply()

2022-07-11 Thread Heinrich Schuchardt
If 'extension apply all' is executed and no extension is found, the return value of do_extension_apply() is undefined. Return CMD_RET_FAILURE in this case. Fixes: 2f84e9cf06d3 ("cmd: add support for a new "extension" command") Signed-off-by: Heinrich Schuchardt --- cmd/extension_board.c | 1 +

Re: [PATCH v1] arm: relocate: Replace ADR instruction with non-pseudo-instruction

2022-07-11 Thread Jesse Taube
On 7/11/22 10:19, Andre Przywara wrote: > On Mon, 11 Jul 2022 15:11:13 +0100 > Andre Przywara wrote: > >> On Mon, 11 Jul 2022 13:57:40 +0100 >> Andre Przywara wrote: >> >> Hi, >> >>> On Sun, 10 Jul 2022 03:09:53 -0400 >>> Jesse Taube wrote: >>> >>> Hi Jesse, >>> In Binutils 2.37 the

Re: [PATCH] rockchip: rk3399: boot_devices: fix eMMC node name

2022-07-11 Thread Xavier Drudis Ferran
El Mon, Jul 11, 2022 at 04:15:33PM +0200, Quentin Schulz deia: > From: Quentin Schulz > > When idbloader.img is flashed on the eMMC, the SPL still tries to load > from SPI-NOR first. > > This is due to an incorrect look-up in the Device Tree. Since commit > 822556a93459 ("arm: dts: sync the

RE: [PATCH v4 1/7] tools: mkimage: Add support to generate FlexSPI Header for i.MX8m

2022-07-11 Thread ZHIZHIKIN Andrey
Hello Mamta, > -Original Message- > From: Mamta Shukla > Sent: Thursday, July 7, 2022 2:45 PM > To: u-boot@lists.denx.de > Cc: sba...@denx.de; peng@nxp.com; aford...@gmail.com; GEO-CHHER-bsp- > development ; feste...@denx.de; > SHUKLA > Mamta Ramendra ; HAEMMERLE Thomas > >

[PATCH] configs: stm32m15: support STM32MP_BOARD_EXTRA_ENV for st boards

2022-07-11 Thread Patrick Delaunay
Correctly handle STM32MP_BOARD_EXTRA_ENV define in stm32mp15_st_common.h; the STM32MP_BOARD_EXTRA_ENV is added in CONFIG_EXTRA_ENV_SETTINGS definition, as it is done "stm32mp15_st_common.h" Without this patch, the content of STM32MP_BOARD_EXTRA_ENV is not used in the default environment for

[PATCH 2/3] doc: environment: Expand on fdt_addr, initrd_addr and loadaddr

2022-07-11 Thread Tom Rini
- Explain why fdt_addr and initrd_addr should not be set to disable relocation normally. - Provide some advice on the typical loadaddr default value. Signed-off-by: Tom Rini --- Changes in v2: - Fix "iamge" typo --- doc/usage/environment.rst | 15 --- 1 file changed, 12

[PATCH 1/3] doc: environment: Drop u-boot_addr_r

2022-07-11 Thread Tom Rini
This variable is never set nor explained why it would be set, drop it. Reviewed-by: Simon Glass Reviewed-by: Heinrich Schuchardt Signed-off-by: Tom Rini --- Changes in v2: - None --- doc/usage/environment.rst | 1 - 1 file changed, 1 deletion(-) diff --git a/doc/usage/environment.rst

[PATCH 3/3] doc: environment: Further expand on Image locations and provide example

2022-07-11 Thread Tom Rini
Start by elaborating on what some of our constraints tend to be with image location values, and document where these external constraints can come from. Provide a new subsection, an example based on the TI ARMv7 OMAP2PLUS families of chips, that gives sample values and explains why we use these

Re: [PATCH 1/7] doc: Migrate CodingStyle wiki page to Sphinx

2022-07-11 Thread Martin Bonner
On // comments: a) I thought I had read recently that they are now accepted. b) Can we stop calling them "C++ style"? They have been legal in standard C for all of this millennium! There are university graduates who were born after // comments became legal in standard C. c) If we don't currently

RE: [PATCH v4 3/7] configs: Add config for enabling FSPI boot option for i.MX8m

2022-07-11 Thread ZHIZHIKIN Andrey
Hello Mamta, > -Original Message- > From: Mamta Shukla > Sent: Thursday, July 7, 2022 2:45 PM > To: u-boot@lists.denx.de > Cc: sba...@denx.de; peng@nxp.com; aford...@gmail.com; GEO-CHHER-bsp- > development ; feste...@denx.de; > SHUKLA > Mamta Ramendra ; HAEMMERLE Thomas > >

Re: [PATCH v1] arm: relocate: Replace ADR instruction with non-pseudo-instruction

2022-07-11 Thread Andre Przywara
On Sun, 10 Jul 2022 03:09:53 -0400 Jesse Taube wrote: Hi Jesse, > In Binutils 2.37 the ADR instruction has changed > use alternate instructions. Can you elaborate on this? What has changed exactly, and why? Looking at the commit you mention below I don't see an immediate problem that would

Re: [PATCH 3/3] doc: environment: Further expand on Image locations and provide example

2022-07-11 Thread Heinrich Schuchardt
On 7/11/22 14:41, Tom Rini wrote: On Mon, Jul 11, 2022 at 08:42:08AM +0200, Heinrich Schuchardt wrote: On 7/10/22 18:17, Tom Rini wrote: On Sun, Jul 10, 2022 at 02:26:04PM +0200, Heinrich Schuchardt wrote: On 6/30/22 12:06, Simon Glass wrote: On Mon, 20 Jun 2022 at 08:32, Tom Rini wrote:

[PATCH] rockchip: rk3399: boot_devices: fix eMMC node name

2022-07-11 Thread Quentin Schulz
From: Quentin Schulz When idbloader.img is flashed on the eMMC, the SPL still tries to load from SPI-NOR first. This is due to an incorrect look-up in the Device Tree. Since commit 822556a93459 ("arm: dts: sync the Rockhip 3399 SoCs from Linux"), the node name (but not label) changed from

[PATCH] rockchip: rk3399: boot_devices: fix eMMC node name

2022-07-11 Thread Quentin Schulz
From: Quentin Schulz When idbloader.img is flashed on the eMMC, the SPL still tries to load from SPI-NOR first. This is due to an incorrect look-up in the Device Tree. Since commit 822556a93459 ("arm: dts: sync the Rockhip 3399 SoCs from Linux"), the node name (but not label) changed from

Re: [PATCH 1/7] doc: Migrate CodingStyle wiki page to Sphinx

2022-07-11 Thread Tom Rini
On Mon, Jul 11, 2022 at 09:02:47PM +0200, Martin Bonner wrote: > On // comments: > a) I thought I had read recently that they are now accepted. > b) Can we stop calling them "C++ style"? They have been legal in standard > C for all of this millennium! There are university graduates who were

Re: [PATCH v2 5/9] board: qualcomm: Add support for dragonboard845c

2022-07-11 Thread Ramon Fried
On Fri, Jul 8, 2022 at 4:14 PM Sumit Garg wrote: > > Add support for 96Boards Dragonboard 845C aka Robotics RB3 development > platform. This board complies with 96Boards Open Platform Specifications. > > Features: > - Qualcomm Snapdragon SDA845 SoC > - 4GiB RAM > - 64GiB UFS drive > > U-boot is

Re: [PATCH v2 2/9] arm64: dts: sdm845: Remove redundant u-boot DT properties

2022-07-11 Thread Ramon Fried
On Fri, Jul 8, 2022 at 4:14 PM Sumit Garg wrote: > > According to u-boot DT recomendation, u-boot specific DT properties belong > to *-uboot.dtsi. Also for starqltechn board (which is the only current > consumer of sdm845.dtsi), the properties are already included in > starqltechn-uboot.dtsi, so

Re: [PATCH 1/8] arm64: dts: sdm845: Remove redundant u-boot DT properties

2022-07-11 Thread Ramon Fried
On Tue, Jul 5, 2022 at 11:57 AM Daniel Thompson wrote: > > On Tue, Jul 05, 2022 at 11:05:04AM +0530, Sumit Garg wrote: > > Hi Daniel, > > > > Thanks for your review. > > > > On Mon, 4 Jul 2022 at 21:28, Daniel Thompson > > wrote: > > > > > > On Mon, Jul 04, 2022 at 06:28:38PM +0530, Sumit Garg

Re: [PATCH v2 1/9] board: starqltechn: Align DT node overrides with sdm845.dtsi

2022-07-11 Thread Ramon Fried
On Fri, Jul 8, 2022 at 4:14 PM Sumit Garg wrote: > > Currently there is a mismatch among DT node overrides in starqltechn > board DTS file and the actual DT nodes in the sdm845.dtsi. So fix that > to align with DT nodes in sdm845.dtsi. > > Signed-off-by: Sumit Garg > --- >

Re: [PATCH v2 4/9] uart: sdm845: Fix debug UART pinmux

2022-07-11 Thread Ramon Fried
On Fri, Jul 8, 2022 at 4:14 PM Sumit Garg wrote: > > Configure debug UART pins as function: "qup9" rather than being regular > gpios. It fixes a hang seen during pinmux setting. > > Signed-off-by: Sumit Garg > --- > arch/arm/dts/sdm845.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [PATCH v2 3/9] clocks: sdm845: Import qcom,gcc-sdm845.h

2022-07-11 Thread Ramon Fried
On Fri, Jul 8, 2022 at 4:14 PM Sumit Garg wrote: > > Rather than using magic numbers as clock ids for peripherals import > qcom,gcc-sdm845.h from Linux to be used standard macros for clock ids. > So start using corresponding clk-id macro for debug UART. > > Signed-off-by: Sumit Garg > --- >

Re: [ANN] U-Boot v2022.07 released

2022-07-11 Thread Simon Glass
Hi Tom, On Mon, 11 Jul 2022 at 07:43, Tom Rini wrote: > > Hey all, > > It's a week after the initially scheduled release day, but the last week > was as quiet as I expected, so now we're releasing v2022.07. At this > point, the DM_ETH migration is 2 years past the deadline, and I have > sent

[PATCH 2/2] mtd: parsers: add Broadcom's U-Boot parser

2022-07-11 Thread Rafał Miłecki
From: Rafał Miłecki Broadcom stores environment variables blocks inside U-Boot partition itself. This driver finds & registers them. Signed-off-by: Rafał Miłecki --- drivers/mtd/parsers/Kconfig | 10 drivers/mtd/parsers/Makefile | 1 + drivers/mtd/parsers/brcm_u-boot.c | 84

[PATCH 1/2] dt-bindings: mtd: partitions: add binding for U-Boot bootloader

2022-07-11 Thread Rafał Miłecki
From: Rafał Miłecki Right now there is no (known) real reason for a custom binding for standard U-Boot partitions. Broadcom's U-Boot however requires extra handling - looking for environment variables subblocks. This commit adds Broadcom specific binding. Signed-off-by: Rafał Miłecki ---

U-Boot SPI DM framework issues

2022-07-11 Thread Andre Przywara
Hi, while trying to debug some nasty SPI flash issue on sunxi, I came across some oddities in the SPI DM framework, and wanted to check some things, since I am not sure whether I miss things here: - When I do a simple "sf probe" call, I see *two* calls to the .claim_bus callback: the first one

[PATCH 3/7] doc: codingstyle: Remove comment about '//' style comments

2022-07-11 Thread Tom Rini
For some time now we've allowed for '//' style comments, which mirrors the Linux kernel. So drop this point here. Cc: Heinrich Schuchardt Signed-off-by: Tom Rini --- Changes in v2: - None --- doc/develop/codingstyle.rst | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH 7/7] process.rst: Modernize the "Workflow of a Custodian" section

2022-07-11 Thread Tom Rini
The "Workflow of a Custodian" section on the wiki had not been changed in quite some time to reflect how the process has been functioning for some time. First, update some links to point to modern and current sources of information. Second, and more overarching, reword much of the section. This

[PATCH 4/7] doc: Migrate Process wiki page to Sphinx

2022-07-11 Thread Tom Rini
Move the current Process wiki page to doc/develop/process.rst. The changes here are for formatting or slight rewording so that it reads well when linking to other Sphinx documents. Cc: Heinrich Schuchardt Signed-off-by: Tom Rini --- Changes in v2: - Assorted wiki -> Sphinx style corrections

[PATCH 5/7] designprinciples.rst: Perform minor cleanups

2022-07-11 Thread Tom Rini
- Remove some missed wiki markup, and escape a "\n" correctly. - Use gender-neutral language to refer to the user, consistently. Cc: Claudius Heine Cc: Heinrich Schuchardt Signed-off-by: Tom Rini --- Changes in v2: - None --- doc/develop/designprinciples.rst | 24 1

[PATCH 6/7] process.rst: Perform minor cleanups

2022-07-11 Thread Tom Rini
- Use gender-neutral language to refer to the user, consistently. - Reword a few places so that they read more naturally. - Make the long standing practice around "Twilight Time" more clear, hopefully. - Replace a reference to MAKEALL with a reference to CI testing as that's the current

[RESEND PATCH v3 1/1] mx7ulp: add base SPL support for mx7ulp

2022-07-11 Thread Oleksandr Suvorov
From: Ricardo Salveti Add a base implementation of mx7ulp SPL config header and soc, and changes in makefiles in order to allow building SPL on mx7ulp based devices. Signed-off-by: Ricardo Salveti Co-developed-by: Oleksandr Suvorov Signed-off-by: Oleksandr Suvorov --- Changes in v3: -

[RESEND PATCH v3 0/1] Support SPL for i.MX7ULP

2022-07-11 Thread Oleksandr Suvorov
I've just realized this only patch was not applied among other patches of the original patchset "Support SPL for i.MX7ULP". Please apply this patch as now "include/configs/mx7ulp_com.h" refers to non-existent "include/configs/imx7ulp_spl.h". Changes in v3: - rebase the patch to the current

Re: [PATCH v1] arm: relocate: Replace ADR instruction with non-pseudo-instruction

2022-07-11 Thread Andre Przywara
On Mon, 11 Jul 2022 13:57:40 +0100 Andre Przywara wrote: Hi, > On Sun, 10 Jul 2022 03:09:53 -0400 > Jesse Taube wrote: > > Hi Jesse, > > > In Binutils 2.37 the ADR instruction has changed > > use alternate instructions. > > Can you elaborate on this? What has changed exactly, and why?

Re: [PATCH v1] arm: relocate: Replace ADR instruction with non-pseudo-instruction

2022-07-11 Thread Jesse Taube
On 7/11/22 08:57, Andre Przywara wrote: > On Sun, 10 Jul 2022 03:09:53 -0400 > Jesse Taube wrote: > > Hi Jesse, > >> In Binutils 2.37 the ADR instruction has changed >> use alternate instructions. > > Can you elaborate on this? What has changed exactly, and why? Looking at > the commit you

[PATCH 2/7] doc: Migrate DesignPrinciples wiki page to Sphinx

2022-07-11 Thread Tom Rini
Move the current DesignPrinciples wiki page to doc/develop/designprinciples.rst. The changes here are for formatting or slight rewording so that it reads well when linking to other Sphinx documents. Cc: Heinrich Schuchardt Signed-off-by: Tom Rini --- Changes in v2: - Assorted wiki -> Sphinx

[PATCH 1/7] doc: Migrate CodingStyle wiki page to Sphinx

2022-07-11 Thread Tom Rini
Move the current CodingStyle wiki page to doc/develop/codingstyle.rst. The changes here are for formatting or slight rewording so that it reads well when linking to other Sphinx documents. Cc: Heinrich Schuchardt Signed-off-by: Tom Rini --- Changes in v2: - Assorted wiki -> Sphinx style

Re: [PATCH v1] arm: relocate: Replace ADR instruction with non-pseudo-instruction

2022-07-11 Thread Tom Rini
On Mon, Jul 11, 2022 at 01:57:40PM +0100, Andre Przywara wrote: > On Sun, 10 Jul 2022 03:09:53 -0400 > Jesse Taube wrote: > > Hi Jesse, > > > In Binutils 2.37 the ADR instruction has changed > > use alternate instructions. > > Can you elaborate on this? What has changed exactly, and why?

Re: [RFC PATCH 1/2] efi_loader: Add SPI I/O protocol support

2022-07-11 Thread Paul Barker
On 11/07/2022 14:12, Ilias Apalodimas wrote: Hi Paul +static efi_status_t +efi_spi_bus_clock(const struct efi_spi_peripheral *spi_peripheral, + u32 *clock_hz) +{ + EFI_ENTRY("%p, %p", spi_peripheral, clock_hz); + return EFI_EXIT(EFI_UNSUPPORTED); +} + All the

Re: [PATCH 2/2] Nokia RX-51: Remove CONFIG_PREBOOT from defconfig

2022-07-11 Thread Tom Rini
On Sun, Jul 10, 2022 at 01:42:56PM +0200, Pali Rohár wrote: > CONFIG_PREBOOT just cause putting "preboot=CONFIG_PREBOOT" into env list. > Value CONFIG_PREBOOT="run preboot" in defconfig is just nonsense and does > not do anything useful (it is infinite recursion). Config file for this > board

Re: [PATCH V4 resend] spl: imx8mm: enlarge SPL_MAX_SIZE

2022-07-11 Thread Fabio Estevam
Hi Peng, On Mon, Jul 11, 2022 at 3:35 AM Peng Fan (OSS) wrote: > > From: Peng Fan > > The CONFIG_SPL_MAX_SIZE could be 0x27000 for i.MX8MM when SPL_TEXT_BASE > set to 0x7E1000. > > The DDR firmware max uses 96KB, there is a 4KB padding header before > SPL_TEXT_BASE, so the SPL MAX SIZE is

[PATCH 10/19] buildman: Incorporate the genboardscfg.py tool

2022-07-11 Thread Simon Glass
Bring this tool into buildman, so we don't have to run it separately. The board.cfg file is still produced as part of the build, to save time when doing another build in the same working directory. If it is out of date with respect to the Kconfig, it is updated. Time to regenerate on a recent

[PATCH 09/19] buildman: Split out Boards into its own file

2022-07-11 Thread Simon Glass
Use a separate file for the Boards class so that its name matches the module name. Fix up the function names to match the pylint style and fix some other warnings. Signed-off-by: Simon Glass --- tools/buildman/board.py | 281 +- tools/buildman/boards.py

[PATCH 14/19] buildman: Return an error if there are maintainer warnings

2022-07-11 Thread Simon Glass
Detect warnings about missing maintain info and return result code 2 in that case. Signed-off-by: Simon Glass --- tools/buildman/README | 3 ++- tools/buildman/boards.py | 22 +- tools/buildman/control.py | 10 +- 3 files changed, 24 insertions(+), 11

[PATCH 13/19] Revert "Revert "global: Remove CONFIG_SYS_EXTRA_OPTIONS support""

2022-07-11 Thread Simon Glass
This is not needed now that CONFIG_SYS_TARGET_NAME is correctly determined when scanning Kconfig. This reverts commit 25b8acee2ea11a9edc100c42a61f5d6187eb6167. Signed-off-by: Simon Glass Suggested-by: Tom Rini --- boot/Kconfig | 13 doc/README.kconfig | 7

[PATCH 17/19] buildman: Allow lines without a symbol

2022-07-11 Thread Simon Glass
The 'nm' tool can produce lines without a symbol, for example: 0004 t Silently skip these and anything else without three fields. Drop the warning since there is nothing the user can do about it. Signed-off-by: Simon Glass Reported-by: Tom Rini --- tools/buildman/builder.py | 20

[PATCH 12/19] buildman: Replace the Options column with config name

2022-07-11 Thread Simon Glass
This appears in boards.cfg but we want to remove it. Drop support for generating it and reading it. Detect an old boards.cfg file that has this field and regenerate it, to avoid problems. Instead, add the config name in that place. This fixes a subtle bug in the generation code, since it uses

imx8m read boot ROM log issue

2022-07-11 Thread Belisko Marek
Hi, I'm trying to add support for reading imx8mq boot ROM event log and got inspired by this patch: https://lists.denx.de/pipermail/u-boot/2021-July/453318.html My question is if the condition for checking log pointers is correct: /* If the ROM event log pointer is not valid. */ if

[PATCH 02/19] buildman: Support running from an IDE

2022-07-11 Thread Simon Glass
Add a flag to allow buildman to behave properly for use from an IDE. This shows error/warning output on stderr and drops all summary and progress information. This should normally only be used when building a single board. Fix up a confusing comment for GetResultSummary() while we are here,

[PATCH 03/19] buildman: Avoid using board as a variable

2022-07-11 Thread Simon Glass
We have a module called 'board'. Sometimes buildman uses 'brd' as an instance variable but sometimes it uses 'board', which is confusing and can mess with the module handling. Update the code to use 'brd' consistently, making it easier for tools to determine when the module is being referenced.

[PATCH 00/19] buildman: Integration of boards.cfg file

2022-07-11 Thread Simon Glass
This series drops the need for the genboardscfg.py script, so that the boards.cfg file is produced (and consumed) entirely within buildman. The file is not entirely removed since it does have some uses and we need some sort of cache for the information. The genboardscfg.py script is effectively

[PATCH 01/19] buildman: Drop -I option

2022-07-11 Thread Simon Glass
This has been deprecated with a notice that it will be removed after April 2021. Drop it now. Signed-off-by: Simon Glass --- tools/buildman/cmdline.py | 3 --- tools/buildman/control.py | 3 --- 2 files changed, 6 deletions(-) diff --git a/tools/buildman/cmdline.py b/tools/buildman/cmdline.py

Re: [RFC PATCH 3/3] eficonfig: add "Delete Key" menu entry

2022-07-11 Thread Takahiro Akashi
On Sun, Jul 10, 2022 at 12:10:13PM +0200, Heinrich Schuchardt wrote: > On 6/19/22 07:20, Masahisa Kojima wrote: > > This commit add the menu-driven interface to delete the > > signature database entry. > > EFI Signature Lists can contain the multiple signature > > entries, this menu can delete the

Re: [PATCH v2 0/9] New boards support: db845c and qcs404-evb

2022-07-11 Thread Peng Fan
On 7/8/2022 9:13 PM, Sumit Garg wrote: Add support for two new boards db845c and qcs404-evb: - db845c is a 96boards compliant platform aka RB3 based on Qualcomm SDM845 SoC. - qcs404-evb is an evaluation board from Qualcomm based on QCS404 SoC. Both these platforms have one thing in common

[PATCH 16/19] Drop genboardscfg.py

2022-07-11 Thread Simon Glass
Now that buildman can generate this with the -R option, drop the script. Signed-off-by: Simon Glass Suggested-by: Tom Rini --- scripts/pylint.base | 1 - tools/genboardscfg.py | 444 -- 2 files changed, 445 deletions(-) delete mode 100755

[PATCH 18/19] buildman: Drop a Python 2.7 comment

2022-07-11 Thread Simon Glass
This is well out of date, but it is still reasonable to use a list. Drop the comment. Signed-off-by: Simon Glass --- tools/buildman/boards.py | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/buildman/boards.py b/tools/buildman/boards.py index b30b344bc8e..8a0971aa407 100644 ---

[PATCH 19/19] buildman: Drop a TODO that is done

2022-07-11 Thread Simon Glass
Buildman now uses worktrees when available, instead of doing a full clone. This was done in this commit: 76de29fc4f buildman: Use git worktrees instead of git clones when possible Drop the TODO. Signed-off-by: Simon Glass --- tools/buildman/README | 3 --- 1 file changed, 3 deletions(-)

[PATCH 15/19] gitlab/azure: Use buildman instead of genboardscfg

2022-07-11 Thread Simon Glass
Use the equivalent buildman functionality to check maintainer info. Signed-off-by: Simon Glass --- .azure-pipelines.yml | 2 +- .gitlab-ci.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml index bc2b437bd99..36ca3cb4626

Re: [PATCH v8 2/9] eficonfig: menu-driven addition of UEFI boot option

2022-07-11 Thread Takahiro Akashi
On Sun, Jul 10, 2022 at 11:03:43AM +0200, Heinrich Schuchardt wrote: > On 6/19/22 06:56, Masahisa Kojima wrote: > > This commit add the "eficonfig" command. > > The "eficonfig" command implements the menu-driven UEFI boot option > > maintenance feature. This commit implements the addition of > >

[PATCH 11/19] buildman: Tidy up pylint problems in boards module

2022-07-11 Thread Simon Glass
Fix all the pylint warnings. Also tidy up the comments so that they show type information, as required. Signed-off-by: Simon Glass --- tools/buildman/boards.py | 352 +++ 1 file changed, 204 insertions(+), 148 deletions(-) diff --git

[PATCH 08/19] buildman: Convert camel case in board.py

2022-07-11 Thread Simon Glass
Convert this file to snake case and update all files which use it. Signed-off-by: Simon Glass --- tools/buildman/board.py | 38 ++--- tools/buildman/control.py | 12 ++-- tools/buildman/func_test.py | 10 +- tools/buildman/test.py | 24

[PATCH 05/19] buildman: Fix use of 'boards' in test

2022-07-11 Thread Simon Glass
We want to create a module called 'boards' so avoid use of this variable name in this module. Change the global to be capitalised, as required by Python style. Signed-off-by: Simon Glass --- tools/buildman/test.py | 42 +- 1 file changed, 21

[PATCH 06/19] buildman: Drop use of 'boards' in control

2022-07-11 Thread Simon Glass
Use brds instead so that we can reserve 'boards' for a module name. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 11 +-- tools/buildman/control.py | 26 +- tools/buildman/func_test.py | 2 +- 3 files changed, 19 insertions(+), 20 deletions(-)

[PATCH 07/19] buildman: Drop use of 'board' in board module

2022-07-11 Thread Simon Glass
Use brds instead so that we can reserve 'boards' and 'board' as module names. Signed-off-by: Simon Glass --- tools/buildman/board.py | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tools/buildman/board.py b/tools/buildman/board.py index

[PATCH 04/19] buildman: Fix use of 'boards' in func_test

2022-07-11 Thread Simon Glass
We want to create a module called 'boards' so avoid use of this variable name in this module. Change the global to be capitalised, as required by Python style. Signed-off-by: Simon Glass --- tools/buildman/func_test.py | 38 ++--- 1 file changed, 19

Re: [PATCH v1] arm: relocate: Replace ADR instruction with non-pseudo-instruction

2022-07-11 Thread Tom Rini
On Sun, Jul 10, 2022 at 03:09:53AM -0400, Jesse Taube wrote: > In Binutils 2.37 the ADR instruction has changed > use alternate instructions. > > The change causes armv7-m to not boot. > > Signed-off-by: Jesse Taube > --- > arch/arm/lib/relocate.S | 8 +++- > 1 file changed, 7

Re: [PATCH 2/2] process.rst: Perform minor cleanups

2022-07-11 Thread Tom Rini
On Mon, Jul 11, 2022 at 10:04:13AM +0200, Claudius Heine wrote: > Hi Tom, > > On 2022-07-08 20:38, Tom Rini wrote: [snip] > > #. Once tests are passed, some agreed time limit expires, the custodian > > - requests that the changes in his public git repository be merged into > > the > > -

Re: [PATCH] spl: mmc: Use correct MMC device when loading image

2022-07-11 Thread Harald Seiler
Hi, On Mon, 2022-07-11 at 12:18 +0200, Quentin Schulz wrote: > Hi Harald, > > On 7/6/22 12:58, Harald Seiler wrote: > > When attempting to load images from multiple MMC devices in sequence, > > spl_mmc_load() chooses the wrong device from the second attempt onwards. > > > > The reason is that

[PATCH v2] spl: mmc: Use correct MMC device when loading image

2022-07-11 Thread Harald Seiler
When attempting to load images from multiple MMC devices in sequence, spl_mmc_load() chooses the wrong device from the second attempt onwards. The reason is that MMC initialization is only done on its first call and spl_mmc_load() will then continue using this same device for all future calls.

Re: [PATCH 3/3] doc: environment: Further expand on Image locations and provide example

2022-07-11 Thread Tom Rini
On Mon, Jul 11, 2022 at 08:42:08AM +0200, Heinrich Schuchardt wrote: > On 7/10/22 18:17, Tom Rini wrote: > > On Sun, Jul 10, 2022 at 02:26:04PM +0200, Heinrich Schuchardt wrote: > > > On 6/30/22 12:06, Simon Glass wrote: > > > > On Mon, 20 Jun 2022 at 08:32, Tom Rini wrote: > > > > > > > > > >

  1   2   >