[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <000101dbdfac$c1a35f40$44ea1dc0$@samsung.com>
Date: Tue, 17 Jun 2025 22:54:57 +0530
From: "Pritam Manohar Sutar" <pritam.sutar@...sung.com>
To: "'Krzysztof Kozlowski'" <krzk@...nel.org>, "'Neil Armstrong'"
<neil.armstrong@...aro.org>
Cc: <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
Hi Krzysztof,
> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@...nel.org>
> Sent: 16 June 2025 01:39 PM
> 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.
we will squash these 2 patches in next version of the patch-set (v4).
>
> >
> > Neil
Thank you.
Regards,
Pritam
Powered by blists - more mailing lists