On Thu, May 12, 2016 at 9:00 PM, chunfeng yun [off-list ref] wrote:

Hi,

On Thu, 2016-05-12 at 19:02 -0400, Alan Cooper wrote:

quoted

On Thu, May 12, 2016 at 3:24 AM, chunfeng yun [off-list ref] wrote:

quoted

quoted

quoted

+ - 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.

Rob