Re: [PATCH v4 00/10] usb: typec: A few more improvements for Intel CHT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 11-09-18 12:10, Heikki Krogerus wrote:
Hi,

This is fourth version of this series. There was one bug in patch 2/10
that Hans noticed. It should be fixed now.

The commit message from v3:

These patches will introduce a few improvements to the USB Type-C
support on Intel CHT platform. In this series I'm preparing Intel CHT
mux handling for DisplayPort Alt Mode support, but please note that,
after these patches the DisplayPort alternate mode will still not work
out of the box on CHT platform. Changes to the fusb302.c, and possibly
tcpm.c are still needed.

This version will only fix the issue Hans pointed out in v2. Instead
of replacing the connection that assigned the mux to the USB Type-C
port with a connection that assigns the mux to the alternate mode
device, we keep all the existing connections and add a new one for the
alternate mode device. That way USB3 devices continue to be enumerated
as USB3 devices instead of USB2 devices.

The origin thread can be read here:
https://www.spinics.net/lists/linux-usb/msg172033.html

Thanks, the entire series looks good to me and has been tested by
me on a GPD win with a fusb302 tcpm controller:

Acked-by: Hans de Goede <hdegoede@xxxxxxxxxx>
Tested-by: Hans de Goede <hdegoede@xxxxxxxxxx>

Greg, the previous version of this series also was:

"
Acked-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>

for PDx86 bits on condition that Hans is fine with them.
"

With the intend that the entire series would be merged
through the USB tree with Andy's ack for the
"platform: x86: ..." patches.

Regards,

Hans








Heikki Krogerus (10):
   usb: typec: Take care of driver module reference counting
   usb: roles: Handle driver reference counting
   platform: x86: intel_cht_int33fe: Add dependency on muxes
   drivers: base: Helpers for adding device connection descriptions
   platform: x86: intel_cht_int33fe: Register all connections at once
   platform: x86: intel_cht_int33fe: Add connection for the DP alt mode
   platform: x86: intel_cht_int33fe: Add connections for the USB Type-C
     port
   usb: typec: class: Don't use port parent for getting mux handles
   platform: x86: intel_cht_int33fe: Remove the old connections for the
     muxes
   usb: typec: fusb302: reorganizing the probe function a little

  drivers/platform/x86/Kconfig             |  2 ++
  drivers/platform/x86/intel_cht_int33fe.c | 27 ++++++++---------
  drivers/usb/common/roles.c               | 15 ++++++++--
  drivers/usb/typec/class.c                | 38 ++++++++++--------------
  drivers/usb/typec/fusb302/fusb302.c      | 25 ++++++----------
  drivers/usb/typec/mux.c                  | 17 ++++++++---
  include/linux/device.h                   | 24 +++++++++++++++
  7 files changed, 87 insertions(+), 61 deletions(-)




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux