linux-kernel - Re: [PATCH] phy: tegra: xusb: Default otg mode to peripheral

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CALHNRZ9s5EdL3vapyJS4TdT=v5v_QG-=n8ABNJDLU4B-7w+wRw@mail.gmail.com>
Date: Sun, 20 Apr 2025 20:44:04 -0500
From: Aaron Kling <webgeek1234@...il.com>
To: webgeek1234@...il.com
Cc: JC Kuo <jckuo@...dia.com>, Vinod Koul <vkoul@...nel.org>, 
	Kishon Vijay Abraham I <kishon@...nel.org>, Thierry Reding <thierry.reding@...il.com>, 
	Jonathan Hunter <jonathanh@...dia.com>, linux-phy@...ts.infradead.org, 
	linux-tegra@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] phy: tegra: xusb: Default otg mode to peripheral

On Sun, Apr 13, 2025 at 11:45 PM Aaron Kling <webgeek1234@...il.com> wrote:
>
> On Fri, Apr 4, 2025 at 3:18 AM Aaron Kling via B4 Relay
> <devnull+webgeek1234.gmail.com@...nel.org> wrote:
> >
> > From: Aaron Kling <webgeek1234@...il.com>
> >
> > Currently, if usb-role-switch is set and role-switch-default-mode is
> > not, a xusb port will be inoperable until that port is hotplugged,
> > because the driver defaults to role none. Instead of requiring all
> > devices to set the default mode, assume that the port is primarily
> > intended for use in device mode. This assumption already has precedence
> > in the synopsys dwc3 driver.
> >
> > Signed-off-by: Aaron Kling <webgeek1234@...il.com>
> > ---
> >  drivers/phy/tegra/xusb.c | 8 +++-----
> >  1 file changed, 3 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/phy/tegra/xusb.c b/drivers/phy/tegra/xusb.c
> > index 79d4814d758d5e1f0e8200d61e131606adbb0e2d..c56e83216d0f566a09b67377172fb04c8406f4cf 100644
> > --- a/drivers/phy/tegra/xusb.c
> > +++ b/drivers/phy/tegra/xusb.c
> > @@ -731,13 +731,11 @@ static void tegra_xusb_parse_usb_role_default_mode(struct tegra_xusb_port *port)
> >
> >         if (mode == USB_DR_MODE_HOST)
> >                 role = USB_ROLE_HOST;
> > -       else if (mode == USB_DR_MODE_PERIPHERAL)
> > +       else
> >                 role = USB_ROLE_DEVICE;
> >
> > -       if (role != USB_ROLE_NONE) {
> > -               usb_role_switch_set_role(port->usb_role_sw, role);
> > -               dev_dbg(&port->dev, "usb role default mode is %s", modes[mode]);
> > -       }
> > +       usb_role_switch_set_role(port->usb_role_sw, role);
> > +       dev_dbg(&port->dev, "usb role default mode is %s", modes[mode]);
> >  }
> >
> >  static int tegra_xusb_usb2_port_parse_dt(struct tegra_xusb_usb2_port *usb2)
> >
> > ---
> > base-commit: 91e5bfe317d8f8471fbaa3e70cf66cae1314a516
> > change-id: 20250404-xusb-peripheral-c45b1637f33b
> >
> > Best regards,
> > --
> > Aaron Kling <webgeek1234@...il.com>
> >
> >
>
> Friendly reminder about this patch.
>
> Sincerely,
> Aaron

Friendly re-reminder about this series.

Sincerely,
Aaron Kling