Re: [PATCH v2] gpio: adp5585: add gpio driver for ADP5585 I/O Expander Controller

2022-10-08 Thread Sean Anderson
Hi Alice, On 10/8/22 23:19, Alice Guo (OSS) wrote: From: Alice Guo Add gpio driver for ADP5585 I/O Expander Controller. The ADP5585 is a 10 input/output port expander and can be used to increase the number of I/Os available to a processor. Signed-off-by: Alice Guo --- Changes for v2: -

[PATCH 0/3] clang-14 fixes

2022-10-08 Thread Simon Glass
This series contains a few clang-14 fixes to partly resolve the problems shown by [1]. The event_dump problem is that the symbol name is not printed by addr2line even though it seems to be present: Without LTO: $ addr2line -e u-boot.clang14.no-lto 4a37f addr2line: DWARF error: invalid or

[PATCH 3/3] test: Drop unwanted option in event_dump.py

2022-10-08 Thread Simon Glass
This option is not used. Drop it. Signed-off-by: Simon Glass --- scripts/event_dump.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/scripts/event_dump.py b/scripts/event_dump.py index 6aadddf28da..d87823f3749 100755 --- a/scripts/event_dump.py +++ b/scripts/event_dump.py @@ -108,8

[PATCH 2/3] event: Drop the path when checking event-list filenames

2022-10-08 Thread Simon Glass
This path does not seem to be present in clang-14 for some reason. Relax the regular expression so that the test works, at least for non-LTO. Signed-off-by: Simon Glass --- test/py/tests/test_event_dump.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 1/3] dm: core: Fix lists_bind_fdt() using non-existent of_match

2022-10-08 Thread Simon Glass
The call to device_bind_with_driver_data() passes id->data but if the entry has no of_match then the id has not been set by the selected driver. Normally this passes unnoticed since a previous driver likely had an of_match value, so the id is set to that. Of course it is not correct to pass the

[PATCH v2] gpio: adp5585: add gpio driver for ADP5585 I/O Expander Controller

2022-10-08 Thread Alice Guo (OSS)
From: Alice Guo Add gpio driver for ADP5585 I/O Expander Controller. The ADP5585 is a 10 input/output port expander and can be used to increase the number of I/Os available to a processor. Signed-off-by: Alice Guo --- Changes for v2: - add a commit log - remove unrelated change - remove

Re: [PATCH] arm64: layerscape: Support SYSRESET

2022-10-08 Thread Peng Fan
On 9/27/2022 12:47 AM, Sean Anderson wrote: CONFIG_SYSRESET provides its own implementation of reset_cpu. Disable our version when it is enabled. Signed-off-by: Sean Anderson Queued in fsl-qoriq/next Thanks, Peng. --- arch/arm/cpu/armv8/fsl-layerscape/cpu.c | 2 ++ 1 file changed,

Re: [PATCH] ls1043ardb: nand driver fixups for revision v7.0 boards

2022-10-08 Thread Peng Fan
On 9/26/2022 4:18 PM, Wei Lu wrote: The LS1043ARDB rev v7.0 board replace nand device MT29F4G08ABBDAH4-AITX:D with MT29F4G08ABBFAH4-AIT:F. Reflecting this change in board_fix_fdt(). CPLD V3.0 is needed for nandboot as the nand device changed. A new macro CPLD_CFG_RCW_SRC_NAND_4K(4Kpage) is

Re: [PATCH] arm64: ls104x: Enable eDMA snooping

2022-10-08 Thread Peng Fan
On 9/24/2022 12:05 AM, Sean Anderson wrote: This enables eDMA snooping on the LS1043A and LS1046A. This will allow marking the I2C, LPUART, and SPI devices on these SoCs as DMA coherent. Oddly, this bit is only documented for the LS1043A, and is marked as "reserved" in the LS1046ARM. I have

[PATCH] Fix typo in CONFIG_USBNET_DEVADDR

2022-10-08 Thread nachopitt
From: Ignacio Zamora Fix typo that was causing the same feature to be splitted into 2 different configuration options Replacing CONFIG_USBNET_DEVADDR with CONFIG_USBNET_DEV_ADDR Signed-off-by: Ignacio Zamora --- drivers/usb/gadget/Kconfig | 2 +- drivers/usb/gadget/ether.c | 2 +- 2 files

[PATCH 1/1] riscv: support building double-float modules

2022-10-08 Thread Heinrich Schuchardt
The riscv32 toolchain for GCC-12 provided by kernel.org contains libgcc.a compiled for double-float. To link to it we have to adjust how we build U-Boot. As U-Boot actually does not use floating point at all this should not make a significant difference for the produced binaries. Signed-off-by:

[PATCH 1/1] xilinx: common: fix board_late_init_xilinx()

2022-10-08 Thread Heinrich Schuchardt
Compiling with GCC-12 leads to an error: +board/xilinx/common/board.c:479:37: error: the comparison will always evaluate as 'true' for the address of 'mac_addr' will never be NULL [-Werror=address] + 479 | if (!desc->mac_addr[i]) + |

[RFC] riscv32 compiler flags

2022-10-08 Thread Heinrich Schuchardt
Hello Rick, hello Leo, in origin master we have moved to compiling riscv32 using the GCC the 32bit toolchain provided by kernel.org. This works fine with GCC-11. Tom now tried to move to GCC-12. Here libgcc.a is compiled with: readelf -h

[PATCH 1/2] efi_loader: internal CloseProtocol

2022-10-08 Thread Heinrich Schuchardt
Allow avoiding using EFI_CALL() when closing a protocol by providing an internal function. Signed-off-by: Heinrich Schuchardt --- include/efi_loader.h | 9 ++-- lib/efi_loader/efi_boottime.c | 90 ++- 2 files changed, 61 insertions(+), 38 deletions(-)

[PATCH 2/2] efi_driver: use efi_close_protocol

2022-10-08 Thread Heinrich Schuchardt
Avoid EFI_CALL() by using efi_close_protocol(). Signed-off-by: Heinrich Schuchardt --- lib/efi_driver/efi_uclass.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git a/lib/efi_driver/efi_uclass.c b/lib/efi_driver/efi_uclass.c index

[PATCH 0/2] efi_loader: internal CloseProtocol

2022-10-08 Thread Heinrich Schuchardt
Get rid of EFI_CALL() for invoking CloseProtocol(). Heinrich Schuchardt (2): efi_loader: internal CloseProtocol efi_driver: use efi_close_protocol include/efi_loader.h | 9 ++-- lib/efi_driver/efi_uclass.c | 25 +- lib/efi_loader/efi_boottime.c | 90

[PATCH 1/1] efi_loader: simplify efi_set_load_options()

2022-10-08 Thread Heinrich Schuchardt
* Replace the OpenProtocol() call by efi_search_protocol(). * Remove the CloseProtocol() call. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_load_options.c | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/lib/efi_loader/efi_load_options.c

[PATCH 1/1] efi_loader: provide agent_handle to efi_disk_add_dev()

2022-10-08 Thread Heinrich Schuchardt
In efi_disk_add_dev() we have to open protocols with BY_DRIVER and BY_CHILD_CONTROLLER. Provide the handle of the EFI block driver. The actual usage of the value will follow in a later patch. Change function descriptions to Sphinx style. Remove a TODO: tag. Signed-off-by: Heinrich Schuchardt

[PATCH 1/1] efi_loader: reformat efi_disk_add_dev()

2022-10-08 Thread Heinrich Schuchardt
Make it clearer why InstallMultipleProtocolInterfaces is invoked with two NULLs: * rename guid to esp_guid * put protocol GUIDs and the related interfaces on same lines * add comment Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_disk.c | 20 1 file changed, 12

Re: [PATCH] efi_driver: fix efi_uc_stop()

2022-10-08 Thread Ilias Apalodimas
On Sat, 8 Oct 2022 at 00:55, Heinrich Schuchardt wrote: > > We must always call EFI_EXIT() when returning from an EFIAPI function. > > Fixes: 05ef48a2484b ("efi_driver: EFI block driver") > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_driver/efi_uclass.c | 5 +++-- > 1 file changed, 3