[PATCH v8 1/6] dt-bindings: phy: samsung,usb3-drd-phy: add ExynosAutov920 HS phy compatible
Pritam Manohar Sutar
pritam.sutar at samsung.com
Thu Sep 4 03:21:57 PDT 2025
Hi Krzysztof,
> -----Original Message-----
> From: Krzysztof Kozlowski <krzk at kernel.org>
> Sent: 04 September 2025 03:12 PM
> To: Pritam Manohar Sutar <pritam.sutar at samsung.com>
> Cc: vkoul at kernel.org; kishon at kernel.org; robh at kernel.org;
> krzk+dt at kernel.org; conor+dt at kernel.org; alim.akhtar at samsung.com;
> andre.draszik at linaro.org; peter.griffin at linaro.org; kauschluss at disroot.org;
> ivo.ivanov.ivanov1 at gmail.com; igor.belwon at mentallysanemainliners.org;
> m.szyprowski at samsung.com; s.nawrocki at samsung.com; linux-
> phy at lists.infradead.org; devicetree at vger.kernel.org; linux-
> kernel at vger.kernel.org; linux-arm-kernel at lists.infradead.org; linux-samsung-
> soc at vger.kernel.org; rosa.pila at samsung.com; dev.tailor at samsung.com;
> faraz.ata at samsung.com; muhammed.ali at samsung.com;
> selvarasu.g at samsung.com
> Subject: Re: [PATCH v8 1/6] dt-bindings: phy: samsung,usb3-drd-phy: add
> ExynosAutov920 HS phy compatible
>
> On 04/09/2025 09:51, Pritam Manohar Sutar wrote:
> > Hi Krzysztof,
> >
> >> -----Original Message-----
> >> From: Krzysztof Kozlowski <krzk at kernel.org>
> >> Sent: 04 September 2025 12:18 PM
> >> To: Pritam Manohar Sutar <pritam.sutar at samsung.com>
> >> Cc: vkoul at kernel.org; kishon at kernel.org; robh at kernel.org;
> >> krzk+dt at kernel.org; conor+dt at kernel.org; alim.akhtar at samsung.com;
> >> andre.draszik at linaro.org; peter.griffin at linaro.org;
> >> kauschluss at disroot.org; ivo.ivanov.ivanov1 at gmail.com;
> >> igor.belwon at mentallysanemainliners.org;
> >> m.szyprowski at samsung.com; s.nawrocki at samsung.com; linux-
> >> phy at lists.infradead.org; devicetree at vger.kernel.org; linux-
> >> kernel at vger.kernel.org; linux-arm-kernel at lists.infradead.org;
> >> linux-samsung- soc at vger.kernel.org; rosa.pila at samsung.com;
> >> dev.tailor at samsung.com; faraz.ata at samsung.com;
> >> muhammed.ali at samsung.com; selvarasu.g at samsung.com
> >> Subject: Re: [PATCH v8 1/6] dt-bindings: phy: samsung,usb3-drd-phy:
> >> add
> >> ExynosAutov920 HS phy compatible
> >>
> >> On Wed, Sep 03, 2025 at 01:08:22PM +0530, Pritam Manohar Sutar wrote:
> >>> Document support for the USB20 phy found on the ExynosAutov920 SoC.
> >>> The
> >>> USB20 phy is functionally identical to that on the Exynos850 SoC, so
> >>> no driver changes are needed to support this phy. However, add a
> >>> dedicated compatible string for USB20 phy found in this SoC.
> >>>
> >>> Signed-off-by: Pritam Manohar Sutar <pritam.sutar at samsung.com>
> >>
> >> You just dropped all tags without explaining why.
> >
> > Regretted inconvenience.
> >
> > There were significant changes in supplies' names in driver and
> > schemas (patch-set v8). This led to make changes in patch no 5. And
> > review for these changes is needed. Hence, removed RB tag in this patch-set.
> >
> > There was a ask for the same https://lore.kernel.org/linux-
> phy/000401dc18cd$ec02a1b0$c407e510$@samsung.com/#:~:text=Let%20me%
> 20know%2C%20because%20of%20above%20changes%2C%20should%20be%20
> removing%20your%20%0A%27reviewed%2Dby%27%20tag%20from%20patch%
> 201%20and%203.
> >
>
>
> Where in the changelog you explained why you dropped the tags?
Along with supplies' names, there were similar commit messages
for patch no 1, 3 as patch no 5 (v7). (though, they were explaining
schema more than h/w). Changed commit messages of the
patch no 1, 3, 5 (v7) as per reference commits and would like
to get them reviewed again, so did not add RB for patch 1 and 3,
which you had given RB (in v7).
Forgot to add these details for dropping the RB tags in changelog.
Do I need to send v9 by retaining RB for patch 1 and 3?
>
> Best regards,
> Krzysztof
Thank you.
Regards,
Pritam
More information about the linux-phy
mailing list