On 23/07/2022 14:15, Peng Fan wrote: > All, > > I am thinking whether we need to modify the fsl,pins as below. Because > fsl,pins use uint32-matrix, so just wanna know we should do this update. > > Anyway there is NO dtbs check warning. If we update fsl,pins, there are > bunches of dts files need update. Do you think we need to update or not? > > pinctrl_ov5640: ov5640grp { > - fsl,pins = < > - MX8MM_IOMUXC_GPIO1_IO07_GPIO1_IO7 0x19 > - MX8MM_IOMUXC_GPIO1_IO06_GPIO1_IO6 0x19 > - MX8MM_IOMUXC_GPIO1_IO14_CCMSRCGPCMIX_CLKO1 0x59 > - >; > + fsl,pins = > + <MX8MM_IOMUXC_GPIO1_IO07_GPIO1_IO7 > 0x19>, > + <MX8MM_IOMUXC_GPIO1_IO06_GPIO1_IO6 > 0x19>, > + <MX8MM_IOMUXC_GPIO1_IO14_CCMSRCGPCMIX_CLKO1 > 0x59>; > }; I believe yes, but as Rob replied, the dtschema has trouble matching the different fsl,pins, if I understood correctly. Best regards, Krzysztof