Re: [PATCH 4/4] sunxi: DRAM: H6: use proper MMIO accessors in mctl_set_addrmap()

2023-10-20 Thread Jernej Škrabec
On Saturday, October 21, 2023 3:10:25 AM CEST Andre Przywara wrote: > For accessing MMIO registers, we must not rely on the compiler to > realise every access to a struct which we made point to some MMIO base > address. From a compiler's point of view, those writes could be > considered pointless,

Re: [PATCH 3/4] sunxi: DRAM: H6: split struct dram_para

2023-10-20 Thread Jernej Škrabec
On Saturday, October 21, 2023 3:10:24 AM CEST Andre Przywara wrote: > Currently there is one DRAM parameter struct for the Allwinner H6 DRAM > "driver". It contains many fields that are compile time constants > (set by Kconfig variables), though there are also some fields that are > probed and

Re: [PATCH 2/4] sunxi: DRAM: H6: const-ify DRAM function parameters

2023-10-20 Thread Jernej Škrabec
On Saturday, October 21, 2023 3:10:23 AM CEST Andre Przywara wrote: > There are quite some functions in the Allwinner H6 DRAM "driver", some > of them actually change the parameters in the structure passed to them, > but many are actually not. > > To increase the optimisation potential for the

Re: [PATCH 1/1] sunxi: dram: Fix incorrect ram size detection for some H6 boards

2023-10-20 Thread Jernej Škrabec
On Saturday, October 21, 2023 1:38:39 AM CEST Andre Przywara wrote: > On Mon, 02 Oct 2023 20:59:34 +0200 > Jernej Škrabec wrote: > > Hi Jernej, > > > Dne ponedeljek, 02. oktober 2023 ob 14:42:40 CEST je Gunjan Gupta napisal(a): > > > > > bool mctl_mem_matches(u32 offset) > > > > > { > > > >

Re: [PATCH] arm: mvebu: AC5/AC5X: use fixed page table size

2023-10-20 Thread Chris Packham
On Sat, 21 Oct 2023, 2:04 am Marc Zyngier, wrote: > On 2023-10-18 21:53, Chris Packham wrote: > > Since commit 6cdf6b7a340d ("arm64: Use FEAT_HAFDBS to track dirty pages > > when available") the default get_page_table_size() sets some flags for > > more efficient handling of dirty page table

Re: [PATCH v2 03/22] sunxi: remove CONFIG_MACPWR

2023-10-20 Thread Samuel Holland
Hi Andre, On 9/28/23 16:54, Andre Przywara wrote: > The CONFIG_MACPWR Kconfig symbol is used to point to a GPIO that enables > the power for the Ethernet "MAC" (mostly PHY, really). > In the DT this is described with the phy-supply property in the MAC DT > node, pointing to a (GPIO controlled)

Re: [PATCH v2 11/22] pinctrl: sunxi: add Allwinner D1 pinctrl description

2023-10-20 Thread Samuel Holland
Hi Andre, On 9/28/23 16:54, Andre Przywara wrote: > Apart from using the new pinctrl MMIO register layout, the Allwinner D1 > and related SoCs still need to usual set of mux values hardcoded in > U-Boot's pinctrl driver. > Add the values we need so far to this list, so that DM based drivers >

Re: [v4 23/24] sandbox: Add

2023-10-20 Thread Sean Anderson
On 10/19/23 11:01, Tom Rini wrote: Add a mostly empty asm/barrier.h file for sandbox where we define nop() to be an empty function. Signed-off-by: Tom Rini --- arch/sandbox/include/asm/barrier.h | 3 +++ 1 file changed, 3 insertions(+) create mode 100644 arch/sandbox/include/asm/barrier.h

Re: [v4 24/24] clk_k210.c: Clean up how we handle nop

2023-10-20 Thread Sean Anderson
On 10/19/23 11:01, Tom Rini wrote: Now that sandbox has and defines nop() there we should include that in our driver for clarity and then remove our local nop() from . Signed-off-by: Tom Rini --- I can see that our ARM should be and updated in a few other ways to match how the kernel is

[PATCH 4/4] sunxi: DRAM: H6: use proper MMIO accessors in mctl_set_addrmap()

2023-10-20 Thread Andre Przywara
For accessing MMIO registers, we must not rely on the compiler to realise every access to a struct which we made point to some MMIO base address. From a compiler's point of view, those writes could be considered pointless, since no code consumes them later on: the compiler would actually be free

[PATCH 3/4] sunxi: DRAM: H6: split struct dram_para

2023-10-20 Thread Andre Przywara
Currently there is one DRAM parameter struct for the Allwinner H6 DRAM "driver". It contains many fields that are compile time constants (set by Kconfig variables), though there are also some fields that are probed and changed over the runtime of the DRAM initialisation. Because of this mixture,

[PATCH 1/4] sunxi: DRAM: H6: add barrier after finishing DRAM setup

2023-10-20 Thread Andre Przywara
From: Gunjan Gupta During the DRAM controller setup, we program its registers for certain configurations (multiple times), then try to access the DRAM array, to detect the number of rows and columns in the used DRAM chips. This requires that all MMIO writes have reached the DRAM controller,

[PATCH 2/4] sunxi: DRAM: H6: const-ify DRAM function parameters

2023-10-20 Thread Andre Przywara
There are quite some functions in the Allwinner H6 DRAM "driver", some of them actually change the parameters in the structure passed to them, but many are actually not. To increase the optimisation potential for the code, mark those functions that just read members of the passed dram_para struct

[PATCH 0/4] sunxi: DRAM: H6: fixes and size reduction

2023-10-20 Thread Andre Przywara
Hi, this contains two fixes for the Allwinner H6 DRAM code: patch 1/4 adds a DSB barrier instruction after the actual DRAM register setup, to make sure that subsequent DRAM accesses actually match the just programmed setup. The last patch makes sure the compiler does not optimise away the MMIO

[PATCH] smbios: arm64: Allow table to be written at a fixed addr

2023-10-20 Thread Simon Glass
U-Boot typically sets up its malloc() pool near the top of memory. On ARM64 systems this can result in an SMBIOS table above 4GB which is not supported by SMBIOSv2. Work around this problem by providing a new option to choose an address just below 4GB, if needed. Signed-off-by: Simon Glass ---

Re: error on h616 ID axp806 legacy kernel

2023-10-20 Thread Andre Przywara
On Thu, 5 Oct 2023 17:37:48 -0300 project-tvbox project-tvbox wrote: Hi, > In the legacy kernel, when starting, the error appears: > the chip id is 0x5d00 > ic cant match axp, please check... Please keep in mind that the BSP code contains a lot of nonsense, the PMIC setup seems to be

Re: H616 DRAM support on mailine kernel

2023-10-20 Thread Andre Przywara
On Thu, 5 Oct 2023 17:23:06 -0300 project-tvbox project-tvbox wrote: Hi, > > How can I edit the drm parameters directly in the mainline kernel, > since the mainline dts does not have these options like the legacy > one, I saw that the dram.c drives in the legacy and mailline are > similar, for

Re: [PATCH 1/1] sunxi: dram: Fix incorrect ram size detection for some H6 boards

2023-10-20 Thread Andre Przywara
On Mon, 02 Oct 2023 20:59:34 +0200 Jernej Škrabec wrote: Hi Jernej, > Dne ponedeljek, 02. oktober 2023 ob 14:42:40 CEST je Gunjan Gupta napisal(a): > > > > bool mctl_mem_matches(u32 offset) > > > > { > > > > + dsb(); > > > This looks a bit odd, do you have an explanation for that? And

[v4.1 2/2] CI, pytest: Add a test for sandbox without LTO

2023-10-20 Thread Tom Rini
The primary motivation for having a sandbox without LTO build in CI is to ensure that we don't have that option break. We now have the ability to run tests of specific options being enabled/disabled, so drop the parts of CI that build and test that configuration specifically and add a build test

[v4.1 1/2] sandbox: Add a test for disabling CONFIG_CMDLINE

2023-10-20 Thread Tom Rini
From: Simon Glass Now that everything is working, add a test to make sure that this builds correctly. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- test/py/tests/test_sandbox_opts.py | 20 1 file changed, 20 insertions(+) create mode 100644

[PATCH 2/3] arm: exynos: Include missing CPU header in gpio.h

2023-10-20 Thread Sam Protsenko
arch/arm/include/asm/arch/gpio.h relies on definitions from cpu.h. Include it explicitly in gpio.h. Otherwise next build error may occur: In file included from ./arch/arm/include/asm/gpio.h:7, from include/cros_ec.h:14, from

[PATCH 1/3] arm: exynos: Include missing CPU header in soc.c

2023-10-20 Thread Sam Protsenko
samsung_get_base_swreset() is called in soc.c, but corresponding header with its prototype is not included. Fix this to avoid possible build errors. Signed-off-by: Sam Protsenko --- arch/arm/mach-exynos/soc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-exynos/soc.c

[PATCH 3/3] watchdog: s5p_wdt: Include missing CPU header

2023-10-20 Thread Sam Protsenko
s5p watchdog driver calls samsung_get_base_watchdog() function, but its prototype is not included. That might lead to build warnings like this: drivers/watchdog/s5p_wdt.c: In function 'wdt_stop': drivers/watchdog/s5p_wdt.c:16:26: warning: implicit declaration of function

[PATCH 0/3] exynos: Include missing CPU headers

2023-10-20 Thread Sam Protsenko
During porting a new Exynos-based board to U-Boot I faced a couple of similar build errors due to missing inclusion. This series make sure it's fixed in all places I found. Sam Protsenko (3): arm: exynos: Include missing CPU header in soc.c arm: exynos: Include missing CPU header in gpio.h

Re: [PATCH v2 1/1] efi_loader: expose the device-tree file name

2023-10-20 Thread Simon Glass
Trying again to add Doug as we chatted briefly about this yesterday. On Fri, Oct 20, 2023, 07:21 Simon Glass wrote: > > +Doug Anderson > > Hi Heinrich, > > On Thu, 19 Oct 2023 at 09:09, Heinrich Schuchardt > wrote: > > > > On 19.10.23 15:55, Simon Glass wrote: > > > Hi Heinrich, > > > > > > On

[PATCH] board: samsung: Remove incorrect SYS_CONFIG_NAME in axy17lte Kconfig

2023-10-20 Thread Sam Protsenko
The global SYS_CONFIG_NAME in axy17lte Kconfig overrides SYS_CONFIG_NAME for all boards specified after this line in arch/arm/mach-exynos/Kconfig: source "board/samsung/axy17lte/Kconfig" Right now it's the last 'source' line there, so the issue is not reproducible. But once some board is

Re: [RESEND PATCH] board: ti: common: board_detect: Fix EEPROM offset read for 1-byte

2023-10-20 Thread Nishanth Menon
On 20:52-20231020, Kumar, Udit wrote: > Thanks Prasanth > > On 10/20/2023 12:31 PM, Prasanth Babu Mantena wrote: > > EEPROM detection logic in ti_i2c_eeprom_get() involves reading the total > > size followed by reading 1-byte size with an offset 1. This commit fixes > >

Re: [RFC] CTAGS prototypes

2023-10-20 Thread Tom Rini
On Fri, Oct 20, 2023 at 12:46:07PM +0200, Philip Richard Oberfichtner wrote: > Hi, > > is there a specific reason why the ctags make target is compiled without > using '--kinds-c=+p'? Maybe it would make sense to (optionally) include > that in the Makefile? > > I personally find it useful to

Re: [PULL] u-boot-sh/master_uart_test

2023-10-20 Thread Tom Rini
On Thu, Oct 19, 2023 at 07:21:42PM +0200, Marek Vasut wrote: > The following changes since commit e463222cce0df86f05a117f06e1737f539841fe6: > > Merge branch 'master' of > https://source.denx.de/u-boot/custodians/u-boot-riscv (2023-10-19 09:40:04 > -0400) > > are available in the Git

Re: Please pull u-boot-marvell/master

2023-10-20 Thread Tom Rini
On Fri, Oct 20, 2023 at 02:51:08PM +0200, Stefan Roese wrote: > Hi Tom, > > please pull this next small batch of Marvell related patches: > After dropping the page table size patch (per other discussions), applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: Please pull u-boot-marvell/master

2023-10-20 Thread Tom Rini
On Fri, Oct 20, 2023 at 06:44:58PM +0200, Stefan Roese wrote: > Hi Tom, > > On 10/20/23 16:45, Tom Rini wrote: > > On Fri, Oct 20, 2023 at 02:51:08PM +0200, Stefan Roese wrote: > > > > > Hi Tom, > > > > > > please pull this next small batch of Marvell related patches: > > > > > >

Re: Please pull u-boot-marvell/master

2023-10-20 Thread Stefan Roese
Hi Tom, On 10/20/23 16:45, Tom Rini wrote: On Fri, Oct 20, 2023 at 02:51:08PM +0200, Stefan Roese wrote: Hi Tom, please pull this next small batch of Marvell related patches: - kirkwood: Pogo v4: Enable LTO (Tony) - mvebu:

Re: [PATCH v2 1/1] efi_loader: expose the device-tree file name

2023-10-20 Thread Tom Rini
On Fri, Oct 20, 2023 at 05:40:03PM +0200, Heinrich Schuchardt wrote: > On 20.10.23 15:21, Simon Glass wrote: > > +Doug Anderson > > > > Hi Heinrich, > > > > On Thu, 19 Oct 2023 at 09:09, Heinrich Schuchardt > > wrote: > > > > > > On 19.10.23 15:55, Simon Glass wrote: > > > > Hi Heinrich, > > >

Re: [PATCH v2 1/1] efi_loader: expose the device-tree file name

2023-10-20 Thread Heinrich Schuchardt
On 20.10.23 15:21, Simon Glass wrote: +Doug Anderson Hi Heinrich, On Thu, 19 Oct 2023 at 09:09, Heinrich Schuchardt wrote: On 19.10.23 15:55, Simon Glass wrote: Hi Heinrich, On Wed, 18 Oct 2023 at 02:15, Heinrich Schuchardt wrote: On 10/18/23 05:33, Simon Glass wrote: Hi Heinrich, On

Re: [RESEND PATCH] board: ti: common: board_detect: Fix EEPROM offset read for 1-byte

2023-10-20 Thread Neha Malcom Francis
Hi Prasanth On 20-Oct-23 12:31 PM, Prasanth Babu Mantena wrote: EEPROM detection logic in ti_i2c_eeprom_get() involves reading the total size followed by reading 1-byte size with an offset 1. This commit fixes the header matching issue in commit 9f393a2d7af8 ("board: ti: common: board_detect:

[PATCH v1] apalis-imx8: add USBH_EN gpio hog

2023-10-20 Thread Andrejs Cainikovs
From: Andrejs Cainikovs USB host interface is not working on some Apalis Toradex carrier boards with Apalis iMX8 SoM. This is due to USBH_EN pin, which powers USB peripherals, having a strong pull-down on some boards, and a weak pull-down on the others. This USBH_EN pin is left unconfigured,

Re: [RESEND PATCH] board: ti: common: board_detect: Fix EEPROM offset read for 1-byte

2023-10-20 Thread Kumar, Udit
Thanks Prasanth On 10/20/2023 12:31 PM, Prasanth Babu Mantena wrote: EEPROM detection logic in ti_i2c_eeprom_get() involves reading the total size followed by reading 1-byte size with an offset 1. This commit fixes the header matching issue in commit 9f393a2d7af8 ("board: ti: common:

Re: [PATCH 30/32] fdt: Allow the devicetree to come from a bloblist

2023-10-20 Thread Simon Glass
Hi Ilias, On Fri, 20 Oct 2023 at 01:22, Ilias Apalodimas wrote: > > Hi Simon, > > On Wed, 18 Oct 2023 at 18:26, Simon Glass wrote: > > > > Hi Ilias, > > > > On Mon, 25 Sept 2023 at 04:19, Ilias Apalodimas > > wrote: > > > > > > Hi Simon, > > > > > > [...] > > > > > > > > > >> +config

Re: Please pull u-boot-marvell/master

2023-10-20 Thread Tom Rini
On Fri, Oct 20, 2023 at 02:51:08PM +0200, Stefan Roese wrote: > Hi Tom, > > please pull this next small batch of Marvell related patches: > > > - kirkwood: Pogo v4: Enable LTO (Tony) > - mvebu: AC5/AC5X: use fixed page table size

[PATCH u-boot-marvell] arm: mvebu: turris_mox: Extend to support RIPE Atlas Probe

2023-10-20 Thread Marek Behún
Extend Turris Mox board code to support CZ.NIC's RIPE Atlas Probe. Signed-off-by: Marek Behún --- arch/arm/dts/Makefile | 1 + arch/arm/dts/armada-3720-ripe-atlas.dts | 91 + arch/arm/mach-mvebu/Kconfig | 6 +-

Re: [PATCH 30/32] fdt: Allow the devicetree to come from a bloblist

2023-10-20 Thread Tom Rini
On Fri, Oct 20, 2023 at 11:21:29AM +0300, Ilias Apalodimas wrote: > Hi Simon, > > On Wed, 18 Oct 2023 at 18:26, Simon Glass wrote: > > > > Hi Ilias, > > > > On Mon, 25 Sept 2023 at 04:19, Ilias Apalodimas > > wrote: > > > > > > Hi Simon, > > > > > > [...] > > > > > > > > > >> +config

Re: [v4 11/24] efi: Rearrange the Kconfig for CMD_BOOTEFI_BOOTMGR

2023-10-20 Thread Tom Rini
On Fri, Oct 20, 2023 at 09:21:00PM +0900, AKASHI Takahiro wrote: > On Thu, Oct 19, 2023 at 11:19:33AM -0400, Tom Rini wrote: > > On Thu, Oct 19, 2023 at 05:16:28PM +0200, Heinrich Schuchardt wrote: > > > On 19.10.23 17:00, Tom Rini wrote: > > > > From: Simon Glass > > > > > > > > The command

Re: [PATCH v2 1/1] efi_loader: expose the device-tree file name

2023-10-20 Thread Tom Rini
On Fri, Oct 20, 2023 at 06:21:33AM -0700, Simon Glass wrote: > +Doug Anderson > > Hi Heinrich, > > On Thu, 19 Oct 2023 at 09:09, Heinrich Schuchardt > wrote: > > > > On 19.10.23 15:55, Simon Glass wrote: > > > Hi Heinrich, > > > > > > On Wed, 18 Oct 2023 at 02:15, Heinrich Schuchardt > > >

Re: [PATCH v2 1/1] efi_loader: expose the device-tree file name

2023-10-20 Thread Simon Glass
+Doug Anderson Hi Heinrich, On Thu, 19 Oct 2023 at 09:09, Heinrich Schuchardt wrote: > > On 19.10.23 15:55, Simon Glass wrote: > > Hi Heinrich, > > > > On Wed, 18 Oct 2023 at 02:15, Heinrich Schuchardt > > wrote: > >> > >> On 10/18/23 05:33, Simon Glass wrote: > >>> Hi Heinrich, > >>> > >>> On

Re: [PATCH 1/1] clk: ti: k3-pll: Add calibration support for non fractional mode

2023-10-20 Thread Bryan Brattlof
Hey Vishal! On October 20, 2023 thus sayeth Vishal Mahaveer: > PLL calibration needs to be enabled when operating in non fractional > mode. Add the sequence to do a fast calibration when using PLL > in this mode. > > Signed-off-by: Vishal Mahaveer Reviewed-by: Bryan Brattlof > --- >

[PATCH] arm_ffa: fix: remove deadcode in ffa_print_error_log()

2023-10-20 Thread Abdellatif El Khlifi
address the CID 464361 Control flow issues [1] [1]: https://lore.kernel.org/all/20230821210927.GL3953269@bill-the-cat/ Signed-off-by: Abdellatif El Khlifi Cc: Tom Rini --- drivers/firmware/arm-ffa/arm-ffa-uclass.c | 2 -- 1 file changed, 2 deletions(-) diff --git

Re: [PATCH] arm: mvebu: AC5/AC5X: use fixed page table size

2023-10-20 Thread Marc Zyngier
On 2023-10-18 21:53, Chris Packham wrote: Since commit 6cdf6b7a340d ("arm64: Use FEAT_HAFDBS to track dirty pages when available") the default get_page_table_size() sets some flags for more efficient handling of dirty page table entries. This causes problems on the AC5/AC5X SoC (specifically a

Please pull u-boot-marvell/master

2023-10-20 Thread Stefan Roese
Hi Tom, please pull this next small batch of Marvell related patches: - kirkwood: Pogo v4: Enable LTO (Tony) - mvebu: AC5/AC5X: use fixed page table size (Chris)

Re: [RESEND PATCH] arm: kirkwood: Pogo v4: Enable LTO

2023-10-20 Thread Stefan Roese
On 10/20/23 08:14, Stefan Roese wrote: On 10/16/23 22:01, Tony Dinh wrote: Enable building Pogo V4 u-boot image with LTO, which results in about 30K reduction in size. Rebased to latest master and resend. Nitpicking: In such cases, please generate a v2 and put this description in the patch

Re: [PATCH] arm: mvebu: AC5/AC5X: use fixed page table size

2023-10-20 Thread Stefan Roese
On 10/20/23 10:21, Chris Packham wrote: On Fri, 20 Oct 2023, 7:18 pm Stefan Roese, > wrote: Hi Chris, On 10/18/23 22:53, Chris Packham wrote: > Since commit 6cdf6b7a340d ("arm64: Use FEAT_HAFDBS to track dirty pages > when available") the default

[PATCH v2] net: eth-uclass: Improve error message when MAC is not found

2023-10-20 Thread Fabio Estevam
From: Fabio Estevam While bringinp up a new board without the MAC fuses programmed, the following error message was observed: Error: ethernet@30bf address not set. Improve the error message to make it clearer the reason of the failure. Signed-off-by: Fabio Estevam --- Changes since v1: -

Re: [v4 11/24] efi: Rearrange the Kconfig for CMD_BOOTEFI_BOOTMGR

2023-10-20 Thread AKASHI Takahiro
On Thu, Oct 19, 2023 at 11:19:33AM -0400, Tom Rini wrote: > On Thu, Oct 19, 2023 at 05:16:28PM +0200, Heinrich Schuchardt wrote: > > On 19.10.23 17:00, Tom Rini wrote: > > > From: Simon Glass > > > > > > The command should not be used to enable library functionality. Add a > > > new

Re: [PATCH v2 1/2] log: select physical address formatting in a generic way

2023-10-20 Thread Abdellatif El Khlifi
Hi Simon, > sets the log formatting according to the platform (64-bit vs 32-bit) > > Signed-off-by: Abdellatif El Khlifi > Cc: Simon Glass > --- > cmd/armffa.c | 8 > include/log.h | 9 + > 2 files changed, 9 insertions(+), 8 deletions(-) > > diff --git a/cmd/armffa.c

Re: Fwd: New Defects reported by Coverity Scan for Das U-Boot

2023-10-20 Thread Abdellatif El Khlifi
Hi Tom, > > *** CID 464361: Control flow issues (DEADCODE) > /drivers/firmware/arm-ffa/arm-ffa-uclass.c: 148 in ffa_print_error_log() > 142 > 143 if (ffa_id < FFA_FIRST_ID ||

[RFC] CTAGS prototypes

2023-10-20 Thread Philip Richard Oberfichtner
Hi, is there a specific reason why the ctags make target is compiled without using '--kinds-c=+p'? Maybe it would make sense to (optionally) include that in the Makefile? I personally find it useful to jump to the header where a function is declared to view the documentary comment. Or am I

[RESEND PATCH] board: ti: common: board_detect: Fix EEPROM offset read for 1-byte

2023-10-20 Thread Prasanth Babu Mantena
EEPROM detection logic in ti_i2c_eeprom_get() involves reading the total size followed by reading 1-byte size with an offset 1. This commit fixes the header matching issue in commit 9f393a2d7af8 ("board: ti: common: board_detect: Fix EEPROM read quirk for 2-byte"). In the previous commit, the

Re: [PATCH 2/3] arm: mvebu: Espressobin: move network setup into a separate function

2023-10-20 Thread Robert Marko
On Fri, Oct 20, 2023 at 8:46 AM Stefan Roese wrote: > > Hi Robert, > > On 9/18/23 08:47, Stefan Roese wrote: > > On 9/14/23 12:16, Robert Marko wrote: > >> Currently, Esspresobin switch is being setup directly in > >> last_stage_init() > >> which makes it hard to add support for any other board

[PATCH v2 3/3] arm: mvebu: eDPU: support new board revision

2023-10-20 Thread Robert Marko
There is a new eDPU revision that uses Marvell 88E6361 switch onboard. We can rely on detecting the switch to enable and fixup the Linux DTS so a single DTS can be used. There is currently no support for the 88E6361 switch and thus no working networking in U-Boot, so we disable both ports.

[PATCH v2 2/3] arm: mvebu: Espressobin: move network setup into a separate function

2023-10-20 Thread Robert Marko
Currently, Esspresobin switch is being setup directly in last_stage_init() which makes it hard to add support for any other board to be setup. So, lets just move the switch setup code to a separate function and call it if compatible matches, there should be no functional change. Signed-off-by:

[PATCH v2 1/3] arm: mvebu: Espressobin: move FDT fixup into a separate function

2023-10-20 Thread Robert Marko
Currently, Esspresobin FDT is being fixed up directly in ft_board_setup() which makes it hard to add support for any other board to be fixed up. So, lets just move the FDT fixup code to a separate function and call it if compatible matches, there should be no functional change. Signed-off-by:

Re: [PATCH 2/3] cmd: mvebu/bubt: move eMMC data-partition uboot from LBA-0 to 4096

2023-10-20 Thread Stefan Roese
On 10/8/23 14:46, Josua Mayer wrote: A38x bootrom only searches 2 sectors when booting from eMMC, irregardless of data or boot partition: 0 & 4096. For eMMC boot partitions sector 0 is fine, but on data partition it conflicts with MBR. Change bubt command default to 4096 for eMMC data

[PATCH v2 3/3] bootcount: Add driver model I2C driver

2023-10-20 Thread Philip Richard Oberfichtner
This adds a generic I2C bootcounter adhering to driver model to replace the previously removed legacy implementation. There is no change in functionality, it can be used on any I2C device. The device tree configuration may look like this for example: bootcount {

[PATCH v2 2/3] i2c: Implement i2c_get_chip_by_phandle()

2023-10-20 Thread Philip Richard Oberfichtner
This new function enhances the i2c_get_chip*() toolbox by implementing a variant that does not require a chip_addr. Instead, the desired device is pointed to by a phandle. Signed-off-by: Philip Richard Oberfichtner --- Changes in v2: new drivers/i2c/i2c-uclass.c | 75

[PATCH v2 1/3] bootcount: Remove legacy I2C driver

2023-10-20 Thread Philip Richard Oberfichtner
The legacy I2C bootcounter will hereby be removed and eventually be replaced by a driver model implementation in the follow-up commit. The legacy driver has the following drawbacks: - It's not adhering to the driver model - Settings are grabbed from Kconfig rather than device tree

[PATCH v2 0/3] bootcount: Replace I2C legacy implementation by driver model

2023-10-20 Thread Philip Richard Oberfichtner
The generic I2C bootcounter driver does not yet adhere to driver model. This patchset intends to replace the legacy implementation. There are currently no upstream boards using the driver, so it should be safe to just remove it. For downstream users it should be straighforward to switch to the

Re: [PATCH v2] make: check if DTC variable is an absolute path

2023-10-20 Thread Rasmus Villemoes
On 19/10/2023 15.04, Richard Marko wrote: > If we try to build using external dtc using > >> make DTC=dtc > > we get a confusing error like > >> make[2]: *** No rule to make target 'arch/x86/dts/bayleybay.dtb', >> needed by 'dtbs'. Stop. > > Workaround is to use > >> make DTC=$( which dtc )

Re: [PATCH 30/32] fdt: Allow the devicetree to come from a bloblist

2023-10-20 Thread Ilias Apalodimas
Hi Simon, On Wed, 18 Oct 2023 at 18:26, Simon Glass wrote: > > Hi Ilias, > > On Mon, 25 Sept 2023 at 04:19, Ilias Apalodimas > wrote: > > > > Hi Simon, > > > > [...] > > > > > > > >> +config OF_BLOBLIST > > > > > >> + bool "DTB is provided by a bloblist" > > > > > >>

Re: [PATCH] arm: mvebu: AC5/AC5X: use fixed page table size

2023-10-20 Thread Chris Packham
On Fri, 20 Oct 2023, 7:18 pm Stefan Roese, wrote: > Hi Chris, > > On 10/18/23 22:53, Chris Packham wrote: > > Since commit 6cdf6b7a340d ("arm64: Use FEAT_HAFDBS to track dirty pages > > when available") the default get_page_table_size() sets some flags for > > more efficient handling of dirty

Re: [PATCH] doc: Replace dm_dump_all() with dm_dump_tree()

2023-10-20 Thread Mattijs Korpershoek
Hi Dylan, Thank you for your patch. On jeu., oct. 19, 2023 at 10:02, Dylan Corrales wrote: > Replace dm_dump_all() with dm_dump_tree() in driver model documentation, > to reflect changes introduced in commit 145287040480 ("dm: core: Rename > dm_dump_all()"). > > Signed-off-by: Dylan Corrales

Re: [PATCH v3 6/6] spi: zynq_qspi: Add parallel memories support in QSPI driver

2023-10-20 Thread Stefan Herbrechtsmeier
Hi, Am 20.10.2023 um 05:18 schrieb Venkatesh Yadav Abbarapu: Add support for parallel memories in zynq_qspi.c driver. In case of parallel memories STRIPE bit is set and sent to the qspi ip, which will send data bits to both the flashes in parallel. However for few commands we should not use

[PATCH] arm: xilinx: Add missing dual parallel flash description

2023-10-20 Thread Michal Simek
Describe flash memories based on the latest DT binding. Signed-off-by: Michal Simek --- arch/arm/dts/zynq-zc770-xm013.dts | 5 +++-- arch/arm/dts/zynqmp-m-a2197-01-revA.dts | 4 +++- arch/arm/dts/zynqmp-m-a2197-02-revA.dts | 4 +++- arch/arm/dts/zynqmp-m-a2197-03-revA.dts | 4 +++- 4

Re: [PATCH] net: eth-uclass: Improve error message when MAC is not found

2023-10-20 Thread Lothar Waßmann
Hi, On Thu, 19 Oct 2023 18:57:20 -0300 Fabio Estevam wrote: > From: Fabio Estevam > > While bringinp up a new board without the MAC fuses programmed, > the following error message was observed: > > Error: ethernet@30bf address not set. > > Improve the error message to make it clearer the

Re: [PATCH 2/3] arm: mvebu: Espressobin: move network setup into a separate function

2023-10-20 Thread Stefan Roese
Hi Robert, On 9/18/23 08:47, Stefan Roese wrote: On 9/14/23 12:16, Robert Marko wrote: Currently, Esspresobin switch is being setup directly in last_stage_init() which makes it hard to add support for any other board to be setup. So, lets just move the switch setup code to a separate

Re: [PATCH] arm: mvebu: AC5/AC5X: use fixed page table size

2023-10-20 Thread Stefan Roese
Hi Chris, On 10/18/23 22:53, Chris Packham wrote: Since commit 6cdf6b7a340d ("arm64: Use FEAT_HAFDBS to track dirty pages when available") the default get_page_table_size() sets some flags for more efficient handling of dirty page table entries. This causes problems on the AC5/AC5X SoC

Re: [RESEND PATCH] arm: kirkwood: Pogo v4: Enable LTO

2023-10-20 Thread Stefan Roese
On 10/16/23 22:01, Tony Dinh wrote: Enable building Pogo V4 u-boot image with LTO, which results in about 30K reduction in size. Rebased to latest master and resend. Nitpicking: In such cases, please generate a v2 and put this description in the patch history below the "---" line. No need to

Re: [PATCH 1/8] tools: mkeficapsule: Add support for parsing capsule params from config file

2023-10-20 Thread Masahisa Kojima
Hi Sughosh, On Fri, 8 Sept 2023 at 21:00, Sughosh Ganu wrote: > > Add support for specifying the parameters needed for capsule > generation through a config file, instead of passing them through > command-line. Parameters for more than a single capsule file can be > specified, resulting in