[PATCH v8 1/6] dt-bindings: phy: samsung,usb3-drd-phy: add ExynosAutov920 HS phy compatible

[Krzysztof Kozlowski]

On 04/09/2025 19:21, Pritam Manohar Sutar wrote:
> 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). 

I do not have time to review the same second time and I find such
request quite a waste of my time. It's v8 so I am surprised to see it
getting changed even after review.


Best regards,
Krzysztof