> Hmm. I think it'd be reasonable to use dr_mode like the other bindings, > and have the default case be decided by the ID pin when dr_mode isn't > specified. Having different defaults for different bindings seems pretty > reasonable to me. As far as I know, there is no USB Spec says the ID value should be used at device or host mode. At USB device and host driver, there should be NO ID related things, that is to say device or host driver should not care ID is grounded or floated, as ID may be grounded at device mode, and high at host mode for different board design. ID interrupt should only be enabled when dr_mode = otg; Board design and customized usage (user may want to use only device mode for otg capable port) makes what USB role will be used at driver, so dr_mode at dts (or pdata->dr_mode at non-dt solution) can make the decision for driver how USB role will be used. Marc's patch is almost OK, but better use below logic at core.c: If there is no dr_mode, it is better to indicate an error message and quit probe. > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
