On Thu, 2016-05-12 at 22:09 -0500, Rob Herring wrote: > On Thu, May 12, 2016 at 9:00 PM, chunfeng yun <chunfeng.yun@xxxxxxxxxxxx> wrote: > > Hi, > > > > On Thu, 2016-05-12 at 19:02 -0400, Alan Cooper wrote: > >> On Thu, May 12, 2016 at 3:24 AM, chunfeng yun <chunfeng.yun@xxxxxxxxxxxx> wrote: > >> >> > + - mediatek,enable-manual-drd : supports manual dual-role switch by sysfs > >> >> > + interface; only used when receptacle is TYPE-A and also wants to support > >> >> > + dual-role mode. > >> >> > >> >> sysfs is a Linux detail that doesn't apply to the binding. Does the > >> >> property mean "the IP block supports or doesn't support switching" or > >> >> "I want to enable switching feature". Only the former belongs in DT. > >> >> > >> > It is the former case, and has little relation with sysfs interface. > >> > I will modify it later, sorry for my unclear description. > >> > >> Could the property name be just "enable-manual-drd" instead of > >> "mediatek,enable-manual-drd"? I expect to have to add this > >> functionality to our USB driver in the near future. > > > > Do you mean provide some APIs to in common USB driver? > > Probably, but first just put the property in a common binding doc. > Ok, I modify it later Thanks. > 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