Re: [PATCH v8 1/6] dt-bindings: phy: samsung,usb3-drd-phy: add ExynosAutov920 HS phy compatible
From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2025-09-04 09:42:32
Also in:
linux-arm-kernel, linux-devicetree, linux-samsung-soc, lkml
On 04/09/2025 09:51, Pritam Manohar Sutar wrote:
Hi Krzysztof,quoted
-----Original Message----- From: Krzysztof Kozlowski <krzk@kernel.org> Sent: 04 September 2025 12:18 PM To: Pritam Manohar Sutar <redacted> Cc: vkoul@kernel.org; kishon@kernel.org; robh@kernel.org; krzk+dt@kernel.org; conor+dt@kernel.org; alim.akhtar@samsung.com; andre.draszik@linaro.org; peter.griffin@linaro.org; kauschluss@disroot.org; ivo.ivanov.ivanov1@gmail.com; igor.belwon@mentallysanemainliners.org; m.szyprowski@samsung.com; s.nawrocki@samsung.com; linux- phy@lists.infradead.org; devicetree@vger.kernel.org; linux- kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-samsung- soc@vger.kernel.org; rosa.pila@samsung.com; dev.tailor@samsung.com; faraz.ata@samsung.com; muhammed.ali@samsung.com; selvarasu.g@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:quoted
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 <redacted>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%20removing%20your%20%0A%27reviewed%2Dby%27%20tag%20from%20patch%201%20and%203 (local).
Where in the changelog you explained why you dropped the tags? Best regards, Krzysztof -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy