On 2021/10/9 5:02, Marcel Ziswiler wrote:
From: Marcel Ziswiler <[email protected]> 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 <[email protected]> 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(-)

