mbox series

[v3,00/10] usb: typec: A few more improvements for Intel CHT

Message ID 20180904112219.89287-1-heikki.krogerus@linux.intel.com (mailing list archive)
Headers show
Series usb: typec: A few more improvements for Intel CHT | expand

Message

Heikki Krogerus Sept. 4, 2018, 11:22 a.m. UTC
Hi,

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


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(-)

Comments

Andy Shevchenko Sept. 4, 2018, 12:13 p.m. UTC | #1
On Tue, Sep 4, 2018 at 2:22 PM Heikki Krogerus
<heikki.krogerus@linux.intel.com> wrote:
>
> Hi,
>
> 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
>
>

It seems you forgot my tag

Acked-by: Andy Shevchenko <andy.shevchenko@gmail.com>

for PDx86 bits on condition that Hans is okay with them.

> 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(-)
>
> --
> 2.18.0
>
Hans de Goede Sept. 4, 2018, 1:43 p.m. UTC | #2
Hi,

On 04-09-18 14:13, Andy Shevchenko wrote:
> On Tue, Sep 4, 2018 at 2:22 PM Heikki Krogerus
> <heikki.krogerus@linux.intel.com> wrote:
>>
>> Hi,
>>
>> 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
>>
>>
> 
> It seems you forgot my tag
> 
> Acked-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> 
> for PDx86 bits on condition that Hans is okay with them.

I'm ok with them now.

The entire series is:

Reviewed-by: Hans de Goede <hdegoede@redhat.com>


Greg, I believe that the intent was for the entire series
to be merged through your tree and I believe that this
series is ready for merging now.

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(-)
>>
>> --
>> 2.18.0
>>
> 
>