usb: typec: ucsi: set USB data role when partner type is power cable/ufp
authorMayank Rana <mrana@codeaurora.org>
Thu, 23 Apr 2020 13:20:53 +0000 (16:20 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 23 Apr 2020 13:33:33 +0000 (15:33 +0200)
Currently UCSI framework doesn't update USB data role when partner type
is reported as power cable or power cable with ufp connected. This
results into no USB host mode functionality. This is valid usecase where
user wants to use legacy type c power cable with type a female connector
to attach different USB devices like mouse, thumb drive etc. Hence update
USB data role as host when partner type is reported as power cable or
power cable with ufp connected.

Signed-off-by: Mayank Rana <mrana@codeaurora.org>
Signed-off-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Link: https://lore.kernel.org/r/20200423132058.6972-2-heikki.krogerus@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/typec/ucsi/ucsi.c

index ddf2ad3..e9baa9a 100644 (file)
@@ -566,6 +566,8 @@ static void ucsi_partner_change(struct ucsi_connector *con)
 
        switch (UCSI_CONSTAT_PARTNER_TYPE(con->status.flags)) {
        case UCSI_CONSTAT_PARTNER_TYPE_UFP:
+       case UCSI_CONSTAT_PARTNER_TYPE_CABLE:
+       case UCSI_CONSTAT_PARTNER_TYPE_CABLE_AND_UFP:
                typec_set_data_role(con->port, TYPEC_HOST);
                break;
        case UCSI_CONSTAT_PARTNER_TYPE_DFP:
@@ -627,6 +629,8 @@ static void ucsi_handle_connector_change(struct work_struct *work)
 
                switch (UCSI_CONSTAT_PARTNER_TYPE(con->status.flags)) {
                case UCSI_CONSTAT_PARTNER_TYPE_UFP:
+               case UCSI_CONSTAT_PARTNER_TYPE_CABLE:
+               case UCSI_CONSTAT_PARTNER_TYPE_CABLE_AND_UFP:
                        typec_set_data_role(con->port, TYPEC_HOST);
                        break;
                case UCSI_CONSTAT_PARTNER_TYPE_DFP:
@@ -927,6 +931,8 @@ static int ucsi_register_port(struct ucsi *ucsi, int index)
 
        switch (UCSI_CONSTAT_PARTNER_TYPE(con->status.flags)) {
        case UCSI_CONSTAT_PARTNER_TYPE_UFP:
+       case UCSI_CONSTAT_PARTNER_TYPE_CABLE:
+       case UCSI_CONSTAT_PARTNER_TYPE_CABLE_AND_UFP:
                typec_set_data_role(con->port, TYPEC_HOST);
                break;
        case UCSI_CONSTAT_PARTNER_TYPE_DFP: