Re: [PATCH 1/2] serial: sh: Fix compile error when lacking HSCIF support

2023-08-23 Thread Paul Barker
On 22/08/2023 22:38, Marek Vasut wrote: > On 8/22/23 20:10, Paul Barker wrote: >> If we attempt to compile serial_sh.c for a system which lacks HSCIF >> support (e.g. R8A7740), we see the following compilation error: >> >> In file included from drivers/serial/serial_sh.c:20: >>

Re: [PATCH 2/2] mtd: nand: raw: atmel: Add error handling when rb-gpios missing

2023-08-23 Thread Eugen Hristev
Hi, On 8/8/23 18:03, Alexander Dahl wrote: Hello Michael, Am Tue, Aug 08, 2023 at 03:49:45PM +0200 schrieb Michael Nazzareno Trimarchi: Hi On Tue, Aug 8, 2023 at 3:03 PM Alexander Dahl wrote: Adapt behaviour to Linux kernel driver. The return value of gpio_request_by_name_nodev() was not

Re: [PATCH 2/2] serial: sh: Tidy up pre-processor directive indentation

2023-08-23 Thread Paul Barker
On 22/08/2023 22:40, Marek Vasut wrote: > On 8/22/23 20:10, Paul Barker wrote: >> Let's make the indentation of pre-processor macros and conditionals in >> serial_sh.h consistent before we add to the confusion with a new SoC. > > What exactly is this patch doing, adding spaces to ifdeffery ? > >

Re: [PATCH V4 4/8] drivers: mmc: am654_sdhci: Update OTAP/ITAP delay

2023-08-23 Thread Nitin Yadav
Hi Mattijs, On 23/08/23 13:36, Mattijs Korpershoek wrote: On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: From: Nitin Yadav U-Boot is fail to boot class U1 UHS SD cards (such as microcenter) due to incorrect OTAP and ITAP delay select values. Update OTAP and ITAP delay select values

Re: [PATCH V4 4/8] drivers: mmc: am654_sdhci: Update OTAP/ITAP delay

2023-08-23 Thread Nitin Yadav
On 23/08/23 15:45, Nitin Yadav wrote: Hi Mattijs, On 23/08/23 13:36, Mattijs Korpershoek wrote: On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: From: Nitin Yadav U-Boot is fail to boot class U1 UHS SD cards (such as microcenter) due to incorrect OTAP and ITAP delay select

Re: [PATCH v1 1/1] usb: host: tegra: implement dts based xcvr setup

2023-08-23 Thread Thierry Reding
On Sun, Aug 20, 2023 at 09:10:17PM +0200, Marek Vasut wrote: > On 8/20/23 20:32, Svyatoslav Ryhel wrote: > > 20 серпня 2023 р. 21:14:15 GMT+03:00, Marek Vasut > > написав(-ла): > > > On 8/20/23 09:13, Svyatoslav Ryhel wrote: > > > > 20 серпня 2023 р. 05:23:14 GMT+03:00, Marek Vasut > > > >

Re: [PATCH v1 0/1] tegra_mmc: get tap and trim from dts

2023-08-23 Thread Thierry Reding
On Sat, Aug 19, 2023 at 06:35:00PM +0300, Svyatoslav Ryhel wrote: > Default-tap and default-trim values are used for eMMC setup > mostly on T114+ devices. As for now, those values are hardcoded > for T210 and ignored for all other Tegra generations. Fix this > by passing tap and trim values from

Re: [PATCH 2/2] mtd: nand: raw: atmel: Add error handling when rb-gpios missing

2023-08-23 Thread Eugen Hristev
On 8/23/23 09:54, Michael Nazzareno Trimarchi wrote: Hi On Wed, Aug 23, 2023 at 8:28 AM Eugen Hristev wrote: Hi, On 8/8/23 18:03, Alexander Dahl wrote: Hello Michael, Am Tue, Aug 08, 2023 at 03:49:45PM +0200 schrieb Michael Nazzareno Trimarchi: Hi On Tue, Aug 8, 2023 at 3:03 PM

Re: [PATCH v1 1/1] mmc: tegra: get default-tap and default-trim from device tree

2023-08-23 Thread Thierry Reding
On Sat, Aug 19, 2023 at 06:35:01PM +0300, Svyatoslav Ryhel wrote: > Default-tap and default-trim values are used for eMMC setup > mostly on T114+ devices. As for now, those values are hardcoded > for T210 and ignored for all other Tegra generations. Fix this > by passing tap and trim values from

[PATCH v2] spl: watchdog: introduce SPL_HW_WATCHDOG

2023-08-23 Thread Oleksandr Suvorov
Add SPL_HW_WATCHDOG Kconfig symbol which can be used to enable non-WDT hardware watchdog in SPL. Co-developed-by: Igor Opaniuk Signed-off-by: Igor Opaniuk Signed-off-by: Oleksandr Suvorov --- Changes in v2: - remove mistakenly included unwanted changes common/spl/Kconfig| 1 -

Re: [PATCH v1 1/1] mmc: tegra: get default-tap and default-trim from device tree

2023-08-23 Thread Svyatoslav Ryhel
23 серпня 2023 р. 14:03:25 GMT+03:00, Thierry Reding написав(-ла): >On Sat, Aug 19, 2023 at 06:35:01PM +0300, Svyatoslav Ryhel wrote: >> Default-tap and default-trim values are used for eMMC setup >> mostly on T114+ devices. As for now, those values are hardcoded >> for T210 and ignored for

Re: [PATCH V4 7/8] board: ti: am62x: Add am62x_beagleplay_* defconfigs and env file

2023-08-23 Thread Mattijs Korpershoek
On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: > Add defconfig fragments for am625 based beagleplay and corresponding > customized environment file for beagleplay. > > Signed-off-by: Nishanth Menon Reviewed-by: Mattijs Korpershoek > --- > Changes: > * new env file to override boot

Re: [PATCH 1/2] net: ti: am65-cpsw-nuss: Workaround for buggy PHY/Board

2023-08-23 Thread Roger Quadros
On 23/08/2023 11:14, Roger Quadros wrote: > > > On 23/08/2023 11:02, Siddharth Vadapalli wrote: >> >> >> On 23/08/23 13:22, Roger Quadros wrote: >>> Hi Siddharth, >>> >>> On 23/08/2023 07:35, Siddharth Vadapalli wrote: Roger, On 22/08/23 17:43, Roger Quadros wrote: >

Re: [PATCH V4 8/8] doc: board: ti: Add BeaglePlay documentation

2023-08-23 Thread Mattijs Korpershoek
Hi Nishanth, On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: > Add base documentation for BeaglePlay > > Signed-off-by: Nishanth Menon This is great! Thanks a lot for the documentation update, especially the "Flash to eMMC" section. Reviewed-by: Mattijs Korpershoek > --- > Cc:

[PATCH 0/2] Add EFI HTTP boot support

2023-08-23 Thread Masahisa Kojima
This series adds the EFI HTTP boot support. User can add the URI device path with "efidebug boot add" command. efibootmgr handles the URI device path, download the specified file using wget, mount the downloaded image with blkmap, then boot with the default file(e.g. EFI/BOOT/BOOTAA64.EFI). To

[PATCH 1/2] cmd: efidebug: add uri device path

2023-08-23 Thread Masahisa Kojima
This adds the URI device path option for 'boot add' subcommand. User can add the URI load option for downloading ISO image file or EFI application through network(e.g. HTTP). Signed-off-by: Masahisa Kojima --- cmd/efidebug.c | 39 +++ 1 file changed, 39

[PATCH 2/2] efi_loader: support boot from URI device path

2023-08-23 Thread Masahisa Kojima
This supports to boot from the URI device path. When user selects the URI device path, bootmgr downloads the file using wget into the address specified by loadaddr env variable. If the file is .iso or .img file, mount the image with blkmap then try to boot with the default file(e.g.

Re: [PATCH v5 00/13] Add video damage tracking

2023-08-23 Thread Alexander Graf
Hey Simon, On 22.08.23 20:56, Simon Glass wrote: Hi Alex, On Tue, 22 Aug 2023 at 01:47, Alexander Graf wrote: On 22.08.23 01:03, Simon Glass wrote: Hi Alex, On Mon, 21 Aug 2023 at 16:40, Alexander Graf wrote: On 22.08.23 00:10, Simon Glass wrote: Hi Alex, On Mon, 21 Aug 2023 at 14:20,

Re: [PATCH v3 1/2] schemas: Add a schema for memory map

2023-08-23 Thread Mark Rutland
On Tue, Aug 22, 2023 at 02:34:42PM -0600, Simon Glass wrote: > The Devicetree specification skips over handling of a logical view of > the memory map, pointing users to the UEFI specification. > > It is common to split firmware into 'Platform Init', which does the > initial hardware setup and a

Re: [PATCH v1 15/19] ARM: tegra: board2: add generic late init

2023-08-23 Thread Thierry Reding
On Tue, Aug 22, 2023 at 02:22:13PM +0300, Svyatoslav Ryhel wrote: > Board specific late init allows vendors to set up different device > or board specific env variables (like serial number, platform name). > In case this information is missing, u-boot will lack info regards > serial or platform. >

Re: [PATCH v1 17/19] ARM: tegra: dt-setup: convert TrustZone remove into config

2023-08-23 Thread Thierry Reding
On Tue, Aug 22, 2023 at 02:22:15PM +0300, Svyatoslav Ryhel wrote: > Remove of TrustZone nodes is required by many product devices > which require repetable calls of same function from device board. > To simplify this, TZ remove is converted into Kconfig option. > > Signed-off-by: Svyatoslav Ryhel

Re: [PATCH 1/2] serial: sh: Fix compile error when lacking HSCIF support

2023-08-23 Thread Marek Vasut
On 8/23/23 08:25, Paul Barker wrote: On 22/08/2023 22:38, Marek Vasut wrote: On 8/22/23 20:10, Paul Barker wrote: If we attempt to compile serial_sh.c for a system which lacks HSCIF support (e.g. R8A7740), we see the following compilation error: In file included from

Re: [PATCH 2/2] serial: sh: Tidy up pre-processor directive indentation

2023-08-23 Thread Marek Vasut
On 8/23/23 08:33, Paul Barker wrote: On 22/08/2023 22:40, Marek Vasut wrote: On 8/22/23 20:10, Paul Barker wrote: Let's make the indentation of pre-processor macros and conditionals in serial_sh.h consistent before we add to the confusion with a new SoC. What exactly is this patch doing,

Re: [PATCH V4 0/8] board: ti: Add support for BeaglePlay

2023-08-23 Thread Mattijs Korpershoek
Hi Nishanth, On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: > Hi, > > Add support for BeaglePlay - rev 4 > > Full series: > https://github.com/nmenon/fix-k3-dt-u-boot/commits/beagleplay-v4.2 > > Caveats: > * networking: pending >

Re: [PATCH V4 1/8] env_default: Allow CONFIG_EXTRA_ENV_TEXT to override CFG_EXTRA_ENV_SETTINGS

2023-08-23 Thread Mattijs Korpershoek
On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: > CFG_EXTRA_ENV_SETTINGS is set in common board config files, This allows > for majority of the settings to be set in a common manner. However, the > minor variations between various board can be addressed by the board.env > files. The

Re: [PATCH V4 2/8] configs: am62x_evm*: Enable EMMC_BOOT configuration

2023-08-23 Thread Mattijs Korpershoek
On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: > Enable EMMC boot support for AM62x evm base configuration. > > Signed-off-by: Nishanth Menon Reviewed-by: Mattijs Korpershoek > --- > New patch > > configs/am62x_evm_a53_defconfig | 1 + > configs/am62x_evm_r5_defconfig | 1 + > 2

Re: [PATCH 1/2] net: ti: am65-cpsw-nuss: Workaround for buggy PHY/Board

2023-08-23 Thread Roger Quadros
Hi Siddharth, On 23/08/2023 07:35, Siddharth Vadapalli wrote: > Roger, > > On 22/08/23 17:43, Roger Quadros wrote: >> Beagleplay has a buggy Ethernet PHY implementation for the Gigabit >> PHY in the sense that it is non responsive over MDIO immediately >> after power-up/reset. >> >> We need to

Re: [PATCH V4 3/8] arm: mach-k3: am625: Add support for UDA FS

2023-08-23 Thread Mattijs Korpershoek
On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: > While boot partition support with EMMC boot is useful, it is > constrained by the size of boot hardware partition itself. > > In the case of K3 devices, tispl images can contain OP-TEE images that > can substantially vary in size and the

Re: [PATCH 1/2] net: ti: am65-cpsw-nuss: Workaround for buggy PHY/Board

2023-08-23 Thread Siddharth Vadapalli
On 23/08/23 13:22, Roger Quadros wrote: > Hi Siddharth, > > On 23/08/2023 07:35, Siddharth Vadapalli wrote: >> Roger, >> >> On 22/08/23 17:43, Roger Quadros wrote: >>> Beagleplay has a buggy Ethernet PHY implementation for the Gigabit >>> PHY in the sense that it is non responsive over MDIO

Re: [PATCH 2/2] net: phy: Change "PHY not found" message to debug()

2023-08-23 Thread Roger Quadros
On 23/08/2023 07:54, Siddharth Vadapalli wrote: > > > On 22/08/23 17:43, Roger Quadros wrote: >> Some boards (e.g. Beagleplay) need multiple attempts to detect the PHY >> and the multiple "PHY not found" prints are not nice. > > I tried grepping for calls to "phy_connect" across the drivers

Re: [PATCH V4 4/8] drivers: mmc: am654_sdhci: Update OTAP/ITAP delay

2023-08-23 Thread Mattijs Korpershoek
On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: > From: Nitin Yadav > > U-Boot is fail to boot class U1 UHS SD cards (such as microcenter) > due to incorrect OTAP and ITAP delay select values. Update OTAP and > ITAP delay select values based on recommeded RIOT values to fix boot > issue.

Re: [PATCH V4 5/8] arm: dts: k3-am625-sk-binman: Add labels for unsigned binary

2023-08-23 Thread Mattijs Korpershoek
On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: > Add labels for unsigned binary to permit over-ride. > > Signed-off-by: Nishanth Menon Reviewed-by: Mattijs Korpershoek > --- > Cc: Simon Glass > > Changes since V3: None > > V3:

Re: [PATCH 1/2] net: ti: am65-cpsw-nuss: Workaround for buggy PHY/Board

2023-08-23 Thread Roger Quadros
On 23/08/2023 11:02, Siddharth Vadapalli wrote: > > > On 23/08/23 13:22, Roger Quadros wrote: >> Hi Siddharth, >> >> On 23/08/2023 07:35, Siddharth Vadapalli wrote: >>> Roger, >>> >>> On 22/08/23 17:43, Roger Quadros wrote: Beagleplay has a buggy Ethernet PHY implementation for the

Re: [PATCH v1 1/1] usb: host: tegra: implement dts based xcvr setup

2023-08-23 Thread Svyatoslav Ryhel
23 серпня 2023 р. 13:49:18 GMT+03:00, Thierry Reding написав(-ла): >On Sun, Aug 20, 2023 at 09:10:17PM +0200, Marek Vasut wrote: >> On 8/20/23 20:32, Svyatoslav Ryhel wrote: >> > 20 серпня 2023 р. 21:14:15 GMT+03:00, Marek Vasut >> > написав(-ла): >> > > On 8/20/23 09:13, Svyatoslav Ryhel

Re: [PATCH v1 0/1] tegra_mmc: get tap and trim from dts

2023-08-23 Thread Svyatoslav Ryhel
23 серпня 2023 р. 13:53:26 GMT+03:00, Thierry Reding написав(-ла): >On Sat, Aug 19, 2023 at 06:35:00PM +0300, Svyatoslav Ryhel wrote: >> Default-tap and default-trim values are used for eMMC setup >> mostly on T114+ devices. As for now, those values are hardcoded >> for T210 and ignored for

Re: [PATCH V4 4/8] drivers: mmc: am654_sdhci: Update OTAP/ITAP delay

2023-08-23 Thread Nishanth Menon
On 15:48-20230823, Nitin Yadav wrote: [...] > > > > -    for (i = MMC_HS; i <= MMC_HS_400; i++) { > > > > +    for (i = MMC_LEGACY; i <= MMC_HS_400; i++) { > > > > > > I'm not super familiar with mmc drivers in general, but it feels like >

Re: [PATCH v1 1/1] mmc: tegra: get default-tap and default-trim from device tree

2023-08-23 Thread Svyatoslav Ryhel
23 серпня 2023 р. 14:03:25 GMT+03:00, Thierry Reding написав(-ла): >On Sat, Aug 19, 2023 at 06:35:01PM +0300, Svyatoslav Ryhel wrote: >> Default-tap and default-trim values are used for eMMC setup >> mostly on T114+ devices. As for now, those values are hardcoded >> for T210 and ignored for

Re: [PATCH v1 15/19] ARM: tegra: board2: add generic late init

2023-08-23 Thread Svyatoslav Ryhel
23 серпня 2023 р. 14:13:11 GMT+03:00, Thierry Reding написав(-ла): >On Tue, Aug 22, 2023 at 02:22:13PM +0300, Svyatoslav Ryhel wrote: >> Board specific late init allows vendors to set up different device >> or board specific env variables (like serial number, platform name). >> In case this

Re: [PATCH V4 1/8] env_default: Allow CONFIG_EXTRA_ENV_TEXT to override CFG_EXTRA_ENV_SETTINGS

2023-08-23 Thread Nishanth Menon
On 20:34-20230822, Simon Glass wrote: > Hi Nishanth, > > On Tue, 22 Aug 2023 at 17:34, Nishanth Menon wrote: > > > > On 17:16-20230822, Simon Glass wrote: > > > Hi Nishanth, > > > > > > On Tue, 22 Aug 2023 at 12:41, Nishanth Menon wrote: > > > > > > > > CFG_EXTRA_ENV_SETTINGS is set in common

Re: [PATCH v1 1/1] usb: host: tegra: implement dts based xcvr setup

2023-08-23 Thread Marek Vasut
On 8/23/23 12:49, Thierry Reding wrote: On Sun, Aug 20, 2023 at 09:10:17PM +0200, Marek Vasut wrote: On 8/20/23 20:32, Svyatoslav Ryhel wrote: 20 серпня 2023 р. 21:14:15 GMT+03:00, Marek Vasut написав(-ла): On 8/20/23 09:13, Svyatoslav Ryhel wrote: 20 серпня 2023 р. 05:23:14 GMT+03:00,

Re: [PATCH v1 17/19] ARM: tegra: dt-setup: convert TrustZone remove into config

2023-08-23 Thread Svyatoslav Ryhel
23 серпня 2023 р. 14:17:37 GMT+03:00, Thierry Reding написав(-ла): >On Tue, Aug 22, 2023 at 02:22:15PM +0300, Svyatoslav Ryhel wrote: >> Remove of TrustZone nodes is required by many product devices >> which require repetable calls of same function from device board. >> To simplify this, TZ

Re: [PATCH 2/2] mtd: nand: raw: atmel: Add error handling when rb-gpios missing

2023-08-23 Thread Michael Nazzareno Trimarchi
Hi On Wed, Aug 23, 2023 at 8:28 AM Eugen Hristev wrote: > > Hi, > > On 8/8/23 18:03, Alexander Dahl wrote: > > Hello Michael, > > > > Am Tue, Aug 08, 2023 at 03:49:45PM +0200 schrieb Michael Nazzareno > > Trimarchi: > >> Hi > >> > >> On Tue, Aug 8, 2023 at 3:03 PM Alexander Dahl wrote: > >>> >

Pull request for doc-2023-10-rc4

2023-08-23 Thread Heinrich Schuchardt
Dear Tom, The following changes since commit 976fb2ffa3875a7bed9866bf5cf939a81c423ef8: Prepare v2023.10-rc3 (2023-08-21 16:19:59 -0400) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-efi.git tags/doc-2023-10-rc4 for you to fetch changes up to

Re: [PATCH 1/2] mtd: nand: raw: atmel: Remove duplicate line

2023-08-23 Thread Eugen Hristev
On 8/8/23 16:48, Michael Nazzareno Trimarchi wrote: Hi On Tue, Aug 8, 2023 at 3:03 PM Alexander Dahl wrote: Signed-off-by: Alexander Dahl Reviewed-by: Michael Trimarchi Applied this one patch to u-boot-at91/next , thanks !

[PATCH v3 0/3] fsl-layerscape: drop support for obsolete PPA secure firmware

2023-08-23 Thread laurentiu . tudor
From: Laurentiu Tudor Support for this in-house secure firmware was discontinued long time ago so remove it. First couple of patches delete the defconfigs that use it (split in two to be <100KB) and the third one removes the actual PPA support. Changes in v3: - fixed some merge conflicts

[PATCH v3 3/3] fsl-layerscape: drop obsolete PPA secure firmware support

2023-08-23 Thread laurentiu . tudor
From: Laurentiu Tudor PPA was a secure firmware developed in-house which is no longer supported and replaced by TF-A quite some years ago. Drop support for it. Signed-off-by: Laurentiu Tudor --- arch/arm/cpu/armv8/fsl-layerscape/Kconfig | 60

[PATCH v3 1/3] configs: layerscape: delete defconfigs using legacy PPA secure FW (part1)

2023-08-23 Thread laurentiu . tudor
From: Laurentiu Tudor PPA was a secure firmware developed in-house which is no longer supported and replaced by TF-A quite some years ago. This makes the defconfigs that make use of PPA obsolete, so remove them. Signed-off-by: Laurentiu Tudor --- configs/ls1012a2g5rdb_qspi_defconfig

[PATCH v3 2/3] configs: layerscape: delete defconfigs using legacy PPA secure FW (part2)

2023-08-23 Thread laurentiu . tudor
From: Laurentiu Tudor PPA was a secure firmware developed in-house which is no longer supported and replaced by TF-A quite some years ago. This makes the defconfigs that make use of PPA obsolete, so remove them. Signed-off-by: Laurentiu Tudor --- configs/ls1046aqds_SECURE_BOOT_defconfig

Re: [PATCH v2 1/2] dm: event: add EVT_DM_POST_INIT_R event type

2023-08-23 Thread Simon Glass
On Mon, 21 Aug 2023 at 21:47, Roland Ruckerbauer wrote: > > Tested on visionfive2 board, it works. Thanks for taking a look into this. > > On 18.08.23 07:11, Chanho Park wrote: > > This patch introduces EVT_DM_POST_INIT_R event type for handling hooks > > after relocation. > > > > Fixes:

Re: [PATCH v2 2/2] riscv: cpu: make riscv_cpu_probe to EVT_DM_POST_INIT_R callback

2023-08-23 Thread Simon Glass
On Thu, 17 Aug 2023 at 23:26, Chanho Park wrote: > > Since the Patch 55171aedda88, VisionFive2 booting has been broken [1]. > VisionFive2 board requires to enable CONFIG_TIMER_EARLY but booting went > to panic from initr_dm_devices due to lack of a timer device. > > - Error logs > initcall

[PATCH 0/7] Correct confusing lmb error message

2023-08-23 Thread Simon Glass
I ran into a very confusing error message about overlapping memory. I found that the message is not correct, so this series refactors the lmb code a little to permit the real message to be displayed, which is that the internal lmb table has overflowed. It also tidies up the code a little. Simon

[PATCH 1/7] lmb: Drop surplus brackets and fix code style

2023-08-23 Thread Simon Glass
Use a blank line before the final return. Avoid using too many brackets to avoid confusion about precedence. Fix some overly long lines. Signed-off-by: Simon Glass --- lib/lmb.c | 17 - 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/lib/lmb.c b/lib/lmb.c index

[PATCH 2/7] lmb: Tidy up structure access

2023-08-23 Thread Simon Glass
In some cases it helps to define a local variable pointing to the structure being accessed. This avoids lots of repeated code. There is no need to individually assign each struct member, so use a structure assignment instead. Signed-off-by: Simon Glass --- lib/lmb.c | 54

[PATCH 3/7] lmb: Avoid long for loop counters and function returns

2023-08-23 Thread Simon Glass
Use int for loop counters since it is more common. Do the same with the return value of some internal functions. Signed-off-by: Simon Glass --- lib/lmb.c | 36 +--- 1 file changed, 17 insertions(+), 19 deletions(-) diff --git a/lib/lmb.c b/lib/lmb.c index

[PATCH 4/7] lmb: Change functions returning long to return int

2023-08-23 Thread Simon Glass
It makes no sense to return long when an int is plenty to provide an error code and a region position. It is just confusing. Update the return-value types to keep to this rule. Signed-off-by: Simon Glass --- include/lmb.h | 10 +- lib/lmb.c | 10 +- 2 files changed, 10

[PATCH 5/7] lmb: Tidy up lmb_overlaps_region()

2023-08-23 Thread Simon Glass
Add a comment to define what this returns. Return a specific error when no overlap is found. Signed-off-by: Simon Glass --- lib/lmb.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/lib/lmb.c b/lib/lmb.c index bf30e0dc90b2..5f2ea45c3ba9 100644 --- a/lib/lmb.c

[PATCH 6/7] lmb: Document and tidy lmb_add_region_flags()

2023-08-23 Thread Simon Glass
Update this to return an error code so we can tell when it just ran out of space in its lmb list. Make lmb_addrs_overlap() return a bool. Add a few function comments while we are here. Signed-off-by: Simon Glass --- lib/lmb.c | 47 +-- 1 file

[PATCH 7/7] fs: Fix a confusing error about overlapping regions

2023-08-23 Thread Simon Glass
When lmb runs out of space in its internal tables, it gives errors on every fs load operation. This is horribly confusing, as the poor user tries different memory regions one at a time. Use the updated API to check the error code and print a helpful message. Also, allow the operation to proceed.

[PATCH] pinctrl: sunxi: add SPI1 pinmuxes for f1c100s

2023-08-23 Thread Iota Hydrae
SPI1 is also usable on port A. But there's nothing about SPI1 pinmux there for now. This should be set when you are using SPI1. According to the manual, the mux value would be 6. Signed-off-by: Iota Hydrae --- drivers/pinctrl/sunxi/pinctrl-sunxi.c | 1 + 1 file changed, 1 insertion(+) diff

Re: [PATCH V4 6/8] arm: dts: Add k3-am625-beagleplay

2023-08-23 Thread Mattijs Korpershoek
On mar., août 22, 2023 at 13:41, Nishanth Menon wrote: > From: Robert Nelson > > BeagleBoard.org BeaglePlay is an easy to use, affordable open source > hardware single board computer based on the Texas Instruments AM625 > SoC that allows you to create connected devices that work even at long >

[PATCH] tools: ensure zeroed padding in external FIT images

2023-08-23 Thread Roman Azarenko
Padding the header of an external FIT image is achieved by growing the existing temporary FIT file to match the given alignment requirement before appending image data. Reusing an existing file this way means that the padding will likely contain a portion of the original data not overwritten by

Re: [PATCH 3/5] thermal: imx_tmu: Select LOG

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 02:26:35PM -0300, Fabio Estevam wrote: > From: Fabio Estevam > > Without LOG being selected, the dev_info() information > from the imx_tmu driver cannot be printed. > > Select the LOG option, so that no dev_info() messages are lost. > > Signed-off-by: Fabio Estevam >

Re: [PATCH 4/5] thermal: imx_tmu: Fix th temperature unit

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 02:26:36PM -0300, Fabio Estevam wrote: > From: Fabio Estevam > > The temperature unit is millidegree Celsius, so divide by 1000 to correctly > print the temperature values in Celsius. > > While at it, also change a typo: "has beyond" to "is beyond". > > Signed-off-by:

Re: Pull request for doc-2023-10-rc4

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 09:04:51AM +0200, Heinrich Schuchardt wrote: > Dear Tom, > > The following changes since commit 976fb2ffa3875a7bed9866bf5cf939a81c423ef8: > > Prepare v2023.10-rc3 (2023-08-21 16:19:59 -0400) > > are available in the Git repository at: > >

Re: [PATCH v5 1/3] Kconfig: Add SPL_SYS_MALLOC_CLEAR_ON_INIT

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 11:21:04AM +0800, Shengyu Qu wrote: > Add SPL version of SYS_MALLOC_CLEAR_ON_INIT, this would help devices > that need to clear ram before use to work correctly. > > Signed-off-by: Bo Gan > Signed-off-by: Shengyu Qu > --- > Kconfig | 11 +++ > 1 file changed,

Re: [PATCH v5 2/3] dlmalloc: Add support for SPL_SYS_MALLOC_CLEAR_ON_INIT

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 11:21:05AM +0800, Shengyu Qu wrote: > To support SPL_SYS_MALLOC_CLEAR_ON_INIT, we have to modify > #ifdef CONFIG_SYS_MALLOC_CLEAR_ON_INIT > to #if CONFIG_IS_ENABLED(SYS_MALLOC_CLEAR_ON_INIT) > > Signed-off-by: Bo Gan > Signed-off-by: Shengyu Qu Reviewed-by: Tom Rini

Re: [PATCH] mmc: spl: select SPL_BLK for SPL_DM_MMC

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 06:45:45PM +0300, Oleksandr Suvorov wrote: > mmc_bind() in mmc-uclass.c calls blk_create_devicef() which is > defined in blk-uclass.c, so SPL_BLK is required by SPL_DM_MMC. > Implicitly select SPL_BLK for SPL_DM_MMC. > > Signed-off-by: Oleksandr Suvorov Reviewed-by: Tom

[PATCH v2 1/5] imx8mm_evk_defconfig: Select CONFIG_IMX_TMU

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam Select the i.MX8MM thermal driver as it is useful for displaying the CPU temperature and its grading: CPU: Commercial temperature grade (0C to 95C) at 38C It also prevents booting when the temperature is above the alert point. Signed-off-by: Fabio Estevam --- Changes

[PATCH v2 2/5] thermal: imx_tmu: Fix the polling default

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam When the 'polling-delay' property is not passed via devicetree, pdata->polling_delay keeps at 0. This causes the imx_tmu driver to get stuck inside the busy while() loop when the CPU temperature is above the alert point. Fix this problem by passing a one second polling

[PATCH v2 3/5] thermal: imx_tmu: Increase the log level for high temperatures

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam dev_info() message is not printed by default. Increase the log level to dev_crit(). This allows the critical messages related to the temperature getting beyong the alert threshold to be displayed. Signed-off-by: Fabio Estevam --- Changes since v1: - Use dev_crit() instead

[PATCH v2 4/5] thermal: imx_tmu: Fix the temperature unit

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam The temperature unit is millidegree Celsius, so divide by 1000 to correctly print the temperature values in Celsius. While at it, also change a typo: "has beyond" to "is beyond". Signed-off-by: Fabio Estevam --- Changes since v1: - Rebased. drivers/thermal/imx_tmu.c | 4

[PATCH v2 5/5] thermal: imx_tmu: Increase the polling interval

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam Polling every second to check whether the CPU has cooled down is too frequent. Allow more time for the CPU to cool down by increasing the polling interval to 5 seconds by defaut. This value is used in the absence of the 'polling-delay' devicetree property. Signed-off-by:

Re: [PATCH v2 3/5] thermal: imx_tmu: Increase the log level for high temperatures

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 02:59:09PM -0300, Fabio Estevam wrote: > From: Fabio Estevam > > dev_info() message is not printed by default. Increase the log level > to dev_crit(). This allows the critical messages related to the temperature > getting beyong the alert threshold to be displayed. > >

Re: [PATCH] tools: ensure zeroed padding in external FIT images

2023-08-23 Thread Marek Vasut
On 8/23/23 14:14, Roman Azarenko wrote: Padding the header of an external FIT image is achieved by growing the existing temporary FIT file to match the given alignment requirement before appending image data. Reusing an existing file this way means that the padding will likely contain a portion

Re: [PATCH v2 1/8] cmd: gpt: Remove confusing help text

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 10:47:37AM -0600, Joshua Watt wrote: > This help text appears to be a fragment of the text shown when > CONFIG_CMD_GPT_RENAME is enabled, but is confusing so remove it. > > Signed-off-by: Joshua Watt Reviewed-by: Tom Rini -- Tom signature.asc Description: PGP

Re: [PATCH v2 2/8] doc: Add gpt command documentation

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 10:47:38AM -0600, Joshua Watt wrote: > Adds initial documentation for the gpt command > > Signed-off-by: Joshua Watt > --- > doc/usage/cmd/gpt.rst | 139 ++ > 1 file changed, 139 insertions(+) > create mode 100644

[PATCH 1/1] doc: describe TPL/VPL/SPL boot

2023-08-23 Thread Heinrich Schuchardt
This is a stub describing how TPL, VPL, and SPL load the next boot stages on a detail level for users. For sure we will need a few patches on top to catch the whole complexity. Signed-off-by: Heinrich Schuchardt --- doc/usage/index.rst| 1 + doc/usage/spl_boot.rst | 309

Re: [PATCH] arm: dts: j7200: dtb sync with Linux 6.5-rc1

2023-08-23 Thread reidt
On 16:36-20230822, Nishanth Menon wrote: > On 13:57-20230822, Reid Tonking wrote: > > Sync j7200 device tree files with Linux 6.5-rc1 > > I understand this is a major step forward, but a still: > > There is more work to do: > a) split the dev-data.c fixup as the first patch > b) See comments in

Re: [PATCH v3 1/2] schemas: Add a schema for memory map

2023-08-23 Thread Simon Glass
Hi, On Wed, 23 Aug 2023 at 08:24, Ard Biesheuvel wrote: > > On Wed, 23 Aug 2023 at 10:59, Mark Rutland wrote: > > > > On Tue, Aug 22, 2023 at 02:34:42PM -0600, Simon Glass wrote: > > > The Devicetree specification skips over handling of a logical view of > > > the memory map, pointing users to

Re: [PATCH 1/1] doc: describe TPL/VPL/SPL boot

2023-08-23 Thread Pali Rohár
On Wednesday 23 August 2023 21:57:28 Heinrich Schuchardt wrote: > +Booting from TPL/SPL > + > + > +The main U-Boot binary may be to large to be loaded directly by the Boot ROM. > +This was the main driver for splitting up U-Boot into multiple boot stages > with > +successively

Re: [PATCH 7/8] spl: x86: Avoid starting up PCI automatically in SPL

2023-08-23 Thread Heinrich Schuchardt
On 8/23/23 20:47, Simon Glass wrote: For x86 platforms, PCI is core to their operation and is managed in arch-specific code. Each platform has its own way of doing this. For TPL and some SPL implementations, the full driver model PCI is not used. A recent change enabled full PCI in TPL/SPL for

Re: [PATCH] usb: dwc3: Fix renaming SPL_USB_HOST_SUPPORT to SPL_USB_HOST

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 05:22:19PM +0300, Oleksandr Suvorov wrote: > In the usb/dwc3-layerscape driver the first option should be renamed > to the latter as well. Do it. > > Fixes: 333e4a621df ("Rename SPL_USB_HOST_SUPPORT to SPL_USB_HOST") > Signed-off-by: Oleksandr Suvorov > --- > >

Re: [PATCH V4 8/8] doc: board: ti: Add BeaglePlay documentation

2023-08-23 Thread Tom Rini
On Wed, Aug 23, 2023 at 09:30:31AM -0600, Simon Glass wrote: > Hi Nishanth, > > On Tue, 22 Aug 2023 at 12:43, Nishanth Menon wrote: > > > > Add base documentation for BeaglePlay > > > > Signed-off-by: Nishanth Menon > > --- > > Cc: Heinrich Schuchardt > > > > Changes Since V3: > > * Updated

[PATCH 1/5] imx8mm_evk_defconfig: Select CONFIG_IMX_TMU

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam Select the i.MX8MM thermal driver as it is useful for displaying the CPU temperature and its grading: CPU: Commercial temperature grade (0C to 95C) at 38C It also prevents booting when the temperature is above the alert point. Signed-off-by: Fabio Estevam ---

[PATCH 2/5] thermal: imx_tmu: Fix the polling default

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam When the 'polling-delay' property is not passed via devicetree, pdata->polling_delay keeps at 0. This causes the imx_tmu driver to get stuck inside the busy while() loop when the CPU temperature is above the alert point. Fix this problem by passing a one second polling

[PATCH 3/5] thermal: imx_tmu: Select LOG

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam Without LOG being selected, the dev_info() information from the imx_tmu driver cannot be printed. Select the LOG option, so that no dev_info() messages are lost. Signed-off-by: Fabio Estevam --- drivers/thermal/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 4/5] thermal: imx_tmu: Fix th temperature unit

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam The temperature unit is millidegree Celsius, so divide by 1000 to correctly print the temperature values in Celsius. While at it, also change a typo: "has beyond" to "is beyond". Signed-off-by: Fabio Estevam --- drivers/thermal/imx_tmu.c | 4 ++-- 1 file changed, 2

[PATCH 5/5] thermal: imx_tmu: Increase the polling interval

2023-08-23 Thread Fabio Estevam
From: Fabio Estevam Polling every second to check whether the CPU has cooled down is too frequent. Allow more time for the CPU to cool down by increasing the polling interval to 5 seconds by defaut. This value is used in the absence of the 'polling-delay' devicetree property. Signed-off-by:

[PATCH v2 0/5] at91: sam9x60-curiosity: Misc improvements

2023-08-23 Thread Alexander Dahl
Hei hei, while working with the sam9x60-curiosity board I noticed some things still have rough edges on U-Boot shell for that device. With these patches (and some more config options enabled), the following commands now work as expected: reset, led, button. The whole series is based on

[PATCH v2 2/5] configs: at91: sam9x60: Switch to new reset driver

2023-08-23 Thread Alexander Dahl
Since commit 61040097a9d1 ("reset: at91: Add reset driver for basic assert/deassert operations") the compatible "microchip,sam9x60-rstc" for the sam9x60 reset controller in sam9x60.dtsi is not handled by CONFIG_SYSRESET_AT91 anymore, but by CONFIG_RESET_AT91 now. This resulted in the following

[PATCH v2 3/5] ARM: dts: at91: sam9x60-curiosity: Sync LED nodes from Linux

2023-08-23 Thread Alexander Dahl
Copied as is from Linux Kernel release v6.4. (dts file is still the same in Linux v6.5-rc7 but was moved to vendor sub-directories with v6.5-rc1.) Signed-off-by: Alexander Dahl --- arch/arm/dts/at91-sam9x60_curiosity.dts | 30 + 1 file changed, 30 insertions(+) diff

[PATCH v2 5/5] ARM: dts: at91: sam9x60-curiosity: Sync gpio button from Linux

2023-08-23 Thread Alexander Dahl
Copied as is from Linux Kernel release v6.4. (dts file is still the same in Linux v6.5-rc7 but was moved to vendor sub-directories with v6.5-rc1.) Button works out of the box now if the following config options are enabled: CONFIG_BUTTON, CONFIG_BUTTON_GPIO, CONFIG_CMD_BUTTON, CONFIG_DM_GPIO.

[PATCH v2 1/5] configs: at91: sam9x60_curiosity: Sync both defconfig variants

2023-08-23 Thread Alexander Dahl
The board has two SD card slots and we have two defconfigs for booting from either the first (micro SD) named 'sam9x60_curiosity_mmc_defconfig' or the second (full size SD) named 'sam9x60_curiosity_mmc1_defconfig'. For comparable Microchip boards (sama5d27-som1-ek, sama5d29-curiosity, sama7g5ek)

[PATCH v2 4/5] board: sam9x60-curiosity: Let LED subsystem init leds if enabled

2023-08-23 Thread Alexander Dahl
If CONFIG_LED and CONFIG_LED_GPIO are enabled, it is not necessary to initialize the RGB LED on the board by manually setting hardcoded GPIOs anymore. Everything is well defined in dts and can be used like on boards of other vendors. Keep the old behaviour as fallback, though. With all this in

Re: [PATCH V4 4/8] drivers: mmc: am654_sdhci: Update OTAP/ITAP delay

2023-08-23 Thread Nishanth Menon
On 06:38-20230823, Nishanth Menon wrote: > On 15:48-20230823, Nitin Yadav wrote: > [...] > > > > > > -    for (i = MMC_HS; i <= MMC_HS_400; i++) { > > > > > +    for (i = MMC_LEGACY; i <= MMC_HS_400; i++) { > > > > > > > >

[PATCH] usb: dwc3: Fix renaming SPL_USB_HOST_SUPPORT to SPL_USB_HOST

2023-08-23 Thread Oleksandr Suvorov
In the usb/dwc3-layerscape driver the first option should be renamed to the latter as well. Do it. Fixes: 333e4a621df ("Rename SPL_USB_HOST_SUPPORT to SPL_USB_HOST") Signed-off-by: Oleksandr Suvorov --- drivers/usb/dwc3/dwc3-layerscape.c | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [PATCH v3 1/2] schemas: Add a schema for memory map

2023-08-23 Thread Ard Biesheuvel
On Wed, 23 Aug 2023 at 10:59, Mark Rutland wrote: > > On Tue, Aug 22, 2023 at 02:34:42PM -0600, Simon Glass wrote: > > The Devicetree specification skips over handling of a logical view of > > the memory map, pointing users to the UEFI specification. > > > > It is common to split firmware into

Re: [PATCH 4/8] disk: Handle partition to block device offset conversion

2023-08-23 Thread Tom Rini
On Mon, Aug 14, 2023 at 01:46:44AM +0200, Marek Vasut wrote: > Convert the read/write/erase offset from one within a partition > to one within a block device, to correctly access the data on > the block device for both write and erase operations. > > Signed-off-by: Marek Vasut Applied to

Re: [PATCH 1/8] disk: Drop always true conditional check

2023-08-23 Thread Tom Rini
On Mon, Aug 14, 2023 at 01:46:41AM +0200, Marek Vasut wrote: > if (device_get_uclass_id(dev) == UCLASS_PARTITION) is always > true, because this disk_blk_read() function calls dev_get_blk() > above and checks its return value for non-NULL. The dev_get_blk() > performs the same

Re: [PATCH] test: acpi: Handle both 32bit and 64bit ACPI tables

2023-08-23 Thread Tom Rini
On Sun, Aug 13, 2023 at 03:25:59AM +0200, Marek Vasut wrote: > Handle both 32bit and 64bit systems, i.e. sandbox and sandbox64 > the same way drivers/cpu/cpu_sandbox.c sets those ACPI tables up. > This fixes "$ ./u-boot -Tc 'ut dm dm_test_acpi_write_tables'" > test failure on sandbox64. > >

  1   2   3   >