Subject: [PATCH] drm: remove redundant 'default n' from Kconfig

'default n' is the default value for any bool or tristate Kconfig
setting so there is no need to write it explicitly.

Also since commit f467c5640c29 ("kconfig: only write '# CONFIG_FOO
is not set' for visible symbols") the Kconfig behavior is the same
regardless of 'default n' being present or not:

...
One side effect of (and the main motivation for) this change is making
the following two definitions behave exactly the same:

config FOO
bool

config FOO
bool
default n

With this change, neither of these will generate a
'# CONFIG_FOO is not set' line (assuming FOO isn't selected/implied).
That might make it clearer to people that a bare 'default n' is
redundant.
...

Signed-off-by: Bartlomiej Zolnierkiewicz <[email protected]>
---
drivers/gpu/drm/Kconfig | 5 -----
drivers/gpu/drm/amd/amdgpu/Kconfig | 1 -
drivers/gpu/drm/arm/Kconfig | 1 -
drivers/gpu/drm/exynos/Kconfig | 2 --
drivers/gpu/drm/i915/Kconfig | 3 ---
drivers/gpu/drm/i915/Kconfig.debug | 13 -------------
drivers/gpu/drm/msm/Kconfig | 2 --
drivers/gpu/drm/nouveau/Kconfig | 2 --
drivers/gpu/drm/omapdrm/Kconfig | 1 -
drivers/gpu/drm/omapdrm/dss/Kconfig | 6 ------
10 files changed, 36 deletions(-)

Index: b/drivers/gpu/drm/Kconfig
===================================================================
--- a/drivers/gpu/drm/Kconfig 2019-04-12 11:42:30.070095359 +0200
+++ b/drivers/gpu/drm/Kconfig 2019-04-12 11:42:30.066095359 +0200
@@ -37,7 +37,6 @@ config DRM_DP_AUX_CHARDEV

config DRM_DEBUG_MM
bool "Insert extra checks and debug info into the DRM range managers"
- default n
depends on DRM=y
depends on STACKTRACE_SUPPORT
select STACKDEPOT
@@ -56,7 +55,6 @@ config DRM_DEBUG_SELFTEST
select PRIME_NUMBERS
select DRM_LIB_RANDOM
select DRM_KMS_HELPER
- default n
help
This option provides kernel modules that can be used to run
various selftests on parts of the DRM api. This option is not
@@ -113,7 +111,6 @@ config DRM_FBDEV_OVERALLOC
config DRM_FBDEV_LEAK_PHYS_SMEM
bool "Shamelessly allow leaking of fbdev physical address (DANGEROUS)"
depends on DRM_FBDEV_EMULATION && EXPERT
- default n
help
In order to keep user-space compatibility, we want in certain
use-cases to keep leaking the fbdev physical address to the
@@ -247,7 +244,6 @@ config DRM_VKMS
tristate "Virtual KMS (EXPERIMENTAL)"
depends on DRM
select DRM_KMS_HELPER
- default n
help
Virtual Kernel Mode-Setting (VKMS) is used for testing or for
running GPU in a headless machines. Choose this option to get
@@ -424,4 +420,3 @@ config DRM_PANEL_ORIENTATION_QUIRKS

config DRM_LIB_RANDOM
bool
- default n
Index: b/drivers/gpu/drm/amd/amdgpu/Kconfig
===================================================================
--- a/drivers/gpu/drm/amd/amdgpu/Kconfig 2019-04-12 11:42:30.070095359 +0200
+++ b/drivers/gpu/drm/amd/amdgpu/Kconfig 2019-04-12 11:42:30.066095359 +0200
@@ -35,7 +35,6 @@ config DRM_AMDGPU_GART_DEBUGFS
bool "Allow GART access through debugfs"
depends on DRM_AMDGPU
depends on DEBUG_FS
- default n
help
Selecting this option creates a debugfs file to inspect the mapped
pages. Uses more memory for housekeeping, enable only for debugging.
Index: b/drivers/gpu/drm/arm/Kconfig
===================================================================
--- a/drivers/gpu/drm/arm/Kconfig 2019-04-12 11:42:30.070095359 +0200
+++ b/drivers/gpu/drm/arm/Kconfig 2019-04-12 11:42:30.066095359 +0200
@@ -16,7 +16,6 @@ config DRM_HDLCD
config DRM_HDLCD_SHOW_UNDERRUN
bool "Show underrun conditions"
depends on DRM_HDLCD
- default n
help
Enable this option to show in red colour the pixels that the
HDLCD device did not fetch from framebuffer due to underrun
Index: b/drivers/gpu/drm/exynos/Kconfig
===================================================================
--- a/drivers/gpu/drm/exynos/Kconfig 2019-04-12 11:42:30.070095359 +0200
+++ b/drivers/gpu/drm/exynos/Kconfig 2019-04-12 11:42:30.066095359 +0200
@@ -46,7 +46,6 @@ config DRM_EXYNOS_DPI
bool "Parallel output"
depends on DRM_EXYNOS_FIMD
select DRM_PANEL
- default n
help
This enables support for Exynos parallel output.

@@ -55,7 +54,6 @@ config DRM_EXYNOS_DSI
depends on DRM_EXYNOS_FIMD || DRM_EXYNOS5433_DECON || DRM_EXYNOS7_DECON
select DRM_MIPI_DSI
select DRM_PANEL
- default n
help
This enables support for Exynos MIPI-DSI device.

Index: b/drivers/gpu/drm/i915/Kconfig
===================================================================
--- a/drivers/gpu/drm/i915/Kconfig 2019-04-12 11:42:30.070095359 +0200
+++ b/drivers/gpu/drm/i915/Kconfig 2019-04-12 11:42:30.066095359 +0200
@@ -45,7 +45,6 @@ config DRM_I915
config DRM_I915_ALPHA_SUPPORT
bool "Enable alpha quality support for new Intel hardware by default"
depends on DRM_I915
- default n
help
Choose this option if you have new Intel hardware and want to enable
the alpha quality i915 driver support for the hardware in this kernel
@@ -99,7 +98,6 @@ config DRM_I915_GVT
bool "Enable Intel GVT-g graphics virtualization host support"
depends on DRM_I915
depends on 64BIT
- default n
help
Choose this option if you want to enable Intel GVT-g graphics
virtualization technology host support with integrated graphics.
@@ -123,7 +121,6 @@ config DRM_I915_GVT_KVMGT
depends on DRM_I915_GVT
depends on KVM
depends on VFIO_MDEV && VFIO_MDEV_DEVICE
- default n
help
Choose this option if you want to enable KVMGT support for
Intel GVT-g.
Index: b/drivers/gpu/drm/i915/Kconfig.debug
===================================================================
--- a/drivers/gpu/drm/i915/Kconfig.debug 2019-04-12 11:42:30.070095359 +0200
+++ b/drivers/gpu/drm/i915/Kconfig.debug 2019-04-12 11:42:30.066095359 +0200
@@ -6,7 +6,6 @@ config DRM_I915_WERROR
# We use the dependency on !COMPILE_TEST to not be enabled in
# allmodconfig or allyesconfig configurations
depends on !COMPILE_TEST
- default n
help
Add -Werror to the build flags for (and only for) i915.ko.
Do not enable this unless you are writing code for the i915.ko module.
@@ -31,7 +30,6 @@ config DRM_I915_DEBUG
select DRM_I915_SW_FENCE_DEBUG_OBJECTS
select DRM_I915_SELFTEST
select DRM_I915_DEBUG_RUNTIME_PM
- default n
help
Choose this option to turn on extra driver debugging that may affect
performance but will catch some internal issues.
@@ -42,7 +40,6 @@ config DRM_I915_DEBUG

config DRM_I915_DEBUG_GEM
bool "Insert extra checks into the GEM internals"
- default n
depends on DRM_I915_WERROR
help
Enable extra sanity checks (including BUGs) along the GEM driver
@@ -54,7 +51,6 @@ config DRM_I915_DEBUG_GEM

config DRM_I915_ERRLOG_GEM
bool "Insert extra logging (very verbose) for common GEM errors"
- default n
depends on DRM_I915_DEBUG_GEM
help
Enable additional logging that may help track down the cause of
@@ -68,7 +64,6 @@ config DRM_I915_TRACE_GEM
bool "Insert extra ftrace output from the GEM internals"
depends on DRM_I915_DEBUG_GEM
select TRACING
- default n
help
Enable additional and verbose debugging output that will spam
ordinary tests, but may be vital for post-mortem debugging when
@@ -82,7 +77,6 @@ config DRM_I915_SW_FENCE_DEBUG_OBJECTS
bool "Enable additional driver debugging for fence objects"
depends on DRM_I915
select DEBUG_OBJECTS
- default n
help
Choose this option to turn on extra driver debugging that may affect
performance but will catch some internal issues.
@@ -94,7 +88,6 @@ config DRM_I915_SW_FENCE_DEBUG_OBJECTS
config DRM_I915_SW_FENCE_CHECK_DAG
bool "Enable additional driver debugging for detecting dependency cycles"
depends on DRM_I915
- default n
help
Choose this option to turn on extra driver debugging that may affect
performance but will catch some internal issues.
@@ -106,7 +99,6 @@ config DRM_I915_SW_FENCE_CHECK_DAG
config DRM_I915_DEBUG_GUC
bool "Enable additional driver debugging for GuC"
depends on DRM_I915
- default n
help
Choose this option to turn on extra driver debugging that may affect
performance but will help resolve GuC related issues.
@@ -118,7 +110,6 @@ config DRM_I915_DEBUG_GUC
config DRM_I915_SELFTEST
bool "Enable selftests upon driver load"
depends on DRM_I915
- default n
select FAULT_INJECTION
select PRIME_NUMBERS
help
@@ -135,7 +126,6 @@ config DRM_I915_SELFTEST_BROKEN
bool "Enable broken and dangerous selftests"
depends on DRM_I915_SELFTEST
depends on BROKEN
- default n
help
This option enables the execution of selftests that are "dangerous"
and may trigger unintended HW side-effects as they break strict
@@ -148,7 +138,6 @@ config DRM_I915_SELFTEST_BROKEN
config DRM_I915_LOW_LEVEL_TRACEPOINTS
bool "Enable low level request tracing events"
depends on DRM_I915
- default n
help
Choose this option to turn on low level request tracing events.
This provides the ability to precisely monitor engine utilisation
@@ -159,7 +148,6 @@ config DRM_I915_LOW_LEVEL_TRACEPOINTS
config DRM_I915_DEBUG_VBLANK_EVADE
bool "Enable extra debug warnings for vblank evasion"
depends on DRM_I915
- default n
help
Choose this option to turn on extra debug warnings for the
vblank evade mechanism. This gives a warning every time the
@@ -172,7 +160,6 @@ config DRM_I915_DEBUG_VBLANK_EVADE
config DRM_I915_DEBUG_RUNTIME_PM
bool "Enable extra state checking for runtime PM"
depends on DRM_I915
- default n
select STACKDEPOT
help
Choose this option to turn on extra state checking for the
Index: b/drivers/gpu/drm/msm/Kconfig
===================================================================
--- a/drivers/gpu/drm/msm/Kconfig 2019-04-12 11:42:30.070095359 +0200
+++ b/drivers/gpu/drm/msm/Kconfig 2019-04-12 11:42:30.066095359 +0200
@@ -24,7 +24,6 @@ config DRM_MSM
config DRM_MSM_REGISTER_LOGGING
bool "MSM DRM register logging"
depends on DRM_MSM
- default n
help
Compile in support for logging register reads/writes in a format
that can be parsed by envytools demsm tool. If enabled, register
@@ -33,7 +32,6 @@ config DRM_MSM_REGISTER_LOGGING
config DRM_MSM_GPU_SUDO
bool "Enable SUDO flag on submits"
depends on DRM_MSM && EXPERT
- default n
help
Enable userspace that has CAP_SYS_RAWIO to submit GPU commands
that are run from RB instead of IB1. This essentially gives
Index: b/drivers/gpu/drm/nouveau/Kconfig
===================================================================
--- a/drivers/gpu/drm/nouveau/Kconfig 2019-04-12 11:42:30.070095359 +0200
+++ b/drivers/gpu/drm/nouveau/Kconfig 2019-04-12 11:43:25.998096767 +0200
@@ -60,7 +60,6 @@ config NOUVEAU_DEBUG_DEFAULT
config NOUVEAU_DEBUG_MMU
bool "Enable additional MMU debugging"
depends on DRM_NOUVEAU
- default n
help
Say Y here if you want to enable verbose MMU debug output.

@@ -79,7 +78,6 @@ config DRM_NOUVEAU_SVM
depends on STAGING
select HMM_MIRROR
select DEVICE_PRIVATE
- default n
help
Say Y here if you want to enable experimental support for
Shared Virtual Memory (SVM).
Index: b/drivers/gpu/drm/omapdrm/Kconfig
===================================================================
--- a/drivers/gpu/drm/omapdrm/Kconfig 2019-04-12 11:42:30.070095359 +0200
+++ b/drivers/gpu/drm/omapdrm/Kconfig 2019-04-12 11:42:30.070095359 +0200
@@ -4,7 +4,6 @@ config DRM_OMAP
depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM
select OMAP2_DSS
select DRM_KMS_HELPER
- default n
help
DRM display driver for OMAP2/3/4 based boards.

Index: b/drivers/gpu/drm/omapdrm/dss/Kconfig
===================================================================
--- a/drivers/gpu/drm/omapdrm/dss/Kconfig 2019-04-12 11:42:30.070095359 +0200
+++ b/drivers/gpu/drm/omapdrm/dss/Kconfig 2019-04-12 11:42:30.070095359 +0200
@@ -17,7 +17,6 @@ if OMAP2_DSS

config OMAP2_DSS_DEBUG
bool "Debug support"
- default n
help
This enables printing of debug messages. Alternatively, debug messages
can also be enabled by setting CONFIG_DYNAMIC_DEBUG and then setting
@@ -26,7 +25,6 @@ config OMAP2_DSS_DEBUG
config OMAP2_DSS_DEBUGFS
bool "Debugfs filesystem support"
depends on DEBUG_FS
- default n
help
This enables debugfs for OMAPDSS at <debugfs>/omapdss. This enables
querying about clock configuration and register configuration of dss,
@@ -35,7 +33,6 @@ config OMAP2_DSS_DEBUGFS
config OMAP2_DSS_COLLECT_IRQ_STATS
bool "Collect DSS IRQ statistics"
depends on OMAP2_DSS_DEBUGFS
- default n
help
Collect DSS IRQ statistics, printable via debugfs.

@@ -75,7 +72,6 @@ config OMAP4_DSS_HDMI_CEC

config OMAP5_DSS_HDMI
bool "HDMI support for OMAP5"
- default n
select OMAP2_DSS_HDMI_COMMON
help
HDMI Interface for OMAP5 and similar cores. This adds the High
@@ -84,7 +80,6 @@ config OMAP5_DSS_HDMI

config OMAP2_DSS_SDI
bool "SDI support"
- default n
help
SDI (Serial Display Interface) support.

@@ -93,7 +88,6 @@ config OMAP2_DSS_SDI

config OMAP2_DSS_DSI
bool "DSI support"
- default n
help
MIPI DSI (Display Serial Interface) support.


2019-04-12 10:12:52

by Jani Nikula

[permalink] [raw]
Subject: Re: [PATCH] drm: remove redundant 'default n' from Kconfig

On Fri, 12 Apr 2019, Bartlomiej Zolnierkiewicz <[email protected]> wrote:
> 'default n' is the default value for any bool or tristate Kconfig
> setting so there is no need to write it explicitly.
>
> Also since commit f467c5640c29 ("kconfig: only write '# CONFIG_FOO
> is not set' for visible symbols") the Kconfig behavior is the same
> regardless of 'default n' being present or not:
>
> ...
> One side effect of (and the main motivation for) this change is making
> the following two definitions behave exactly the same:
>
> config FOO
> bool
>
> config FOO
> bool
> default n
>
> With this change, neither of these will generate a
> '# CONFIG_FOO is not set' line (assuming FOO isn't selected/implied).
> That might make it clearer to people that a bare 'default n' is
> redundant.
> ...
>
> Signed-off-by: Bartlomiej Zolnierkiewicz <[email protected]>
> ---
> drivers/gpu/drm/Kconfig | 5 -----
> drivers/gpu/drm/amd/amdgpu/Kconfig | 1 -
> drivers/gpu/drm/arm/Kconfig | 1 -
> drivers/gpu/drm/exynos/Kconfig | 2 --
> drivers/gpu/drm/i915/Kconfig | 3 ---
> drivers/gpu/drm/i915/Kconfig.debug | 13 -------------

For i915,

Acked-by: Jani Nikula <[email protected]>

and can be merged through whichever tree you like.

> drivers/gpu/drm/msm/Kconfig | 2 --
> drivers/gpu/drm/nouveau/Kconfig | 2 --
> drivers/gpu/drm/omapdrm/Kconfig | 1 -
> drivers/gpu/drm/omapdrm/dss/Kconfig | 6 ------
> 10 files changed, 36 deletions(-)
>
> Index: b/drivers/gpu/drm/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -37,7 +37,6 @@ config DRM_DP_AUX_CHARDEV
>
> config DRM_DEBUG_MM
> bool "Insert extra checks and debug info into the DRM range managers"
> - default n
> depends on DRM=y
> depends on STACKTRACE_SUPPORT
> select STACKDEPOT
> @@ -56,7 +55,6 @@ config DRM_DEBUG_SELFTEST
> select PRIME_NUMBERS
> select DRM_LIB_RANDOM
> select DRM_KMS_HELPER
> - default n
> help
> This option provides kernel modules that can be used to run
> various selftests on parts of the DRM api. This option is not
> @@ -113,7 +111,6 @@ config DRM_FBDEV_OVERALLOC
> config DRM_FBDEV_LEAK_PHYS_SMEM
> bool "Shamelessly allow leaking of fbdev physical address (DANGEROUS)"
> depends on DRM_FBDEV_EMULATION && EXPERT
> - default n
> help
> In order to keep user-space compatibility, we want in certain
> use-cases to keep leaking the fbdev physical address to the
> @@ -247,7 +244,6 @@ config DRM_VKMS
> tristate "Virtual KMS (EXPERIMENTAL)"
> depends on DRM
> select DRM_KMS_HELPER
> - default n
> help
> Virtual Kernel Mode-Setting (VKMS) is used for testing or for
> running GPU in a headless machines. Choose this option to get
> @@ -424,4 +420,3 @@ config DRM_PANEL_ORIENTATION_QUIRKS
>
> config DRM_LIB_RANDOM
> bool
> - default n
> Index: b/drivers/gpu/drm/amd/amdgpu/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/amd/amdgpu/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/amd/amdgpu/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -35,7 +35,6 @@ config DRM_AMDGPU_GART_DEBUGFS
> bool "Allow GART access through debugfs"
> depends on DRM_AMDGPU
> depends on DEBUG_FS
> - default n
> help
> Selecting this option creates a debugfs file to inspect the mapped
> pages. Uses more memory for housekeeping, enable only for debugging.
> Index: b/drivers/gpu/drm/arm/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/arm/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/arm/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -16,7 +16,6 @@ config DRM_HDLCD
> config DRM_HDLCD_SHOW_UNDERRUN
> bool "Show underrun conditions"
> depends on DRM_HDLCD
> - default n
> help
> Enable this option to show in red colour the pixels that the
> HDLCD device did not fetch from framebuffer due to underrun
> Index: b/drivers/gpu/drm/exynos/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/exynos/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/exynos/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -46,7 +46,6 @@ config DRM_EXYNOS_DPI
> bool "Parallel output"
> depends on DRM_EXYNOS_FIMD
> select DRM_PANEL
> - default n
> help
> This enables support for Exynos parallel output.
>
> @@ -55,7 +54,6 @@ config DRM_EXYNOS_DSI
> depends on DRM_EXYNOS_FIMD || DRM_EXYNOS5433_DECON || DRM_EXYNOS7_DECON
> select DRM_MIPI_DSI
> select DRM_PANEL
> - default n
> help
> This enables support for Exynos MIPI-DSI device.
>
> Index: b/drivers/gpu/drm/i915/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/i915/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/i915/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -45,7 +45,6 @@ config DRM_I915
> config DRM_I915_ALPHA_SUPPORT
> bool "Enable alpha quality support for new Intel hardware by default"
> depends on DRM_I915
> - default n
> help
> Choose this option if you have new Intel hardware and want to enable
> the alpha quality i915 driver support for the hardware in this kernel
> @@ -99,7 +98,6 @@ config DRM_I915_GVT
> bool "Enable Intel GVT-g graphics virtualization host support"
> depends on DRM_I915
> depends on 64BIT
> - default n
> help
> Choose this option if you want to enable Intel GVT-g graphics
> virtualization technology host support with integrated graphics.
> @@ -123,7 +121,6 @@ config DRM_I915_GVT_KVMGT
> depends on DRM_I915_GVT
> depends on KVM
> depends on VFIO_MDEV && VFIO_MDEV_DEVICE
> - default n
> help
> Choose this option if you want to enable KVMGT support for
> Intel GVT-g.
> Index: b/drivers/gpu/drm/i915/Kconfig.debug
> ===================================================================
> --- a/drivers/gpu/drm/i915/Kconfig.debug 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/i915/Kconfig.debug 2019-04-12 11:42:30.066095359 +0200
> @@ -6,7 +6,6 @@ config DRM_I915_WERROR
> # We use the dependency on !COMPILE_TEST to not be enabled in
> # allmodconfig or allyesconfig configurations
> depends on !COMPILE_TEST
> - default n
> help
> Add -Werror to the build flags for (and only for) i915.ko.
> Do not enable this unless you are writing code for the i915.ko module.
> @@ -31,7 +30,6 @@ config DRM_I915_DEBUG
> select DRM_I915_SW_FENCE_DEBUG_OBJECTS
> select DRM_I915_SELFTEST
> select DRM_I915_DEBUG_RUNTIME_PM
> - default n
> help
> Choose this option to turn on extra driver debugging that may affect
> performance but will catch some internal issues.
> @@ -42,7 +40,6 @@ config DRM_I915_DEBUG
>
> config DRM_I915_DEBUG_GEM
> bool "Insert extra checks into the GEM internals"
> - default n
> depends on DRM_I915_WERROR
> help
> Enable extra sanity checks (including BUGs) along the GEM driver
> @@ -54,7 +51,6 @@ config DRM_I915_DEBUG_GEM
>
> config DRM_I915_ERRLOG_GEM
> bool "Insert extra logging (very verbose) for common GEM errors"
> - default n
> depends on DRM_I915_DEBUG_GEM
> help
> Enable additional logging that may help track down the cause of
> @@ -68,7 +64,6 @@ config DRM_I915_TRACE_GEM
> bool "Insert extra ftrace output from the GEM internals"
> depends on DRM_I915_DEBUG_GEM
> select TRACING
> - default n
> help
> Enable additional and verbose debugging output that will spam
> ordinary tests, but may be vital for post-mortem debugging when
> @@ -82,7 +77,6 @@ config DRM_I915_SW_FENCE_DEBUG_OBJECTS
> bool "Enable additional driver debugging for fence objects"
> depends on DRM_I915
> select DEBUG_OBJECTS
> - default n
> help
> Choose this option to turn on extra driver debugging that may affect
> performance but will catch some internal issues.
> @@ -94,7 +88,6 @@ config DRM_I915_SW_FENCE_DEBUG_OBJECTS
> config DRM_I915_SW_FENCE_CHECK_DAG
> bool "Enable additional driver debugging for detecting dependency cycles"
> depends on DRM_I915
> - default n
> help
> Choose this option to turn on extra driver debugging that may affect
> performance but will catch some internal issues.
> @@ -106,7 +99,6 @@ config DRM_I915_SW_FENCE_CHECK_DAG
> config DRM_I915_DEBUG_GUC
> bool "Enable additional driver debugging for GuC"
> depends on DRM_I915
> - default n
> help
> Choose this option to turn on extra driver debugging that may affect
> performance but will help resolve GuC related issues.
> @@ -118,7 +110,6 @@ config DRM_I915_DEBUG_GUC
> config DRM_I915_SELFTEST
> bool "Enable selftests upon driver load"
> depends on DRM_I915
> - default n
> select FAULT_INJECTION
> select PRIME_NUMBERS
> help
> @@ -135,7 +126,6 @@ config DRM_I915_SELFTEST_BROKEN
> bool "Enable broken and dangerous selftests"
> depends on DRM_I915_SELFTEST
> depends on BROKEN
> - default n
> help
> This option enables the execution of selftests that are "dangerous"
> and may trigger unintended HW side-effects as they break strict
> @@ -148,7 +138,6 @@ config DRM_I915_SELFTEST_BROKEN
> config DRM_I915_LOW_LEVEL_TRACEPOINTS
> bool "Enable low level request tracing events"
> depends on DRM_I915
> - default n
> help
> Choose this option to turn on low level request tracing events.
> This provides the ability to precisely monitor engine utilisation
> @@ -159,7 +148,6 @@ config DRM_I915_LOW_LEVEL_TRACEPOINTS
> config DRM_I915_DEBUG_VBLANK_EVADE
> bool "Enable extra debug warnings for vblank evasion"
> depends on DRM_I915
> - default n
> help
> Choose this option to turn on extra debug warnings for the
> vblank evade mechanism. This gives a warning every time the
> @@ -172,7 +160,6 @@ config DRM_I915_DEBUG_VBLANK_EVADE
> config DRM_I915_DEBUG_RUNTIME_PM
> bool "Enable extra state checking for runtime PM"
> depends on DRM_I915
> - default n
> select STACKDEPOT
> help
> Choose this option to turn on extra state checking for the
> Index: b/drivers/gpu/drm/msm/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/msm/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/msm/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -24,7 +24,6 @@ config DRM_MSM
> config DRM_MSM_REGISTER_LOGGING
> bool "MSM DRM register logging"
> depends on DRM_MSM
> - default n
> help
> Compile in support for logging register reads/writes in a format
> that can be parsed by envytools demsm tool. If enabled, register
> @@ -33,7 +32,6 @@ config DRM_MSM_REGISTER_LOGGING
> config DRM_MSM_GPU_SUDO
> bool "Enable SUDO flag on submits"
> depends on DRM_MSM && EXPERT
> - default n
> help
> Enable userspace that has CAP_SYS_RAWIO to submit GPU commands
> that are run from RB instead of IB1. This essentially gives
> Index: b/drivers/gpu/drm/nouveau/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/nouveau/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/nouveau/Kconfig 2019-04-12 11:43:25.998096767 +0200
> @@ -60,7 +60,6 @@ config NOUVEAU_DEBUG_DEFAULT
> config NOUVEAU_DEBUG_MMU
> bool "Enable additional MMU debugging"
> depends on DRM_NOUVEAU
> - default n
> help
> Say Y here if you want to enable verbose MMU debug output.
>
> @@ -79,7 +78,6 @@ config DRM_NOUVEAU_SVM
> depends on STAGING
> select HMM_MIRROR
> select DEVICE_PRIVATE
> - default n
> help
> Say Y here if you want to enable experimental support for
> Shared Virtual Memory (SVM).
> Index: b/drivers/gpu/drm/omapdrm/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/omapdrm/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/omapdrm/Kconfig 2019-04-12 11:42:30.070095359 +0200
> @@ -4,7 +4,6 @@ config DRM_OMAP
> depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM
> select OMAP2_DSS
> select DRM_KMS_HELPER
> - default n
> help
> DRM display driver for OMAP2/3/4 based boards.
>
> Index: b/drivers/gpu/drm/omapdrm/dss/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/omapdrm/dss/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/omapdrm/dss/Kconfig 2019-04-12 11:42:30.070095359 +0200
> @@ -17,7 +17,6 @@ if OMAP2_DSS
>
> config OMAP2_DSS_DEBUG
> bool "Debug support"
> - default n
> help
> This enables printing of debug messages. Alternatively, debug messages
> can also be enabled by setting CONFIG_DYNAMIC_DEBUG and then setting
> @@ -26,7 +25,6 @@ config OMAP2_DSS_DEBUG
> config OMAP2_DSS_DEBUGFS
> bool "Debugfs filesystem support"
> depends on DEBUG_FS
> - default n
> help
> This enables debugfs for OMAPDSS at <debugfs>/omapdss. This enables
> querying about clock configuration and register configuration of dss,
> @@ -35,7 +33,6 @@ config OMAP2_DSS_DEBUGFS
> config OMAP2_DSS_COLLECT_IRQ_STATS
> bool "Collect DSS IRQ statistics"
> depends on OMAP2_DSS_DEBUGFS
> - default n
> help
> Collect DSS IRQ statistics, printable via debugfs.
>
> @@ -75,7 +72,6 @@ config OMAP4_DSS_HDMI_CEC
>
> config OMAP5_DSS_HDMI
> bool "HDMI support for OMAP5"
> - default n
> select OMAP2_DSS_HDMI_COMMON
> help
> HDMI Interface for OMAP5 and similar cores. This adds the High
> @@ -84,7 +80,6 @@ config OMAP5_DSS_HDMI
>
> config OMAP2_DSS_SDI
> bool "SDI support"
> - default n
> help
> SDI (Serial Display Interface) support.
>
> @@ -93,7 +88,6 @@ config OMAP2_DSS_SDI
>
> config OMAP2_DSS_DSI
> bool "DSI support"
> - default n
> help
> MIPI DSI (Display Serial Interface) support.
>

--
Jani Nikula, Intel Open Source Graphics Center

2019-04-12 16:59:26

by Alex Deucher

[permalink] [raw]
Subject: Re: [PATCH] drm: remove redundant 'default n' from Kconfig

On Fri, Apr 12, 2019 at 5:56 AM Bartlomiej Zolnierkiewicz
<[email protected]> wrote:
>
> 'default n' is the default value for any bool or tristate Kconfig
> setting so there is no need to write it explicitly.
>
> Also since commit f467c5640c29 ("kconfig: only write '# CONFIG_FOO
> is not set' for visible symbols") the Kconfig behavior is the same
> regardless of 'default n' being present or not:
>
> ...
> One side effect of (and the main motivation for) this change is making
> the following two definitions behave exactly the same:
>
> config FOO
> bool
>
> config FOO
> bool
> default n
>
> With this change, neither of these will generate a
> '# CONFIG_FOO is not set' line (assuming FOO isn't selected/implied).
> That might make it clearer to people that a bare 'default n' is
> redundant.
> ...
>
> Signed-off-by: Bartlomiej Zolnierkiewicz <[email protected]>

Acked-by: Alex Deucher <[email protected]>
for amdgpu and drm.

> ---
> drivers/gpu/drm/Kconfig | 5 -----
> drivers/gpu/drm/amd/amdgpu/Kconfig | 1 -
> drivers/gpu/drm/arm/Kconfig | 1 -
> drivers/gpu/drm/exynos/Kconfig | 2 --
> drivers/gpu/drm/i915/Kconfig | 3 ---
> drivers/gpu/drm/i915/Kconfig.debug | 13 -------------
> drivers/gpu/drm/msm/Kconfig | 2 --
> drivers/gpu/drm/nouveau/Kconfig | 2 --
> drivers/gpu/drm/omapdrm/Kconfig | 1 -
> drivers/gpu/drm/omapdrm/dss/Kconfig | 6 ------
> 10 files changed, 36 deletions(-)
>
> Index: b/drivers/gpu/drm/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -37,7 +37,6 @@ config DRM_DP_AUX_CHARDEV
>
> config DRM_DEBUG_MM
> bool "Insert extra checks and debug info into the DRM range managers"
> - default n
> depends on DRM=y
> depends on STACKTRACE_SUPPORT
> select STACKDEPOT
> @@ -56,7 +55,6 @@ config DRM_DEBUG_SELFTEST
> select PRIME_NUMBERS
> select DRM_LIB_RANDOM
> select DRM_KMS_HELPER
> - default n
> help
> This option provides kernel modules that can be used to run
> various selftests on parts of the DRM api. This option is not
> @@ -113,7 +111,6 @@ config DRM_FBDEV_OVERALLOC
> config DRM_FBDEV_LEAK_PHYS_SMEM
> bool "Shamelessly allow leaking of fbdev physical address (DANGEROUS)"
> depends on DRM_FBDEV_EMULATION && EXPERT
> - default n
> help
> In order to keep user-space compatibility, we want in certain
> use-cases to keep leaking the fbdev physical address to the
> @@ -247,7 +244,6 @@ config DRM_VKMS
> tristate "Virtual KMS (EXPERIMENTAL)"
> depends on DRM
> select DRM_KMS_HELPER
> - default n
> help
> Virtual Kernel Mode-Setting (VKMS) is used for testing or for
> running GPU in a headless machines. Choose this option to get
> @@ -424,4 +420,3 @@ config DRM_PANEL_ORIENTATION_QUIRKS
>
> config DRM_LIB_RANDOM
> bool
> - default n
> Index: b/drivers/gpu/drm/amd/amdgpu/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/amd/amdgpu/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/amd/amdgpu/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -35,7 +35,6 @@ config DRM_AMDGPU_GART_DEBUGFS
> bool "Allow GART access through debugfs"
> depends on DRM_AMDGPU
> depends on DEBUG_FS
> - default n
> help
> Selecting this option creates a debugfs file to inspect the mapped
> pages. Uses more memory for housekeeping, enable only for debugging.
> Index: b/drivers/gpu/drm/arm/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/arm/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/arm/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -16,7 +16,6 @@ config DRM_HDLCD
> config DRM_HDLCD_SHOW_UNDERRUN
> bool "Show underrun conditions"
> depends on DRM_HDLCD
> - default n
> help
> Enable this option to show in red colour the pixels that the
> HDLCD device did not fetch from framebuffer due to underrun
> Index: b/drivers/gpu/drm/exynos/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/exynos/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/exynos/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -46,7 +46,6 @@ config DRM_EXYNOS_DPI
> bool "Parallel output"
> depends on DRM_EXYNOS_FIMD
> select DRM_PANEL
> - default n
> help
> This enables support for Exynos parallel output.
>
> @@ -55,7 +54,6 @@ config DRM_EXYNOS_DSI
> depends on DRM_EXYNOS_FIMD || DRM_EXYNOS5433_DECON || DRM_EXYNOS7_DECON
> select DRM_MIPI_DSI
> select DRM_PANEL
> - default n
> help
> This enables support for Exynos MIPI-DSI device.
>
> Index: b/drivers/gpu/drm/i915/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/i915/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/i915/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -45,7 +45,6 @@ config DRM_I915
> config DRM_I915_ALPHA_SUPPORT
> bool "Enable alpha quality support for new Intel hardware by default"
> depends on DRM_I915
> - default n
> help
> Choose this option if you have new Intel hardware and want to enable
> the alpha quality i915 driver support for the hardware in this kernel
> @@ -99,7 +98,6 @@ config DRM_I915_GVT
> bool "Enable Intel GVT-g graphics virtualization host support"
> depends on DRM_I915
> depends on 64BIT
> - default n
> help
> Choose this option if you want to enable Intel GVT-g graphics
> virtualization technology host support with integrated graphics.
> @@ -123,7 +121,6 @@ config DRM_I915_GVT_KVMGT
> depends on DRM_I915_GVT
> depends on KVM
> depends on VFIO_MDEV && VFIO_MDEV_DEVICE
> - default n
> help
> Choose this option if you want to enable KVMGT support for
> Intel GVT-g.
> Index: b/drivers/gpu/drm/i915/Kconfig.debug
> ===================================================================
> --- a/drivers/gpu/drm/i915/Kconfig.debug 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/i915/Kconfig.debug 2019-04-12 11:42:30.066095359 +0200
> @@ -6,7 +6,6 @@ config DRM_I915_WERROR
> # We use the dependency on !COMPILE_TEST to not be enabled in
> # allmodconfig or allyesconfig configurations
> depends on !COMPILE_TEST
> - default n
> help
> Add -Werror to the build flags for (and only for) i915.ko.
> Do not enable this unless you are writing code for the i915.ko module.
> @@ -31,7 +30,6 @@ config DRM_I915_DEBUG
> select DRM_I915_SW_FENCE_DEBUG_OBJECTS
> select DRM_I915_SELFTEST
> select DRM_I915_DEBUG_RUNTIME_PM
> - default n
> help
> Choose this option to turn on extra driver debugging that may affect
> performance but will catch some internal issues.
> @@ -42,7 +40,6 @@ config DRM_I915_DEBUG
>
> config DRM_I915_DEBUG_GEM
> bool "Insert extra checks into the GEM internals"
> - default n
> depends on DRM_I915_WERROR
> help
> Enable extra sanity checks (including BUGs) along the GEM driver
> @@ -54,7 +51,6 @@ config DRM_I915_DEBUG_GEM
>
> config DRM_I915_ERRLOG_GEM
> bool "Insert extra logging (very verbose) for common GEM errors"
> - default n
> depends on DRM_I915_DEBUG_GEM
> help
> Enable additional logging that may help track down the cause of
> @@ -68,7 +64,6 @@ config DRM_I915_TRACE_GEM
> bool "Insert extra ftrace output from the GEM internals"
> depends on DRM_I915_DEBUG_GEM
> select TRACING
> - default n
> help
> Enable additional and verbose debugging output that will spam
> ordinary tests, but may be vital for post-mortem debugging when
> @@ -82,7 +77,6 @@ config DRM_I915_SW_FENCE_DEBUG_OBJECTS
> bool "Enable additional driver debugging for fence objects"
> depends on DRM_I915
> select DEBUG_OBJECTS
> - default n
> help
> Choose this option to turn on extra driver debugging that may affect
> performance but will catch some internal issues.
> @@ -94,7 +88,6 @@ config DRM_I915_SW_FENCE_DEBUG_OBJECTS
> config DRM_I915_SW_FENCE_CHECK_DAG
> bool "Enable additional driver debugging for detecting dependency cycles"
> depends on DRM_I915
> - default n
> help
> Choose this option to turn on extra driver debugging that may affect
> performance but will catch some internal issues.
> @@ -106,7 +99,6 @@ config DRM_I915_SW_FENCE_CHECK_DAG
> config DRM_I915_DEBUG_GUC
> bool "Enable additional driver debugging for GuC"
> depends on DRM_I915
> - default n
> help
> Choose this option to turn on extra driver debugging that may affect
> performance but will help resolve GuC related issues.
> @@ -118,7 +110,6 @@ config DRM_I915_DEBUG_GUC
> config DRM_I915_SELFTEST
> bool "Enable selftests upon driver load"
> depends on DRM_I915
> - default n
> select FAULT_INJECTION
> select PRIME_NUMBERS
> help
> @@ -135,7 +126,6 @@ config DRM_I915_SELFTEST_BROKEN
> bool "Enable broken and dangerous selftests"
> depends on DRM_I915_SELFTEST
> depends on BROKEN
> - default n
> help
> This option enables the execution of selftests that are "dangerous"
> and may trigger unintended HW side-effects as they break strict
> @@ -148,7 +138,6 @@ config DRM_I915_SELFTEST_BROKEN
> config DRM_I915_LOW_LEVEL_TRACEPOINTS
> bool "Enable low level request tracing events"
> depends on DRM_I915
> - default n
> help
> Choose this option to turn on low level request tracing events.
> This provides the ability to precisely monitor engine utilisation
> @@ -159,7 +148,6 @@ config DRM_I915_LOW_LEVEL_TRACEPOINTS
> config DRM_I915_DEBUG_VBLANK_EVADE
> bool "Enable extra debug warnings for vblank evasion"
> depends on DRM_I915
> - default n
> help
> Choose this option to turn on extra debug warnings for the
> vblank evade mechanism. This gives a warning every time the
> @@ -172,7 +160,6 @@ config DRM_I915_DEBUG_VBLANK_EVADE
> config DRM_I915_DEBUG_RUNTIME_PM
> bool "Enable extra state checking for runtime PM"
> depends on DRM_I915
> - default n
> select STACKDEPOT
> help
> Choose this option to turn on extra state checking for the
> Index: b/drivers/gpu/drm/msm/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/msm/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/msm/Kconfig 2019-04-12 11:42:30.066095359 +0200
> @@ -24,7 +24,6 @@ config DRM_MSM
> config DRM_MSM_REGISTER_LOGGING
> bool "MSM DRM register logging"
> depends on DRM_MSM
> - default n
> help
> Compile in support for logging register reads/writes in a format
> that can be parsed by envytools demsm tool. If enabled, register
> @@ -33,7 +32,6 @@ config DRM_MSM_REGISTER_LOGGING
> config DRM_MSM_GPU_SUDO
> bool "Enable SUDO flag on submits"
> depends on DRM_MSM && EXPERT
> - default n
> help
> Enable userspace that has CAP_SYS_RAWIO to submit GPU commands
> that are run from RB instead of IB1. This essentially gives
> Index: b/drivers/gpu/drm/nouveau/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/nouveau/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/nouveau/Kconfig 2019-04-12 11:43:25.998096767 +0200
> @@ -60,7 +60,6 @@ config NOUVEAU_DEBUG_DEFAULT
> config NOUVEAU_DEBUG_MMU
> bool "Enable additional MMU debugging"
> depends on DRM_NOUVEAU
> - default n
> help
> Say Y here if you want to enable verbose MMU debug output.
>
> @@ -79,7 +78,6 @@ config DRM_NOUVEAU_SVM
> depends on STAGING
> select HMM_MIRROR
> select DEVICE_PRIVATE
> - default n
> help
> Say Y here if you want to enable experimental support for
> Shared Virtual Memory (SVM).
> Index: b/drivers/gpu/drm/omapdrm/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/omapdrm/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/omapdrm/Kconfig 2019-04-12 11:42:30.070095359 +0200
> @@ -4,7 +4,6 @@ config DRM_OMAP
> depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM
> select OMAP2_DSS
> select DRM_KMS_HELPER
> - default n
> help
> DRM display driver for OMAP2/3/4 based boards.
>
> Index: b/drivers/gpu/drm/omapdrm/dss/Kconfig
> ===================================================================
> --- a/drivers/gpu/drm/omapdrm/dss/Kconfig 2019-04-12 11:42:30.070095359 +0200
> +++ b/drivers/gpu/drm/omapdrm/dss/Kconfig 2019-04-12 11:42:30.070095359 +0200
> @@ -17,7 +17,6 @@ if OMAP2_DSS
>
> config OMAP2_DSS_DEBUG
> bool "Debug support"
> - default n
> help
> This enables printing of debug messages. Alternatively, debug messages
> can also be enabled by setting CONFIG_DYNAMIC_DEBUG and then setting
> @@ -26,7 +25,6 @@ config OMAP2_DSS_DEBUG
> config OMAP2_DSS_DEBUGFS
> bool "Debugfs filesystem support"
> depends on DEBUG_FS
> - default n
> help
> This enables debugfs for OMAPDSS at <debugfs>/omapdss. This enables
> querying about clock configuration and register configuration of dss,
> @@ -35,7 +33,6 @@ config OMAP2_DSS_DEBUGFS
> config OMAP2_DSS_COLLECT_IRQ_STATS
> bool "Collect DSS IRQ statistics"
> depends on OMAP2_DSS_DEBUGFS
> - default n
> help
> Collect DSS IRQ statistics, printable via debugfs.
>
> @@ -75,7 +72,6 @@ config OMAP4_DSS_HDMI_CEC
>
> config OMAP5_DSS_HDMI
> bool "HDMI support for OMAP5"
> - default n
> select OMAP2_DSS_HDMI_COMMON
> help
> HDMI Interface for OMAP5 and similar cores. This adds the High
> @@ -84,7 +80,6 @@ config OMAP5_DSS_HDMI
>
> config OMAP2_DSS_SDI
> bool "SDI support"
> - default n
> help
> SDI (Serial Display Interface) support.
>
> @@ -93,7 +88,6 @@ config OMAP2_DSS_SDI
>
> config OMAP2_DSS_DSI
> bool "DSI support"
> - default n
> help
> MIPI DSI (Display Serial Interface) support.
>
> _______________________________________________
> amd-gfx mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx