Re: Troubles booting kernel with new imx8 board

2023-05-26 Thread marc
Hi, Thanks for responses, some more info: - imx8mp based system, 1Gb LPDDR, uses same PMIC as other boards (PCA9450) - barebox master - u-boot is NXP fork - kernel is NXP fork, 5.15.31 > > I'm trying to get barebox going for a new imx8mp based board. I can > > successfully power on and get to a b

Re: [PATCH 1/2] imx-usb-loader: fix compilation if prepare make target was not called before

2023-05-26 Thread Ahmad Fatoum
On 26.05.23 18:59, Johannes Zink wrote: > As pointed out in [1], compilation failed if the prepare make > target was not called before compiling the imx-usb-loader, which caused > the generated file not to exist when building for the sandbox target and > broke the build. > > After a discussion on

[PATCH 2/2] imx-usb-loader: also add version information on target tool

2023-05-26 Thread Johannes Zink
The version information was not added to the target tool, which then always showed "undefined" as version string. Fixed. Fixes: c1b50061f4b3 ("imx-usb-loader: add commandline option for displaying version number") Signed-off-by: Johannes Zink --- scripts/imx/Makefile | 2 +- 1 file changed, 1 i

[PATCH 1/2] imx-usb-loader: fix compilation if prepare make target was not called before

2023-05-26 Thread Johannes Zink
As pointed out in [1], compilation failed if the prepare make target was not called before compiling the imx-usb-loader, which caused the generated file not to exist when building for the sandbox target and broke the build. After a discussion on the original thread, a different approach has been p

[PATCH 0/2] imx-usb-loader: fixes in displaying version number

2023-05-26 Thread Johannes Zink
As Ahmad pointed out in [1], compilation failed if the prepare make target was not called before compiling the imx-usb-loader, which caused the generated file not to exist when building for the sandbox target and broke the build. Instead of continuing to pursue the approach discussed in [2], a rat

Re: [PATCH master] i.MX: HABv4: fix left-over old arch_remap_range prototype use

2023-05-26 Thread Sascha Hauer
On Fri, May 26, 2023 at 02:47:12PM +0200, Ahmad Fatoum wrote: > The first arch_remap_range occurrence in the file was correctly changed > to remap_range, but the second that was added for v3 wasn't adjusted. > Fix that. > > Fixes: 9845563450b8 ("treewide: use remap_range instead of arch_remap_rang

Re: [PATCH 1/1] Documentation: boards: nxp-imx8mp-evk; clarify firmware folder

2023-05-26 Thread Sascha Hauer
On Fri, May 26, 2023 at 06:15:41PM +1000, Marc Reilly wrote: > The imx8mp-bp31.bin file is expected to be in the firmware/ folder > > Signed-off-by: Marc Reilly > --- > Documentation/boards/imx/nxp-imx8mp-evk.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Applied, thanks Sascha >

Re: [PATCH master 4/4] ARM: mmu: invalidate when mapping range uncached

2023-05-26 Thread Ahmad Fatoum
On 26.05.23 08:33, Ahmad Fatoum wrote: > memtest can call remap_range to map regions being tested as uncached, > but remap_range did not take care to evict any stale cache lines. > Do this now. > > This fixes an issue of SELFTEST_MMU failing on an i.MX8MN, when running > memtest on an uncached reg

[PATCH master] i.MX: HABv4: fix left-over old arch_remap_range prototype use

2023-05-26 Thread Ahmad Fatoum
The first arch_remap_range occurrence in the file was correctly changed to remap_range, but the second that was added for v3 wasn't adjusted. Fix that. Fixes: 9845563450b8 ("treewide: use remap_range instead of arch_remap_range") Signed-off-by: Ahmad Fatoum --- drivers/hab/habv4.c | 2 +- 1 file

Re: Troubles booting kernel with new imx8 board

2023-05-26 Thread Ahmad Fatoum
On 26.05.23 12:46, m...@cpdesign.com.au wrote: > - boots kernel ok using u-boot (u-boot version provided by solidrun, based on > NXP versions) As Sascha said, logs would be useful. Two more questions: - Are you using the mainline kernel or NXP's fork? - Do you have an i.MX8MP or some stripped do

Re: Troubles booting kernel with new imx8 board

2023-05-26 Thread Sascha Hauer
On Fri, May 26, 2023 at 08:46:40PM +1000, m...@cpdesign.com.au wrote: > Hi all, > > I'm trying to get barebox going for a new imx8mp based board. I can > successfully power on and get to a barebox prompt etc and fiddle with gpios, > files on sd, memtest etc, but when booting to kernel it will h

Troubles booting kernel with new imx8 board

2023-05-26 Thread marc
Hi all, I'm trying to get barebox going for a new imx8mp based board. I can successfully power on and get to a barebox prompt etc and fiddle with gpios, files on sd, memtest etc, but when booting to kernel it will hang. Note though that the board boots ok with u-boot (using exact same kernel i

Re: [RFC PATCH 1/1] commands: add pwm manipulation command

2023-05-26 Thread Johannes Zink
Hi Marc, thanks for your patch. On 5/26/23 10:23, Marc Reilly wrote: This introduces a command to set parameters for a pwm device. Signed-off-by: Marc Reilly --- commands/Kconfig | 11 + commands/Makefile | 1 + commands/pwm.c| 111 +

[RFC PATCH 1/1] commands: add pwm manipulation command

2023-05-26 Thread Marc Reilly
This introduces a command to set parameters for a pwm device. Signed-off-by: Marc Reilly --- commands/Kconfig | 11 + commands/Makefile | 1 + commands/pwm.c| 111 ++ 3 files changed, 123 insertions(+) create mode 100644 commands/pwm.c di

Adding a pwm command

2023-05-26 Thread Marc Reilly
Hi all, I've been playing around with adding a new board to barebox, and its been a while so I didn't realise that properties could be set on a driver instance directly (eg 'pwm0.enable=1' or 'pwm0.period_ns=1') I started this pwm_set command, if it's useful (ability to set frequency is help

Re: [PATCH] fixup! imx-usb-loader: add commandline option for displaying version number

2023-05-26 Thread Ahmad Fatoum
On 26.05.23 10:19, Johannes Zink wrote: > Hi Ahmad, > > On 5/26/23 09:50, Ahmad Fatoum wrote: >> On 26.05.23 09:08, Johannes Zink wrote: >>> As Ahmad pointed out in [1], compilation failed if the prepare make >>> target was not called before compiling the imx-usb-loader, which caused >>> the gener

Re: [PATCH 1/1] Documentation: boards: nxp-imx8mp-evk; clarify firmware folder

2023-05-26 Thread Ahmad Fatoum
On 26.05.23 10:15, Marc Reilly wrote: > The imx8mp-bp31.bin file is expected to be in the firmware/ folder > > Signed-off-by: Marc Reilly Reviewed-by: Ahmad Fatoum > --- > Documentation/boards/imx/nxp-imx8mp-evk.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Doc

Re: [PATCH] fixup! imx-usb-loader: add commandline option for displaying version number

2023-05-26 Thread Johannes Zink
Hi Ahmad, On 5/26/23 09:50, Ahmad Fatoum wrote: On 26.05.23 09:08, Johannes Zink wrote: As Ahmad pointed out in [1], compilation failed if the prepare make target was not called before compiling the imx-usb-loader, which caused the generated file not to exist when building for the sandbox targe

[PATCH 1/1] Documentation: boards: nxp-imx8mp-evk; clarify firmware folder

2023-05-26 Thread Marc Reilly
The imx8mp-bp31.bin file is expected to be in the firmware/ folder Signed-off-by: Marc Reilly --- Documentation/boards/imx/nxp-imx8mp-evk.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/boards/imx/nxp-imx8mp-evk.rst b/Documentation/boards/imx/nxp-imx8mp-evk

Re: [PATCH] fixup! imx-usb-loader: add commandline option for displaying version number

2023-05-26 Thread Ahmad Fatoum
On 26.05.23 09:08, Johannes Zink wrote: > As Ahmad pointed out in [1], compilation failed if the prepare make > target was not called before compiling the imx-usb-loader, which caused > the generated file not to exist when building for the sandbox target and > broke the build. This needs to be v2

Re: [PATCH v2 3/7] include: sync with Linux

2023-05-26 Thread Ahmad Fatoum
On 26.05.23 08:54, Sascha Hauer wrote: > On Fri, May 26, 2023 at 08:37:42AM +0200, Ahmad Fatoum wrote: >> Linux has added new SYM_ macros in the assembly code and deprecated >> ENTRY/PROC. Import the necessary definitions to make kernel code >> porting easier. >> >> Signed-off-by: Ahmad Fatoum >>

Re: [PATCH master 1/4] Revert "imx-usb-loader: add commandline option for displaying version number"

2023-05-26 Thread Johannes Zink
Hi Ahmad, On 5/26/23 08:20, Ahmad Fatoum wrote: [snip] Agreed. I am looking forward to a v2 that doesn't break the build. ;) ack - just sent out a fixup. Do you know of a flag that is set when the files have been generated? Maybe we can include the file conditionally if and only if it has

[PATCH] fixup! imx-usb-loader: add commandline option for displaying version number

2023-05-26 Thread Johannes Zink
As Ahmad pointed out in [1], compilation failed if the prepare make target was not called before compiling the imx-usb-loader, which caused the generated file not to exist when building for the sandbox target and broke the build. As the included utsrelease file only contains a UTS_RELEASE macro ge