| Message ID | 20230112221609.540754-4-pmalani@chromium.org |
|---|---|
| State | New |
| Headers | show |
| Series | [1/3] usb: typec: Add retimer handle to port altmode | expand |
On Thu, Jan 12, 2023 at 10:16:08PM +0000, Prashant Malani wrote: > Since ports can have retimers associated with them, update the Type-C > alternate mode bus code to also set retimer state when the switch state > is updated. > > While we are here, make the typec_retimer_dev_type declaration in the > retimer.h file as extern, so that the header file can be successfully > included in the bus code without redeclaration compilation errors. > > Signed-off-by: Prashant Malani <pmalani@chromium.org> Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com> > --- > drivers/usb/typec/bus.c | 22 ++++++++++++++++++++++ > drivers/usb/typec/retimer.h | 2 +- > 2 files changed, 23 insertions(+), 1 deletion(-) > > diff --git a/drivers/usb/typec/bus.c b/drivers/usb/typec/bus.c > index 9f1bbd26ca47..0c8d554240be 100644 > --- a/drivers/usb/typec/bus.c > +++ b/drivers/usb/typec/bus.c > @@ -11,6 +11,22 @@ > #include "bus.h" > #include "class.h" > #include "mux.h" > +#include "retimer.h" > + > +static inline int > +typec_altmode_set_retimer(struct altmode *alt, unsigned long conf, void *data) > +{ > + struct typec_retimer_state state; > + > + if (!alt->retimer) > + return 0; > + > + state.alt = &alt->adev; > + state.mode = conf; > + state.data = data; > + > + return typec_retimer_set(alt->retimer, &state); > +} > > static inline int > typec_altmode_set_mux(struct altmode *alt, unsigned long conf, void *data) > @@ -31,6 +47,12 @@ typec_altmode_set_mux(struct altmode *alt, unsigned long conf, void *data) > static inline int > typec_altmode_set_switches(struct altmode *alt, unsigned long conf, void *data) > { > + int ret; > + > + ret = typec_altmode_set_retimer(alt, conf, data); > + if (ret) > + return ret; > + > return typec_altmode_set_mux(alt, conf, data); > } > > diff --git a/drivers/usb/typec/retimer.h b/drivers/usb/typec/retimer.h > index e34bd23323be..d6a5ef9881e1 100644 > --- a/drivers/usb/typec/retimer.h > +++ b/drivers/usb/typec/retimer.h > @@ -12,7 +12,7 @@ struct typec_retimer { > > #define to_typec_retimer(_dev_) container_of(_dev_, struct typec_retimer, dev) > > -const struct device_type typec_retimer_dev_type; > +extern const struct device_type typec_retimer_dev_type; > > #define is_typec_retimer(dev) ((dev)->type == &typec_retimer_dev_type) > > -- > 2.39.0.314.g84b9a713c41-goog
diff --git a/drivers/usb/typec/bus.c b/drivers/usb/typec/bus.c index 9f1bbd26ca47..0c8d554240be 100644 --- a/drivers/usb/typec/bus.c +++ b/drivers/usb/typec/bus.c @@ -11,6 +11,22 @@ #include "bus.h" #include "class.h" #include "mux.h" +#include "retimer.h" + +static inline int +typec_altmode_set_retimer(struct altmode *alt, unsigned long conf, void *data) +{ + struct typec_retimer_state state; + + if (!alt->retimer) + return 0; + + state.alt = &alt->adev; + state.mode = conf; + state.data = data; + + return typec_retimer_set(alt->retimer, &state); +} static inline int typec_altmode_set_mux(struct altmode *alt, unsigned long conf, void *data) @@ -31,6 +47,12 @@ typec_altmode_set_mux(struct altmode *alt, unsigned long conf, void *data) static inline int typec_altmode_set_switches(struct altmode *alt, unsigned long conf, void *data) { + int ret; + + ret = typec_altmode_set_retimer(alt, conf, data); + if (ret) + return ret; + return typec_altmode_set_mux(alt, conf, data); } diff --git a/drivers/usb/typec/retimer.h b/drivers/usb/typec/retimer.h index e34bd23323be..d6a5ef9881e1 100644 --- a/drivers/usb/typec/retimer.h +++ b/drivers/usb/typec/retimer.h @@ -12,7 +12,7 @@ struct typec_retimer { #define to_typec_retimer(_dev_) container_of(_dev_, struct typec_retimer, dev) -const struct device_type typec_retimer_dev_type; +extern const struct device_type typec_retimer_dev_type; #define is_typec_retimer(dev) ((dev)->type == &typec_retimer_dev_type)
Since ports can have retimers associated with them, update the Type-C alternate mode bus code to also set retimer state when the switch state is updated. While we are here, make the typec_retimer_dev_type declaration in the retimer.h file as extern, so that the header file can be successfully included in the bus code without redeclaration compilation errors. Signed-off-by: Prashant Malani <pmalani@chromium.org> --- drivers/usb/typec/bus.c | 22 ++++++++++++++++++++++ drivers/usb/typec/retimer.h | 2 +- 2 files changed, 23 insertions(+), 1 deletion(-)