On Tue, Dec 19, 2017 at 1:49 PM, Stefan Riedmueller <s.riedmueller@xxxxxxxxx> wrote: > + > +/ { > + model = "PHYTEC phyCORE-i.MX 6"; > + compatible = "phytec,imx6qdl-pcm058", "fsl,imx6qdl"; > + > + aliases { > + ipu0 = &ipu1; Is this alias needed? > + rtc1 = &da9062_rtc; > + rtc2 = &snvs_rtc; > + }; > + > + /* > + * Set the minimum memory size here and > + * let the bootloader set the real size. > + */ > + memory { memory@10000000 so that you don't get warnings when building with W=1. Also, please build W=1 and make sure to not introduce any build warning in this series. > +&i2c3 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_i2c3>; > + clock-frequency = <400000>; > + status = "okay"; > + > + eeprom: eeprom@50 { > + compatible = "cat,24c32"; This compatible is not documented. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html