On Fri, Dec 2, 2016 at 4:39 AM, Bai Ping [off-list ref] wrote:

Add basic dts support for imx6sll EVK baoard.

s/baord/board

+       battery: max8903 at 0 {
+               compatible = "fsl,max8903-charger";
+               pinctrl-names = "default";
+               dok_input = <&gpio4 13 1>;
+               uok_input = <&gpio4 13 1>;
+               chg_input = <&gpio4 15 1>;
+               flt_input = <&gpio4 14 1>;
+               fsl,dcm_always_high;
+               fsl,dc_valid;
+               fsl,adc_disable;

These properties do not exist in mainline, please remove them.

+               status = "okay";
+       };
+
+       pxp_v4l2_out {
+               compatible = "fsl,imx6sl-pxp-v4l2";
+               status = "okay";
+       };

We don't have pxp support in mainline kernel, please remove it.

+
+       regulators {
+               compatible = "simple-bus";
+               #address-cells = <1>;
+               #size-cells = <0>;

Please remove it and place the regulator nodes directly as below:

+
+               reg_usb_otg1_vbus: regulator at 0 {
+                       compatible = "regulator-fixed";
+                       reg = <0>;
+                       regulator-name = "usb_otg1_vbus";
+                       regulator-min-microvolt = <5000000>;
+                       regulator-max-microvolt = <5000000>;
+                       gpio = <&gpio4 0 GPIO_ACTIVE_HIGH>;
+                       enable-active-high;
+               };

    reg_usb_otg1_vbus: regulator-usb-otg1-vbus {
          compatible = "regulator-fixed";
          regulator-name = "usb_otg1_vbus";
          regulator-min-microvolt = <5000000>;
          regulator-max-microvolt = <5000000>;
          gpio = <&gpio4 0 GPIO_ACTIVE_HIGH>;
          enable-active-high;
    };

+&cpu0 {
+       arm-supply = <&sw1a_reg>;
+       soc-supply = <&sw1c_reg>;
+};

This is only for LDO bypass mode, right? We don't support LDO-bypass
in mainline.

+&gpc {
+       fsl,ldo-bypass = <1>;

We don't support ldo-bypass in mainline, please remove it.

+&pxp {
+       status = "okay";
+};

We don't support PXP in mainline, please remove it.