Commit 2000016c authored by Hans de Goede's avatar Hans de Goede Committed by Greg Kroah-Hartman

usb: typec: tcpm: Use new Type-C switch/mux and usb-role-switch functions

Remove the unused (not implemented anywhere) tcpc_mux_dev abstraction
and replace it with calling the new typec_set_orientation,
usb_role_switch_set and typec_set_mode functions.
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
Reviewed-by: default avatarGuenter Roeck <linux@roeck-us.net>
Reviewed-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: default avatarHeikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent c6962c29
...@@ -48,6 +48,7 @@ if TYPEC ...@@ -48,6 +48,7 @@ if TYPEC
config TYPEC_TCPM config TYPEC_TCPM
tristate "USB Type-C Port Controller Manager" tristate "USB Type-C Port Controller Manager"
depends on USB depends on USB
select USB_ROLE_SWITCH
help help
The Type-C Port Controller Manager provides a USB PD and USB Type-C The Type-C Port Controller Manager provides a USB PD and USB Type-C
state machine for use with Type-C Port Controllers. state machine for use with Type-C Port Controllers.
......
...@@ -1239,7 +1239,6 @@ static void init_tcpc_dev(struct tcpc_dev *fusb302_tcpc_dev) ...@@ -1239,7 +1239,6 @@ static void init_tcpc_dev(struct tcpc_dev *fusb302_tcpc_dev)
fusb302_tcpc_dev->set_roles = tcpm_set_roles; fusb302_tcpc_dev->set_roles = tcpm_set_roles;
fusb302_tcpc_dev->start_drp_toggling = tcpm_start_drp_toggling; fusb302_tcpc_dev->start_drp_toggling = tcpm_start_drp_toggling;
fusb302_tcpc_dev->pd_transmit = tcpm_pd_transmit; fusb302_tcpc_dev->pd_transmit = tcpm_pd_transmit;
fusb302_tcpc_dev->mux = NULL;
} }
static const char * const cc_polarity_name[] = { static const char * const cc_polarity_name[] = {
......
...@@ -20,6 +20,7 @@ ...@@ -20,6 +20,7 @@
#include <linux/usb/pd.h> #include <linux/usb/pd.h>
#include <linux/usb/pd_bdo.h> #include <linux/usb/pd_bdo.h>
#include <linux/usb/pd_vdo.h> #include <linux/usb/pd_vdo.h>
#include <linux/usb/role.h>
#include <linux/usb/tcpm.h> #include <linux/usb/tcpm.h>
#include <linux/usb/typec.h> #include <linux/usb/typec.h>
#include <linux/workqueue.h> #include <linux/workqueue.h>
...@@ -176,6 +177,7 @@ struct tcpm_port { ...@@ -176,6 +177,7 @@ struct tcpm_port {
struct typec_port *typec_port; struct typec_port *typec_port;
struct tcpc_dev *tcpc; struct tcpc_dev *tcpc;
struct usb_role_switch *role_sw;
enum typec_role vconn_role; enum typec_role vconn_role;
enum typec_role pwr_role; enum typec_role pwr_role;
...@@ -604,18 +606,25 @@ void tcpm_pd_transmit_complete(struct tcpm_port *port, ...@@ -604,18 +606,25 @@ void tcpm_pd_transmit_complete(struct tcpm_port *port,
EXPORT_SYMBOL_GPL(tcpm_pd_transmit_complete); EXPORT_SYMBOL_GPL(tcpm_pd_transmit_complete);
static int tcpm_mux_set(struct tcpm_port *port, enum tcpc_mux_mode mode, static int tcpm_mux_set(struct tcpm_port *port, enum tcpc_mux_mode mode,
enum usb_role usb_role) enum usb_role usb_role,
enum typec_orientation orientation)
{ {
int ret = 0; int ret;
tcpm_log(port, "Requesting mux mode %d, usb-role %d, polarity %d", tcpm_log(port, "Requesting mux mode %d, usb-role %d, orientation %d",
mode, usb_role, port->polarity); mode, usb_role, orientation);
if (port->tcpc->mux) ret = typec_set_orientation(port->typec_port, orientation);
ret = port->tcpc->mux->set(port->tcpc->mux, mode, usb_role, if (ret)
port->polarity); return ret;
return ret; if (port->role_sw) {
ret = usb_role_switch_set_role(port->role_sw, usb_role);
if (ret)
return ret;
}
return typec_set_mode(port->typec_port, mode);
} }
static int tcpm_set_polarity(struct tcpm_port *port, static int tcpm_set_polarity(struct tcpm_port *port,
...@@ -728,15 +737,21 @@ static int tcpm_set_attached_state(struct tcpm_port *port, bool attached) ...@@ -728,15 +737,21 @@ static int tcpm_set_attached_state(struct tcpm_port *port, bool attached)
static int tcpm_set_roles(struct tcpm_port *port, bool attached, static int tcpm_set_roles(struct tcpm_port *port, bool attached,
enum typec_role role, enum typec_data_role data) enum typec_role role, enum typec_data_role data)
{ {
enum typec_orientation orientation;
enum usb_role usb_role; enum usb_role usb_role;
int ret; int ret;
if (port->polarity == TYPEC_POLARITY_CC1)
orientation = TYPEC_ORIENTATION_NORMAL;
else
orientation = TYPEC_ORIENTATION_REVERSE;
if (data == TYPEC_HOST) if (data == TYPEC_HOST)
usb_role = USB_ROLE_HOST; usb_role = USB_ROLE_HOST;
else else
usb_role = USB_ROLE_DEVICE; usb_role = USB_ROLE_DEVICE;
ret = tcpm_mux_set(port, TYPEC_MUX_USB, usb_role); ret = tcpm_mux_set(port, TYPEC_MUX_USB, usb_role, orientation);
if (ret < 0) if (ret < 0)
return ret; return ret;
...@@ -2029,7 +2044,8 @@ static int tcpm_src_attach(struct tcpm_port *port) ...@@ -2029,7 +2044,8 @@ static int tcpm_src_attach(struct tcpm_port *port)
out_disable_pd: out_disable_pd:
port->tcpc->set_pd_rx(port->tcpc, false); port->tcpc->set_pd_rx(port->tcpc, false);
out_disable_mux: out_disable_mux:
tcpm_mux_set(port, TYPEC_MUX_NONE, USB_ROLE_NONE); tcpm_mux_set(port, TYPEC_MUX_NONE, USB_ROLE_NONE,
TYPEC_ORIENTATION_NONE);
return ret; return ret;
} }
...@@ -2073,7 +2089,8 @@ static void tcpm_reset_port(struct tcpm_port *port) ...@@ -2073,7 +2089,8 @@ static void tcpm_reset_port(struct tcpm_port *port)
tcpm_init_vconn(port); tcpm_init_vconn(port);
tcpm_set_current_limit(port, 0, 0); tcpm_set_current_limit(port, 0, 0);
tcpm_set_polarity(port, TYPEC_POLARITY_CC1); tcpm_set_polarity(port, TYPEC_POLARITY_CC1);
tcpm_mux_set(port, TYPEC_MUX_NONE, USB_ROLE_NONE); tcpm_mux_set(port, TYPEC_MUX_NONE, USB_ROLE_NONE,
TYPEC_ORIENTATION_NONE);
tcpm_set_attached_state(port, false); tcpm_set_attached_state(port, false);
port->try_src_count = 0; port->try_src_count = 0;
port->try_snk_count = 0; port->try_snk_count = 0;
...@@ -3653,6 +3670,12 @@ struct tcpm_port *tcpm_register_port(struct device *dev, struct tcpc_dev *tcpc) ...@@ -3653,6 +3670,12 @@ struct tcpm_port *tcpm_register_port(struct device *dev, struct tcpc_dev *tcpc)
port->partner_desc.identity = &port->partner_ident; port->partner_desc.identity = &port->partner_ident;
port->port_type = tcpc->config->type; port->port_type = tcpc->config->type;
port->role_sw = usb_role_switch_get(port->dev);
if (IS_ERR(port->role_sw)) {
err = PTR_ERR(port->role_sw);
goto out_destroy_wq;
}
port->typec_port = typec_register_port(port->dev, &port->typec_caps); port->typec_port = typec_register_port(port->dev, &port->typec_caps);
if (IS_ERR(port->typec_port)) { if (IS_ERR(port->typec_port)) {
err = PTR_ERR(port->typec_port); err = PTR_ERR(port->typec_port);
...@@ -3688,6 +3711,7 @@ struct tcpm_port *tcpm_register_port(struct device *dev, struct tcpc_dev *tcpc) ...@@ -3688,6 +3711,7 @@ struct tcpm_port *tcpm_register_port(struct device *dev, struct tcpc_dev *tcpc)
return port; return port;
out_destroy_wq: out_destroy_wq:
usb_role_switch_put(port->role_sw);
destroy_workqueue(port->wq); destroy_workqueue(port->wq);
return ERR_PTR(err); return ERR_PTR(err);
} }
......
...@@ -16,7 +16,6 @@ ...@@ -16,7 +16,6 @@
#define __LINUX_USB_TCPM_H #define __LINUX_USB_TCPM_H
#include <linux/bitops.h> #include <linux/bitops.h>
#include <linux/usb/role.h>
#include <linux/usb/typec.h> #include <linux/usb/typec.h>
#include "pd.h" #include "pd.h"
...@@ -113,14 +112,6 @@ enum tcpc_mux_mode { ...@@ -113,14 +112,6 @@ enum tcpc_mux_mode {
TCPC_MUX_DP_ENABLED, TCPC_MUX_DP_ENABLED,
}; };
struct tcpc_mux_dev {
int (*set)(struct tcpc_mux_dev *dev, enum tcpc_mux_mode mux_mode,
enum usb_role usb_role,
enum typec_cc_polarity polarity);
bool dfp_only;
void *priv_data;
};
/** /**
* struct tcpc_dev - Port configuration and callback functions * struct tcpc_dev - Port configuration and callback functions
* @config: Pointer to port configuration * @config: Pointer to port configuration
...@@ -172,7 +163,6 @@ struct tcpc_dev { ...@@ -172,7 +163,6 @@ struct tcpc_dev {
int (*try_role)(struct tcpc_dev *dev, int role); int (*try_role)(struct tcpc_dev *dev, int role);
int (*pd_transmit)(struct tcpc_dev *dev, enum tcpm_transmit_type type, int (*pd_transmit)(struct tcpc_dev *dev, enum tcpm_transmit_type type,
const struct pd_message *msg); const struct pd_message *msg);
struct tcpc_mux_dev *mux;
}; };
struct tcpm_port; struct tcpm_port;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment