Re: [PATCH] arm64: zynqmp: Setup default si570 frequency to 156.25MHz

2023-08-31 Thread Michal Simek
On 8/25/23 09:11, Michal Simek wrote: All si570 mgt chips have factory default 156.25MHz but DT changed it to 148.5MHz. After tracking it is pretty much c&p fault taken from Zynq zc702/zc706 boards where 148.5MHz was setup as default because it was requirement for AD7511 chip available on thes

Re: [PATCH] arm64: zynqmp: Add support for K26 rev2 boards

2023-08-31 Thread Michal Simek
On 8/25/23 10:10, Michal Simek wrote: Revision 2 is SW compatible with revision 1 but it is necessary to reflect it in model and compatible properties which are parsed by user space. Rev 2 has improved a power on boot reset and MIO34 shutdown glich improvement done via an additional filter in

Re: [PATCH v2] fpga: define dummy fpga_load function for debug build

2023-08-31 Thread Michal Simek
On 8/31/23 09:52, Chanho Park wrote: This fixes below build error when CC_OPTIMIZE_FOR_DEBUG is enabled and CONFIG_FPGA or CONFIG_SPL_FPGA are not enabled. When CC_OPTIMIZE_FOR_DEBUG is enabled, unused code will not be optimized out. Hence, fpga_load function must have a dummy implementation t

Re: [PATCH v2 14/14] fs: Fix a confusing error about overlapping regions

2023-08-31 Thread Heinrich Schuchardt
On 9/1/23 03:13, Simon Glass wrote: 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. A

Re: [PATCH v2 02/14] lmb: Rename interior piece to area

2023-08-31 Thread Heinrich Schuchardt
On 9/1/23 03:13, Simon Glass wrote: The lmb data structures use the word 'region' to describe the region in which the allocations happen, as well as the individual allocations in that region. The interior structure is called lmb_property but is described as a region. This is quite confusing. One

[PATCH 1/2] thermal: k3-j72xx-bandgap: Add support for vtm

2023-08-31 Thread Udit Kumar
Add support for VTM (Voltage and Thermal Manager), which is part of TI's K3 series SOC. Signed-off-by: Udit Kumar --- drivers/thermal/Kconfig| 6 + drivers/thermal/Makefile | 1 + drivers/thermal/k3_j72xx_bandgap.c | 539 + 3 files changed, 5

[PATCH 2/2] DONOTMERGE: j7200: device tree and defconfig update

2023-08-31 Thread Udit Kumar
Enable VTM node in device tree and update defconfig for vtm. Signed-off-by: Udit Kumar --- arch/arm/dts/k3-j7200-mcu-wakeup.dtsi | 9 + configs/j7200_evm_a72_defconfig | 3 +++ 2 files changed, 12 insertions(+) diff --git a/arch/arm/dts/k3-j7200-mcu-wakeup.dtsi b/arch/arm/dts/k3-

[PATCH 0/2] thermal: Add VTM driver support for J72xx SOCs

2023-08-31 Thread Udit Kumar
This series adds VTM (Voltage and Thermal Manager) driver support for J72xx SOCs. Basically this driver is ported from kernel v6.5 to u-boot. Main purpose of this driver to program thermal shutdown value and provide SOC temperature. To use this driver, update in device tree and defconfig are nee

Re: [PATCH 5/6] net: zynq: Use generic_phy_valid() helper

2023-08-31 Thread Michal Simek
On 9/1/23 00:16, Jonas Karlman wrote: The documentation for struct phy state that "The content of the structure is managed solely by the PHY API and PHY drivers". Change to use the generic_phy_valid() helper to check if phy is valid. Fixes: 10c50b1facbf ("net: zynq: Add support for PHY confi

Re: [PATCH 4/6] scsi: ceva: Use generic_phy_valid() helper

2023-08-31 Thread Michal Simek
On 9/1/23 00:16, Jonas Karlman wrote: The documentation for struct phy state that "The content of the structure is managed solely by the PHY API and PHY drivers". Change to use the generic_phy_valid() helper to check if phy is valid. Fixes: f6f5451d469b ("scsi: ceva: Enable PHY and reset sup

Re: [PATCH RESEND 1/2] arm: dts: rockchip: rk3588, rk3588s: sync with Linux

2023-08-31 Thread Jonas Karlman
Hi Naoki, On 2023-09-01 06:58, FUKAUMI Naoki wrote: > Sync the devicetree with linux-next tag: next-20230831 With the soc-dt-6.6 tag now merged into torvalds/linux tree [1] we can probably call this a sync from v6.6-rc1. However, this patch misses some things: - It does not remove synced p

[PATCH] tools: Fix patman launcher script.

2023-08-31 Thread Maxim Cournoyer
There is no "run_patman" procedure in patman's __main__.py file, which would cause the following error at execution: "AttributeError: module 'patman.__main__' has no attribute 'run_patman'" Signed-off-by: Maxim Cournoyer --- tools/patman/pyproject.toml | 2 +- 1 file changed, 1 insertion(+),

[PATCH RESEND 1/2] arm: dts: rockchip: rk3588, rk3588s: sync with Linux

2023-08-31 Thread FUKAUMI Naoki
Sync the devicetree with linux-next tag: next-20230831 Signed-off-by: FUKAUMI Naoki --- arch/arm/dts/rk3588.dtsi | 215 +++ arch/arm/dts/rk3588s.dtsi | 367 + include/dt-bindings/ata/ahci.h | 20 ++ 3 files changed, 602 insertions

[PATCH RESEND 2/2] arm: dts: rockchip: rock-5b: add support for PCIe3 and NVMe

2023-08-31 Thread FUKAUMI Naoki
this patch adds support for PCIe3 (M.2 M key) and enables NVMe. => pci BusDevFun VendorId DeviceId Device Class Sub-Class _ 00.00.00 0x1d87 0x3588 Bridge device 0x04 01.00.00 0x10ec 0x8125 Net

[PATCH 2/2] arm: dts: rockchip: rock-5b: add support for PCIe3 and NVMe

2023-08-31 Thread FUKAUMI Naoki
this patch adds support for PCIe3 (M.2 M key) and enables NVMe. => pci BusDevFun VendorId DeviceId Device Class Sub-Class _ 00.00.00 0x1d87 0x3588 Bridge device 0x04 01.00.00 0x10ec 0x8125 Network

[PATCH 1/2] arm: dts: rockchip: rk3588, rk3588s: sync with Linux

2023-08-31 Thread FUKAUMI Naoki
Sync the devicetree with linux-next tag: next-20230831 Signed-off-by: FUKAUMI Naoki --- arch/arm/dts/rk3588.dtsi | 215 +++ arch/arm/dts/rk3588s.dtsi | 367 + include/dt-bindings/ata/ahci.h | 20 ++ 3 files changed, 602 insertions

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

2023-08-31 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 v2 13/14] lmb: Document and tidy lmb_add_region_flags()

2023-08-31 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 --- (no changes since v1) lib/lmb.c | 47

[PATCH v2 12/14] lmb: Tidy up lmb_overlaps_region()

2023-08-31 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 --- (no changes since v1) lib/lmb.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/lib/lmb.c b/lib/lmb.c index 0c6244ca47f8..8a299c97b3ad 100

[PATCH v2 11/14] lmb: Change functions returning long to return int

2023-08-31 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 --- (no changes since v1) include/lmb.h | 10 +- lib/lmb.c | 10 +- 2

[PATCH v2 10/14] lmb: Avoid long for loop counters and function returns

2023-08-31 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 --- (no changes since v1) lib/lmb.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/lib/lmb.c b/lib

[PATCH v2 09/14] lmb: Tidy up structure access

2023-08-31 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 --- (no changes since v1) lib/lmb.c | 58

[PATCH v2 08/14] lmb: Update use of region in the C file

2023-08-31 Thread Simon Glass
Change places that should say area to do so. Change the name of the 'rgn' variable to 'area' in places where it refers to an area number. Fix the 'beginging' typo while here. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch to update use of region in the C file lib/lmb.c | 126 ++

[PATCH v2 07/14] lmb: Update use of region in the header file

2023-08-31 Thread Simon Glass
Tidy up places in the header file where it says region but now means area. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch to update use of region in the header file include/lmb.h | 44 ++-- 1 file changed, 22 insertions(+), 22 deletions(-)

[PATCH v2 06/14] lmb: Rename LMB_MAX_REGIONS and LMB_USE_MAX_REGIONS

2023-08-31 Thread Simon Glass
These refer to the maximum number of areas, so rename them. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch to rename LMB_MAX_REGIONS and LMB_USE_MAX_REGIONS configs/a3y17lte_defconfig | 2 +- configs/a5y17lte_defconfig | 2 +- configs/a7y17lte_defconfig

[PATCH v2 05/14] lmb: Rename LMB_MAX_REGIONS and LMB_USE_MAX_REGIONS

2023-08-31 Thread Simon Glass
These are the number of areas within a region, so rename them. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch to rename LMB_MAX_REGIONS and LMB_USE_MAX_REGIONS configs/stm32mp13_defconfig | 4 ++-- configs/stm32mp15_basic_defconfig | 4 ++-- configs/stm32mp15_defconf

[PATCH v2 04/14] lmb: Rename memory/reserved_regions to area

2023-08-31 Thread Simon Glass
Rename these two fields since they are areas, not regions. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch to rename memory/reserved_regions to area include/lmb.h | 8 lib/lmb.c | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/include/lmb.h b/

[PATCH v2 03/14] lmb: Rename region array to area

2023-08-31 Thread Simon Glass
Adjust this so that the struct member matches its new name. For now, some of the comments are wrong, but it difficult to do everything at once. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch to rename region array to area include/lmb.h | 14 +++--- lib/lmb.c | 112 +

[PATCH v2 02/14] lmb: Rename interior piece to area

2023-08-31 Thread Simon Glass
The lmb data structures use the word 'region' to describe the region in which the allocations happen, as well as the individual allocations in that region. The interior structure is called lmb_property but is described as a region. This is quite confusing. One of the reviewers in v1 asked why we a

[PATCH v2 01/14] lmb: Drop surplus brackets and fix code style

2023-08-31 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 --- (no changes since v1) lib/lmb.c | 17 - 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/lib/lmb

[PATCH v2 00/14] Correct confusing lmb error message

2023-08-31 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. Feedbac

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

2023-08-31 Thread Simon Glass
Hi Ard, On Thu, 31 Aug 2023 at 16:39, Ard Biesheuvel wrote: > > On Fri, 1 Sept 2023 at 00:17, Simon Glass wrote: > > > > Hi Ard, > > > > On Thu, 31 Aug 2023 at 15:48, Ard Biesheuvel wrote: > > > > > > On Thu, 31 Aug 2023 at 21:03, Simon Glass wrote: > > > > > > > > Hi Ard, > > > > > > > > On T

Re: [PATCH 0/6] Attempt to enforce standard extensions for build output

2023-08-31 Thread Simon Glass
Hi Tom, On Thu, 31 Aug 2023 at 13:24, Tom Rini wrote: > > On Thu, Aug 31, 2023 at 01:22:13PM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Thu, 31 Aug 2023 at 13:07, Tom Rini wrote: > > > > > > On Thu, Aug 31, 2023 at 01:01:59PM -0600, Simon Glass wrote: > > > > Hi Alper, > > > > > > > > On T

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

2023-08-31 Thread Simon Glass
Hi Tom, On Wed, 23 Aug 2023 at 09:14, Tom Rini wrote: > > On Wed, Aug 23, 2023 at 07:42:03AM -0600, Simon Glass wrote: > > > 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 r

[PATCH v2 3/3] phy: Refactor generic_{setup, shutdown}_phy() to reduce complexity

2023-08-31 Thread Jonas Karlman
Refactor generic_{setup,shutdown}_phy() to reduce complexity and indentation. This have no intended functional change. Signed-off-by: Jonas Karlman --- v2: - Split code refactor into own patch drivers/phy/phy-uclass.c | 41 1 file changed, 16 insertions(

[PATCH v2 2/3] phy: Return success from generic_setup_phy() when phy is not found

2023-08-31 Thread Jonas Karlman
Restore the old behavior of ehci_setup_phy() and ohci_setup_phy() to return success when generic_phy_get_by_index() return -ENOENT. Fixes: 84e561407a5f ("phy: Add generic_{setup,shutdown}_phy() helpers") Fixes: 10005004db73 ("usb: ohci: Make usage of generic_{setup,shutdown}_phy() helpers") Fixes

[PATCH v2 1/3] phy: Fix generic_setup_phy() return value on power on failure

2023-08-31 Thread Jonas Karlman
generic_phy_exit() typically return 0 for a struct phy that has been initialized with a generic_phy_init() call. generic_setup_phy() returns the value from a generic_phy_exit() call when generic_phy_power_on() fails. This hides the failed state of the power_on ops from the caller of generic_setup_

[PATCH v2 0/3] phy: Fix generic_setup_phy return value on power on failure

2023-08-31 Thread Jonas Karlman
This series fixes two issues related to the return value in case of error from generic_setup_phy(). Patch 1 fixes that the error code from power_on gets returned. Patch 2 fixes an regression so that success is returned instead of ENOENT. Patch 3 refactor generic_{setup,shutdown}_phy() to improve r

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

2023-08-31 Thread Ard Biesheuvel
On Fri, 1 Sept 2023 at 00:17, Simon Glass wrote: > > Hi Ard, > > On Thu, 31 Aug 2023 at 15:48, Ard Biesheuvel wrote: > > > > On Thu, 31 Aug 2023 at 21:03, Simon Glass wrote: > > > > > > Hi Ard, > > > > > > On Thu, 31 Aug 2023 at 06:28, Ard Biesheuvel wrote: > > > > > > > > On Wed, 30 Aug 2023 a

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

2023-08-31 Thread Simon Glass
Hi Ard, On Thu, 31 Aug 2023 at 15:48, Ard Biesheuvel wrote: > > On Thu, 31 Aug 2023 at 21:03, Simon Glass wrote: > > > > Hi Ard, > > > > On Thu, 31 Aug 2023 at 06:28, Ard Biesheuvel wrote: > > > > > > On Wed, 30 Aug 2023 at 23:11, Simon Glass wrote: > > > > > > > > Hi Ard, > > > > > > > > On T

[PATCH 6/6] video: rockchip: dw_mipi_dsi: Use generic_phy_valid() helper

2023-08-31 Thread Jonas Karlman
The documentation for struct phy state that "The content of the structure is managed solely by the PHY API and PHY drivers". Change to use the generic_phy_valid() helper to check if phy is valid. Fixes: b7d8d40346f2 ("video: rockchip: dw_mipi_dsi: Fix external phy existence check") Signed-off-by

[PATCH 5/6] net: zynq: Use generic_phy_valid() helper

2023-08-31 Thread Jonas Karlman
The documentation for struct phy state that "The content of the structure is managed solely by the PHY API and PHY drivers". Change to use the generic_phy_valid() helper to check if phy is valid. Fixes: 10c50b1facbf ("net: zynq: Add support for PHY configuration in SGMII mode") Signed-off-by: Jo

[PATCH 4/6] scsi: ceva: Use generic_phy_valid() helper

2023-08-31 Thread Jonas Karlman
The documentation for struct phy state that "The content of the structure is managed solely by the PHY API and PHY drivers". Change to use the generic_phy_valid() helper to check if phy is valid. Fixes: f6f5451d469b ("scsi: ceva: Enable PHY and reset support") Signed-off-by: Jonas Karlman --- d

[PATCH 3/6] usb: dwc3: Use generic_phy_valid() helper

2023-08-31 Thread Jonas Karlman
The documentation for struct phy state that "The content of the structure is managed solely by the PHY API and PHY drivers". Change to use the generic_phy_valid() helper to check if phy is valid. Also remove setting phy->dev to NULL now that generic_phy_get_by_name() properly initialize phy->dev t

[PATCH 2/6] phy: Set phy->dev to NULL when generic_phy_get_by_index_nodev() fails

2023-08-31 Thread Jonas Karlman
Generic phy helpers typically use generic_phy_valid() to determine if the helper should perform its function on a passed struct phy. generic_phy_valid() treat any struct phy having phy->dev set as valid. With generic_phy_get_by_index_nodev() setting phy->dev to a valid struct udevice early, there

[PATCH 1/6] phy: Set phy->dev to NULL when generic_phy_get_by_name() fails

2023-08-31 Thread Jonas Karlman
generic_phy_get_by_name() does not initialize phy->dev to NULL before returning when dev_read_stringlist_search() fails. This can lead to an uninitialized or reused struct phy erroneously be report as valid by generic_phy_valid(). Fix this issue by initializing phy->dev to NULL, also extend the dm

[PATCH 0/6] phy: Fix use of generic_phy_valid() helper

2023-08-31 Thread Jonas Karlman
The documentation for struct phy state that "The content of the structure is managed solely by the PHY API and PHY drivers". Change to use the generic_phy_valid() helper to check if phy is valid. Patch 1-2 fixes issues where generic_phy_valid() erroneously report a phy that has failed to be initi

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

2023-08-31 Thread Ard Biesheuvel
On Thu, 31 Aug 2023 at 21:03, Simon Glass wrote: > > Hi Ard, > > On Thu, 31 Aug 2023 at 06:28, Ard Biesheuvel wrote: > > > > On Wed, 30 Aug 2023 at 23:11, Simon Glass wrote: > > > > > > Hi Ard, > > > > > > On Tue, 29 Aug 2023 at 15:32, Ard Biesheuvel wrote: > > > > > > > > On Tue, 29 Aug 2023 a

Re: [PATCH 00/14] event: Replace some more init hooks

2023-08-31 Thread Tom Rini
On Mon, 21 Aug 2023 21:16:47 -0600, Simon Glass wrote: > This series replaces some more of the init hooks in board_f.c and > board_r.c with events. Notably it converts last_state_init() over. > > It also provides a 'simple' event spy, which takes no arguments. It turns > out that this is quite a

[PATCH RESEND] sunxi: add axp313a support

2023-08-31 Thread SASANO Takayoshi
There is no response from previous post, I think this patch is not reached to suitable person and not reviewed yet. axp313a code is a part of power/sunxi. This is needed for Mango Pi MQ-Quad and Orange Pi Zero3. From 9127de42691f86ef5ed22dcf5fa0b44b03288a07 Mon Sep 17 00:00:00 2001 From: SASANO

Re: [PATCH v1 0/4] Port gen_compile_commands.py from Linux to U-Boot

2023-08-31 Thread Tom Rini
On Thu, Aug 31, 2023 at 09:21:19PM +0200, João Marcos Costa wrote: > Hello Simon, > > Em seg., 21 de ago. de 2023 às 21:13, Simon Glass > escreveu: > > > Hi Joao, > > > > Can you also please bring over the documentation for this feature? > > > > Actually, I couldn't find any documentation per s

Re: [PATCH 0/6] Attempt to enforce standard extensions for build output

2023-08-31 Thread Tom Rini
On Thu, Aug 31, 2023 at 01:22:13PM -0600, Simon Glass wrote: > Hi Tom, > > On Thu, 31 Aug 2023 at 13:07, Tom Rini wrote: > > > > On Thu, Aug 31, 2023 at 01:01:59PM -0600, Simon Glass wrote: > > > Hi Alper, > > > > > > On Thu, 31 Aug 2023 at 04:20, Alper Nebi Yasak > > > wrote: > > > > > > > > O

Re: [PATCH 0/6] Attempt to enforce standard extensions for build output

2023-08-31 Thread Simon Glass
Hi Tom, On Thu, 31 Aug 2023 at 13:07, Tom Rini wrote: > > On Thu, Aug 31, 2023 at 01:01:59PM -0600, Simon Glass wrote: > > Hi Alper, > > > > On Thu, 31 Aug 2023 at 04:20, Alper Nebi Yasak > > wrote: > > > > > > On 2023-08-28 20:54 +03:00, Simon Glass wrote: > > > > Hi Alper, > > > > > > > > On

Re: [PATCH v1 0/4] Port gen_compile_commands.py from Linux to U-Boot

2023-08-31 Thread João Marcos Costa
Hello Yannic, Em seg., 28 de ago. de 2023 às 10:09, Yannic Moog escreveu: > > Can you also add a patch to add the compile_commands.json to > .gitignore, please? > > Yannic > Absolutely. I will add such patch in the v2 series. Thanks for the suggestion. -- Atenciosamente, João Marcos Costa ww

Re: [PATCH v1 0/4] Port gen_compile_commands.py from Linux to U-Boot

2023-08-31 Thread João Marcos Costa
Hello Simon, Em seg., 21 de ago. de 2023 às 21:13, Simon Glass escreveu: > Hi Joao, > > Can you also please bring over the documentation for this feature? > Actually, I couldn't find any documentation per se (e.g. in linux/Documentation) besides what is already documented in the actual code, as

Re: [PATCH 0/6] Attempt to enforce standard extensions for build output

2023-08-31 Thread Tom Rini
On Thu, Aug 31, 2023 at 01:01:59PM -0600, Simon Glass wrote: > Hi Alper, > > On Thu, 31 Aug 2023 at 04:20, Alper Nebi Yasak > wrote: > > > > On 2023-08-28 20:54 +03:00, Simon Glass wrote: > > > Hi Alper, > > > > > > On Sun, 27 Aug 2023 at 13:17, Alper Nebi Yasak > > > wrote: > > >> > > >> On 2

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

2023-08-31 Thread Simon Glass
Hi Ard, On Thu, 31 Aug 2023 at 06:28, Ard Biesheuvel wrote: > > On Wed, 30 Aug 2023 at 23:11, Simon Glass wrote: > > > > Hi Ard, > > > > On Tue, 29 Aug 2023 at 15:32, Ard Biesheuvel wrote: > > > > > > On Tue, 29 Aug 2023 at 21:18, Simon Glass wrote: > > > > > > > > Hi Ard, > > > > > > > > On T

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

2023-08-31 Thread Simon Glass
Hi Ilias, On Thu, 31 Aug 2023 at 01:06, Ilias Apalodimas wrote: > > Hi Simon, > > On Wed, Aug 30, 2023 at 12:05:01PM -0600, Simon Glass wrote: > > Standard passage provides for a bloblist to be passed from one firmware > > phase to the next. That can be used to pass the devicetree along as well.

Re: [PATCH 0/6] Attempt to enforce standard extensions for build output

2023-08-31 Thread Simon Glass
Hi Alper, On Thu, 31 Aug 2023 at 04:20, Alper Nebi Yasak wrote: > > On 2023-08-28 20:54 +03:00, Simon Glass wrote: > > Hi Alper, > > > > On Sun, 27 Aug 2023 at 13:17, Alper Nebi Yasak > > wrote: > >> > >> On 2023-08-24 06:02 +03:00, Simon Glass wrote: > >>> In this early stage of using binman t

Re: [PATCH v2 2/2] xilinx: board: Add support to pick bootscr flash offset/size from DT

2023-08-31 Thread Simon Glass
On Thu, 31 Aug 2023 at 01:04, Michal Simek wrote: > > Location of bootscript in flash can be specified via /options/u-boot DT > node by using bootscr-flash-offset and bootscr-flash-size properties. > Values should be saved to script_offset_f and script_size_f variables. > Variables are described i

Re: [PATCH v2 4/4] doc: qemu: arm: Add a section on booting Linux distros

2023-08-31 Thread Simon Glass
Hi Alper, On Thu, 31 Aug 2023 at 03:57, Alper Nebi Yasak wrote: > > On 2023-08-28 20:54 +03:00, Simon Glass wrote: > > Hi Alper, > > > > On Sun, 27 Aug 2023 at 09:49, Alper Nebi Yasak > > wrote: > >> > >> On 2023-08-15 01:43 +03:00, Simon Glass wrote: > >>> Hi Alper, > >>> > >>> On Mon, 14 Aug

Re: [PATCH v2 1/4] fdt: common API to populate kaslr seed

2023-08-31 Thread Simon Glass
Hi Sean, On Tue, 29 Aug 2023 at 14:37, wrote: > > From: Dhananjay Phadke > > fdt_fixup_kaslr_seed() will update given ofnode with random seed value. > Source for random seed can be TPM or RNG driver in u-boot or sec > firmware (ARM). > > Signed-off-by: Dhananjay Phadke > Signed-off-by: Sean Edm

Re: [PATCH] driver: rng: Add DM_RNG interface for ARMv8.5 RNDR registers

2023-08-31 Thread Simon Glass
Hi Andre, On Thu, 31 Aug 2023 at 06:43, Andre Przywara wrote: > > On Wed, 30 Aug 2023 20:49:18 -0600 > Simon Glass wrote: > > Hi Simon, > > > On Wed, 30 Aug 2023 at 05:32, Andre Przywara wrote: > > > > > > The ARMv8.5 architecture extension defines architectural RNDR/RNDRRS > > > system registe

Re: [PATCH v2 3/4] cmd: kaslrseed: Use common API to fixup FDT

2023-08-31 Thread Simon Glass
Hi Sean, On Tue, 29 Aug 2023 at 14:37, wrote: > > From: Sean Edmond > > Use the newly introduced common API fdt_fixup_kaslr_seed() in the > kaslrseed command. > > Signed-off-by: Sean Edmond > --- > cmd/kaslrseed.c | 22 -- > 1 file changed, 8 insertions(+), 14 deletions(-)

Re: [PATCH v2 2/4] fdt: kaslr seed from tpm entropy

2023-08-31 Thread Simon Glass
Hi Sean, On Tue, 29 Aug 2023 at 14:37, wrote: > > From: Dhananjay Phadke > > Add support for KASLR seed from TPM device. Invokes tpm_get_random() > API to read 8-bytes of random bytes for KASLR. > > Signed-off-by: Dhananjay Phadke > Signed-off-by: Drew Kluemke > Signed-off-by: Sean Edmond > -

Re: [PATCH 1/2] Allow Python packages to be dropped

2023-08-31 Thread Simon Glass
Hi Tom, On Thu, 31 Aug 2023 at 11:48, Tom Rini wrote: > > On Thu, Aug 31, 2023 at 11:20:52AM -0600, Simon Glass wrote: > > > When building in a portage chroot, we do not have the environment needed > > to build pylibfdt. It is instead build as a separate package. > > > > Provide a build option to

Re: [PATCH] efi: Correct handling of frame buffer

2023-08-31 Thread Simon Glass
Hi Heinrich, On Fri, 25 Aug 2023 at 16:06, Heinrich Schuchardt wrote: > > On 8/25/23 21:28, Simon Glass wrote: > > The efi_gop driver uses private fields from the video uclass to obtain a > > pointer to the frame buffer. Use the platform data instead. > > > > Check the VIDEO_COPY setting to deter

Re: [PATCH] efi: Correct handling of frame buffer

2023-08-31 Thread Simon Glass
Hi, On Sun, 27 Aug 2023 at 12:41, Alper Nebi Yasak wrote: > > On 2023-08-27 19:32 +03:00, Heinrich Schuchardt wrote: > > Am 27. August 2023 16:56:51 MESZ schrieb Alper Nebi Yasak > > : > >> On 2023-08-25 22:28 +03:00, Simon Glass wrote: > >>> so we must allow EFI to write directly to the hardwar

Re: [PATCH v2] cmd: dm: allow for selecting uclass and device

2023-08-31 Thread Simon Glass
Hi, On Wed, 30 Aug 2023 at 22:34, AKASHI Takahiro wrote: > > On Wed, Aug 30, 2023 at 09:48:48PM -0600, Simon Glass wrote: > > Hi AKASHI, > > > > On Tue, 22 Aug 2023 at 19:50, AKASHI Takahiro > > wrote: > > > > > > The output from "dm tree" or "dm uclass" is a bit annoying > > > if the number of

Re: [PATCH 05/32] spl: mx6: powerpc: Drop the condition on timer_init()

2023-08-31 Thread Tom Rini
On Wed, Aug 30, 2023 at 12:04:36PM -0600, Simon Glass wrote: > It doesn't make sense to have some boards do this differently. Drop the > condition in the hope that the maintainers can figure out any run-time > problems. > > Signed-off-by: Simon Glass > --- > > common/spl/spl.c | 6 -- > 1

Re: [PATCH 1/2] Allow Python packages to be dropped

2023-08-31 Thread Tom Rini
On Thu, Aug 31, 2023 at 11:20:52AM -0600, Simon Glass wrote: > When building in a portage chroot, we do not have the environment needed > to build pylibfdt. It is instead build as a separate package. > > Provide a build option to tell U-Boot to skip this part of the build. We > still need it to u

[PATCH 2/2] x86: coreboot: Avoid a declaration after a label

2023-08-31 Thread Simon Glass
Declare the global_data pointer at the top of the file, to avoid an error: arch/x86/include/asm/global_data.h:143:35: error: a label can only be part of a statement and a declaration is not a statement board/coreboot/coreboot/coreboot.c:60:2: note: in expansion of macro ‘DECLARE_

[PATCH 1/2] Allow Python packages to be dropped

2023-08-31 Thread Simon Glass
When building in a portage chroot, we do not have the environment needed to build pylibfdt. It is instead build as a separate package. Provide a build option to tell U-Boot to skip this part of the build. We still need it to use binman, etc. but don't need it to build its dependencies. Signed-off

[PATCH v5 8/8] cmd: gpt: Add command to swap partition order

2023-08-31 Thread Joshua Watt
Adds a command called "gpt transpose" which will swap the order two partition table entries in the GPT partition table (but leaves them pointing to the same locations on disk). This can be useful for swapping bootloaders in systems that use an A/B partitioning scheme where the bootrom is hard code

[PATCH v5 7/8] cmd: gpt: Preserve bootable flag

2023-08-31 Thread Joshua Watt
Sets the bootable flag when constructing the partition string from the current partition configuration. This ensures that when the partitions are written back (for example, when renaming a partition), the flag is preserved. Signed-off-by: Joshua Watt --- cmd/gpt.c | 3 +++ test/p

[PATCH v5 6/8] cmd: gpt: Preserve type GUID if enabled

2023-08-31 Thread Joshua Watt
If CONFIG_PARTITION_TYPE_GUID is enabled, the type GUID will be preserved when writing out the partition string. It was already respected when writing out partitions; this ensures that if you capture the current partition layout and write it back (such as when renaming), the type GUIDs are preserve

[PATCH v5 5/8] cmd: gpt: Add command to set bootable flags

2023-08-31 Thread Joshua Watt
Adds a command that can be used to modify the GPT partition table to indicate which partitions should have the bootable flag set Signed-off-by: Joshua Watt --- cmd/gpt.c | 80 +++ doc/usage/cmd/gpt.rst | 12 ++ test/py/tests/test_gpt.py

[PATCH v5 4/8] cmd: gpt: Add gpt_partition_bootable variable

2023-08-31 Thread Joshua Watt
Adds an additional variable called gpt_partition_bootable that indicates if the given partition is bootable or not. Signed-off-by: Joshua Watt --- cmd/gpt.c | 9 +++-- doc/usage/cmd/gpt.rst | 5 + test/py/tests/test_gpt.py | 33 + 3 f

[PATCH v5 3/8] tests: gpt: Remove test order dependency

2023-08-31 Thread Joshua Watt
Re-create a clean disk image for each test to prevent modifications from one test affecting another Signed-off-by: Joshua Watt --- test/py/tests/test_gpt.py | 20 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/test/py/tests/test_gpt.py b/test/py/tests/test_gp

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

2023-08-31 Thread Joshua Watt
Adds initial documentation for the gpt command Signed-off-by: Joshua Watt --- doc/usage/cmd/gpt.rst | 184 ++ doc/usage/index.rst | 1 + 2 files changed, 185 insertions(+) create mode 100644 doc/usage/cmd/gpt.rst diff --git a/doc/usage/cmd/gpt.rst b/

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

2023-08-31 Thread Joshua Watt
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 --- cmd/gpt.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/cmd/gpt.c b/cmd/gpt.c index 964056bd28..fe9e06681c 100644 --- a/cmd/gpt.c

[PATCH v5 0/8] cmd: gpt: GPT manipulation improvements

2023-08-31 Thread Joshua Watt
Adds several improvements and additions to the gpt command processing, specifically (although not exclusively) for the purpose of supporting "ping-pong" booting when doing A/B boot partitions with u-boot itself. In this mechanism, u-boot must boot up, and then check if the correct boot partition i

Re: [PATCH] net: wget: Avoid packet queue overflow

2023-08-31 Thread Tom Rini
On Thu, Aug 31, 2023 at 12:27:59PM +0200, Richard Weinberger wrote: > - Ursprüngliche Mail - > > Von: "richard" > > An: u-boot@lists.denx.de > > CC: "richard" , "Joe Hershberger" , > > "Ramon Fried" > > Gesendet: Donnerstag, 20. Juli 2023 14:51:56 > > Betreff: [PATCH] net: wget: Avoid pa

Re: [PATCH] fwu: Initialize global fwu library state during CI test

2023-08-31 Thread Tom Rini
On Wed, Aug 23, 2023 at 02:16:52AM +0200, Marek Vasut wrote: > The current CI test worked by sheer luck, the g_dev global pointer > in the fwu library was never initialized and the test equally well > failed on sandbox64. Trigger the main loop in sandbox tests too to > initialize that global state

Re: [PATCH] tools: image-host: print error messages to stderr

2023-08-31 Thread Tom Rini
On Thu, Aug 17, 2023 at 06:36:10PM +0300, Oleksandr Suvorov wrote: > The make by default cuts off the stdout output from external tools, > so all error messages from the image-host are not shown in a make > output. Besides that, it is a common approach to use stderr stream > for error messages. >

Re: [PATCH v2 5/5] configs: sandbox: Enable NVMXIP QSPI driver

2023-08-31 Thread Tom Rini
On Wed, Aug 23, 2023 at 02:18:21AM +0200, Marek Vasut wrote: > Enable NVMXIP QSPI driver on sandbox, since it is already enabled > on sandbox64. > > Signed-off-by: Marek Vasut Applied to u-boot/next, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v2 4/5] drivers/mtd/nvmxip: Move sandbox_set_enable_memio() to test

2023-08-31 Thread Tom Rini
On Wed, Aug 23, 2023 at 02:18:20AM +0200, Marek Vasut wrote: > The sandbox_set_enable_memio() should only ever be set during > sandbox testing, not within driver itself, move it back to test/ . > > Signed-off-by: Marek Vasut Applied to u-boot/next, thanks! -- Tom signature.asc Description:

Re: [PATCH v2 3/5] drivers/mtd/nvmxip: Print phys_addr_t without warnings on both 32bit and 64bit systems

2023-08-31 Thread Tom Rini
On Wed, Aug 23, 2023 at 02:18:19AM +0200, Marek Vasut wrote: > Cast the address such that it can be printed without warnings > on both 32bit and 64bit systems. This really should use some > better print formatter, but for the lack of it, do it this way. > > Signed-off-by: Marek Vasut Applied to

Re: [PATCH v2 2/5] drivers/mtd/nvmxip: Rework the read accessor to support 32bit systems

2023-08-31 Thread Tom Rini
On Wed, Aug 23, 2023 at 02:18:18AM +0200, Marek Vasut wrote: > Get rid of nvmxip_mmio_rawread() and just implement the readl()/readq() > reader loop within nvmxip_blk_read(). Cast the destination buffer as > needed and increment the read by either 4 or 8 bytes depending on if > this is systemd wit

Re: [PATCH v2 1/5] drivers/mtd/nvmxip: Trigger post bind as probe on driver level

2023-08-31 Thread Tom Rini
On Wed, Aug 23, 2023 at 02:18:17AM +0200, Marek Vasut wrote: > Perform all the block device creation only once, after the driver itself > successfully bound. Do not do this in uclass post bind, as this might be > triggered multiple times. For example the ut_dm_host test triggers this > and trigger

Re: [PATCH v1] serial-uclass: reset gd->cur_serial_dev to NULL if serial not found

2023-08-31 Thread Tom Rini
On Fri, Aug 18, 2023 at 12:34:30PM +0300, Maksim Kiselev wrote: > Reset gd->cur_serial_dev pointer to avoid calling non-relocated code > from relocated code if a serial driver is not found and > CONFIG_REQUIRE_SERIAL_CONSOLE is disabled. > > Here is detailed explanation of what this patch is tryi

Re: [PATCH] pci: pcie-brcmstb: do not rely on CLKREQ# signal

2023-08-31 Thread Tom Rini
On Wed, Aug 16, 2023 at 03:27:53PM -0700, Sam Edwards wrote: > When the Broadcom STB PCIe controller is initialized, it must be set > into one of three CLKREQ# modes: "none"/"aspm"/"l1ss". The Linux driver, > through today, hard-codes "aspm" since the vast majority of boards using > this driver ha

Re: [PATCH] pci: pcie-brcmstb: bring over some robustness improvements from Linux

2023-08-31 Thread Tom Rini
On Mon, Aug 14, 2023 at 04:34:13PM -0600, Sam Edwards wrote: > Since the initial U-Boot driver was ported here from Linux, the latter > has had a few changes for robustness/stability. This patch brings over > two of them: > - Do not attempt to access the configuration space of a PCIe device if >

Re: [PATCH] imx: hab: Use CONFIG_SPL_LOAD_FIT_ADDRESS in the CSF example

2023-08-31 Thread Fabio Estevam
On 31/08/2023 11:56, Marek Vasut wrote: The SPL authenticates image starting from CONFIG_SPL_LOAD_FIT_ADDRESS address, update the csf_fit.txt to match. Signed-off-by: Marek Vasut --- Cc: "NXP i.MX U-Boot Team" Cc: Fabio Estevam Cc: Peng Fan Cc: Rasmus Villemoes Cc: Stefano Babic Cc: Tim Ha

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

2023-08-31 Thread Tom Rini
On Thu, Aug 31, 2023 at 09:39:52AM +, Roman Azarenko wrote: > > On Mon, 28 Aug 2023 at 02:00, Roman Azarenko > wrote: > > > > > > On Fri, 2023-08-25 at 12:06 -0600, Simon Glass wrote: > > > > > @@ -564,9 +564,13 @@ static int fit_extract_data(struct > > > > > image_tool_params *params, const

Re: [RFC] make sandbox UT more generic

2023-08-31 Thread Simon Glass
Hi, On Wed, 30 Aug 2023 at 23:28, AKASHI Takahiro wrote: > > Hi Simon, > > On Wed, Aug 30, 2023 at 08:49:05PM -0600, Simon Glass wrote: > > Hi, > > > > On Wed, 30 Aug 2023 at 18:38, AKASHI Takahiro > > wrote: > > > > > > Hi, > > > > > > I'm working on implementing SCMI-based pinctrl/gpio driver,

[PATCH] imx: hab: Use size parameter

2023-08-31 Thread Marek Vasut
The current code works by sheer coincidence, because (see HABv4 API documentation, section 3.4) the RVT authenticate_image call updates the size that is passed in with the actual size ROM code pulls from IVT/CSF . So if the input size is larger, that is "fine" . Pass in size instead to make this re

[PATCH] imx: hab: Use CONFIG_SPL_LOAD_FIT_ADDRESS in the CSF example

2023-08-31 Thread Marek Vasut
The SPL authenticates image starting from CONFIG_SPL_LOAD_FIT_ADDRESS address, update the csf_fit.txt to match. Signed-off-by: Marek Vasut --- Cc: "NXP i.MX U-Boot Team" Cc: Fabio Estevam Cc: Peng Fan Cc: Rasmus Villemoes Cc: Stefano Babic Cc: Tim Harvey --- doc/imx/habv4/csf_examples/mx8m

[PATCH 2/2] ARM: imx: Use default SAVED_DRAM_TIMING_BASE on Data Modul i.MX8M Plus eDM SBC

2023-08-31 Thread Marek Vasut
Use default SAVED_DRAM_TIMING_BASE as that is what upstream TFA expects. Without this change, the board will fail to suspend/resume e.g. in Linux. Signed-off-by: Marek Vasut --- Cc: Fabio Estevam Cc: Peng Fan Cc: Stefano Babic --- configs/imx8mp_data_modul_edm_sbc_defconfig | 1 - 1 file chan

[PATCH 1/2] ARM: imx: Use default SAVED_DRAM_TIMING_BASE on Data Modul i.MX8M Mini eDM SBC

2023-08-31 Thread Marek Vasut
Use default SAVED_DRAM_TIMING_BASE as that is what upstream TFA expects. Without this change, the board will fail to suspend/resume e.g. in Linux. Signed-off-by: Marek Vasut --- Cc: Fabio Estevam Cc: Peng Fan Cc: Stefano Babic --- configs/imx8mm_data_modul_edm_sbc_defconfig | 1 - 1 file chan

  1   2   >