Re: [TF-A] Re: [RFC] Proposed location to host the firmware handoff specification.

2022-09-17 Thread Simon Glass
Hi, I discussed this with Jose a white back. I am OK with this as an interim measure to get the initial doc agreed, so long as we move it to a more independent place when available. Regards, Simon On Tue, 13 Sept 2022 at 09:48, Dan Handley wrote: > > Hi all > > Just picking up this old

Re: [PATCH v2] usb: Kconfig: Fix typo in SPL_DM_USB text

2022-09-17 Thread Marek Vasut
On 9/18/22 01:28, Fabio Estevam wrote: Change a typo in "USB host mode". Applied, thanks.

[PATCH v2] usb: Kconfig: Fix typo in SPL_DM_USB text

2022-09-17 Thread Fabio Estevam
Change a typo in "USB host mode". Signed-off-by: Fabio Estevam --- Changes since v1: - most->mode (Marek). drivers/usb/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/Kconfig b/drivers/usb/Kconfig index fd13cf31b5..3afb45d5cc 100644 ---

Re: [PATCH] usb: Kconfig: Fix typo in SPL_DM_USB text

2022-09-17 Thread Marek Vasut
On 9/14/22 23:00, Fabio Estevam wrote: From: Fabio Estevam There is an extra "most" word in the SPL_DM_USB text that makes it confusing. Remove it. Signed-off-by: Fabio Estevam --- drivers/usb/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/Kconfig

Re: [PATCH V2] Kconfig: enlarge CONFIG_SYS_MALLOC_F_LEN

2022-09-17 Thread Heiko Thiery
Hi Stefano, Hi Tom, Am Do., 15. Sept. 2022 um 02:53 Uhr schrieb Peng Fan (OSS) : > > From: Peng Fan > > "alloc space exhausted" happens in very early stage, which could be seen > with DEBUG_UART options enabled and leeds to an non-functional board. > > kontron_pitx_imx8m: >

Re: [PATCH v2 1/3] lib: Add common semihosting library

2022-09-17 Thread Sean Anderson
On 9/16/22 04:12, Kautuk Consul wrote: We factor out the arch-independent parts of the ARM semihosting implementation as a common library so that it can be shared with RISC-V. Signed-off-by: Kautuk Consul --- arch/arm/Kconfig | 2 + arch/arm/lib/semihosting.c | 179

Re: [PATCH v3 1/1] blk: Rename if_type to uclass_id

2022-09-17 Thread Simon Glass
Hi Takahiro, On Sat, 17 Sept 2022 at 04:52, AKASHI Takahiro wrote: > > On Sat, Sep 17, 2022 at 02:25:50AM -0600, Simon Glass wrote: > > Use the word 'uclass' instead of 'if_type' to complete the conversion. > > NAK. > You seem to have missed out some of my comments. > See: >

Re: [PATCH v2 0/3] Add riscv semihosting support in u-boot

2022-09-17 Thread Sean Anderson
On 9/16/22 10:32, Tom Rini wrote: On Fri, Sep 16, 2022 at 09:11:11AM -0400, Sean Anderson wrote: Hi Pali, On 9/16/22 05:12, Pali Rohár wrote: That is strange because I'm not aware of the fact that I'm riscv maintainer. get_maintainer will pick up anyone who has touched a file recently, even

Re: [PATCH v3] dm: core: Do not stop uclass iteration on error

2022-09-17 Thread Michal Suchánek
Hello, On Sat, Sep 17, 2022 at 09:02:53AM -0600, Simon Glass wrote: > Hi Michal, > > On Wed, 31 Aug 2022 at 11:44, Simon Glass wrote: > > > > Hi Michal, > > > > On Wed, 31 Aug 2022 at 01:39, Michal Suchánek wrote: > > > > > > Hello, > > > > > > On Tue, Aug 30, 2022 at 09:15:12PM -0600, Simon

Re: [PATCH 1/2] smbios: Simplify reporting of unknown values

2022-09-17 Thread Sean Anderson
On 9/16/22 16:30, Ilias Apalodimas wrote: Hi Simon, [...] Signed-off-by: Ilias Apalodimas --- lib/smbios.c | 17 +++-- 1 file changed, 3 insertions(+), 14 deletions(-) Perhaps a better fix is to drop the smbios info? Unfortunately there's a ton of userspace tools still

[PATCH v4 1/1] blk: Rename if_type to uclass_id

2022-09-17 Thread Simon Glass
Use the word 'uclass' instead of 'if_type' to complete the conversion. Signed-off-by: Simon Glass --- Changes in v4: - Fix comments from Takahiro Akashi missed on the previous revision Changes in v3: - Drop previously applied patches - Rebase on -next after EFI changes Changes in v2: - Use

[PATCH v4 0/1] blk: Rationalise the block interface

2022-09-17 Thread Simon Glass
The block interface has two separate implementations, one using driver model and one not. The latter is really only needed for SPL, where size constraints allegedly don't allow use of driver model. Of course we still need space for filesystems and other code, so it isn't clear that driver model is

Re: [PATCH v3] dm: core: Do not stop uclass iteration on error

2022-09-17 Thread Simon Glass
Hi Michal, On Wed, 31 Aug 2022 at 11:44, Simon Glass wrote: > > Hi Michal, > > On Wed, 31 Aug 2022 at 01:39, Michal Suchánek wrote: > > > > Hello, > > > > On Tue, Aug 30, 2022 at 09:15:12PM -0600, Simon Glass wrote: > > > Hi Michal, > > > > > > On Tue, 30 Aug 2022 at 10:48, Michal Suchánek

[PATCH v3] binman: Get futility by building it

2022-09-17 Thread Simon Glass
A binary download is not great, since it depends on libraries being present in the system. Build futility from source instead. Signed-off-by: Simon Glass --- Changes in v3: - Drop the branch, just use master instead - Explain why we are using a mirror - Drop mention of upstream patches as they

Re: [PATCH v2 0/6] Migrate watchdog reset to cyclic infrastructure

2022-09-17 Thread Tom Rini
On Sat, Sep 17, 2022 at 10:14:44AM +0200, Stefan Roese wrote: > This patchset migrates the watchdog triggering (WATCHDOG_RESET calls) > to the newly introduced cyclic execution framework. For this, the > watchdog driver now registers a cyclic execution function for each > WDT device that needs to

Re: [PATCH v2] pci: Do not enable PCIe GEN3 link retrain workaround by default

2022-09-17 Thread Maciej W. Rozycki
On Tue, 30 Aug 2022, Pali Rohár wrote: > > Agreed. But I also understand the reasoning from Maciej, at least in > > parts. Thinking a bit more about this, my preference would be to still > > include this workaround per default in U-Boot proper though. To not > > make things too complicated here.

Re: [PATCH v2] pci: Do not enable PCIe GEN3 link retrain workaround by default

2022-09-17 Thread Maciej W. Rozycki
On Tue, 30 Aug 2022, Stefan Roese wrote: > > I think I wrote it. One issue is that it is increasing size of SPL image > > and we really should not include into SPL things which are not required > > for all target platforms. Lot of boards have size constrained memory > > requirements and

Re: [PATCH v2] pci: Do not enable PCIe GEN3 link retrain workaround by default

2022-09-17 Thread Maciej W. Rozycki
On Tue, 30 Aug 2022, Pali Rohár wrote: > > > Moreover this workaround is enabled for all existing hardware and also all > > > future PCIe hardware, which opens a hole that other PCIe vendors may > > > introduce same HW issue as on systems where this workaround is required > > > and > > > nobody

Re: [PATCH v3 1/1] blk: Rename if_type to uclass_id

2022-09-17 Thread AKASHI Takahiro
On Sat, Sep 17, 2022 at 02:25:50AM -0600, Simon Glass wrote: > Use the word 'uclass' instead of 'if_type' to complete the conversion. NAK. You seem to have missed out some of my comments. See: https://lists.denx.de/pipermail/u-boot/2022-September/494583.html (against efi_api.h, efi_device_path.c,

Re: [PATCH 6/7] arm: mvebu: Add RD-AC5X board

2022-09-17 Thread Chris Packham
On Fri, 16 Sep 2022, 10:58 PM Pali Rohár, wrote: > On Friday 16 September 2022 16:54:22 Chris Packham wrote: > > + { > > + status = "okay"; > > + > > + spiflash0: flash@0 { > > + compatible = "jedec,spi-nor"; > > + spi-max-frequency = <5000>; > > +

[PATCH v3 1/1] blk: Rename if_type to uclass_id

2022-09-17 Thread Simon Glass
Use the word 'uclass' instead of 'if_type' to complete the conversion. Signed-off-by: Simon Glass --- Changes in v3: - Drop previously applied patches - Rebase on -next after EFI changes Changes in v2: - Use conv_uclass_id() instead of the confusing uclass_id_to_uclass_id()

[PATCH v2 3/6] cyclic: Use schedule() instead of WATCHDOG_RESET()

2022-09-17 Thread Stefan Roese
Globally replace all occurances of WATCHDOG_RESET() with schedule(), which handles the HW_WATCHDOG functionality and the cyclic infrastructure. Signed-off-by: Stefan Roese Reviewed-by: Simon Glass --- v1: - Add Simon's RB tag arch/arm/mach-at91/phy.c | 2 +-

[PATCH v2 4/6] watchdog: Get rid of ASSEMBLY hacks

2022-09-17 Thread Stefan Roese
Only one occurance of WATCHDOG_RESET is left in one assembler file. This patch changes this occurance to a direct call to watchdog_reset and then removes all the ASSEMBLY ifdef'ery in watchdog.h, as it's not needed any more to clean this mess a bit up. Signed-off-by: Stefan Roese --- v1: - No

[PATCH v2 2/6] cyclic: Introduce schedule() function

2022-09-17 Thread Stefan Roese
This patch introduces a schedule() function, which shall be used instead of the old WATCHDOG_RESET. Follow-up patches will make sure, that this new function is used. Signed-off-by: Stefan Roese Reviewed-by: Simon Glass --- v2: - Add some checks in schedule(), that the cyclic IF is ready before

[PATCH v2 6/6] watchdog: Further cleanup

2022-09-17 Thread Stefan Roese
Remove some now unused macros and #ifdef's. Signed-off-by: Stefan Roese Reviewed-by: Simon Glass --- v1: - Add Simon's RB tag include/watchdog.h | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/include/watchdog.h b/include/watchdog.h index 91506d3327a8..ac5f11e376fa

[PATCH v2 5/6] watchdog: Remove WATCHDOG_RESET macro

2022-09-17 Thread Stefan Roese
Now that we've globally replaced all WATCHDOG_RESET occurances, let's remove the ugly macro itself in the header. Signed-off-by: Stefan Roese Reviewed-by: Simon Glass --- v1: - Add Simon's RB tag include/watchdog.h | 36 1 file changed, 36 deletions(-)

[PATCH v2 1/6] watchdog: Integrate watchdog triggering into the cyclic framework

2022-09-17 Thread Stefan Roese
This patch integrates the watchdog triggering into the recently added cyclic infrastructure. Each watchdog device that shall be triggered registers it's own cyclic function. This way, multiple watchdog devices are still supported, each via a cyclic function with separate trigger intervals.

[PATCH v2 0/6] Migrate watchdog reset to cyclic infrastructure

2022-09-17 Thread Stefan Roese
This patchset migrates the watchdog triggering (WATCHDOG_RESET calls) to the newly introduced cyclic execution framework. For this, the watchdog driver now registers a cyclic execution function for each WDT device that needs to get serviced. Additionally the cyclic IF is a bit extended to also

Re: Please pull u-boot-watchdog/next

2022-09-17 Thread Stefan Roese
Hi Tom, On 16.09.22 23:48, Tom Rini wrote: Thanks for testing. I do have one last experiment for tonight. Please give the attached v2 a try. Thanks, Stefan From 2f61bc2cf011190eedbc0be34b4d61f342e7e5a5 Mon Sep 17 00:00:00 2001 From: Stefan Roese Date: Fri, 16 Sep 2022 21:08:51 +0200