Re: [PATCH] usb: typec: hd3ss3220: Check if regulator needs to be switched

From: Greg Kroah-Hartman

Date: Tue Jan 27 2026 - 09:26:32 EST


On Mon, Jan 26, 2026 at 03:22:01PM +0000, Jan Remmet wrote:
> Am 26.01.26 um 15:32 schrieb Heikki Krogerus:
> > Mon, Jan 26, 2026 at 01:06:36PM +0100, Jan Remmet kirjoitti:
> >> Check regulator state as peripheral and detach can disable vbus.
> >> Without this check we will try to disable the regulator twice if
> >> we disconnect host and then connect as device.
> >>
> >> Signed-off-by: Jan Remmet <j.remmet@xxxxxxxxx>
> >> ---
> >> This is a fixup from
> >> - Link to v1: https://lore.kernel.org/r/20260115-wip-jremmet-hd3ss3220_vbus-v1-1-b7d9adfbe346@xxxxxxxxx
> >> To
> >> - Link to v2: https://lore.kernel.org/r/20260123-wip-jremmet-hd3ss3220_vbus-v2-1-bcad313ce92b@xxxxxxxxx
> >
> > I don't understand that, but I think you are fixing a commit in Greg's
> > tree, right?
> Yes
> >
> > If that's the case, then you need the Fixes tag.
>
> Sorry, I'm not familar with the workflow here. Should it be a "--fixup"
> git commit against the usb-next commit id?
>
> Or
> Fixes: 7e7025811579 ("usb: typec: hd3ss3220: Check if regulator needs to
> be switched")
>
> In the commit message (again with commit sha from Greg's usb-next

This one, Fixes: is the correct way to do it.

thanks,

greg k-h