Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-08-18 Thread Marcel Ziswiler
Hi Fabio and Frieder On Mon, 2021-08-16 at 12:10 -0300, Fabio Estevam wrote: > Hi Frieder, > > On Mon, Aug 16, 2021 at 11:18 AM Frieder Schrempf > wrote: > > > > If you prepare a patch I would prefer to have separate names for the SPL > > > image and the final concatenated > > > image. So we d

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-08-16 Thread Fabio Estevam
Hi Frieder, On Mon, Aug 16, 2021 at 11:18 AM Frieder Schrempf wrote: > > If you prepare a patch I would prefer to have separate names for the SPL > > image and the final concatenated image. So we don't overwrite the SPL image > > and can still use it on its own if needed. > > But rather than c

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-08-16 Thread Frieder Schrempf
On 16.08.21 16:09, Frieder Schrempf wrote: > Hi Fabio, > > On 16.08.21 15:56, Fabio Estevam wrote: >> Hi Heiko, >> >> On Mon, Aug 16, 2021 at 7:27 AM Heiko Schocher wrote: >> >>> I am not a binman expert, but may you try to add in your u-boot.dtsi >>> in your "binman" node at the end: >>> >>>

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-08-16 Thread Frieder Schrempf
Hi Fabio, On 16.08.21 15:56, Fabio Estevam wrote: > Hi Heiko, > > On Mon, Aug 16, 2021 at 7:27 AM Heiko Schocher wrote: > >> I am not a binman expert, but may you try to add in your u-boot.dtsi >> in your "binman" node at the end: >> >> imx-boot { >> filename = "imx-boot

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-08-16 Thread Fabio Estevam
Hi Heiko, On Mon, Aug 16, 2021 at 7:27 AM Heiko Schocher wrote: > I am not a binman expert, but may you try to add in your u-boot.dtsi > in your "binman" node at the end: > > imx-boot { > filename = "imx-boot.bin"; > pad-byte = <0x00>; > >

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-08-16 Thread Heiko Schocher
Hello Fabio, On 13.08.21 14:55, Fabio Estevam wrote: > Hi Heiko, > > On Sat, Jul 10, 2021 at 9:23 AM Heiko Schocher wrote: > >>> Doesn't this change from having a single flash.bin encompasing the SPL >>> and U-Boot proper to having split files? I noticed that happened with >>> imx8mm_evk for ex

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-08-13 Thread Fabio Estevam
Hi Heiko, On Sat, Jul 10, 2021 at 9:23 AM Heiko Schocher wrote: > > Doesn't this change from having a single flash.bin encompasing the SPL > > and U-Boot proper to having split files? I noticed that happened with > > imx8mm_evk for example when it switched to binman. > > Yes, but you can easy ge

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-18 Thread Heiko Schocher
Hi Tim, Simon, On 13.07.21 22:17, Simon Glass wrote: > Hi Tim, > > On Mon, 12 Jul 2021 at 14:58, Tim Harvey wrote: >> >> On Mon, Jul 12, 2021 at 12:44 PM Simon Glass wrote: >>> >>> Hi Tim, >>> >>> On Mon, 12 Jul 2021 at 10:42, Tim Harvey wrote: On Sat, Jul 10, 2021 at 5:23 AM Heiko S

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-18 Thread Heiko Schocher
Hello Tim, On 12.07.21 18:42, Tim Harvey wrote: > On Sat, Jul 10, 2021 at 5:23 AM Heiko Schocher wrote: >> >> Hello Tim, >> >> On 09.07.21 16:47, Tim Harvey wrote: >>> On Wed, Jul 7, 2021 at 5:58 AM Teresa Remmet wrote: Factor out the common node settings for dm-spl and dm-pre-reloc >>

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-13 Thread Simon Glass
Hi Tim, On Mon, 12 Jul 2021 at 14:58, Tim Harvey wrote: > > On Mon, Jul 12, 2021 at 12:44 PM Simon Glass wrote: > > > > Hi Tim, > > > > On Mon, 12 Jul 2021 at 10:42, Tim Harvey wrote: > > > > > > On Sat, Jul 10, 2021 at 5:23 AM Heiko Schocher wrote: > > > > > > > > Hello Tim, > > > > > > > > O

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-12 Thread Tim Harvey
On Mon, Jul 12, 2021 at 12:44 PM Simon Glass wrote: > > Hi Tim, > > On Mon, 12 Jul 2021 at 10:42, Tim Harvey wrote: > > > > On Sat, Jul 10, 2021 at 5:23 AM Heiko Schocher wrote: > > > > > > Hello Tim, > > > > > > On 09.07.21 16:47, Tim Harvey wrote: > > > > On Wed, Jul 7, 2021 at 5:58 AM Teresa

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-12 Thread Simon Glass
Hi Tim, On Mon, 12 Jul 2021 at 10:42, Tim Harvey wrote: > > On Sat, Jul 10, 2021 at 5:23 AM Heiko Schocher wrote: > > > > Hello Tim, > > > > On 09.07.21 16:47, Tim Harvey wrote: > > > On Wed, Jul 7, 2021 at 5:58 AM Teresa Remmet wrote: > > >> > > >> Factor out the common node settings for dm-sp

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-12 Thread Tim Harvey
On Sat, Jul 10, 2021 at 5:23 AM Heiko Schocher wrote: > > Hello Tim, > > On 09.07.21 16:47, Tim Harvey wrote: > > On Wed, Jul 7, 2021 at 5:58 AM Teresa Remmet wrote: > >> > >> Factor out the common node settings for dm-spl and dm-pre-reloc > >> and move them to imx8mp-u-boot.dtsi > >> > >> Signed

[PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-10 Thread sbabic
> Factor out the common node settings for dm-spl and dm-pre-reloc > and move them to imx8mp-u-boot.dtsi > Signed-off-by: Teresa Remmet > Reviewed-by: Fabio Estevam > Reviewed-by: Heiko Schocher Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic --

[PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-10 Thread sbabic
> Factor out the common node settings for dm-spl and dm-pre-reloc > and move them to imx8mp-u-boot.dtsi > Signed-off-by: Teresa Remmet > Reviewed-by: Fabio Estevam > Reviewed-by: Heiko Schocher Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic --

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-10 Thread Heiko Schocher
Hello Tim, On 09.07.21 16:47, Tim Harvey wrote: > On Wed, Jul 7, 2021 at 5:58 AM Teresa Remmet wrote: >> >> Factor out the common node settings for dm-spl and dm-pre-reloc >> and move them to imx8mp-u-boot.dtsi >> >> Signed-off-by: Teresa Remmet >> Reviewed-by: Fabio Estevam >> Reviewed-by: Hei

Re: [PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-09 Thread Tim Harvey
On Wed, Jul 7, 2021 at 5:58 AM Teresa Remmet wrote: > > Factor out the common node settings for dm-spl and dm-pre-reloc > and move them to imx8mp-u-boot.dtsi > > Signed-off-by: Teresa Remmet > Reviewed-by: Fabio Estevam > Reviewed-by: Heiko Schocher > --- > Changes in v3: > - Moved binman nodes

[PATCH v3 2/9] arm: dts: imx8mp: Add common u-boot dtsi

2021-07-07 Thread Teresa Remmet
Factor out the common node settings for dm-spl and dm-pre-reloc and move them to imx8mp-u-boot.dtsi Signed-off-by: Teresa Remmet Reviewed-by: Fabio Estevam Reviewed-by: Heiko Schocher --- Changes in v3: - Moved binman nodes to common imx8mp-u-boot.dtsi Changes in v2: - none arch/arm/dts/imx8m