On Thu, Jul 13, 2017 at 03:29:43PM -0700, Stephen Boyd wrote: > Quoting Peter Rosin (2017-07-11 23:45:24) > > On 2017-07-12 03:02, Stephen Boyd wrote: > > > @@ -102,4 +107,7 @@ Example: > > > rx-burst-size-dword = <0x10>; > > > extcon = <0>, <&usb_id>; > > > phy-clkgate-delay-us = <400>; > > > + mux-controls = <&usb_switch>; > > > + mux-control-names = "usb_switch"; Pointless to have a name when there is only 1. > > > + usb-switch-states = <0>, <1>; > > > > I don't see the need for usb-switch-states? Just assume states 0/1 and > > if someone later needs some other states, make them add a property that > > overrides the defaults. Just document that 0 is host and 1 is device. > > > > Fine by me. Rob H do you agree? Yes. Rob -- 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