Hi Shawn,
On 12/22/15 14:17, Shawn Guo wrote:
> On Tue, Dec 22, 2015 at 10:15:19AM +0200, Ilya Ledvich wrote:
[...]
>> +/ {
>> + model = "CompuLab CL-SOM-iMX7";
>> + compatible = "compulab,cl-som-imx7", "fsl,imx7d";
>> +
>> + memory {
>> + reg = <0x80000000 0x10000000>; /* 256 MB - minimal
>> configuration */
>> + };
>> +
>> + regulators {
>> + compatible = "simple-bus";
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>
> DT maintainers dislike this fake "simple-bus" container, and we were
> asked to put fixed regulator directly under root. I fixed it up like
> below:
>
> reg_usb_otg1_vbus: regulator-vbus {
> compatible = "regulator-fixed";
> regulator-name = "usb_otg1_vbus";
> regulator-min-microvolt = <5000000>;
> regulator-max-microvolt = <5000000>;
> gpio = <&gpio1 5 GPIO_ACTIVE_HIGH>;
> enable-active-high;
> };
>
Looks good! Thanks for adjusting this.
[...]
--
Regards,
Igor.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
