| Message ID | 20240305004222.622-1-laurent.pinchart@ideasonboard.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Add DT bindings and device tree for Compulab SB-UCM-iMX8MPLUS | expand |
On Tue, 05 Mar 2024 02:42:17 +0200, Laurent Pinchart wrote: > Hello, > > This small patch series is a drive-by addition of the Compulab > SB-UCM-iMX8MPLUS to the Linux kernel device tree sources. While porting > the device tree from the Compulab BSP kernel to mainline, I thought I > could as well mainline it, along with related conversion of text DT > bindings to YAML. > > The SB-UCM-iMX8MPLUS is a carrier board designed as a reference to > evaluate the Compulab UCM-iMX8MPLUS SoM. The SoM integrates the bare > minimal peripherals (DRAM, eMMC, ethernet PHY, EEPROM and RTC), while > the carrier board includes a much wider range of peripherals. I have > only enabled support for the ones I am interested in, or, as a strech > goal, the ones I could easily test. > > The first patch in the series adds compatible strings for the SoM and > the board to the ARM FSL bindings. The next patch then converts text DT > bindings to a YAML schema for the RTC present on the SoM. Finally, the > last two patches add DT sources for the SoM and the carrier board. > > Please see individual patches for details. > > I have checked the validity of the bindings and the device tree sources. > The only warnings in the DT sources are due to issues in imx8mp.dtsi. > > Laurent Pinchart (4): > dt-bindings: arm: fsl: Add Compulab SB-UCM-iMX8MPLUS carrier board > dt-bindings: rtc: abx80x: Convert text bindings to YAML > arm64: dts: freescale: Add device tree for Compulab UCM-iMX8M-Plus > arm64: dts: freescale: Add device tree for Compulab SB-UCM-iMX8MPLUS > > .../devicetree/bindings/arm/fsl.yaml | 6 + > .../bindings/rtc/abracon,abx80x.txt | 31 -- > .../bindings/rtc/abracon,abx80x.yaml | 89 +++++ > arch/arm64/boot/dts/freescale/Makefile | 1 + > .../boot/dts/freescale/imx8mp-sb-ucm.dts | 284 ++++++++++++++++ > arch/arm64/boot/dts/freescale/imx8mp-ucm.dtsi | 309 ++++++++++++++++++ > 6 files changed, 689 insertions(+), 31 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/rtc/abracon,abx80x.txt > create mode 100644 Documentation/devicetree/bindings/rtc/abracon,abx80x.yaml > create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dts > create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-ucm.dtsi > > -- > Regards, > > Laurent Pinchart > > > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade New warnings running 'make CHECK_DTBS=y freescale/imx8mp-sb-ucm.dtb' for 20240305004222.622-1-laurent.pinchart@ideasonboard.com: Error: arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dts:68.1-10 Label or path hdmi_pvi not found Error: arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dts:72.1-9 Label or path hdmi_tx not found Error: arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dts:86.1-13 Label or path hdmi_tx_phy not found Error: arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dts:208.1-8 Label or path lcdif3 not found FATAL ERROR: Syntax error parsing input tree make[3]: *** [scripts/Makefile.lib:419: arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dtb] Error 1 make[2]: *** [scripts/Makefile.build:481: arch/arm64/boot/dts/freescale] Error 2 make[2]: Target 'arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dtb' not remade because of errors. make[1]: *** [/home/rob/proj/linux-dt-testing/Makefile:1385: freescale/imx8mp-sb-ucm.dtb] Error 2 make: *** [Makefile:240: __sub-make] Error 2 make: Target 'freescale/imx8mp-sb-ucm.dtb' not remade because of errors.
Hi Rob, On Tue, Mar 05, 2024 at 07:20:21AM -0600, Rob Herring wrote: > On Tue, 05 Mar 2024 02:42:17 +0200, Laurent Pinchart wrote: > > Hello, > > > > This small patch series is a drive-by addition of the Compulab > > SB-UCM-iMX8MPLUS to the Linux kernel device tree sources. While porting > > the device tree from the Compulab BSP kernel to mainline, I thought I > > could as well mainline it, along with related conversion of text DT > > bindings to YAML. > > > > The SB-UCM-iMX8MPLUS is a carrier board designed as a reference to > > evaluate the Compulab UCM-iMX8MPLUS SoM. The SoM integrates the bare > > minimal peripherals (DRAM, eMMC, ethernet PHY, EEPROM and RTC), while > > the carrier board includes a much wider range of peripherals. I have > > only enabled support for the ones I am interested in, or, as a strech > > goal, the ones I could easily test. > > > > The first patch in the series adds compatible strings for the SoM and > > the board to the ARM FSL bindings. The next patch then converts text DT > > bindings to a YAML schema for the RTC present on the SoM. Finally, the > > last two patches add DT sources for the SoM and the carrier board. > > > > Please see individual patches for details. > > > > I have checked the validity of the bindings and the device tree sources. > > The only warnings in the DT sources are due to issues in imx8mp.dtsi. > > > > Laurent Pinchart (4): > > dt-bindings: arm: fsl: Add Compulab SB-UCM-iMX8MPLUS carrier board > > dt-bindings: rtc: abx80x: Convert text bindings to YAML > > arm64: dts: freescale: Add device tree for Compulab UCM-iMX8M-Plus > > arm64: dts: freescale: Add device tree for Compulab SB-UCM-iMX8MPLUS > > > > .../devicetree/bindings/arm/fsl.yaml | 6 + > > .../bindings/rtc/abracon,abx80x.txt | 31 -- > > .../bindings/rtc/abracon,abx80x.yaml | 89 +++++ > > arch/arm64/boot/dts/freescale/Makefile | 1 + > > .../boot/dts/freescale/imx8mp-sb-ucm.dts | 284 ++++++++++++++++ > > arch/arm64/boot/dts/freescale/imx8mp-ucm.dtsi | 309 ++++++++++++++++++ > > 6 files changed, 689 insertions(+), 31 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/rtc/abracon,abx80x.txt > > create mode 100644 Documentation/devicetree/bindings/rtc/abracon,abx80x.yaml > > create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dts > > create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-ucm.dtsi > > My bot found new DTB warnings on the .dts files added or changed in this > series. > > Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings > are fixed by another series. Ultimately, it is up to the platform > maintainer whether these warnings are acceptable or not. No need to reply > unless the platform maintainer has comments. > > If you already ran DT checks and didn't see these error(s), then > make sure dt-schema is up to date: > > pip3 install dtschema --upgrade > > > New warnings running 'make CHECK_DTBS=y freescale/imx8mp-sb-ucm.dtb' for 20240305004222.622-1-laurent.pinchart@ideasonboard.com: > > Error: arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dts:68.1-10 Label or path hdmi_pvi not found > Error: arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dts:72.1-9 Label or path hdmi_tx not found > Error: arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dts:86.1-13 Label or path hdmi_tx_phy not found > Error: arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dts:208.1-8 Label or path lcdif3 not found I forgot to mention in the cover letter that this depends on i.MX8MP HDMI integration in DT that is currently pending. As it's not in linux-next yet, I'll send a v2 that splits HDMI support in a separate patch. Depending on when the HDMI integration gets merged, the base SB-UCM support can go in by itself or with HDMI support in v6.10. > FATAL ERROR: Syntax error parsing input tree > make[3]: *** [scripts/Makefile.lib:419: arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dtb] Error 1 > make[2]: *** [scripts/Makefile.build:481: arch/arm64/boot/dts/freescale] Error 2 > make[2]: Target 'arch/arm64/boot/dts/freescale/imx8mp-sb-ucm.dtb' not remade because of errors. > make[1]: *** [/home/rob/proj/linux-dt-testing/Makefile:1385: freescale/imx8mp-sb-ucm.dtb] Error 2 > make: *** [Makefile:240: __sub-make] Error 2 > make: Target 'freescale/imx8mp-sb-ucm.dtb' not remade because of errors.