Re: [PATCH v2 1/6] ARM: dts: ifc6410: Add pwrseq support for WLAN

From: Srinivas Kandagatla
Date: Thu Sep 03 2015 - 11:58:33 EST




On 25/08/15 22:33, Stephen Boyd wrote:
On 08/18/2015 06:06 AM, Srinivas Kandagatla wrote:
@@ -10,6 +11,20 @@
serial1 = &gsbi6_serial;
};
+ pwrseq {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;

Why do we need any of these three properties?

Yep, you are right I will remove it and give it a try.

+ compatible = "simple-bus";
+
+ sdcc4_pwrseq: sdcc4_pwrseq {
+ pinctrl-names = "default";
+ pinctrl-0 = <&wlan_default_gpios>;
+ compatible = "mmc-pwrseq-simple";
+ reset-gpios = <&pm8921_gpio 43 GPIO_ACTIVE_LOW>;

Especially because this node doesn't have a reg property.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/