From: Krzysztof Kozlowski <krzk@kernel.org>
To: Marek Vasut <marex@nabladev.com>
Cc: linux-usb@vger.kernel.org, Pankaj Dev <pankaj.dev@st.com>,
Fabrice Gasnier <fabrice.gasnier@foss.st.com>,
Rahul Kumar <rahul.kumar05@st.com>,
Alexandre Torgue <alexandre.torgue@foss.st.com>,
Christian Bruel <christian.bruel@foss.st.com>,
Conor Dooley <conor+dt@kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Rob Herring <robh@kernel.org>, Rosen Penev <rosenp@gmail.com>,
Thinh Nguyen <Thinh.Nguyen@synopsys.com>,
Vinod Koul <vkoul@kernel.org>,
devicetree@vger.kernel.org, kernel@dh-electronics.com,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org,
linux-stm32@st-md-mailman.stormreply.com
Subject: Re: [PATCH 1/8] dt-bindings: phy: Document ST STM32MP25 USB2-FEMTO PHY
Date: Wed, 12 Aug 2026 13:22:55 +0200 [thread overview]
Message-ID: <ff957fbb-b8ed-485d-adaa-18258ed693a6@kernel.org> (raw)
In-Reply-To: <ff691741-a110-44e8-90e7-ac54a96de1af@nabladev.com>
On 12/08/2026 12:56, Marek Vasut wrote:
> On 8/12/26 12:15 PM, Krzysztof Kozlowski wrote:
>> On Wed, Aug 12, 2026 at 02:55:41AM +0200, Marek Vasut wrote:
>>> From: Pankaj Dev <pankaj.dev@st.com>
>>>
>>> Document USB2 PHY for STM32MP25 USB2 controllers, which includes the
>>> USB2.0 host-only controller and USB 2.0 part of the DWC3 controller.
>>> Two such PHYs in total are present in STM32MP25 SoC, they both are
>>> slightly different, therefore they use different compatible string
>>
>> What is slightly different? This should be either here, or better, in
>> "description" part.
>
> They are both Synopsys FEMTO-PHY , but they are each attached to a
> different controller (one to USB 2.0 host , the other to DWC3 DRD) , and
> their control bits in the system controller register are different for
> each variant.
>
> I extended the description in V2.
Depending on the actual difference, this rather belongs to the phandle
to syscon, instead of separate compatible.
>
>>> to discern them.
>>>
>>> Signed-off-by: Clément Le Goffic <clement.legoffic@foss.st.com>
>>> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@foss.st.com>
>>> Signed-off-by: Marek Vasut <marex@nabladev.com>
>>> Signed-off-by: Pankaj Dev <pankaj.dev@st.com>
>>> Signed-off-by: Rahul Kumar <rahul.kumar05@st.com>
>>
>> Incomplete/incorrect DCO chain. Who sends it? Rahul?
> This is lifted from the ST downstream kernel fork, it's a combination of
> a bunch of commits there and then cleaned up. Pankaj and Fabrice did
> most of the downstream work, I did the clean up for upstreaming. I
> collected the SoB lines from all the commits from ST downstream fork and
> added mine, hence this list. How shall I address this correctly, with
> some Co-developed-by tag ? Who gets which tag ?
Everything explained in submitting patches. The SoBs represent the
chronological lifecyle of the patch. Person sending the patch is
obviously the last one touching. How Rahul could touch the code AFTER
you sent it?
Best regards,
Krzysztof
next prev parent reply other threads:[~2026-08-12 11:23 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-12 0:55 [PATCH 0/8] arm64: dts: phy: st: usb: Add STM32MP2 USB support Marek Vasut
2026-08-12 0:55 ` [PATCH 1/8] dt-bindings: phy: Document ST STM32MP25 USB2-FEMTO PHY Marek Vasut
2026-08-12 2:45 ` Rob Herring (Arm)
2026-08-12 10:15 ` Krzysztof Kozlowski
2026-08-12 10:56 ` Marek Vasut
2026-08-12 11:22 ` Krzysztof Kozlowski [this message]
2026-08-12 0:55 ` [PATCH 2/8] phy: stm32: Add support for " Marek Vasut
2026-08-12 0:55 ` [PATCH 3/8] dt-bindings: usb: generic-ehci: Document access-controllers property Marek Vasut
2026-08-12 0:55 ` [PATCH 4/8] dt-bindings: usb: generic-ohci: " Marek Vasut
2026-08-12 0:55 ` [PATCH 5/8] dt-bindings: usb: dwc3: Document ST STM32MP2 DWC3 xHCI USB controller Marek Vasut
2026-08-12 0:55 ` [PATCH 6/8] usb: dwc3: dwc3-generic-plat: Add ST STM32MP2 DWC3 xHCI USB controller glue Marek Vasut
2026-08-12 0:55 ` [PATCH 7/8] arm64: dts: st: Add USB nodes on stm32mp231 Marek Vasut
2026-08-12 0:55 ` [PATCH 8/8] arm64: dts: st: Add USB nodes on stm32mp251 Marek Vasut
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ff957fbb-b8ed-485d-adaa-18258ed693a6@kernel.org \
--to=krzk@kernel.org \
--cc=Thinh.Nguyen@synopsys.com \
--cc=alexandre.torgue@foss.st.com \
--cc=christian.bruel@foss.st.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=fabrice.gasnier@foss.st.com \
--cc=gregkh@linuxfoundation.org \
--cc=kernel@dh-electronics.com \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-phy@lists.infradead.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=linux-usb@vger.kernel.org \
--cc=marex@nabladev.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=neil.armstrong@linaro.org \
--cc=pankaj.dev@st.com \
--cc=rahul.kumar05@st.com \
--cc=robh@kernel.org \
--cc=rosenp@gmail.com \
--cc=vkoul@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox