Skip to content

Commit 8b0beb4

Browse files
krzkarndb
authored andcommitted
Documentation/process: maintainer-soc: Document purpose of defconfigs
Common mistake in commit messages of patches on mailing list adding CONFIG options to arm/multi_v7 or arm64/defconfig is saying what that patch is doing, e.g. "Enable driver foo". That is obvious from the diff part, thus explaining it does not bring any value. What brings value is to understand why "driver foo" should be in a shared, upstream defconfig, especially considering that distros have their own defconfigs and we do not care about non-upstream trees. Signed-off-by: Krzysztof Kozlowski <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Arnd Bergmann <[email protected]>
1 parent f325b23 commit 8b0beb4

1 file changed

Lines changed: 10 additions & 0 deletions

File tree

Documentation/process/maintainer-soc.rst

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -207,3 +207,13 @@ The subject line of a pull request should begin with "[GIT PULL]" and made using
207207
a signed tag, rather than a branch. This tag should contain a short description
208208
summarising the changes in the pull request. For more detail on sending pull
209209
requests, please see Documentation/maintainer/pull-requests.rst.
210+
211+
Defconfigs purpose
212+
~~~~~~~~~~~~~~~~~~
213+
214+
Defconfigs are primarily used by the kernel developers, because distros have
215+
their own configs. A change adding new CONFIG options to a defconfig should
216+
explain why the kernel developers in general would want such option, e.g. by
217+
providing a name of an upstream-supported machine/board using that new option.
218+
This implies that enabling options in defconfig for non-upstream machines shall
219+
not be accepted.

0 commit comments

Comments
 (0)