[PATCH] uncompress: simplify prototype of uncompress()

2023-11-13 Thread Uwe Kleine-König
All callers apart from lib/uncompress.c itself only use memory-to-memory decompression. Simplify the calls accordingly. Note that two of three callers passed error_fn=NULL. As the uncompress function calls error_fn() unconditionally on error, this might yield undefined behaviour and so the new unc

Re: [PATCH 1/2] console: provide best-effort clk_get_for_console helper

2023-11-13 Thread Ahmad Fatoum
On 13.11.23 14:03, Sascha Hauer wrote: > On Thu, Nov 09, 2023 at 12:47:06PM +0100, Ahmad Fatoum wrote: >> From: Ahmad Fatoum >> >> clk_get will return -EPROBE_DEFER if clock provider hasn't yet been >> probed. In a system with deep probe enabled, dependencies are probed >> on demand, so a -EPROBE_

Re: [PATCH 1/2] console: provide best-effort clk_get_for_console helper

2023-11-13 Thread Sascha Hauer
On Thu, Nov 09, 2023 at 12:47:06PM +0100, Ahmad Fatoum wrote: > From: Ahmad Fatoum > > clk_get will return -EPROBE_DEFER if clock provider hasn't yet been > probed. In a system with deep probe enabled, dependencies are probed > on demand, so a -EPROBE_DEFER return is final and the console probe >

Re: [PATCH v2 1/3] glob: drop needless ifdeffery in {glob,fnmatch}.h

2023-11-13 Thread Sascha Hauer
On Thu, Nov 09, 2023 at 01:25:49PM +0100, Ahmad Fatoum wrote: > The implementation of the glob and fnmatch functions is taken from glibc > and it shows in the amount of #ifdefs. We don't need that in barebox and > can even get the glob_t size down by dropping unused members, so let's > do that. >

Re: [PATCH 17/18] of: define of_devices_ensure_probed_by_compatible

2023-11-13 Thread Sascha Hauer
On Fri, Nov 10, 2023 at 10:44:20PM +0100, Ahmad Fatoum wrote: > We will need to ensure probe of OP-TEE from SCMI code in a follow-up > commit, so add a small helper to facilitate this. > > Signed-off-by: Ahmad Fatoum > --- > include/of.h | 10 ++ > 1 file changed, 10 insertions(+) > > d

Re: [PATCH 05/18] asm-generic: split off typeconfused readl and friends

2023-11-13 Thread Sascha Hauer
On Fri, Nov 10, 2023 at 10:44:08PM +0100, Ahmad Fatoum wrote: > The MMIO accessors in barebox like readl have the error prone > peculiarity of accepting integer arguments to pointers automatically, > which makes it easy to confuse address and data. > > We will add an alternative less error-probe w

Re: [PATCH 15/18] include: add blocking notifier aliases

2023-11-13 Thread Sascha Hauer
On Fri, Nov 10, 2023 at 10:44:18PM +0100, Ahmad Fatoum wrote: > All notifiers are blocking in barebox, but to avoid needless mechanical > fixups during porting, just provide the stubs in a new > header. While at it, also provide NOTIFY_DONE and > NOTIFY_DONE for use in the notifier callbacks. The

Re: [PATCH 14/18] include: implement dev_warn_once and friends

2023-11-13 Thread Sascha Hauer
On Fri, Nov 10, 2023 at 10:44:17PM +0100, Ahmad Fatoum wrote: > We already have pr_once, so duplicate it for dev_warn and friends as > well. > > Signed-off-by: Ahmad Fatoum > --- > drivers/firmware/arm_scmi/driver.c | 4 ++-- > include/linux/printk.h | 29 +++

Re: [PATCH master] ARM: i.MX8M: bootrom: fix load image size

2023-11-13 Thread Marco Felsch
Hi all, On 23-11-13, Marco Felsch wrote: > Without the proper aligned image size we may encounter host load errors > like: > > | found i.MX8MP USB device [1fc9:0146] > | No dcd table in this ivt > | dl_command err=-1, last_trans=-1 > | 4 in err=-4, last_trans=0 00 00 00 00 > > The reason for th

[PATCH master] ARM: i.MX8M: bootrom: fix load image size

2023-11-13 Thread Marco Felsch
Without the proper aligned image size we may encounter host load errors like: | found i.MX8MP USB device [1fc9:0146] | No dcd table in this ivt | dl_command err=-1, last_trans=-1 | 4 in err=-4, last_trans=0 00 00 00 00 The reason for this is that we asked only for the required bytes while the ho

[PATCH] ARM: i.MX8M: bootrom: fix load image size

2023-11-13 Thread Marco Felsch
Without the proper aligned image size we may encounter host load errors like: | found i.MX8MP USB device [1fc9:0146] | No dcd table in this ivt | dl_command err=-1, last_trans=-1 | 4 in err=-4, last_trans=0 00 00 00 00 The reason for this is that we asked only for the required bytes while the ho

Re: [PATCH 03/25] ARM: i.MX: Add i.MX93 trdc support

2023-11-13 Thread Sascha Hauer
On Mon, Nov 13, 2023 at 08:24:21AM +0100, Ahmad Fatoum wrote: > On 10.11.23 13:57, Sascha Hauer wrote: > > Signed-off-by: Sascha Hauer > > TRDC is presumably Tsomething Resource Domain Controller? > Would be nice to have a commit message spelling it out and describing > why this needs to be done