Messages in this thread | | | Date | Thu, 11 Feb 2016 16:36:16 +0200 | From | Heikki Krogerus <> | Subject | Re: [PATCH 1/3] usb: USB Type-C Connector Class |
| |
Hi Felipe,
> >> 4. partner_alt_modes - Lists partner's Alternate Modes when connected > > partner_alternate_modes ? (it's a file name, we can spell it out) > > >> 5. partner_type - Can be USB, Charger, Alt Mode or Accessory > >> 6. data_role - The current data role, host or device > >> 7. data_roles - Data roles supported by the connector > > current_data_role > available_data_roles > > >> 8. power_role - Connector's current power role, source or sink > >> 9. power_roles - Power roles supported by the connector > > ditto... > > >> 10. power_operation_mode - The current power level in use > >> 11. usb_pd - yes if the connector supports USB PD. > > supports_usb_power_delivery ? > > >> 12. audio_accessory - yes if the connector supports Audio Accessory > > supports_audio_accessory > > >> 13. debug_accessory - yes if the connector supports Debug Accessory > > supports_debug_accessory
Those all make sense.
> > Doesn't this need locking? > > And why wouldn't user space want to preselect a mode? > > isn't USB always the default mode ?
Just in case there are no misunderstandings here, Oliver was commenting on alternate_mode_store function, and USB of course is not Alternate Mode.
Thanks,
-- heikki
| |