Re: [PATCH 0/7] arm: dts: Create common imx8mn-u-boot
On 21.10.22 14:28, Adam Ford wrote: On Fri, Oct 21, 2022 at 7:27 AM Stefano Babic wrote: Hi Adam, On 21.10.22 14:10, Adam Ford wrote: On Sun, Sep 11, 2022 at 11:36 AM Adam Ford wrote: On Mon, Aug 15, 2022 at 6:43 AM Adam Ford wrote: On Sun, Aug 14, 2022 at 5:57 PM Fabio Estevam wrote: Hi Adam, On Sun, Jul 31, 2022 at 8:46 PM Adam Ford wrote: Every imx8mn board has a bunch of similar entries on their respective board-u-boot.dtsi file to make the board bootable. Instead of maintaining multiple files with duplicate code, have them all point to a new, common file. This file includes the necessary nodes that were common to nearly all boards and added spba1 to help faciliate SPL_DM_SERIAL. This also adds support for CONFIG_FSPI_CONF_HEADER which can be used to generate flash.bin files for booting from FlexSPI. Adam Ford (7): arm: dts: imx8mn-u-boot: Create common imx8mn-u-boot.dtsi arm: dts: imx8mn-beacon-kit: Consolidate with imx8mn-u-boot arm: dts: imx8mn-bsh-smm-s2: Consolidate with imx8mn-u-boot arm: dts: imx8mn-ddr4-evk: Consolidate with imx8mn-u-boot arm: dts: imx8mn-evk: Consolidate with imx8mn-u-boot arm: dts: imx8mn-var-som-symphony: Consolidate with imx8mn-u-boot arm: dts: imx8mn-venice: Consolidate with imx8mn-u-boot For the series: Reviewed-by: Fabio Estevam Stefano, It's been over a month, and I haven't heard any feedback from you and it doesn't appear this series has been accepted. Without it, Nano boards with DM_SERIAL in SPL won't' boot because of the device tree change which moved the UART's into a subnode. Are you able to apply to this series? As I mentioned before, I can squash the dependent patch into this series and push a V2 if you want. With a pending release in a few weeks, it would be nice to have the Nano boards booting again. thanks! adam Thanks! Stefano, If you want me to squash the 1 dependent patch, I can squash it into this series and submit a V2, but getting this series applied should fix a bunch of boards which don't boot properly. Stefano, Gentle nudge on this series. I posted it back in July and several people have responded with positive feedback. Do you think it can be merged? I have applied a second bunch of patches, this will be in the third one. However, I have fixed a couple of small things during first merge. I cannot apply them clean, can you check and rebase on top of u-boot-imx ? Thanks ! Yes, I'll send out a V2 rebased this afternoon Thanks, CI is testing. Stefano adam Stefano thanks, adam adam -- = DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, 82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de = -- = DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, 82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de =
Re: [PATCH 0/7] arm: dts: Create common imx8mn-u-boot
On Fri, Oct 21, 2022 at 7:27 AM Stefano Babic wrote: > > Hi Adam, > > On 21.10.22 14:10, Adam Ford wrote: > > On Sun, Sep 11, 2022 at 11:36 AM Adam Ford wrote: > >> > >> On Mon, Aug 15, 2022 at 6:43 AM Adam Ford wrote: > >>> > >>> On Sun, Aug 14, 2022 at 5:57 PM Fabio Estevam wrote: > > Hi Adam, > > On Sun, Jul 31, 2022 at 8:46 PM Adam Ford wrote: > > > > Every imx8mn board has a bunch of similar entries on their > > respective board-u-boot.dtsi file to make the board bootable. > > Instead of maintaining multiple files with duplicate code, > > have them all point to a new, common file. This file includes > > the necessary nodes that were common to nearly all boards > > and added spba1 to help faciliate SPL_DM_SERIAL. This also > > adds support for CONFIG_FSPI_CONF_HEADER which can be used > > to generate flash.bin files for booting from FlexSPI. > > > > Adam Ford (7): > >arm: dts: imx8mn-u-boot: Create common imx8mn-u-boot.dtsi > >arm: dts: imx8mn-beacon-kit: Consolidate with imx8mn-u-boot > >arm: dts: imx8mn-bsh-smm-s2: Consolidate with imx8mn-u-boot > >arm: dts: imx8mn-ddr4-evk: Consolidate with imx8mn-u-boot > >arm: dts: imx8mn-evk: Consolidate with imx8mn-u-boot > >arm: dts: imx8mn-var-som-symphony: Consolidate with imx8mn-u-boot > >arm: dts: imx8mn-venice: Consolidate with imx8mn-u-boot > > For the series: > > Reviewed-by: Fabio Estevam > >>> > >> > >> Stefano, > >> > >> It's been over a month, and I haven't heard any feedback from you and > >> it doesn't appear this series has been accepted. Without it, Nano > >> boards with DM_SERIAL in SPL won't' boot because of the device tree > >> change which moved the UART's into a subnode. Are you able to apply > >> to this series? As I mentioned before, I can squash the dependent > >> patch into this series and push a V2 if you want. With a pending > >> release in a few weeks, it would be nice to have the Nano boards > >> booting again. > >> > >> thanks! > >> > >> adam > >> > >>> Thanks! > >>> > >>> Stefano, > >>> > >>> If you want me to squash the 1 dependent patch, I can squash it into > >>> this series and submit a V2, but getting this series applied should > >>> fix a bunch of boards which don't boot properly. > > > > Stefano, > > > > Gentle nudge on this series. I posted it back in July and several > > people have responded with positive feedback. Do you think it can be > > merged? > > > > I have applied a second bunch of patches, this will be in the third one. > However, I have fixed a couple of small things during first merge. I > cannot apply them clean, can you check and rebase on top of u-boot-imx ? > Thanks ! Yes, I'll send out a V2 rebased this afternoon adam > > Stefano > > > thanks, > > > > adam > > > >>> > >>> adam > > -- > = > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, 82194 Groebenzell, Germany > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de > = >
Re: [PATCH 0/7] arm: dts: Create common imx8mn-u-boot
Hi Adam, On 21.10.22 14:10, Adam Ford wrote: On Sun, Sep 11, 2022 at 11:36 AM Adam Ford wrote: On Mon, Aug 15, 2022 at 6:43 AM Adam Ford wrote: On Sun, Aug 14, 2022 at 5:57 PM Fabio Estevam wrote: Hi Adam, On Sun, Jul 31, 2022 at 8:46 PM Adam Ford wrote: Every imx8mn board has a bunch of similar entries on their respective board-u-boot.dtsi file to make the board bootable. Instead of maintaining multiple files with duplicate code, have them all point to a new, common file. This file includes the necessary nodes that were common to nearly all boards and added spba1 to help faciliate SPL_DM_SERIAL. This also adds support for CONFIG_FSPI_CONF_HEADER which can be used to generate flash.bin files for booting from FlexSPI. Adam Ford (7): arm: dts: imx8mn-u-boot: Create common imx8mn-u-boot.dtsi arm: dts: imx8mn-beacon-kit: Consolidate with imx8mn-u-boot arm: dts: imx8mn-bsh-smm-s2: Consolidate with imx8mn-u-boot arm: dts: imx8mn-ddr4-evk: Consolidate with imx8mn-u-boot arm: dts: imx8mn-evk: Consolidate with imx8mn-u-boot arm: dts: imx8mn-var-som-symphony: Consolidate with imx8mn-u-boot arm: dts: imx8mn-venice: Consolidate with imx8mn-u-boot For the series: Reviewed-by: Fabio Estevam Stefano, It's been over a month, and I haven't heard any feedback from you and it doesn't appear this series has been accepted. Without it, Nano boards with DM_SERIAL in SPL won't' boot because of the device tree change which moved the UART's into a subnode. Are you able to apply to this series? As I mentioned before, I can squash the dependent patch into this series and push a V2 if you want. With a pending release in a few weeks, it would be nice to have the Nano boards booting again. thanks! adam Thanks! Stefano, If you want me to squash the 1 dependent patch, I can squash it into this series and submit a V2, but getting this series applied should fix a bunch of boards which don't boot properly. Stefano, Gentle nudge on this series. I posted it back in July and several people have responded with positive feedback. Do you think it can be merged? I have applied a second bunch of patches, this will be in the third one. However, I have fixed a couple of small things during first merge. I cannot apply them clean, can you check and rebase on top of u-boot-imx ? Thanks ! Stefano thanks, adam adam -- = DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, 82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de =
Re: [PATCH 0/7] arm: dts: Create common imx8mn-u-boot
Hi On Fri, Oct 21, 2022 at 2:10 PM Adam Ford wrote: > > On Sun, Sep 11, 2022 at 11:36 AM Adam Ford wrote: > > > > On Mon, Aug 15, 2022 at 6:43 AM Adam Ford wrote: > > > > > > On Sun, Aug 14, 2022 at 5:57 PM Fabio Estevam wrote: > > > > > > > > Hi Adam, > > > > > > > > On Sun, Jul 31, 2022 at 8:46 PM Adam Ford wrote: > > > > > > > > > > Every imx8mn board has a bunch of similar entries on their > > > > > respective board-u-boot.dtsi file to make the board bootable. > > > > > Instead of maintaining multiple files with duplicate code, > > > > > have them all point to a new, common file. This file includes > > > > > the necessary nodes that were common to nearly all boards > > > > > and added spba1 to help faciliate SPL_DM_SERIAL. This also > > > > > adds support for CONFIG_FSPI_CONF_HEADER which can be used > > > > > to generate flash.bin files for booting from FlexSPI. > > > > > > > > > > Adam Ford (7): > > > > > arm: dts: imx8mn-u-boot: Create common imx8mn-u-boot.dtsi > > > > > arm: dts: imx8mn-beacon-kit: Consolidate with imx8mn-u-boot > > > > > arm: dts: imx8mn-bsh-smm-s2: Consolidate with imx8mn-u-boot > > > > > arm: dts: imx8mn-ddr4-evk: Consolidate with imx8mn-u-boot > > > > > arm: dts: imx8mn-evk: Consolidate with imx8mn-u-boot > > > > > arm: dts: imx8mn-var-som-symphony: Consolidate with imx8mn-u-boot > > > > > arm: dts: imx8mn-venice: Consolidate with imx8mn-u-boot > > > > > > > > For the series: > > > > > > > > Reviewed-by: Fabio Estevam > > > > > > > Stefano, > > > > It's been over a month, and I haven't heard any feedback from you and > > it doesn't appear this series has been accepted. Without it, Nano > > boards with DM_SERIAL in SPL won't' boot because of the device tree > > change which moved the UART's into a subnode. Are you able to apply > > to this series? As I mentioned before, I can squash the dependent > > patch into this series and push a V2 if you want. With a pending > > release in a few weeks, it would be nice to have the Nano boards > > booting again. > > > > thanks! > > > > adam > > > > > Thanks! > > > > > > Stefano, > > > > > > If you want me to squash the 1 dependent patch, I can squash it into > > > this series and submit a V2, but getting this series applied should > > > fix a bunch of boards which don't boot properly. > > Stefano, > > Gentle nudge on this series. I posted it back in July and several > people have responded with positive feedback. Do you think it can be > merged? > It should. Can they need to be rebased? Michael > thanks, > > adam > > > > > > > adam -- Michael Nazzareno Trimarchi Co-Founder & Chief Executive Officer M. +39 347 913 2170 mich...@amarulasolutions.com __ Amarula Solutions BV Joop Geesinkweg 125, 1114 AB, Amsterdam, NL T. +31 (0)85 111 9172 i...@amarulasolutions.com www.amarulasolutions.com
Re: [PATCH 0/7] arm: dts: Create common imx8mn-u-boot
On Sun, Sep 11, 2022 at 11:36 AM Adam Ford wrote: > > On Mon, Aug 15, 2022 at 6:43 AM Adam Ford wrote: > > > > On Sun, Aug 14, 2022 at 5:57 PM Fabio Estevam wrote: > > > > > > Hi Adam, > > > > > > On Sun, Jul 31, 2022 at 8:46 PM Adam Ford wrote: > > > > > > > > Every imx8mn board has a bunch of similar entries on their > > > > respective board-u-boot.dtsi file to make the board bootable. > > > > Instead of maintaining multiple files with duplicate code, > > > > have them all point to a new, common file. This file includes > > > > the necessary nodes that were common to nearly all boards > > > > and added spba1 to help faciliate SPL_DM_SERIAL. This also > > > > adds support for CONFIG_FSPI_CONF_HEADER which can be used > > > > to generate flash.bin files for booting from FlexSPI. > > > > > > > > Adam Ford (7): > > > > arm: dts: imx8mn-u-boot: Create common imx8mn-u-boot.dtsi > > > > arm: dts: imx8mn-beacon-kit: Consolidate with imx8mn-u-boot > > > > arm: dts: imx8mn-bsh-smm-s2: Consolidate with imx8mn-u-boot > > > > arm: dts: imx8mn-ddr4-evk: Consolidate with imx8mn-u-boot > > > > arm: dts: imx8mn-evk: Consolidate with imx8mn-u-boot > > > > arm: dts: imx8mn-var-som-symphony: Consolidate with imx8mn-u-boot > > > > arm: dts: imx8mn-venice: Consolidate with imx8mn-u-boot > > > > > > For the series: > > > > > > Reviewed-by: Fabio Estevam > > > > Stefano, > > It's been over a month, and I haven't heard any feedback from you and > it doesn't appear this series has been accepted. Without it, Nano > boards with DM_SERIAL in SPL won't' boot because of the device tree > change which moved the UART's into a subnode. Are you able to apply > to this series? As I mentioned before, I can squash the dependent > patch into this series and push a V2 if you want. With a pending > release in a few weeks, it would be nice to have the Nano boards > booting again. > > thanks! > > adam > > > Thanks! > > > > Stefano, > > > > If you want me to squash the 1 dependent patch, I can squash it into > > this series and submit a V2, but getting this series applied should > > fix a bunch of boards which don't boot properly. Stefano, Gentle nudge on this series. I posted it back in July and several people have responded with positive feedback. Do you think it can be merged? thanks, adam > > > > adam
Re: [PATCH 0/7] arm: dts: Create common imx8mn-u-boot
On Mon, Aug 15, 2022 at 6:43 AM Adam Ford wrote: > > On Sun, Aug 14, 2022 at 5:57 PM Fabio Estevam wrote: > > > > Hi Adam, > > > > On Sun, Jul 31, 2022 at 8:46 PM Adam Ford wrote: > > > > > > Every imx8mn board has a bunch of similar entries on their > > > respective board-u-boot.dtsi file to make the board bootable. > > > Instead of maintaining multiple files with duplicate code, > > > have them all point to a new, common file. This file includes > > > the necessary nodes that were common to nearly all boards > > > and added spba1 to help faciliate SPL_DM_SERIAL. This also > > > adds support for CONFIG_FSPI_CONF_HEADER which can be used > > > to generate flash.bin files for booting from FlexSPI. > > > > > > Adam Ford (7): > > > arm: dts: imx8mn-u-boot: Create common imx8mn-u-boot.dtsi > > > arm: dts: imx8mn-beacon-kit: Consolidate with imx8mn-u-boot > > > arm: dts: imx8mn-bsh-smm-s2: Consolidate with imx8mn-u-boot > > > arm: dts: imx8mn-ddr4-evk: Consolidate with imx8mn-u-boot > > > arm: dts: imx8mn-evk: Consolidate with imx8mn-u-boot > > > arm: dts: imx8mn-var-som-symphony: Consolidate with imx8mn-u-boot > > > arm: dts: imx8mn-venice: Consolidate with imx8mn-u-boot > > > > For the series: > > > > Reviewed-by: Fabio Estevam > Stefano, It's been over a month, and I haven't heard any feedback from you and it doesn't appear this series has been accepted. Without it, Nano boards with DM_SERIAL in SPL won't' boot because of the device tree change which moved the UART's into a subnode. Are you able to apply to this series? As I mentioned before, I can squash the dependent patch into this series and push a V2 if you want. With a pending release in a few weeks, it would be nice to have the Nano boards booting again. thanks! adam > Thanks! > > Stefano, > > If you want me to squash the 1 dependent patch, I can squash it into > this series and submit a V2, but getting this series applied should > fix a bunch of boards which don't boot properly. > > adam
Re: [PATCH 0/7] arm: dts: Create common imx8mn-u-boot
On Sun, Aug 14, 2022 at 5:57 PM Fabio Estevam wrote: > > Hi Adam, > > On Sun, Jul 31, 2022 at 8:46 PM Adam Ford wrote: > > > > Every imx8mn board has a bunch of similar entries on their > > respective board-u-boot.dtsi file to make the board bootable. > > Instead of maintaining multiple files with duplicate code, > > have them all point to a new, common file. This file includes > > the necessary nodes that were common to nearly all boards > > and added spba1 to help faciliate SPL_DM_SERIAL. This also > > adds support for CONFIG_FSPI_CONF_HEADER which can be used > > to generate flash.bin files for booting from FlexSPI. > > > > Adam Ford (7): > > arm: dts: imx8mn-u-boot: Create common imx8mn-u-boot.dtsi > > arm: dts: imx8mn-beacon-kit: Consolidate with imx8mn-u-boot > > arm: dts: imx8mn-bsh-smm-s2: Consolidate with imx8mn-u-boot > > arm: dts: imx8mn-ddr4-evk: Consolidate with imx8mn-u-boot > > arm: dts: imx8mn-evk: Consolidate with imx8mn-u-boot > > arm: dts: imx8mn-var-som-symphony: Consolidate with imx8mn-u-boot > > arm: dts: imx8mn-venice: Consolidate with imx8mn-u-boot > > For the series: > > Reviewed-by: Fabio Estevam Thanks! Stefano, If you want me to squash the 1 dependent patch, I can squash it into this series and submit a V2, but getting this series applied should fix a bunch of boards which don't boot properly. adam
Re: [PATCH 0/7] arm: dts: Create common imx8mn-u-boot
Hi Adam, On Sun, Jul 31, 2022 at 8:46 PM Adam Ford wrote: > > Every imx8mn board has a bunch of similar entries on their > respective board-u-boot.dtsi file to make the board bootable. > Instead of maintaining multiple files with duplicate code, > have them all point to a new, common file. This file includes > the necessary nodes that were common to nearly all boards > and added spba1 to help faciliate SPL_DM_SERIAL. This also > adds support for CONFIG_FSPI_CONF_HEADER which can be used > to generate flash.bin files for booting from FlexSPI. > > Adam Ford (7): > arm: dts: imx8mn-u-boot: Create common imx8mn-u-boot.dtsi > arm: dts: imx8mn-beacon-kit: Consolidate with imx8mn-u-boot > arm: dts: imx8mn-bsh-smm-s2: Consolidate with imx8mn-u-boot > arm: dts: imx8mn-ddr4-evk: Consolidate with imx8mn-u-boot > arm: dts: imx8mn-evk: Consolidate with imx8mn-u-boot > arm: dts: imx8mn-var-som-symphony: Consolidate with imx8mn-u-boot > arm: dts: imx8mn-venice: Consolidate with imx8mn-u-boot For the series: Reviewed-by: Fabio Estevam
Re: [PATCH 0/7] arm: dts: Create common imx8mn-u-boot
On Sun, Jul 31, 2022 at 4:46 PM Adam Ford wrote: > > Every imx8mn board has a bunch of similar entries on their > respective board-u-boot.dtsi file to make the board bootable. > Instead of maintaining multiple files with duplicate code, > have them all point to a new, common file. This file includes > the necessary nodes that were common to nearly all boards > and added spba1 to help faciliate SPL_DM_SERIAL. This also > adds support for CONFIG_FSPI_CONF_HEADER which can be used > to generate flash.bin files for booting from FlexSPI. > > Adam Ford (7): > arm: dts: imx8mn-u-boot: Create common imx8mn-u-boot.dtsi > arm: dts: imx8mn-beacon-kit: Consolidate with imx8mn-u-boot > arm: dts: imx8mn-bsh-smm-s2: Consolidate with imx8mn-u-boot > arm: dts: imx8mn-ddr4-evk: Consolidate with imx8mn-u-boot > arm: dts: imx8mn-evk: Consolidate with imx8mn-u-boot > arm: dts: imx8mn-var-som-symphony: Consolidate with imx8mn-u-boot > arm: dts: imx8mn-venice: Consolidate with imx8mn-u-boot > > arch/arm/dts/imx8mn-beacon-kit-u-boot.dtsi| 180 + > .../dts/imx8mn-bsh-smm-s2-u-boot-common.dtsi | 172 +--- > arch/arm/dts/imx8mn-ddr4-evk-u-boot.dtsi | 180 + > arch/arm/dts/imx8mn-evk-u-boot.dtsi | 108 > arch/arm/dts/imx8mn-u-boot.dtsi | 248 ++ > .../dts/imx8mn-var-som-symphony-u-boot.dtsi | 188 + > arch/arm/dts/imx8mn-venice-u-boot.dtsi| 188 + > 7 files changed, 254 insertions(+), 1010 deletions(-) > create mode 100644 arch/arm/dts/imx8mn-u-boot.dtsi > > -- > 2.34.1 > Thanks Adam, For the series: Reviewed-By: Tim Harvey Tested-By: Tim Harvey #imx8mn-venice-gw7902 Best Regards, Tim
[PATCH 0/7] arm: dts: Create common imx8mn-u-boot
Every imx8mn board has a bunch of similar entries on their respective board-u-boot.dtsi file to make the board bootable. Instead of maintaining multiple files with duplicate code, have them all point to a new, common file. This file includes the necessary nodes that were common to nearly all boards and added spba1 to help faciliate SPL_DM_SERIAL. This also adds support for CONFIG_FSPI_CONF_HEADER which can be used to generate flash.bin files for booting from FlexSPI. Adam Ford (7): arm: dts: imx8mn-u-boot: Create common imx8mn-u-boot.dtsi arm: dts: imx8mn-beacon-kit: Consolidate with imx8mn-u-boot arm: dts: imx8mn-bsh-smm-s2: Consolidate with imx8mn-u-boot arm: dts: imx8mn-ddr4-evk: Consolidate with imx8mn-u-boot arm: dts: imx8mn-evk: Consolidate with imx8mn-u-boot arm: dts: imx8mn-var-som-symphony: Consolidate with imx8mn-u-boot arm: dts: imx8mn-venice: Consolidate with imx8mn-u-boot arch/arm/dts/imx8mn-beacon-kit-u-boot.dtsi| 180 + .../dts/imx8mn-bsh-smm-s2-u-boot-common.dtsi | 172 +--- arch/arm/dts/imx8mn-ddr4-evk-u-boot.dtsi | 180 + arch/arm/dts/imx8mn-evk-u-boot.dtsi | 108 arch/arm/dts/imx8mn-u-boot.dtsi | 248 ++ .../dts/imx8mn-var-som-symphony-u-boot.dtsi | 188 + arch/arm/dts/imx8mn-venice-u-boot.dtsi| 188 + 7 files changed, 254 insertions(+), 1010 deletions(-) create mode 100644 arch/arm/dts/imx8mn-u-boot.dtsi -- 2.34.1
