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

[Alex Deucher]

On Fri, Apr 12, 2019 at 5:56 AM Bartlomiej Zolnierkiewicz
<b.zolnierkie@xxxxxxxxxxx> 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 <b.zolnierkie@xxxxxxxxxxx>

Acked-by: Alex Deucher <alexander.deucher@xxxxxxx>
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
> amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx