Re: [PATCH 1/3] usb: USB Type-C Connector Class
From: Heikki Krogerus
Date: Thu Feb 11 2016 - 09:38:42 EST
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