Re: [PATCH v2 4/6] usb: Add environment based device blocklist

2024-03-17 Thread Marek Vasut
On 3/17/24 7:15 PM, Janne Grunau wrote: Hej, Hi, On Sun, Mar 17, 2024, at 17:18, Marek Vasut wrote: On 3/17/24 12:07 PM, Janne Grunau via B4 Relay wrote: From: Janne Grunau Add the environment variable "usb_blocklist" to prevent USB devices listed in it from being used. This allows to

RE: [PATCH] efi_loader: accept append write with valid size and data

2024-03-17 Thread kojima.masahisa
Hi Heinrich, > -Original Message- > From: Heinrich Schuchardt > Sent: Friday, March 15, 2024 4:58 PM > To: Kojima, Masahisa/小島 雅久 > Cc: u-boot@lists.denx.de; Ilias Apalodimas > Subject: Re: [PATCH] efi_loader: accept append write with valid size and data > > On 3/15/24 08:03, Ilias

[PATCH] rpi: Copy eth MAC address from fw DT to loaded DT

2024-03-17 Thread Martin Wetterwald
Raspberry Pi B models before model 4 don't have an EEPROM nor an OTP to store the permanent factory MAC address of the NIC. So the firmware that runs initially computes the factory MAC address of the board and patches the DTB to give that information to the next stage. The MAC is put in the

Re: [PATCH] ARM: dts: renesas: Stop using the -u-boot DTs for build

2024-03-17 Thread Adam Ford
On Sun, Mar 17, 2024 at 1:24 AM Marek Vasut wrote: > > The U-Boot build system can automatically paste -u-boot.dtsi at the > end of matching .dts during build. Stop emulating this behavior and > rename the -u-boot.dts files to -u-boot.dtsi, drop "#include...dts" > from those new u-boot.dtsi

Re: [PATCH v1] arm: imx: imx8m: soc: Fix NPU/VPU fdt disable fixup

2024-03-17 Thread Fabio Estevam
On Fri, Mar 15, 2024 at 11:45 AM Vitor Soares wrote: > > From: Vitor Soares > > On imx8m[m|p|q].dtsi, upstream Linux uses different names for NPU/VPU > IP block nodes. It leads variants without such HW block having it > enabled by default. > > This patch adds the upstream Linux node's paths to

Re: [PATCH] apalis-imx8: Fix sc_misc_otp_fuse_read() error check

2024-03-17 Thread Fabio Estevam
On Tue, Mar 12, 2024 at 9:59 PM Fabio Estevam wrote: > > Commit bfb3409d676f ("imx: toradex/apalis-imx8: correct SCU API usage") > made an incorrect logic change in the error code check of > sc_misc_otp_fuse_read(): > > - if (scierr == SC_ERR_NONE) { > + if (scierr) { >

Re: [PATCH] colibri-imx8x: Fix sc_misc_otp_fuse_read() error check

2024-03-17 Thread Fabio Estevam
On Tue, Mar 12, 2024 at 9:36 PM Fabio Estevam wrote: > > Commit aa6e698a7acd ("imx: toradex/colibri-imx8x: correct SCU API usage") > made an incorrect logic change in the error code check of > sc_misc_otp_fuse_read(): > > - if (sc_err == SC_ERR_NONE) { > + if (sc_err) { >

Re: [PATCH] imx8m*_venice: move venice to OF_UPSTREAM

2024-03-17 Thread Fabio Estevam
On Tue, Mar 12, 2024 at 10:16 PM Fabio Estevam wrote: > > Hi Tim, > > On Tue, Mar 12, 2024 at 4:05 PM Tim Harvey wrote: > > > > Move to imx8m{m,n,p}-venice to OF_UPSTREAM: > > - replace the non-upstream generic imx8m{m,n,p}-venice dt with one of the > >dt's from the OF_LIST > > - handle

Re: [PATCH v2 0/5] Add RAUC boot logic for phycore_imx8mp

2024-03-17 Thread Fabio Estevam
On Tue, Mar 12, 2024 at 6:00 AM Leonard Anderweit wrote: > > This series adds RAUC boot logic for the phycore_imx8mp. > The first patch converts the environment from a CFG_EXTRA_ENV_SETTINGS #define > to a text environment for better readability and maintainability. > The second patch moves the

Re: [PATCH] board: phytec: define get_som_type also when SoM detection is disabled

2024-03-17 Thread Fabio Estevam
On Tue, Mar 12, 2024 at 6:39 AM Benjamin Hahn wrote: > > define the phytec_get_som_type function also when the SoM detection is > disabled. > > Fixes: > commit 110d321a56c3 ("board: phytec: common: phytec_som_detection: Add > phytec_get_som_type") > > Signed-off-by: Benjamin Hahn Applied for

Re: [PATCH v1] configs: colibri-imx7: enable watchdog

2024-03-17 Thread Fabio Estevam
On Thu, Mar 7, 2024 at 12:23 PM Parth Pancholi wrote: > > From: Parth Pancholi > > Enable watchdog functionality for Toradex's Colibri iMX7 (NAND/EMMC) > modules. > > Signed-off-by: Parth Pancholi Applied for u-boot-imx/next, thanks.

Re: [PATCH] drivers: imx_tmu: Select polling-rate from cpu-thermal devicetree node

2024-03-17 Thread Fabio Estevam
On Mon, Mar 4, 2024 at 8:49 AM Benjamin Hahn wrote: > > The polling rate is already specified in some devicetrees, like > imx8mp.dtsi for example, but was not selected so far. For the > trippoints, the cpu-thermal node is used. Also get the polling rate from > this node. Use the default of 5000ms

[GIT PULL] Please pull u-boot-imx-next-20240317

2024-03-17 Thread Fabio Estevam
) are available in the Git repository at: https://gitlab.denx.de/u-boot/custodians/u-boot-imx.git tags/u-boot-imx-next-20240317 for you to fetch changes up to 86b79cf131b64eadae023a127921893d30503093: imx8m*_venice: move venice to OF_UPSTREAM (2024-03-17 18:39:54 -0300) u-boot-imx-next-20240317

[GIT PULL] Please pull u-boot-imx-master-20240317

2024-03-17 Thread Fabio Estevam
at: https://gitlab.denx.de/u-boot/custodians/u-boot-imx.git tags/u-boot-imx-master-20240317 for you to fetch changes up to e648c4a3455a4d1880efe121602ed90a0bc9b53f: arm: imx: imx8m: soc: Fix NPU/VPU fdt disable fixup (2024-03-17 18:00:04 -0300) u-boot-imx-master-20240317

Re: [PATCH v2 4/6] usb: Add environment based device blocklist

2024-03-17 Thread E Shattow
Should be usb_denylist, since "block devices" is ambiguous meaning if it is a list of data chunks (blocks) or if it blocks (deny). There is no question what a denylist is. On Sun, Mar 17, 2024 at 4:07 AM Janne Grunau via B4 Relay wrote: > > From: Janne Grunau > > Add the environment variable

[PATCH 3/3] rockchip: io-domain: Add support for RK3328

2024-03-17 Thread Jonas Karlman
Port the RK3328 part of the Rockchip IO-domain driver from linux. This differs from linux version in that pmu io iodomain bit is enabled in the write ops instead of in an init ops as in linux, this way we can avoid keeping a full state of all supply that have been configured. Enable by default

[PATCH 1/3] rockchip: rk3328: Sort imply statements alphabetically

2024-03-17 Thread Jonas Karlman
Sort imply statements under ROCKCHIP_RK3328 alphabetically and remove ENABLE_ARM_SOC_BOOT0_HOOK, DEBUG_UART_BOARD_INIT and SYS_NS16550, they are already implyed or selected by ARCH_ROCKCHIP. Signed-off-by: Jonas Karlman --- arch/arm/mach-rockchip/Kconfig | 11 --- 1 file changed, 4

[PATCH 2/3] rockchip: rk3328: Enable ARMv8 crypto extensions

2024-03-17 Thread Jonas Karlman
The RK3328 SoC support ARMv8 Cryptography Extensions and use of the ARMv8 crypto extensions help speed up FIT checksum validation in SPL. Imply ARMV8_SET_SMPEN and ARMV8_CRYPTO to take advantage of the crypto extensions for SHA256 when validating checksum of FIT images. Also imply OF_LIVE to

[PATCH 0/3] rockchip: rk3328: Add IO-domain driver and speed up boot

2024-03-17 Thread Jonas Karlman
This series adds support for RK3328 to the IO-domain driver, it also enabled ARMv8 crypto extensions and OF_LIVE to speed up boot on rk3328 boards. Before this series init time is around 4 seconds on a Rock64 v2.0: => bootstage report Timer summary in microseconds (9 records): Mark

Re: [PATCH v2 4/6] usb: Add environment based device blocklist

2024-03-17 Thread Janne Grunau
Hej, On Sun, Mar 17, 2024, at 17:18, Marek Vasut wrote: > On 3/17/24 12:07 PM, Janne Grunau via B4 Relay wrote: >> From: Janne Grunau >> >> Add the environment variable "usb_blocklist" to prevent USB devices >> listed in it from being used. This allows to ignore devices which >> trigger bugs in

Re: [PATCH 1/3] boot: fdt: Change type of env_get_bootm_low() to phys_addr_t

2024-03-17 Thread Heinrich Schuchardt
On 17.03.24 17:58, Marek Vasut wrote: On 3/17/24 10:08 AM, Heinrich Schuchardt wrote: On 3/17/24 07:16, Marek Vasut wrote: Change type of ulong env_get_bootm_low() to phys_addr_t env_get_bootm_low(). The PPC/LS systems already treat env_get_bootm_low() result as phys_addr_t, while the function

Re: [PATCH 2/3] boot: fdt: Drop lmb_alloc*() typecasts

2024-03-17 Thread Marek Vasut
On 3/17/24 10:16 AM, Heinrich Schuchardt wrote: On 3/17/24 07:16, Marek Vasut wrote: The lmb_alloc_base() returns phys_addr_t , map_sysmem() accepts phys_addr_t as first parameter. Declare $addr as phys_addr_t and %s/$addr/addr/ I'm not sure this helps readability, I want to delimit the

Re: [PATCH 3/3] boot: fdt: Move usable variable below updated comment

2024-03-17 Thread Marek Vasut
On 3/17/24 12:29 PM, Laurent Pinchart wrote: Hi Marek, Thank you for the patch. On Sun, Mar 17, 2024 at 07:16:31AM +0100, Marek Vasut wrote: Move the variable below comment which explains what the variable means. Update the comment. No functional change. Suggested-by: Laurent Pinchart

Re: [PATCH 2/3] boot: fdt: Drop lmb_alloc*() typecasts

2024-03-17 Thread Marek Vasut
On 3/17/24 12:26 PM, Laurent Pinchart wrote: Hi Marek, Thank you for the patch. On Sun, Mar 17, 2024 at 07:16:30AM +0100, Marek Vasut wrote: The lmb_alloc_base() returns phys_addr_t , map_sysmem() accepts phys_addr_t as first parameter. Declare $addr as phys_addr_t and s/$addr/addr/ ? get

Re: [PATCH 1/3] boot: fdt: Change type of env_get_bootm_low() to phys_addr_t

2024-03-17 Thread Marek Vasut
On 3/17/24 12:18 PM, Laurent Pinchart wrote: Hi Marek, Thank you for the patch. On Sun, Mar 17, 2024 at 07:16:29AM +0100, Marek Vasut wrote: Change type of ulong env_get_bootm_low() to phys_addr_t env_get_bootm_low(). The PPC/LS systems already treat env_get_bootm_low() result as phys_addr_t,

Re: [PATCH 1/3] boot: fdt: Change type of env_get_bootm_low() to phys_addr_t

2024-03-17 Thread Marek Vasut
On 3/17/24 10:08 AM, Heinrich Schuchardt wrote: On 3/17/24 07:16, Marek Vasut wrote: Change type of ulong env_get_bootm_low() to phys_addr_t env_get_bootm_low(). The PPC/LS systems already treat env_get_bootm_low() result as phys_addr_t, while the function itself still returns ulong. This is

iMX8MP - Falcon Mode in QSPI Boot Implementation

2024-03-17 Thread Nitzan Tavor
Dear U-Boot community list members, I have a NXP i.MX8M Plus EVB which I'm trying to enable Falcon mode in secure boot from QSPI on it. I saw that there is no defconfig file for this boot method (using FlexSPI) so I took imx8mp_evk_defconfig and modified it manually to the attached

Re: [PATCH v2 6/6] usb: kbd: Add probe quirk for Apple and Keychron keyboards

2024-03-17 Thread Marek Vasut
On 3/17/24 12:07 PM, Janne Grunau via B4 Relay wrote: From: Janne Grunau Those keyboards do not return the current device state. Polling will timeout unless there are key presses. This is not a problem during operation but the inital device state query during probing will fail. Skip this step

Re: [PATCH v2 5/6] usb: kbd: support Apple Magic Keyboards (2021)

2024-03-17 Thread Marek Vasut
On 3/17/24 12:07 PM, Janne Grunau via B4 Relay wrote: From: Janne Grunau Apple USB keyboards (Magic Keyboard from 2021 (product id 0x029c)) carry the HID keyboard boot protocol on the second interface descriptor. Probe via vendor and product IDs since the class/subclass/protocol match uses the

Re: [PATCH v2 4/6] usb: Add environment based device blocklist

2024-03-17 Thread Marek Vasut
On 3/17/24 12:07 PM, Janne Grunau via B4 Relay wrote: From: Janne Grunau Add the environment variable "usb_blocklist" to prevent USB devices listed in it from being used. This allows to ignore devices which trigger bugs in u-boot's USB stack or are undesirable for other reasons. Devices

Re: [PATCH v2 4/6] usb: Add environment based device blocklist

2024-03-17 Thread Marek Vasut
On 3/17/24 12:34 PM, Janne Grunau wrote: Hej, On Sun, Mar 17, 2024, at 12:07, Janne Grunau via B4 Relay wrote: From: Janne Grunau Add the environment variable "usb_blocklist" to prevent USB devices listed in it from being used. This allows to ignore devices which trigger bugs in u-boot's USB

Re: [PATCH v2 3/6] usb: xhci: Abort transfers with unallocated rings

2024-03-17 Thread Marek Vasut
On 3/17/24 12:07 PM, Janne Grunau via B4 Relay wrote: From: Janne Grunau Discovered while trying to use the second interface in the USB keyboard driver necessary on Apple USB keyboards. Signed-off-by: Janne Grunau Reviewed-by: Marek Vasut

Re: [PATCH 0/4] configs: apple: Switch to standard boot + small adjustments

2024-03-17 Thread Neal Gompa
++ > include/configs/apple.h| 24 > 3 files changed, 8 insertions(+), 21 deletions(-) > --- > base-commit: f3c979dd0053c082d2df170446923e7ce5edbc2d > change-id: 20240317-apple_config-981fcd9028b9 > LGTM. Reviewed-by: Neal Gompa -- 真実はいつも一つ!/ Always, there's only one truth!

[PATCH 1/4] apple_m1_defconfig: Turn on CONFIG_SYS_64BIT_LBA

2024-03-17 Thread Janne Grunau via B4 Relay
From: Hector Martin This makes USB HDDs >2TiB work. The only reason this hasn't bitten us for the internal NVMe yet is the 4K sector size, because the largest SSD Apple sells is 8TB and we can handle up to 16TiB with that sector size. Close call. Signed-off-by: Hector Martin Signed-off-by:

[PATCH 3/4] configs: apple: Enable CMD_SELECT_FONT and FONT_16X32

2024-03-17 Thread Janne Grunau via B4 Relay
From: Janne Grunau Apple devices have high DPI displays so the larger fonts are preferable for improved readability. This does not yet change the used font based on the display's pixel density so the standard 8x16 font is still used by default. Signed-off-by: Janne Grunau ---

[PATCH 2/4] configs: apple: Use "vidconsole,serial" as stdout/stderr

2024-03-17 Thread Janne Grunau via B4 Relay
From: Janne Grunau The display size querying in efi_console relies on this order. The display should be the primary output device and should be used to display more than 80x25 chars. Signed-off-by: Janne Grunau --- include/configs/apple.h | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH 4/4] arm: apple: Switch to standard boot

2024-03-17 Thread Janne Grunau via B4 Relay
From: Janne Grunau Use standard boot instead of the distro boot scripts. Signed-off-by: Janne Grunau --- arch/arm/Kconfig| 2 +- include/configs/apple.h | 20 ++-- 2 files changed, 3 insertions(+), 19 deletions(-) diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig

[PATCH 0/4] configs: apple: Switch to standard boot + small adjustments

2024-03-17 Thread Janne Grunau via B4 Relay
change-id: 20240317-apple_config-981fcd9028b9 Best regards, -- Janne Grunau

Re: [PATCH v2 4/6] usb: Add environment based device blocklist

2024-03-17 Thread Janne Grunau
Hej, On Sun, Mar 17, 2024, at 12:07, Janne Grunau via B4 Relay wrote: > From: Janne Grunau > > Add the environment variable "usb_blocklist" to prevent USB devices > listed in it from being used. This allows to ignore devices which > trigger bugs in u-boot's USB stack or are undesirable for other

Re: [PATCH 3/3] boot: fdt: Move usable variable below updated comment

2024-03-17 Thread Laurent Pinchart
Hi Marek, Thank you for the patch. On Sun, Mar 17, 2024 at 07:16:31AM +0100, Marek Vasut wrote: > Move the variable below comment which explains what the variable means. > Update the comment. No functional change. > > Suggested-by: Laurent Pinchart > Signed-off-by: Marek Vasut > --- > Cc:

Re: [PATCH v2 0/6] USB keyboard improvements for asahi / desktop systems

2024-03-17 Thread Neal Gompa
On Sun, Mar 17, 2024 at 4:07 AM Janne Grunau via B4 Relay wrote: > > Apple USB Keyboards from 2021 need quirks to be useable. The boot HID > keyboard protocol is unfortunately not described in the first interface > descriptor but the second. This needs several changes. The USB keyboard > driver

Re: [PATCH 2/3] boot: fdt: Drop lmb_alloc*() typecasts

2024-03-17 Thread Laurent Pinchart
Hi Marek, Thank you for the patch. On Sun, Mar 17, 2024 at 07:16:30AM +0100, Marek Vasut wrote: > The lmb_alloc_base() returns phys_addr_t , map_sysmem() accepts > phys_addr_t as first parameter. Declare $addr as phys_addr_t and s/$addr/addr/ ? > get rid of the casts. > > Reported-by: Laurent

Re: [PATCH 1/3] boot: fdt: Change type of env_get_bootm_low() to phys_addr_t

2024-03-17 Thread Laurent Pinchart
Hi Marek, Thank you for the patch. On Sun, Mar 17, 2024 at 07:16:29AM +0100, Marek Vasut wrote: > Change type of ulong env_get_bootm_low() to phys_addr_t env_get_bootm_low(). > The PPC/LS systems already treat env_get_bootm_low() result as phys_addr_t, > while the function itself still returns

[PATCH v2 0/6] USB keyboard improvements for asahi / desktop systems

2024-03-17 Thread Janne Grunau via B4 Relay
Apple USB Keyboards from 2021 need quirks to be useable. The boot HID keyboard protocol is unfortunately not described in the first interface descriptor but the second. This needs several changes. The USB keyboard driver has to look at all (2) interface descriptors during probing. Since I didn't

[PATCH v2 5/6] usb: kbd: support Apple Magic Keyboards (2021)

2024-03-17 Thread Janne Grunau via B4 Relay
From: Janne Grunau Apple USB keyboards (Magic Keyboard from 2021 (product id 0x029c)) carry the HID keyboard boot protocol on the second interface descriptor. Probe via vendor and product IDs since the class/subclass/protocol match uses the first interface descriptor. Probe the two first

[PATCH v2 1/6] usb: xhci: refactor xhci_set_configuration

2024-03-17 Thread Janne Grunau via B4 Relay
From: Janne Grunau In the next step endpoints for multiple interfaces are set up. Move most of the per endpoint initialization to separate function to avoid another identation level. Reviewed-by: Marek Vasut Signed-off-by: Janne Grunau --- drivers/usb/host/xhci.c | 119

[PATCH v2 6/6] usb: kbd: Add probe quirk for Apple and Keychron keyboards

2024-03-17 Thread Janne Grunau via B4 Relay
From: Janne Grunau Those keyboards do not return the current device state. Polling will timeout unless there are key presses. This is not a problem during operation but the inital device state query during probing will fail. Skip this step in usb_kbd_probe_dev() to make these devices useable.

[PATCH v2 2/6] usb: xhci: Set up endpoints for the first 2 interfaces

2024-03-17 Thread Janne Grunau via B4 Relay
From: Janne Grunau The xhci driver currently only does the necessary initialization for endpoints found in the first interface descriptor. Apple USB keyboards (released 2021) use the second interface descriptor for the HID keyboard boot protocol. To allow USB drivers to use endpoints from other

[PATCH v2 4/6] usb: Add environment based device blocklist

2024-03-17 Thread Janne Grunau via B4 Relay
From: Janne Grunau Add the environment variable "usb_blocklist" to prevent USB devices listed in it from being used. This allows to ignore devices which trigger bugs in u-boot's USB stack or are undesirable for other reasons. Devices emulating keyboards are one example. U-boot currently supports

[PATCH v2 3/6] usb: xhci: Abort transfers with unallocated rings

2024-03-17 Thread Janne Grunau via B4 Relay
From: Janne Grunau Discovered while trying to use the second interface in the USB keyboard driver necessary on Apple USB keyboards. Signed-off-by: Janne Grunau --- drivers/usb/host/xhci-ring.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/usb/host/xhci-ring.c

Re: [PATCH v3 7/7] efi_selftest: Update StrToFat() unit test after CP473 map extension

2024-03-17 Thread Heinrich Schuchardt
On 3/16/24 22:50, Janne Grunau via B4 Relay wrote: From: Janne Grunau Test that Unicode code points which map to CP437 code points 1-31 are converted to '_'. This ensures no FAT file names do not contain chars which are control characters in other code pages (CP 1250 for example).

Re: [PATCH v3 6/7] efi_selftest: Add geometric shapes character selftest

2024-03-17 Thread Heinrich Schuchardt
On 3/16/24 22:50, Janne Grunau via B4 Relay wrote: From: Janne Grunau Draw symbols from Unicode's "Geometric shapes" page which translate to code page 437 code points 1-31. These are used by UEFI applications to draw user interfaces using EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL. The output has to be

Re: [PATCH v3 5/7] efi_selftest: Add box drawing character selftest

2024-03-17 Thread Heinrich Schuchardt
On 3/16/24 22:50, Janne Grunau via B4 Relay wrote: From: Andre Przywara UEFI applications rely on Unicode output capability, and might use that for drawing pseudo-graphical interfaces using Unicode defined box drawing characters. Add a simple test to display the most basic box characters,

Re: [PATCH v3 1/7] lib: charset: Fix and extend utf8_to_utf32_stream() documentation

2024-03-17 Thread Heinrich Schuchardt
On 3/16/24 22:50, Janne Grunau via B4 Relay wrote: From: Janne Grunau Suggested-by: Heinrich Schuchardt Reviewed-by: Heinrich Schuchardt Signed-off-by: Janne Grunau This patch was already merged. No need to resend it. Best regards Heinrich --- include/charset.h | 14 --

Re: [PATCH v3 4/7] efi_selftest: Add international characters test

2024-03-17 Thread Heinrich Schuchardt
On 3/16/24 22:50, Janne Grunau via B4 Relay wrote: From: Andre Przywara UEFI relies entirely on unicode output, which actual fonts displayed on the screen might not be ready for. Add a test displaying some international characters, to reveal missing glyphs, especially in our builtin fonts.

Re: [PATCH 2/3] boot: fdt: Drop lmb_alloc*() typecasts

2024-03-17 Thread Heinrich Schuchardt
On 3/17/24 07:16, Marek Vasut wrote: The lmb_alloc_base() returns phys_addr_t , map_sysmem() accepts phys_addr_t as first parameter. Declare $addr as phys_addr_t and %s/$addr/addr/ get rid of the casts. Reported-by: Laurent Pinchart Signed-off-by: Marek Vasut --- Cc: Heinrich Schuchardt

Re: [PATCH 1/3] boot: fdt: Change type of env_get_bootm_low() to phys_addr_t

2024-03-17 Thread Heinrich Schuchardt
On 3/17/24 07:16, Marek Vasut wrote: Change type of ulong env_get_bootm_low() to phys_addr_t env_get_bootm_low(). The PPC/LS systems already treat env_get_bootm_low() result as phys_addr_t, while the function itself still returns ulong. This is potentially dangerous on 64bit systems, where ulong

[PATCH] ARM: dts: renesas: Stop using the -u-boot DTs for build

2024-03-17 Thread Marek Vasut
The U-Boot build system can automatically paste -u-boot.dtsi at the end of matching .dts during build. Stop emulating this behavior and rename the -u-boot.dts files to -u-boot.dtsi, drop "#include...dts" from those new u-boot.dtsi files, and update board configuration accordingly. The rename,

[PATCH 3/3] boot: fdt: Move usable variable below updated comment

2024-03-17 Thread Marek Vasut
Move the variable below comment which explains what the variable means. Update the comment. No functional change. Suggested-by: Laurent Pinchart Signed-off-by: Marek Vasut --- Cc: Heinrich Schuchardt Cc: Kuninori Morimoto Cc: Laurent Pinchart Cc: Simon Glass Cc: Tom Rini ---

[PATCH 2/3] boot: fdt: Drop lmb_alloc*() typecasts

2024-03-17 Thread Marek Vasut
The lmb_alloc_base() returns phys_addr_t , map_sysmem() accepts phys_addr_t as first parameter. Declare $addr as phys_addr_t and get rid of the casts. Reported-by: Laurent Pinchart Signed-off-by: Marek Vasut --- Cc: Heinrich Schuchardt Cc: Kuninori Morimoto Cc: Laurent Pinchart Cc: Simon

[PATCH 1/3] boot: fdt: Change type of env_get_bootm_low() to phys_addr_t

2024-03-17 Thread Marek Vasut
Change type of ulong env_get_bootm_low() to phys_addr_t env_get_bootm_low(). The PPC/LS systems already treat env_get_bootm_low() result as phys_addr_t, while the function itself still returns ulong. This is potentially dangerous on 64bit systems, where ulong might not be large enough to hold the

Re: [PATCH 1/3] mmc: Convert hs400_tuning flag from u8 to bool

2024-03-17 Thread Marek Vasut
On 2/20/24 3:41 PM, Paul Barker wrote: On 20/02/2024 11:27, Marek Vasut wrote: On 2/20/24 11:57, Paul Barker wrote: On 20/02/2024 08:37, Marek Vasut wrote: This hs400_tuning is a flag, make it bool. No functional change. This will be useful in the following patch, which adds another more