Hi Rob, thanks for your feedback, the changes will be included in V2 of the patch-series. Jan > Von: Rob Herring > Gesendet: Sonntag, 26. November 2017 20:24 > On Thu, Nov 23, 2017 at 01:55:54PM +0100, Jan Tuerk wrote: > > This patch adds support for the emtrion GmbH emCON-MX6 modules. > > They are available with imx.6 Solo, Dual-Lite, Dual and Quad > > equipped with Memory from 512MB to 2GB (configured by U-Boot). [...] > > +&i2c3 { > > + clock-frequency = <100000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_i2c3>; > > + status = "okay"; > > + > > + sgtl5000: sgtl5000@0a { > > audio-codec@a > > same errors repeat below. changed for v2 > > > + compatible = "fsl,sgtl5000"; > > + reg = <0x0a>; > > + clocks = <&sndosc>; > > + VDDA-supply = <&base3p3>; > > + VDDIO-supply = <&base3p3>; > > + }; > > + > > + boardID: pca8754a@3A { > > ...@3a changed in V2 > > > + compatible = "nxp,pca8574"; > > + reg = <0x3A>; > > 0x3a > > > + gpio-controller; > > + #gpio-cells = <1>; > > + }; > > + > > + captouch: edt-ft5x06@38 { > > touchscreen@38 > > > + reg = <0x38>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_irq_touch2 &pinctrl_emcon_gpio4>; > > + interrupt-parent = <&gpio6>; > > + interrupts = <31 IRQ_TYPE_EDGE_FALLING>; > > + compatible = "edt,edt-ft5406"; > > + wake-gpios = <&gpio2 3 GPIO_ACTIVE_HIGH>; > > + wakeup-source; > > + }; > > +}; > > + > > +&ssi2 { > > + status = "okay"; > > +}; > > + > > +&rgb_encoder { > > + status = "okay"; > > +}; > > + [...] > > + > > + rgb_panel: panel { > > + backlight = <&rgb_backlight>; > > + power-supply = <®_prallel_disp>; > > + port { > > + rgb_panel_in: endpoint { > > + remote-endpoint = <&rgb_encoder_out>; > > + }; > > + }; > > + }; > > + > > + rgb_backlight: rgb_backlight { > > rgb-backlight { [...] > > + > > + lvds_backlight: lvds_backlight { > > lvds-backlight { Ok, I'll change that in V2 [...] > > + > > +&i2c1 { > > + clock-frequency = <100000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_i2c1>; > > + status = "okay"; > > + > > + rtc: rtc@68 { > > + compatible = "dallas,ds1307"; > > + reg = <0x68>; > > + }; > > + > > + da9063: da9063@58 { > > pmic@38 changed > > > + compatible = "dlg,da9063"; > > + reg = <0x58>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_pmic>; > > + interrupt-parent = <&gpio2>; > > + [....] > > + > > + mdio { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + ksz9031: phy { > > phy@0 > > Building with W=2 or W=1 will tell you this. thanks for the hint, it will be also included in v2 > > > + reg = <0>; > > + compatible = "ethernet-phy-ieee802.3-c22"; > > + interrupt-parent = <&gpio1>; > > + interrupts = <30 IRQ_TYPE_EDGE_FALLING>; > > + rxdv-skew-ps = <480>; > > + txen-skew-ps = <480>; [...] emtrion GmbH Kreativpark - Alter Schlachthof 45 76131 Karlsruhe GERMANY http://www.emtrion.de _______________________________________ Amtsgericht Mannheim HRB 110 300 Geschäftsführer: Dieter Baur, Ramona Maurer _______________________________________ -- 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