Re: [PATCH 2/2] extcon: usbc-tusb320: Add USB TYPE-C support

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

 



On 8/23/22 11:49, Alvin Šipraga wrote:
Hi Marek,

Hi,

On Sat, Jul 30, 2022 at 08:05:00PM +0200, Marek Vasut wrote:
The TI TUSB320 seems like a better fit for USB TYPE-C subsystem,
which can expose details collected by the TUSB320 in a far more
precise way than extcon. Since there are existing users in the
kernel and in DT which depend on the extcon interface, keep it
for now.

Add TYPE-C interface and expose the supported supply current,
direction and connector polarity via the TYPE-C interface.

Signed-off-by: Marek Vasut <marex@xxxxxxx>
---
Cc: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>
Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
Cc: Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxx>
Cc: Yassine Oudjana <y.oudjana@xxxxxxxxxxxxxx>
To: linux-usb@xxxxxxxxxxxxxxx
---
  drivers/extcon/Kconfig               |   2 +-
  drivers/extcon/extcon-usbc-tusb320.c | 159 +++++++++++++++++++++++++++
  2 files changed, 160 insertions(+), 1 deletion(-)

Happy to see I'm not the only one that observed this. I wonder if you
saw also my previous stab at this:

https://lore.kernel.org/linux-usb/20220301132010.115258-1-alvin@xxxxxxx/

I have not.

I had some issues with the dt-bindings which I could not reconcile, but
the basic problem was how to describe a typec accessory mode mux
connected to the TUSB320. Perhaps you have a better intuition for how
this should look?

One thing that is missing from your implementation that we are using on
our end is the USB role switch. I set this from the typec driver via
usb_role_switch_set_role().

I only use this chip to detect charger type (and cable polarity), the device where this is integrated is always peripheral and cannot charge other devices or become host.

But I think those aforementioned requirements could be extended on top of this patch, can they not ? I recall I looked at least at the direction detection and that could be added easily. I have no way of testing any of that functionality, so I didn't add them as part of the patch.



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

  Powered by Linux