[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250616-perfect-fervent-robin-a1bc3c@kuoka>
Date: Mon, 16 Jun 2025 10:09:10 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Neil Armstrong <neil.armstrong@...aro.org>
Cc: Pritam Manohar Sutar <pritam.sutar@...sung.com>, vkoul@...nel.org,
kishon@...nel.org, robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
alim.akhtar@...sung.com, andre.draszik@...aro.org, peter.griffin@...aro.org,
kauschluss@...root.org, ivo.ivanov.ivanov1@...il.com, m.szyprowski@...sung.com,
s.nawrocki@...sung.com, linux-phy@...ts.infradead.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-samsung-soc@...r.kernel.org, rosa.pila@...sung.com, dev.tailor@...sung.com,
faraz.ata@...sung.com, muhammed.ali@...sung.com, selvarasu.g@...sung.com
Subject: Re: [PATCH v3 9/9] arm64: dts: exynos: ExynosAutov920: add USB and
USB SS combo phy nodes
On Fri, Jun 13, 2025 at 11:12:26AM GMT, neil.armstrong@...aro.org wrote:
> On 13/06/2025 07:56, Pritam Manohar Sutar wrote:
> > usbdrd31_hsphy: phy@...90000 {
> > compatible = "samsung,exynosautov920-usbdrd-hsphy";
> > reg = <0x16490000 0x0200>;
> > @@ -1109,8 +1120,8 @@ usbdrd31_dwc3: usb@0 {
> > <&cmu_hsi1 CLK_MOUT_HSI1_USBDRD>;
> > clock-names = "ref", "susp_clk";
> > interrupts = <GIC_SPI 491 IRQ_TYPE_LEVEL_HIGH>;
> > - phys = <&usbdrd31_hsphy 0>;
> > - phy-names = "usb2-phy";
> > + phys = <&usbdrd31_hsphy 0>, <&usbdrd31_ssphy 0>;
> > + phy-names = "usb2-phy", "usb3-phy";
> > snps,has-lpm-erratum;
> > snps,dis_u2_susphy_quirk;
> > snps,dis_u3_susphy_quirk;
>
> I think at least patch 6 & 9 should be squashed.
Yes. Changing lines which were just added is a strong hint, that
patchset is incorrectly organized.
>
> Neil
Powered by blists - more mailing lists