* [PATCH] dt-bindings: marvell: move MMP boards to common marvell directory
@ 2023-12-01 13:23 Krzysztof Kozlowski
2023-12-01 13:51 ` Andrew Lunn
0 siblings, 1 reply; 7+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-01 13:23 UTC (permalink / raw)
To: Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Lubomir Rintel,
linux-arm-kernel, devicetree, linux-kernel
Cc: Krzysztof Kozlowski
Marvell board bindings are spread over arm/marvell/ and arm/mrvl/
directories. Move MMP board bindings from the latter to the former, to
keep all of them together.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../bindings/arm/{mrvl/mrvl.yaml => marvell/marvell,mmp.yaml} | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
rename Documentation/devicetree/bindings/arm/{mrvl/mrvl.yaml => marvell/marvell,mmp.yaml} (89%)
diff --git a/Documentation/devicetree/bindings/arm/mrvl/mrvl.yaml b/Documentation/devicetree/bindings/arm/marvell/marvell,mmp.yaml
similarity index 89%
rename from Documentation/devicetree/bindings/arm/mrvl/mrvl.yaml
rename to Documentation/devicetree/bindings/arm/marvell/marvell,mmp.yaml
index 4c43eaf3632e..92a667e27ec8 100644
--- a/Documentation/devicetree/bindings/arm/mrvl/mrvl.yaml
+++ b/Documentation/devicetree/bindings/arm/marvell/marvell,mmp.yaml
@@ -1,10 +1,10 @@
# SPDX-License-Identifier: GPL-2.0
%YAML 1.2
---
-$id: http://devicetree.org/schemas/arm/mrvl/mrvl.yaml#
+$id: http://devicetree.org/schemas/arm/marvell/marvell,mmp.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#
-title: Marvell Platforms
+title: Marvell MMP Platforms
maintainers:
- Lubomir Rintel <lkundrak@v3.sk>
--
2.34.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] dt-bindings: marvell: move MMP boards to common marvell directory
2023-12-01 13:23 [PATCH] dt-bindings: marvell: move MMP boards to common marvell directory Krzysztof Kozlowski
@ 2023-12-01 13:51 ` Andrew Lunn
2023-12-01 14:21 ` Krzysztof Kozlowski
0 siblings, 1 reply; 7+ messages in thread
From: Andrew Lunn @ 2023-12-01 13:51 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Gregory Clement, Sebastian Hesselbarth, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Lubomir Rintel,
linux-arm-kernel, devicetree, linux-kernel
On Fri, Dec 01, 2023 at 02:23:06PM +0100, Krzysztof Kozlowski wrote:
> Marvell board bindings are spread over arm/marvell/ and arm/mrvl/
> directories. Move MMP board bindings from the latter to the former, to
> keep all of them together.
Hi Krzysztof
Did you test get_maintainers.pl? MMP has a different maintainer to
many of the other Marvell SoCs. We want emails going to the correct
Maintainers, and ideally not spamming the others.
Andrew
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] dt-bindings: marvell: move MMP boards to common marvell directory
2023-12-01 13:51 ` Andrew Lunn
@ 2023-12-01 14:21 ` Krzysztof Kozlowski
2023-12-01 14:34 ` Andrew Lunn
0 siblings, 1 reply; 7+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-01 14:21 UTC (permalink / raw)
To: Andrew Lunn
Cc: Gregory Clement, Sebastian Hesselbarth, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Lubomir Rintel,
linux-arm-kernel, devicetree, linux-kernel
On 01/12/2023 14:51, Andrew Lunn wrote:
> On Fri, Dec 01, 2023 at 02:23:06PM +0100, Krzysztof Kozlowski wrote:
>> Marvell board bindings are spread over arm/marvell/ and arm/mrvl/
>> directories. Move MMP board bindings from the latter to the former, to
>> keep all of them together.
>
> Hi Krzysztof
>
> Did you test get_maintainers.pl? MMP has a different maintainer to
> many of the other Marvell SoCs. We want emails going to the correct
> Maintainers, and ideally not spamming the others.
The old binding was not referenced in MAINTAINERS, at least I could not
find it.
My change does not affect status quo - orphaned files.
OTOH, some entries like Orion list specific files. Others like Marvell
list entire directory, which is their mistake in the first place.
There is a mess in this approach, but the mess exists before my patch.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] dt-bindings: marvell: move MMP boards to common marvell directory
2023-12-01 14:21 ` Krzysztof Kozlowski
@ 2023-12-01 14:34 ` Andrew Lunn
2023-12-01 14:38 ` Krzysztof Kozlowski
0 siblings, 1 reply; 7+ messages in thread
From: Andrew Lunn @ 2023-12-01 14:34 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Gregory Clement, Sebastian Hesselbarth, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Lubomir Rintel,
linux-arm-kernel, devicetree, linux-kernel
On Fri, Dec 01, 2023 at 03:21:05PM +0100, Krzysztof Kozlowski wrote:
> On 01/12/2023 14:51, Andrew Lunn wrote:
> > On Fri, Dec 01, 2023 at 02:23:06PM +0100, Krzysztof Kozlowski wrote:
> >> Marvell board bindings are spread over arm/marvell/ and arm/mrvl/
> >> directories. Move MMP board bindings from the latter to the former, to
> >> keep all of them together.
> >
> > Hi Krzysztof
> >
> > Did you test get_maintainers.pl? MMP has a different maintainer to
> > many of the other Marvell SoCs. We want emails going to the correct
> > Maintainers, and ideally not spamming the others.
>
> The old binding was not referenced in MAINTAINERS, at least I could not
> find it.
> My change does not affect status quo - orphaned files.
>
> OTOH, some entries like Orion list specific files. Others like Marvell
> list entire directory, which is their mistake in the first place.
>
> There is a mess in this approach, but the mess exists before my patch.
I think these moved files will now match:
ARM/Marvell Kirkwood and Armada 370, 375, 38x, 39x, XP, 3700, 7K/8K, CN9130 SOC support
M: Andrew Lunn <andrew@lunn.ch>
M: Gregory Clement <gregory.clement@bootlin.com>
M: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
S: Maintained
T: git git://git.kernel.org/pub/scm/linux/kernel/git/gclement/mvebu.git
F: Documentation/devicetree/bindings/arm/marvell/
But these files are not Gregory or my problem.
If they were orphaned before, i would prefer they are either orphaned
after the move, or associated to the correct maintainer. Being
associated to the wrong maintainers is worse than having no
maintainers at all.
Andrew
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] dt-bindings: marvell: move MMP boards to common marvell directory
2023-12-01 14:34 ` Andrew Lunn
@ 2023-12-01 14:38 ` Krzysztof Kozlowski
2023-12-01 16:17 ` Andrew Lunn
0 siblings, 1 reply; 7+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-01 14:38 UTC (permalink / raw)
To: Andrew Lunn
Cc: Gregory Clement, Sebastian Hesselbarth, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Lubomir Rintel,
linux-arm-kernel, devicetree, linux-kernel
On 01/12/2023 15:34, Andrew Lunn wrote:
> On Fri, Dec 01, 2023 at 03:21:05PM +0100, Krzysztof Kozlowski wrote:
>> On 01/12/2023 14:51, Andrew Lunn wrote:
>>> On Fri, Dec 01, 2023 at 02:23:06PM +0100, Krzysztof Kozlowski wrote:
>>>> Marvell board bindings are spread over arm/marvell/ and arm/mrvl/
>>>> directories. Move MMP board bindings from the latter to the former, to
>>>> keep all of them together.
>>>
>>> Hi Krzysztof
>>>
>>> Did you test get_maintainers.pl? MMP has a different maintainer to
>>> many of the other Marvell SoCs. We want emails going to the correct
>>> Maintainers, and ideally not spamming the others.
>>
>> The old binding was not referenced in MAINTAINERS, at least I could not
>> find it.
>> My change does not affect status quo - orphaned files.
>>
>> OTOH, some entries like Orion list specific files. Others like Marvell
>> list entire directory, which is their mistake in the first place.
>>
>> There is a mess in this approach, but the mess exists before my patch.
>
> I think these moved files will now match:
>
> ARM/Marvell Kirkwood and Armada 370, 375, 38x, 39x, XP, 3700, 7K/8K, CN9130 SOC support
> M: Andrew Lunn <andrew@lunn.ch>
> M: Gregory Clement <gregory.clement@bootlin.com>
> M: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> S: Maintained
> T: git git://git.kernel.org/pub/scm/linux/kernel/git/gclement/mvebu.git
> F: Documentation/devicetree/bindings/arm/marvell/
>
> But these files are not Gregory or my problem.
Yeah, that's what I meant by listing entire directory.
>
> If they were orphaned before, i would prefer they are either orphaned
> after the move, or associated to the correct maintainer. Being
> associated to the wrong maintainers is worse than having no
> maintainers at all.
I can add this to existing maintainers entry. Which one? I see only two
ARM/Marvell.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] dt-bindings: marvell: move MMP boards to common marvell directory
2023-12-01 14:38 ` Krzysztof Kozlowski
@ 2023-12-01 16:17 ` Andrew Lunn
2023-12-07 13:57 ` Krzysztof Kozlowski
0 siblings, 1 reply; 7+ messages in thread
From: Andrew Lunn @ 2023-12-01 16:17 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Gregory Clement, Sebastian Hesselbarth, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Lubomir Rintel,
linux-arm-kernel, devicetree, linux-kernel
On Fri, Dec 01, 2023 at 03:38:36PM +0100, Krzysztof Kozlowski wrote:
> On 01/12/2023 15:34, Andrew Lunn wrote:
> > On Fri, Dec 01, 2023 at 03:21:05PM +0100, Krzysztof Kozlowski wrote:
> >> On 01/12/2023 14:51, Andrew Lunn wrote:
> >>> On Fri, Dec 01, 2023 at 02:23:06PM +0100, Krzysztof Kozlowski wrote:
> >>>> Marvell board bindings are spread over arm/marvell/ and arm/mrvl/
> >>>> directories. Move MMP board bindings from the latter to the former, to
> >>>> keep all of them together.
> >>>
> >>> Hi Krzysztof
> >>>
> >>> Did you test get_maintainers.pl? MMP has a different maintainer to
> >>> many of the other Marvell SoCs. We want emails going to the correct
> >>> Maintainers, and ideally not spamming the others.
> >>
> >> The old binding was not referenced in MAINTAINERS, at least I could not
> >> find it.
> >> My change does not affect status quo - orphaned files.
> >>
> >> OTOH, some entries like Orion list specific files. Others like Marvell
> >> list entire directory, which is their mistake in the first place.
> >>
> >> There is a mess in this approach, but the mess exists before my patch.
> >
> > I think these moved files will now match:
> >
> > ARM/Marvell Kirkwood and Armada 370, 375, 38x, 39x, XP, 3700, 7K/8K, CN9130 SOC support
> > M: Andrew Lunn <andrew@lunn.ch>
> > M: Gregory Clement <gregory.clement@bootlin.com>
> > M: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> > L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> > S: Maintained
> > T: git git://git.kernel.org/pub/scm/linux/kernel/git/gclement/mvebu.git
> > F: Documentation/devicetree/bindings/arm/marvell/
> >
> > But these files are not Gregory or my problem.
>
> Yeah, that's what I meant by listing entire directory.
>
> >
> > If they were orphaned before, i would prefer they are either orphaned
> > after the move, or associated to the correct maintainer. Being
> > associated to the wrong maintainers is worse than having no
> > maintainers at all.
>
> I can add this to existing maintainers entry. Which one? I see only two
> ARM/Marvell.
MMP SUPPORT
R: Lubomir Rintel <lkundrak@v3.sk>
L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
S: Odd Fixes
T: git git://git.kernel.org/pub/scm/linux/kernel/git/lkundrak/linux-mmp.git
F: arch/arm/boot/dts/marvell/mmp*
F: arch/arm/mach-mmp/
F: include/linux/soc/mmp/
But you also need to modify the ARM/Marvell Kirkwood and ... so that
it excludes these MMP files.
Andrew
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] dt-bindings: marvell: move MMP boards to common marvell directory
2023-12-01 16:17 ` Andrew Lunn
@ 2023-12-07 13:57 ` Krzysztof Kozlowski
0 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-07 13:57 UTC (permalink / raw)
To: Andrew Lunn
Cc: Gregory Clement, Sebastian Hesselbarth, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Lubomir Rintel,
linux-arm-kernel, devicetree, linux-kernel
On 01/12/2023 17:17, Andrew Lunn wrote:
> On Fri, Dec 01, 2023 at 03:38:36PM +0100, Krzysztof Kozlowski wrote:
>> On 01/12/2023 15:34, Andrew Lunn wrote:
>>> On Fri, Dec 01, 2023 at 03:21:05PM +0100, Krzysztof Kozlowski wrote:
>>>> On 01/12/2023 14:51, Andrew Lunn wrote:
>>>>> On Fri, Dec 01, 2023 at 02:23:06PM +0100, Krzysztof Kozlowski wrote:
>>>>>> Marvell board bindings are spread over arm/marvell/ and arm/mrvl/
>>>>>> directories. Move MMP board bindings from the latter to the former, to
>>>>>> keep all of them together.
>>>>>
>>>>> Hi Krzysztof
>>>>>
>>>>> Did you test get_maintainers.pl? MMP has a different maintainer to
>>>>> many of the other Marvell SoCs. We want emails going to the correct
>>>>> Maintainers, and ideally not spamming the others.
>>>>
>>>> The old binding was not referenced in MAINTAINERS, at least I could not
>>>> find it.
>>>> My change does not affect status quo - orphaned files.
>>>>
>>>> OTOH, some entries like Orion list specific files. Others like Marvell
>>>> list entire directory, which is their mistake in the first place.
>>>>
>>>> There is a mess in this approach, but the mess exists before my patch.
>>>
>>> I think these moved files will now match:
>>>
>>> ARM/Marvell Kirkwood and Armada 370, 375, 38x, 39x, XP, 3700, 7K/8K, CN9130 SOC support
>>> M: Andrew Lunn <andrew@lunn.ch>
>>> M: Gregory Clement <gregory.clement@bootlin.com>
>>> M: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
>>> L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>>> S: Maintained
>>> T: git git://git.kernel.org/pub/scm/linux/kernel/git/gclement/mvebu.git
>>> F: Documentation/devicetree/bindings/arm/marvell/
>>>
>>> But these files are not Gregory or my problem.
>>
>> Yeah, that's what I meant by listing entire directory.
>>
>>>
>>> If they were orphaned before, i would prefer they are either orphaned
>>> after the move, or associated to the correct maintainer. Being
>>> associated to the wrong maintainers is worse than having no
>>> maintainers at all.
>>
>> I can add this to existing maintainers entry. Which one? I see only two
>> ARM/Marvell.
>
> MMP SUPPORT
> R: Lubomir Rintel <lkundrak@v3.sk>
> L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> S: Odd Fixes
> T: git git://git.kernel.org/pub/scm/linux/kernel/git/lkundrak/linux-mmp.git
> F: arch/arm/boot/dts/marvell/mmp*
> F: arch/arm/mach-mmp/
> F: include/linux/soc/mmp/
>
> But you also need to modify the ARM/Marvell Kirkwood and ... so that
> it excludes these MMP files.
Let's make it then in subdirectory? There are like 20 items from
Marvell/Kirkwood, so I imagine all apply to it. At least currently they
are applying.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-12-07 13:58 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-01 13:23 [PATCH] dt-bindings: marvell: move MMP boards to common marvell directory Krzysztof Kozlowski
2023-12-01 13:51 ` Andrew Lunn
2023-12-01 14:21 ` Krzysztof Kozlowski
2023-12-01 14:34 ` Andrew Lunn
2023-12-01 14:38 ` Krzysztof Kozlowski
2023-12-01 16:17 ` Andrew Lunn
2023-12-07 13:57 ` 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).