Mention the SoC maintainer profile in P: entry.
Signed-off-by: Krzysztof Kozlowski <[email protected]>
Reviewed-by: Randy Dunlap <[email protected]>
---
Changes in v3:
1. None
Changes in v2:
1. Add Rb tag.
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 3dfa40abc885..7624eb231b65 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1570,6 +1570,7 @@ M: Olof Johansson <[email protected]>
M: [email protected]
L: [email protected] (moderated for non-subscribers)
S: Maintained
+P: Documentation/process/maintainer-soc.rst
C: irc://irc.libera.chat/armlinux
T: git git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git
F: Documentation/process/maintainer-soc.rst
--
2.34.1
Samsung ARM/ARM64 SoCs (including legacy S3C64xx and S5PV210) are also
expected not to bring any new dtbs_check warnings. In fact this have
been already enforced and tested since few release.
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
Changes in v3:
1. None
Changes in v2:
1. None
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 9fe3870300f2..4b299e39111d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2646,6 +2646,7 @@ R: Alim Akhtar <[email protected]>
L: [email protected] (moderated for non-subscribers)
L: [email protected]
S: Maintained
+P: Documentation/process/maintainer-soc-clean-dts.rst
Q: https://patchwork.kernel.org/project/linux-samsung-soc/list/
B: mailto:[email protected]
C: irc://irc.libera.chat/linux-exynos
--
2.34.1
Some SoC platforms require that commits must not bring any new
dtbs_check warnings. Maintainers of such platforms usually have some
automation set, so any new warning will be spotted sooner or later.
Worst case: they run the tests themselves. Document requirements for
such platforms, so contributors can expect their patches being dropped
or ignored, if they bring new warnings for existing boards.
Signed-off-by: Krzysztof Kozlowski <[email protected]>
Reviewed-by: Conor Dooley <[email protected]>
---
Changes in v3:
1. Rephrase according to Rob's comments (this document when referenced
in maintainers, "... and dtc Compliance", existing warnings are not
new warnings).
Changes in v2:
1. Add Rb tag.
2. Implement Conor's feedback: change doc title, follow->should follow,
minor style changes.
---
.../process/maintainer-handbooks.rst | 1 +
.../process/maintainer-soc-clean-dts.rst | 25 +++++++++++++++++++
MAINTAINERS | 2 +-
3 files changed, 27 insertions(+), 1 deletion(-)
create mode 100644 Documentation/process/maintainer-soc-clean-dts.rst
diff --git a/Documentation/process/maintainer-handbooks.rst b/Documentation/process/maintainer-handbooks.rst
index 9992bfd7eaa3..976391cec528 100644
--- a/Documentation/process/maintainer-handbooks.rst
+++ b/Documentation/process/maintainer-handbooks.rst
@@ -17,5 +17,6 @@ Contents:
maintainer-netdev
maintainer-soc
+ maintainer-soc-clean-dts
maintainer-tip
maintainer-kvm-x86
diff --git a/Documentation/process/maintainer-soc-clean-dts.rst b/Documentation/process/maintainer-soc-clean-dts.rst
new file mode 100644
index 000000000000..1b32430d0cfc
--- /dev/null
+++ b/Documentation/process/maintainer-soc-clean-dts.rst
@@ -0,0 +1,25 @@
+.. SPDX-License-Identifier: GPL-2.0
+
+==============================================
+SoC Platforms with DTS Compliance Requirements
+==============================================
+
+Overview
+--------
+
+SoC platforms or subarchitectures should follow all the rules from
+Documentation/process/maintainer-soc.rst. This document referenced in
+MAINTAINERS impose additional requirements listed below.
+
+Strict DTS DT Schema and dtc Compliance
+---------------------------------------
+
+No changes to the SoC platform Devicetree sources (DTS files) should introduce
+new ``make dtbs_check W=1`` warnings. Warnings in a new board DTS, which are
+results of issues in an included DTSI file, are considered existing, not new
+warnings. The platform maintainers have automation in place which should point
+out any new warnings.
+
+If a commit introducing new warnings gets accepted somehow, the resulting
+issues shall be fixed in reasonable time (e.g. within one release) or the
+commit reverted.
diff --git a/MAINTAINERS b/MAINTAINERS
index 7624eb231b65..9fe3870300f2 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1573,7 +1573,7 @@ S: Maintained
P: Documentation/process/maintainer-soc.rst
C: irc://irc.libera.chat/armlinux
T: git git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git
-F: Documentation/process/maintainer-soc.rst
+F: Documentation/process/maintainer-soc*.rst
F: arch/arm/boot/dts/Makefile
F: arch/arm64/boot/dts/Makefile
--
2.34.1
> -----Original Message-----
> From: Krzysztof Kozlowski <[email protected]>
> Sent: Sunday, July 23, 2023 6:49 PM
> To: Rob Herring <[email protected]>; Krzysztof Kozlowski
> <[email protected]>; Conor Dooley <[email protected]>;
> Alim Akhtar <[email protected]>; Andy Gross <[email protected]>;
> Bjorn Andersson <[email protected]>; Konrad Dybcio
> <[email protected]>; [email protected]; linux-arm-
> [email protected]; [email protected]; linux-
> [email protected]; [email protected]; Jonathan Corbet
> <[email protected]>; Arnd Bergmann <[email protected]>; Olof Johansson
> <[email protected]>; [email protected]; [email protected]; linux-
> [email protected]; [email protected]
> Cc: Krzysztof Kozlowski <[email protected]>
> Subject: [PATCH v3 3/3] MAINTAINER: samsung: document dtbs_check
> requirement for Samsung
>
> Samsung ARM/ARM64 SoCs (including legacy S3C64xx and S5PV210) are also
> expected not to bring any new dtbs_check warnings. In fact this have been
> already enforced and tested since few release.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
Reviewed-by: Alim Akhtar <[email protected]>
> ---
>
> Changes in v3:
> 1. None
>
> Changes in v2:
> 1. None
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 9fe3870300f2..4b299e39111d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2646,6 +2646,7 @@ R: Alim Akhtar <[email protected]>
> L: [email protected] (moderated for non-subscribers)
> L: [email protected]
> S: Maintained
> +P: Documentation/process/maintainer-soc-clean-dts.rst
> Q: https://patchwork.kernel.org/project/linux-samsung-soc/list/
> B: mailto:[email protected]
> C: irc://irc.libera.chat/linux-exynos
> --
> 2.34.1
Krzysztof Kozlowski <[email protected]> writes:
> Mention the SoC maintainer profile in P: entry.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> Reviewed-by: Randy Dunlap <[email protected]>
I've applied this series, thanks.
jon