[PATCH v2 0/7] arm64: dts: imx8mm: add common -binman.dtsi and further clean-up

[Peng Fan (OSS)]

On 2021/10/9 5:02, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler at toradex.com>
> 
> 
> With the move to using binman to generate SPL aka u-boot-spl-ddr.bin and
> U-Boot proper aka u-boot.itb every board now covers such configuration
> in its own U-Boot specific device tree include. Move the comon part of
> that configuration to the common imx8mm-u-boot.dtsi include file.
> 
> The initial patch fixes an issue with intermediate binary naming for the
> imx8mm-cl-iot-gate. And subsequent patches further clean up that dtsi.
> 
> This series is based on Peng's binman conversion of late [1], my Verdin
> iMX8M Mini target refresh [2], Fabio's generating a single bootable
> flash.bin again for imx8mm-evk [3] and Tim's switching imx8mm_venice to
> using binman to pack images [4].
> 
> This series has been run-time tested on Verdin iMX8M Mini. The other
> targets were only compile tested.

Thanks for the work.

Reviewed-by: Peng Fan <peng.fan at nxp.com> for the patchset.

> 
> [1] https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmarc.info%2F%3Fl%3Du-boot%26m%3D163366377832026&data=04%7C01%7Cpeng.fan%40nxp.com%7Cec53721a8d7145e87cb708d98a9efc40%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C1%7C637693237708724959%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=KlZvffel%2BO8mTBpGW5w0nEM%2BtP4DQFmNt4PglFz1mv4%3D&reserved=0
> [2] https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmarc.info%2F%3Fl%3Du-boot%26m%3D163365299927718&data=04%7C01%7Cpeng.fan%40nxp.com%7Cec53721a8d7145e87cb708d98a9efc40%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C1%7C637693237708724959%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=X46kkPPpZWis0XForTOOLjsEfmJFabeZP1E0N%2Bpf%2BDc%3D&reserved=0
> [3] https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmarc.info%2F%3Fl%3Du-boot%26m%3D162972343709264&data=04%7C01%7Cpeng.fan%40nxp.com%7Cec53721a8d7145e87cb708d98a9efc40%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C1%7C637693237708724959%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=4pjzBjyuys3PRE1X0mArki0%2Fq%2FG%2BvhsgCoxFM6I5PK0%3D&reserved=0
> [4] https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmarc.info%2F%3Fl%3Du-boot%26m%3D162983187605276&data=04%7C01%7Cpeng.fan%40nxp.com%7Cec53721a8d7145e87cb708d98a9efc40%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C1%7C637693237708724959%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=m61TR7FDPnl7Weoz2Md%2FDeeMDqytq6VWS%2F7CeNv4Ais%3D&reserved=0
> 
> Changes in v2:
> - New patch preparing cl-iot-gate.
> - Morph common binman dtsi into regular common u-boot dtsi as discussed
>    on mailing list.
> - Re-based.
> - Rather than doing that clean-up to the separate binman dtsi now do it
>    to the common u-boot dtsi one.
> 
> Changes in v1:
> - This was suggested by Simon on my earlier patch set upon which we
>    decided to first break this up into a common binman dtsi and then
>    subsequently do such further improvements.
> 
> Marcel Ziswiler (7):
>    imx8mm-cl-iot-gate: fix imximage intermediate binary naming
>    arm64: dts: imx8mm-cl-iot-gate-u-boot.dtsi: alphabetically re-order
>    arm64: dts: imx8mm-cl-iot-gate-u-boot.dtsi: use common
>      imx8mm-u-boot.dtsi
>    arm64: dts: imx8mm: use common binman configuration
>    arm64: dts: imx8mm-u-boot.dtsi: alphabetically re-order properties
>    arm64: dts: imx8mm-u-boot.dtsi: explicitly add spl filename
>    arm64: dts: imx8mm-u-boot.dtsi: improve odd blob-ext naming
> 
>   .../dts/imx8mm-cl-iot-gate-optee-u-boot.dtsi  | 209 ++++--------------
>   arch/arm/dts/imx8mm-cl-iot-gate-u-boot.dtsi   | 188 +++-------------
>   arch/arm/dts/imx8mm-evk-u-boot.dtsi           | 123 -----------
>   arch/arm/dts/imx8mm-u-boot.dtsi               | 149 ++++++++++++-
>   arch/arm/dts/imx8mm-venice-u-boot.dtsi        | 119 ----------
>   arch/arm/dts/imx8mm-verdin-u-boot.dtsi        | 122 +---------
>   .../imximage-8mm-lpddr4.cfg                   |   2 +-
>   7 files changed, 212 insertions(+), 700 deletions(-)
>