Re: [PATCH 0/6] usb: typec: add Qualcomm PMI8998 USB Type-C role-switch support

From: Dmitry Baryshkov

Date: Wed May 20 2026 - 11:52:41 EST


On Tue, May 19, 2026 at 12:43:13AM +0100, Bryan O'Donoghue wrote:
> On 19/05/2026 00:37, Максим Фурман wrote:
> > Three shapes for adding a second port handler to the existing driver:
> >   (a) extracted ops -- pmic_typec_port_ops with chip-specific get_cc /
> >       get_vbus / set_mode / set_vconn / start_toggling, refactor PM8150B
> >       code into the ops, add parallel PMI8998 implementation
> >   (b) parallel port file -- keep PM8150B code untouched, add a sibling
> >       source file with the same tcpc_dev callback surface

2c, my preference would be this option.

> >   (c) per-chip register table + per-chip decode helpers in the same file
> >
> > Bryan, do you have a preference? I'd rather match your taste than guess,
> > since you'll be maintaining the merged result.
>
> Taste - loaded with honey, chilli and salt please chef and add the whole
> clove of garlic....
>
> My feeling is sibling file but please use your own taste/discretion.
>
> ---
> bod

--
With best wishes
Dmitry