* [PATCH] dt-bindings: usb: Add Marvell ac5 @ 2024-01-07 15:25 Elad Nachman 2024-01-08 17:15 ` Conor Dooley 0 siblings, 1 reply; 7+ messages in thread From: Elad Nachman @ 2024-01-07 15:25 UTC (permalink / raw) To: gregkh, robh+dt, krzysztof.kozlowski+dt, conor+dt, devicetree, linux-usb, linux-kernel Cc: enachman From: Elad Nachman <enachman@marvell.com> Add Marvell ac5 device tree bindings to generic EHCI. This compatible enables the Marvell Orion platform code to properly configure the DMA mask for the Marvell AC5 SOC. Signed-off-by: Elad Nachman <enachman@marvell.com> --- Documentation/devicetree/bindings/usb/generic-ehci.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml b/Documentation/devicetree/bindings/usb/generic-ehci.yaml index 87986c45be88..2ed178f16a78 100644 --- a/Documentation/devicetree/bindings/usb/generic-ehci.yaml +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml @@ -77,6 +77,7 @@ properties: - const: usb-ehci - enum: - generic-ehci + - marvell,ac5-ehci - marvell,armada-3700-ehci - marvell,orion-ehci - nuvoton,npcm750-ehci -- 2.25.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] dt-bindings: usb: Add Marvell ac5 2024-01-07 15:25 [PATCH] dt-bindings: usb: Add Marvell ac5 Elad Nachman @ 2024-01-08 17:15 ` Conor Dooley 2024-01-08 17:47 ` [EXT] " Elad Nachman 0 siblings, 1 reply; 7+ messages in thread From: Conor Dooley @ 2024-01-08 17:15 UTC (permalink / raw) To: Elad Nachman Cc: gregkh, robh+dt, krzysztof.kozlowski+dt, conor+dt, devicetree, linux-usb, linux-kernel [-- Attachment #1: Type: text/plain, Size: 1258 bytes --] On Sun, Jan 07, 2024 at 05:25:57PM +0200, Elad Nachman wrote: > From: Elad Nachman <enachman@marvell.com> > > Add Marvell ac5 device tree bindings to generic EHCI. > This compatible enables the Marvell Orion platform code > to properly configure the DMA mask for the Marvell AC5 SOC. > > Signed-off-by: Elad Nachman <enachman@marvell.com> Maybe I am dumb, but I don't understand how this does what the commit message says. There's no fallback to generic-ehci or any other compatible that is already understood by the driver. What am I missing? > --- > Documentation/devicetree/bindings/usb/generic-ehci.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > index 87986c45be88..2ed178f16a78 100644 > --- a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > @@ -77,6 +77,7 @@ properties: > - const: usb-ehci > - enum: > - generic-ehci > + - marvell,ac5-ehci > - marvell,armada-3700-ehci > - marvell,orion-ehci > - nuvoton,npcm750-ehci > -- > 2.25.1 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 2024-01-08 17:15 ` Conor Dooley @ 2024-01-08 17:47 ` Elad Nachman 2024-01-08 18:01 ` Conor Dooley 0 siblings, 1 reply; 7+ messages in thread From: Elad Nachman @ 2024-01-08 17:47 UTC (permalink / raw) To: Conor Dooley Cc: gregkh@linuxfoundation.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, devicetree@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org > -----Original Message----- > From: Conor Dooley <conor@kernel.org> > Sent: Monday, January 8, 2024 7:15 PM > To: Elad Nachman <enachman@marvell.com> > Cc: gregkh@linuxfoundation.org; robh+dt@kernel.org; > krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; > devicetree@vger.kernel.org; linux-usb@vger.kernel.org; linux- > kernel@vger.kernel.org > Subject: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 > > External Email > > ---------------------------------------------------------------------- > On Sun, Jan 07, 2024 at 05:25:57PM +0200, Elad Nachman wrote: > > From: Elad Nachman <enachman@marvell.com> > > > > Add Marvell ac5 device tree bindings to generic EHCI. > > This compatible enables the Marvell Orion platform code to properly > > configure the DMA mask for the Marvell AC5 SOC. > > > > Signed-off-by: Elad Nachman <enachman@marvell.com> > > Maybe I am dumb, but I don't understand how this does what the commit > message says. There's no fallback to generic-ehci or any other compatible > that is already understood by the driver. > > What am I missing? Following guidance by Andrew Lunn on my mmc patchset, I have split the patchset Into a single patch for dt-bindings review and another for USB driver review. The USB driver patch utilizing this new dt bindings compatible string is here: https://patchwork.kernel.org/project/linux-usb/patch/20240107163307.3677347-1-enachman@marvell.com/ > > > --- > > Documentation/devicetree/bindings/usb/generic-ehci.yaml | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > index 87986c45be88..2ed178f16a78 100644 > > --- a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > @@ -77,6 +77,7 @@ properties: > > - const: usb-ehci > > - enum: > > - generic-ehci > > + - marvell,ac5-ehci > > - marvell,armada-3700-ehci > > - marvell,orion-ehci > > - nuvoton,npcm750-ehci > > -- > > 2.25.1 > > Elad. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 2024-01-08 17:47 ` [EXT] " Elad Nachman @ 2024-01-08 18:01 ` Conor Dooley 2024-01-08 18:15 ` Elad Nachman 0 siblings, 1 reply; 7+ messages in thread From: Conor Dooley @ 2024-01-08 18:01 UTC (permalink / raw) To: Elad Nachman Cc: gregkh@linuxfoundation.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, devicetree@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org [-- Attachment #1: Type: text/plain, Size: 2636 bytes --] On Mon, Jan 08, 2024 at 05:47:24PM +0000, Elad Nachman wrote: > > > > -----Original Message----- > > From: Conor Dooley <conor@kernel.org> > > Sent: Monday, January 8, 2024 7:15 PM > > To: Elad Nachman <enachman@marvell.com> > > Cc: gregkh@linuxfoundation.org; robh+dt@kernel.org; > > krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; > > devicetree@vger.kernel.org; linux-usb@vger.kernel.org; linux- > > kernel@vger.kernel.org > > Subject: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 > > > > External Email > > > > ---------------------------------------------------------------------- > > On Sun, Jan 07, 2024 at 05:25:57PM +0200, Elad Nachman wrote: > > > From: Elad Nachman <enachman@marvell.com> > > > > > > Add Marvell ac5 device tree bindings to generic EHCI. > > > This compatible enables the Marvell Orion platform code to properly > > > configure the DMA mask for the Marvell AC5 SOC. > > > > > > Signed-off-by: Elad Nachman <enachman@marvell.com> > > > > Maybe I am dumb, but I don't understand how this does what the commit > > message says. There's no fallback to generic-ehci or any other compatible > > that is already understood by the driver. > > > > What am I missing? > > Following guidance by Andrew Lunn on my mmc patchset, I have split the patchset > Into a single patch for dt-bindings review and another for USB driver review. I don't understand that advice. dt-bindings should (almost) always be with the driver. Otherwise you just confuse people like me. Perhaps Andrew meant sending dt*s* patches separately from the driver & bindings? Cheers, Conor. > > The USB driver patch utilizing this new dt bindings compatible string is here: > > https://patchwork.kernel.org/project/linux-usb/patch/20240107163307.3677347-1-enachman@marvell.com/ > > > > > > --- > > > Documentation/devicetree/bindings/usb/generic-ehci.yaml | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > index 87986c45be88..2ed178f16a78 100644 > > > --- a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > @@ -77,6 +77,7 @@ properties: > > > - const: usb-ehci > > > - enum: > > > - generic-ehci > > > + - marvell,ac5-ehci > > > - marvell,armada-3700-ehci > > > - marvell,orion-ehci > > > - nuvoton,npcm750-ehci > > > -- > > > 2.25.1 > > > > > Elad. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 2024-01-08 18:01 ` Conor Dooley @ 2024-01-08 18:15 ` Elad Nachman 2024-01-08 18:23 ` Conor Dooley 2024-01-09 9:32 ` Krzysztof Kozlowski 0 siblings, 2 replies; 7+ messages in thread From: Elad Nachman @ 2024-01-08 18:15 UTC (permalink / raw) To: Conor Dooley Cc: gregkh@linuxfoundation.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, devicetree@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org > -----Original Message----- > From: Conor Dooley <conor@kernel.org> > Sent: Monday, January 8, 2024 8:01 PM > To: Elad Nachman <enachman@marvell.com> > Cc: gregkh@linuxfoundation.org; robh+dt@kernel.org; > krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; > devicetree@vger.kernel.org; linux-usb@vger.kernel.org; linux- > kernel@vger.kernel.org > Subject: Re: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 > > On Mon, Jan 08, 2024 at 05:47:24PM +0000, Elad Nachman wrote: > > > > > > > -----Original Message----- > > > From: Conor Dooley <conor@kernel.org> > > > Sent: Monday, January 8, 2024 7:15 PM > > > To: Elad Nachman <enachman@marvell.com> > > > Cc: gregkh@linuxfoundation.org; robh+dt@kernel.org; > > > krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; > > > devicetree@vger.kernel.org; linux-usb@vger.kernel.org; linux- > > > kernel@vger.kernel.org > > > Subject: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 > > > > > > External Email > > > > > > -------------------------------------------------------------------- > > > -- On Sun, Jan 07, 2024 at 05:25:57PM +0200, Elad Nachman wrote: > > > > From: Elad Nachman <enachman@marvell.com> > > > > > > > > Add Marvell ac5 device tree bindings to generic EHCI. > > > > This compatible enables the Marvell Orion platform code to > > > > properly configure the DMA mask for the Marvell AC5 SOC. > > > > > > > > Signed-off-by: Elad Nachman <enachman@marvell.com> > > > > > > Maybe I am dumb, but I don't understand how this does what the > > > commit message says. There's no fallback to generic-ehci or any > > > other compatible that is already understood by the driver. > > > > > > What am I missing? > > > > Following guidance by Andrew Lunn on my mmc patchset, I have split the > > patchset Into a single patch for dt-bindings review and another for USB > driver review. > > I don't understand that advice. dt-bindings should (almost) always be with > the driver. Otherwise you just confuse people like me. > > Perhaps Andrew meant sending dt*s* patches separately from the driver & > bindings? The rational was different people making the actual merge, into different intermediate git REPOs, before mering into linux-next. > > Cheers, > Conor. > > > > > The USB driver patch utilizing this new dt bindings compatible string is here: > > > > https://patchwork.kernel.org/project/linux-usb/patch/20240107163307.36 > > 77347-1-enachman@marvell.com/ > > > > > > > > > --- > > > > Documentation/devicetree/bindings/usb/generic-ehci.yaml | 1 + > > > > 1 file changed, 1 insertion(+) > > > > > > > > diff --git > > > > a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > > b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > > index 87986c45be88..2ed178f16a78 100644 > > > > --- a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > > +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > > @@ -77,6 +77,7 @@ properties: > > > > - const: usb-ehci > > > > - enum: > > > > - generic-ehci > > > > + - marvell,ac5-ehci > > > > - marvell,armada-3700-ehci > > > > - marvell,orion-ehci > > > > - nuvoton,npcm750-ehci > > > > -- > > > > 2.25.1 > > > > > > > > Elad. Elad. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 2024-01-08 18:15 ` Elad Nachman @ 2024-01-08 18:23 ` Conor Dooley 2024-01-09 9:32 ` Krzysztof Kozlowski 1 sibling, 0 replies; 7+ messages in thread From: Conor Dooley @ 2024-01-08 18:23 UTC (permalink / raw) To: Elad Nachman Cc: gregkh@linuxfoundation.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, devicetree@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org [-- Attachment #1: Type: text/plain, Size: 2836 bytes --] On Mon, Jan 08, 2024 at 06:15:51PM +0000, Elad Nachman wrote: > > -----Original Message----- > > From: Conor Dooley <conor@kernel.org> > > On Mon, Jan 08, 2024 at 05:47:24PM +0000, Elad Nachman wrote: > > > > -----Original Message----- > > > > From: Conor Dooley <conor@kernel.org> > > > > -- On Sun, Jan 07, 2024 at 05:25:57PM +0200, Elad Nachman wrote: > > > > > From: Elad Nachman <enachman@marvell.com> > > > > > > > > > > Add Marvell ac5 device tree bindings to generic EHCI. > > > > > This compatible enables the Marvell Orion platform code to > > > > > properly configure the DMA mask for the Marvell AC5 SOC. > > > > > > > > > > Signed-off-by: Elad Nachman <enachman@marvell.com> > > > > > > > > Maybe I am dumb, but I don't understand how this does what the > > > > commit message says. There's no fallback to generic-ehci or any > > > > other compatible that is already understood by the driver. > > > > > > > > What am I missing? > > > > > > Following guidance by Andrew Lunn on my mmc patchset, I have split the > > > patchset Into a single patch for dt-bindings review and another for USB > > driver review. > > > > I don't understand that advice. dt-bindings should (almost) always be with > > the driver. Otherwise you just confuse people like me. > > > > Perhaps Andrew meant sending dt*s* patches separately from the driver & > > bindings? > > The rational was different people making the actual merge, into different > intermediate git REPOs, before mering into linux-next. Which almost certainly means he was talking about dts patches, not dt-bindings. bindings (almost) always go through the same tree as the driver that uses them. Cheers, Conor. > > > The USB driver patch utilizing this new dt bindings compatible string is here: > > > > > > https://patchwork.kernel.org/project/linux-usb/patch/20240107163307.36 > > > 77347-1-enachman@marvell.com/ > > > > > > > > > > > > --- > > > > > Documentation/devicetree/bindings/usb/generic-ehci.yaml | 1 + > > > > > 1 file changed, 1 insertion(+) > > > > > > > > > > diff --git > > > > > a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > > > b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > > > index 87986c45be88..2ed178f16a78 100644 > > > > > --- a/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > > > +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml > > > > > @@ -77,6 +77,7 @@ properties: > > > > > - const: usb-ehci > > > > > - enum: > > > > > - generic-ehci > > > > > + - marvell,ac5-ehci > > > > > - marvell,armada-3700-ehci > > > > > - marvell,orion-ehci > > > > > - nuvoton,npcm750-ehci > > > > > -- > > > > > 2.25.1 > > > > > > > > > > > Elad. > > Elad. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 2024-01-08 18:15 ` Elad Nachman 2024-01-08 18:23 ` Conor Dooley @ 2024-01-09 9:32 ` Krzysztof Kozlowski 1 sibling, 0 replies; 7+ messages in thread From: Krzysztof Kozlowski @ 2024-01-09 9:32 UTC (permalink / raw) To: Elad Nachman, Conor Dooley Cc: gregkh@linuxfoundation.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, devicetree@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org On 08/01/2024 19:15, Elad Nachman wrote: > > >> -----Original Message----- >> From: Conor Dooley <conor@kernel.org> >> Sent: Monday, January 8, 2024 8:01 PM >> To: Elad Nachman <enachman@marvell.com> >> Cc: gregkh@linuxfoundation.org; robh+dt@kernel.org; >> krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; >> devicetree@vger.kernel.org; linux-usb@vger.kernel.org; linux- >> kernel@vger.kernel.org >> Subject: Re: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 >> >> On Mon, Jan 08, 2024 at 05:47:24PM +0000, Elad Nachman wrote: >>> >>> >>>> -----Original Message----- >>>> From: Conor Dooley <conor@kernel.org> >>>> Sent: Monday, January 8, 2024 7:15 PM >>>> To: Elad Nachman <enachman@marvell.com> >>>> Cc: gregkh@linuxfoundation.org; robh+dt@kernel.org; >>>> krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; >>>> devicetree@vger.kernel.org; linux-usb@vger.kernel.org; linux- >>>> kernel@vger.kernel.org >>>> Subject: [EXT] Re: [PATCH] dt-bindings: usb: Add Marvell ac5 >>>> >>>> External Email >>>> >>>> -------------------------------------------------------------------- >>>> -- On Sun, Jan 07, 2024 at 05:25:57PM +0200, Elad Nachman wrote: >>>>> From: Elad Nachman <enachman@marvell.com> >>>>> >>>>> Add Marvell ac5 device tree bindings to generic EHCI. >>>>> This compatible enables the Marvell Orion platform code to >>>>> properly configure the DMA mask for the Marvell AC5 SOC. >>>>> >>>>> Signed-off-by: Elad Nachman <enachman@marvell.com> >>>> >>>> Maybe I am dumb, but I don't understand how this does what the >>>> commit message says. There's no fallback to generic-ehci or any >>>> other compatible that is already understood by the driver. >>>> >>>> What am I missing? >>> >>> Following guidance by Andrew Lunn on my mmc patchset, I have split the >>> patchset Into a single patch for dt-bindings review and another for USB >> driver review. >> >> I don't understand that advice. dt-bindings should (almost) always be with >> the driver. Otherwise you just confuse people like me. >> >> Perhaps Andrew meant sending dt*s* patches separately from the driver & >> bindings? > > The rational was different people making the actual merge, into different intermediate git REPOs, before mering into linux-next. Sorry, but NAK. The submitting-patches for bindings expresses this as well: bindings go with the driver. There was never an advice from Andrew about splitting binding. He clearly said that FIRST PATCH, so the DTS, goes via separate tree. Absolutely nothing about splitting the binding. https://lore.kernel.org/all/cfbbe706-5953-488c-9bff-f72f6d00b44f@lunn.ch/ Best regards, Krzysztof ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-01-09 9:32 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-01-07 15:25 [PATCH] dt-bindings: usb: Add Marvell ac5 Elad Nachman 2024-01-08 17:15 ` Conor Dooley 2024-01-08 17:47 ` [EXT] " Elad Nachman 2024-01-08 18:01 ` Conor Dooley 2024-01-08 18:15 ` Elad Nachman 2024-01-08 18:23 ` Conor Dooley 2024-01-09 9:32 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).