All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3] drm: Move radeon and amdgpu Kconfig options into their directories
@ 2022-10-26 16:19 Andrew Davis
  2022-10-27 18:11   ` Alex Deucher
  0 siblings, 1 reply; 3+ messages in thread
From: Andrew Davis @ 2022-10-26 16:19 UTC (permalink / raw)
  To: David Airlie, Daniel Vetter, Christian König, Randy Dunlap,
	dri-devel, linux-kernel
  Cc: Andrew Davis

Most Kconfig options to enable a driver are in the Kconfig file
inside the relevant directory, move these two to the same.

Signed-off-by: Andrew Davis <afd@ti.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
---

Changes from v2:
 - Rebased on latest

Changes from v1:
 - Fix whitespace issue pointed out by Randy

 drivers/gpu/drm/Kconfig            | 57 ------------------------------
 drivers/gpu/drm/amd/amdgpu/Kconfig | 29 +++++++++++++++
 drivers/gpu/drm/radeon/Kconfig     | 30 ++++++++++++++++
 3 files changed, 59 insertions(+), 57 deletions(-)

diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig
index fa986075e8fb..9c2d9495cb3c 100644
--- a/drivers/gpu/drm/Kconfig
+++ b/drivers/gpu/drm/Kconfig
@@ -233,65 +233,8 @@ source "drivers/gpu/drm/i2c/Kconfig"
 
 source "drivers/gpu/drm/arm/Kconfig"
 
-config DRM_RADEON
-	tristate "ATI Radeon"
-	depends on DRM && PCI && MMU
-	depends on AGP || !AGP
-	select FW_LOADER
-	select DRM_DISPLAY_DP_HELPER
-	select DRM_DISPLAY_HELPER
-        select DRM_KMS_HELPER
-        select DRM_TTM
-	select DRM_TTM_HELPER
-	select SND_HDA_COMPONENT if SND_HDA_CORE
-	select POWER_SUPPLY
-	select HWMON
-	select BACKLIGHT_CLASS_DEVICE
-	select INTERVAL_TREE
-	# radeon depends on ACPI_VIDEO when ACPI is enabled, for select to work
-	# ACPI_VIDEO's dependencies must also be selected.
-	select INPUT if ACPI
-	select ACPI_VIDEO if ACPI
-	# On x86 ACPI_VIDEO also needs ACPI_WMI
-	select X86_PLATFORM_DEVICES if ACPI && X86
-	select ACPI_WMI if ACPI && X86
-	help
-	  Choose this option if you have an ATI Radeon graphics card.  There
-	  are both PCI and AGP versions.  You don't need to choose this to
-	  run the Radeon in plain VGA mode.
-
-	  If M is selected, the module will be called radeon.
-
 source "drivers/gpu/drm/radeon/Kconfig"
 
-config DRM_AMDGPU
-	tristate "AMD GPU"
-	depends on DRM && PCI && MMU
-	select FW_LOADER
-	select DRM_DISPLAY_DP_HELPER
-	select DRM_DISPLAY_HDMI_HELPER
-	select DRM_DISPLAY_HELPER
-	select DRM_KMS_HELPER
-	select DRM_SCHED
-	select DRM_TTM
-	select DRM_TTM_HELPER
-	select POWER_SUPPLY
-	select HWMON
-	select BACKLIGHT_CLASS_DEVICE
-	select INTERVAL_TREE
-	select DRM_BUDDY
-	# amdgpu depends on ACPI_VIDEO when ACPI is enabled, for select to work
-	# ACPI_VIDEO's dependencies must also be selected.
-	select INPUT if ACPI
-	select ACPI_VIDEO if ACPI
-	# On x86 ACPI_VIDEO also needs ACPI_WMI
-	select X86_PLATFORM_DEVICES if ACPI && X86
-	select ACPI_WMI if ACPI && X86
-	help
-	  Choose this option if you have a recent AMD Radeon graphics card.
-
-	  If M is selected, the module will be called amdgpu.
-
 source "drivers/gpu/drm/amd/amdgpu/Kconfig"
 
 source "drivers/gpu/drm/nouveau/Kconfig"
diff --git a/drivers/gpu/drm/amd/amdgpu/Kconfig b/drivers/gpu/drm/amd/amdgpu/Kconfig
index 7777d55275de..5fcd510f1abb 100644
--- a/drivers/gpu/drm/amd/amdgpu/Kconfig
+++ b/drivers/gpu/drm/amd/amdgpu/Kconfig
@@ -1,4 +1,33 @@
 # SPDX-License-Identifier: MIT
+
+config DRM_AMDGPU
+	tristate "AMD GPU"
+	depends on DRM && PCI && MMU
+	select FW_LOADER
+	select DRM_DISPLAY_DP_HELPER
+	select DRM_DISPLAY_HDMI_HELPER
+	select DRM_DISPLAY_HELPER
+	select DRM_KMS_HELPER
+	select DRM_SCHED
+	select DRM_TTM
+	select DRM_TTM_HELPER
+	select POWER_SUPPLY
+	select HWMON
+	select BACKLIGHT_CLASS_DEVICE
+	select INTERVAL_TREE
+	select DRM_BUDDY
+	# amdgpu depends on ACPI_VIDEO when ACPI is enabled, for select to work
+	# ACPI_VIDEO's dependencies must also be selected.
+	select INPUT if ACPI
+	select ACPI_VIDEO if ACPI
+	# On x86 ACPI_VIDEO also needs ACPI_WMI
+	select X86_PLATFORM_DEVICES if ACPI && X86
+	select ACPI_WMI if ACPI && X86
+	help
+	  Choose this option if you have a recent AMD Radeon graphics card.
+
+	  If M is selected, the module will be called amdgpu.
+
 config DRM_AMDGPU_SI
 	bool "Enable amdgpu support for SI parts"
 	depends on DRM_AMDGPU
diff --git a/drivers/gpu/drm/radeon/Kconfig b/drivers/gpu/drm/radeon/Kconfig
index 52819e7f1fca..2267c501f724 100644
--- a/drivers/gpu/drm/radeon/Kconfig
+++ b/drivers/gpu/drm/radeon/Kconfig
@@ -1,4 +1,34 @@
 # SPDX-License-Identifier: MIT
+
+config DRM_RADEON
+	tristate "ATI Radeon"
+	depends on DRM && PCI && MMU
+	depends on AGP || !AGP
+	select FW_LOADER
+	select DRM_DISPLAY_DP_HELPER
+	select DRM_DISPLAY_HELPER
+	select DRM_KMS_HELPER
+	select DRM_TTM
+	select DRM_TTM_HELPER
+	select SND_HDA_COMPONENT if SND_HDA_CORE
+	select POWER_SUPPLY
+	select HWMON
+	select BACKLIGHT_CLASS_DEVICE
+	select INTERVAL_TREE
+	# radeon depends on ACPI_VIDEO when ACPI is enabled, for select to work
+	# ACPI_VIDEO's dependencies must also be selected.
+	select INPUT if ACPI
+	select ACPI_VIDEO if ACPI
+	# On x86 ACPI_VIDEO also needs ACPI_WMI
+	select X86_PLATFORM_DEVICES if ACPI && X86
+	select ACPI_WMI if ACPI && X86
+	help
+	  Choose this option if you have an ATI Radeon graphics card.  There
+	  are both PCI and AGP versions.  You don't need to choose this to
+	  run the Radeon in plain VGA mode.
+
+	  If M is selected, the module will be called radeon.
+
 config DRM_RADEON_USERPTR
 	bool "Always enable userptr support"
 	depends on DRM_RADEON
-- 
2.37.3


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v3] drm: Move radeon and amdgpu Kconfig options into their directories
  2022-10-26 16:19 [PATCH v3] drm: Move radeon and amdgpu Kconfig options into their directories Andrew Davis
@ 2022-10-27 18:11   ` Alex Deucher
  0 siblings, 0 replies; 3+ messages in thread
From: Alex Deucher @ 2022-10-27 18:11 UTC (permalink / raw)
  To: Andrew Davis
  Cc: David Airlie, Daniel Vetter, Christian König, Randy Dunlap,
	dri-devel, linux-kernel

Applied.  Thanks!

Alex

On Wed, Oct 26, 2022 at 12:31 PM Andrew Davis <afd@ti.com> wrote:
>
> Most Kconfig options to enable a driver are in the Kconfig file
> inside the relevant directory, move these two to the same.
>
> Signed-off-by: Andrew Davis <afd@ti.com>
> Reviewed-by: Christian König <christian.koenig@amd.com>
> ---
>
> Changes from v2:
>  - Rebased on latest
>
> Changes from v1:
>  - Fix whitespace issue pointed out by Randy
>
>  drivers/gpu/drm/Kconfig            | 57 ------------------------------
>  drivers/gpu/drm/amd/amdgpu/Kconfig | 29 +++++++++++++++
>  drivers/gpu/drm/radeon/Kconfig     | 30 ++++++++++++++++
>  3 files changed, 59 insertions(+), 57 deletions(-)
>
> diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig
> index fa986075e8fb..9c2d9495cb3c 100644
> --- a/drivers/gpu/drm/Kconfig
> +++ b/drivers/gpu/drm/Kconfig
> @@ -233,65 +233,8 @@ source "drivers/gpu/drm/i2c/Kconfig"
>
>  source "drivers/gpu/drm/arm/Kconfig"
>
> -config DRM_RADEON
> -       tristate "ATI Radeon"
> -       depends on DRM && PCI && MMU
> -       depends on AGP || !AGP
> -       select FW_LOADER
> -       select DRM_DISPLAY_DP_HELPER
> -       select DRM_DISPLAY_HELPER
> -        select DRM_KMS_HELPER
> -        select DRM_TTM
> -       select DRM_TTM_HELPER
> -       select SND_HDA_COMPONENT if SND_HDA_CORE
> -       select POWER_SUPPLY
> -       select HWMON
> -       select BACKLIGHT_CLASS_DEVICE
> -       select INTERVAL_TREE
> -       # radeon depends on ACPI_VIDEO when ACPI is enabled, for select to work
> -       # ACPI_VIDEO's dependencies must also be selected.
> -       select INPUT if ACPI
> -       select ACPI_VIDEO if ACPI
> -       # On x86 ACPI_VIDEO also needs ACPI_WMI
> -       select X86_PLATFORM_DEVICES if ACPI && X86
> -       select ACPI_WMI if ACPI && X86
> -       help
> -         Choose this option if you have an ATI Radeon graphics card.  There
> -         are both PCI and AGP versions.  You don't need to choose this to
> -         run the Radeon in plain VGA mode.
> -
> -         If M is selected, the module will be called radeon.
> -
>  source "drivers/gpu/drm/radeon/Kconfig"
>
> -config DRM_AMDGPU
> -       tristate "AMD GPU"
> -       depends on DRM && PCI && MMU
> -       select FW_LOADER
> -       select DRM_DISPLAY_DP_HELPER
> -       select DRM_DISPLAY_HDMI_HELPER
> -       select DRM_DISPLAY_HELPER
> -       select DRM_KMS_HELPER
> -       select DRM_SCHED
> -       select DRM_TTM
> -       select DRM_TTM_HELPER
> -       select POWER_SUPPLY
> -       select HWMON
> -       select BACKLIGHT_CLASS_DEVICE
> -       select INTERVAL_TREE
> -       select DRM_BUDDY
> -       # amdgpu depends on ACPI_VIDEO when ACPI is enabled, for select to work
> -       # ACPI_VIDEO's dependencies must also be selected.
> -       select INPUT if ACPI
> -       select ACPI_VIDEO if ACPI
> -       # On x86 ACPI_VIDEO also needs ACPI_WMI
> -       select X86_PLATFORM_DEVICES if ACPI && X86
> -       select ACPI_WMI if ACPI && X86
> -       help
> -         Choose this option if you have a recent AMD Radeon graphics card.
> -
> -         If M is selected, the module will be called amdgpu.
> -
>  source "drivers/gpu/drm/amd/amdgpu/Kconfig"
>
>  source "drivers/gpu/drm/nouveau/Kconfig"
> diff --git a/drivers/gpu/drm/amd/amdgpu/Kconfig b/drivers/gpu/drm/amd/amdgpu/Kconfig
> index 7777d55275de..5fcd510f1abb 100644
> --- a/drivers/gpu/drm/amd/amdgpu/Kconfig
> +++ b/drivers/gpu/drm/amd/amdgpu/Kconfig
> @@ -1,4 +1,33 @@
>  # SPDX-License-Identifier: MIT
> +
> +config DRM_AMDGPU
> +       tristate "AMD GPU"
> +       depends on DRM && PCI && MMU
> +       select FW_LOADER
> +       select DRM_DISPLAY_DP_HELPER
> +       select DRM_DISPLAY_HDMI_HELPER
> +       select DRM_DISPLAY_HELPER
> +       select DRM_KMS_HELPER
> +       select DRM_SCHED
> +       select DRM_TTM
> +       select DRM_TTM_HELPER
> +       select POWER_SUPPLY
> +       select HWMON
> +       select BACKLIGHT_CLASS_DEVICE
> +       select INTERVAL_TREE
> +       select DRM_BUDDY
> +       # amdgpu depends on ACPI_VIDEO when ACPI is enabled, for select to work
> +       # ACPI_VIDEO's dependencies must also be selected.
> +       select INPUT if ACPI
> +       select ACPI_VIDEO if ACPI
> +       # On x86 ACPI_VIDEO also needs ACPI_WMI
> +       select X86_PLATFORM_DEVICES if ACPI && X86
> +       select ACPI_WMI if ACPI && X86
> +       help
> +         Choose this option if you have a recent AMD Radeon graphics card.
> +
> +         If M is selected, the module will be called amdgpu.
> +
>  config DRM_AMDGPU_SI
>         bool "Enable amdgpu support for SI parts"
>         depends on DRM_AMDGPU
> diff --git a/drivers/gpu/drm/radeon/Kconfig b/drivers/gpu/drm/radeon/Kconfig
> index 52819e7f1fca..2267c501f724 100644
> --- a/drivers/gpu/drm/radeon/Kconfig
> +++ b/drivers/gpu/drm/radeon/Kconfig
> @@ -1,4 +1,34 @@
>  # SPDX-License-Identifier: MIT
> +
> +config DRM_RADEON
> +       tristate "ATI Radeon"
> +       depends on DRM && PCI && MMU
> +       depends on AGP || !AGP
> +       select FW_LOADER
> +       select DRM_DISPLAY_DP_HELPER
> +       select DRM_DISPLAY_HELPER
> +       select DRM_KMS_HELPER
> +       select DRM_TTM
> +       select DRM_TTM_HELPER
> +       select SND_HDA_COMPONENT if SND_HDA_CORE
> +       select POWER_SUPPLY
> +       select HWMON
> +       select BACKLIGHT_CLASS_DEVICE
> +       select INTERVAL_TREE
> +       # radeon depends on ACPI_VIDEO when ACPI is enabled, for select to work
> +       # ACPI_VIDEO's dependencies must also be selected.
> +       select INPUT if ACPI
> +       select ACPI_VIDEO if ACPI
> +       # On x86 ACPI_VIDEO also needs ACPI_WMI
> +       select X86_PLATFORM_DEVICES if ACPI && X86
> +       select ACPI_WMI if ACPI && X86
> +       help
> +         Choose this option if you have an ATI Radeon graphics card.  There
> +         are both PCI and AGP versions.  You don't need to choose this to
> +         run the Radeon in plain VGA mode.
> +
> +         If M is selected, the module will be called radeon.
> +
>  config DRM_RADEON_USERPTR
>         bool "Always enable userptr support"
>         depends on DRM_RADEON
> --
> 2.37.3
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v3] drm: Move radeon and amdgpu Kconfig options into their directories
@ 2022-10-27 18:11   ` Alex Deucher
  0 siblings, 0 replies; 3+ messages in thread
From: Alex Deucher @ 2022-10-27 18:11 UTC (permalink / raw)
  To: Andrew Davis
  Cc: David Airlie, Randy Dunlap, linux-kernel, dri-devel,
	Christian König

Applied.  Thanks!

Alex

On Wed, Oct 26, 2022 at 12:31 PM Andrew Davis <afd@ti.com> wrote:
>
> Most Kconfig options to enable a driver are in the Kconfig file
> inside the relevant directory, move these two to the same.
>
> Signed-off-by: Andrew Davis <afd@ti.com>
> Reviewed-by: Christian König <christian.koenig@amd.com>
> ---
>
> Changes from v2:
>  - Rebased on latest
>
> Changes from v1:
>  - Fix whitespace issue pointed out by Randy
>
>  drivers/gpu/drm/Kconfig            | 57 ------------------------------
>  drivers/gpu/drm/amd/amdgpu/Kconfig | 29 +++++++++++++++
>  drivers/gpu/drm/radeon/Kconfig     | 30 ++++++++++++++++
>  3 files changed, 59 insertions(+), 57 deletions(-)
>
> diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig
> index fa986075e8fb..9c2d9495cb3c 100644
> --- a/drivers/gpu/drm/Kconfig
> +++ b/drivers/gpu/drm/Kconfig
> @@ -233,65 +233,8 @@ source "drivers/gpu/drm/i2c/Kconfig"
>
>  source "drivers/gpu/drm/arm/Kconfig"
>
> -config DRM_RADEON
> -       tristate "ATI Radeon"
> -       depends on DRM && PCI && MMU
> -       depends on AGP || !AGP
> -       select FW_LOADER
> -       select DRM_DISPLAY_DP_HELPER
> -       select DRM_DISPLAY_HELPER
> -        select DRM_KMS_HELPER
> -        select DRM_TTM
> -       select DRM_TTM_HELPER
> -       select SND_HDA_COMPONENT if SND_HDA_CORE
> -       select POWER_SUPPLY
> -       select HWMON
> -       select BACKLIGHT_CLASS_DEVICE
> -       select INTERVAL_TREE
> -       # radeon depends on ACPI_VIDEO when ACPI is enabled, for select to work
> -       # ACPI_VIDEO's dependencies must also be selected.
> -       select INPUT if ACPI
> -       select ACPI_VIDEO if ACPI
> -       # On x86 ACPI_VIDEO also needs ACPI_WMI
> -       select X86_PLATFORM_DEVICES if ACPI && X86
> -       select ACPI_WMI if ACPI && X86
> -       help
> -         Choose this option if you have an ATI Radeon graphics card.  There
> -         are both PCI and AGP versions.  You don't need to choose this to
> -         run the Radeon in plain VGA mode.
> -
> -         If M is selected, the module will be called radeon.
> -
>  source "drivers/gpu/drm/radeon/Kconfig"
>
> -config DRM_AMDGPU
> -       tristate "AMD GPU"
> -       depends on DRM && PCI && MMU
> -       select FW_LOADER
> -       select DRM_DISPLAY_DP_HELPER
> -       select DRM_DISPLAY_HDMI_HELPER
> -       select DRM_DISPLAY_HELPER
> -       select DRM_KMS_HELPER
> -       select DRM_SCHED
> -       select DRM_TTM
> -       select DRM_TTM_HELPER
> -       select POWER_SUPPLY
> -       select HWMON
> -       select BACKLIGHT_CLASS_DEVICE
> -       select INTERVAL_TREE
> -       select DRM_BUDDY
> -       # amdgpu depends on ACPI_VIDEO when ACPI is enabled, for select to work
> -       # ACPI_VIDEO's dependencies must also be selected.
> -       select INPUT if ACPI
> -       select ACPI_VIDEO if ACPI
> -       # On x86 ACPI_VIDEO also needs ACPI_WMI
> -       select X86_PLATFORM_DEVICES if ACPI && X86
> -       select ACPI_WMI if ACPI && X86
> -       help
> -         Choose this option if you have a recent AMD Radeon graphics card.
> -
> -         If M is selected, the module will be called amdgpu.
> -
>  source "drivers/gpu/drm/amd/amdgpu/Kconfig"
>
>  source "drivers/gpu/drm/nouveau/Kconfig"
> diff --git a/drivers/gpu/drm/amd/amdgpu/Kconfig b/drivers/gpu/drm/amd/amdgpu/Kconfig
> index 7777d55275de..5fcd510f1abb 100644
> --- a/drivers/gpu/drm/amd/amdgpu/Kconfig
> +++ b/drivers/gpu/drm/amd/amdgpu/Kconfig
> @@ -1,4 +1,33 @@
>  # SPDX-License-Identifier: MIT
> +
> +config DRM_AMDGPU
> +       tristate "AMD GPU"
> +       depends on DRM && PCI && MMU
> +       select FW_LOADER
> +       select DRM_DISPLAY_DP_HELPER
> +       select DRM_DISPLAY_HDMI_HELPER
> +       select DRM_DISPLAY_HELPER
> +       select DRM_KMS_HELPER
> +       select DRM_SCHED
> +       select DRM_TTM
> +       select DRM_TTM_HELPER
> +       select POWER_SUPPLY
> +       select HWMON
> +       select BACKLIGHT_CLASS_DEVICE
> +       select INTERVAL_TREE
> +       select DRM_BUDDY
> +       # amdgpu depends on ACPI_VIDEO when ACPI is enabled, for select to work
> +       # ACPI_VIDEO's dependencies must also be selected.
> +       select INPUT if ACPI
> +       select ACPI_VIDEO if ACPI
> +       # On x86 ACPI_VIDEO also needs ACPI_WMI
> +       select X86_PLATFORM_DEVICES if ACPI && X86
> +       select ACPI_WMI if ACPI && X86
> +       help
> +         Choose this option if you have a recent AMD Radeon graphics card.
> +
> +         If M is selected, the module will be called amdgpu.
> +
>  config DRM_AMDGPU_SI
>         bool "Enable amdgpu support for SI parts"
>         depends on DRM_AMDGPU
> diff --git a/drivers/gpu/drm/radeon/Kconfig b/drivers/gpu/drm/radeon/Kconfig
> index 52819e7f1fca..2267c501f724 100644
> --- a/drivers/gpu/drm/radeon/Kconfig
> +++ b/drivers/gpu/drm/radeon/Kconfig
> @@ -1,4 +1,34 @@
>  # SPDX-License-Identifier: MIT
> +
> +config DRM_RADEON
> +       tristate "ATI Radeon"
> +       depends on DRM && PCI && MMU
> +       depends on AGP || !AGP
> +       select FW_LOADER
> +       select DRM_DISPLAY_DP_HELPER
> +       select DRM_DISPLAY_HELPER
> +       select DRM_KMS_HELPER
> +       select DRM_TTM
> +       select DRM_TTM_HELPER
> +       select SND_HDA_COMPONENT if SND_HDA_CORE
> +       select POWER_SUPPLY
> +       select HWMON
> +       select BACKLIGHT_CLASS_DEVICE
> +       select INTERVAL_TREE
> +       # radeon depends on ACPI_VIDEO when ACPI is enabled, for select to work
> +       # ACPI_VIDEO's dependencies must also be selected.
> +       select INPUT if ACPI
> +       select ACPI_VIDEO if ACPI
> +       # On x86 ACPI_VIDEO also needs ACPI_WMI
> +       select X86_PLATFORM_DEVICES if ACPI && X86
> +       select ACPI_WMI if ACPI && X86
> +       help
> +         Choose this option if you have an ATI Radeon graphics card.  There
> +         are both PCI and AGP versions.  You don't need to choose this to
> +         run the Radeon in plain VGA mode.
> +
> +         If M is selected, the module will be called radeon.
> +
>  config DRM_RADEON_USERPTR
>         bool "Always enable userptr support"
>         depends on DRM_RADEON
> --
> 2.37.3
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-10-27 18:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-26 16:19 [PATCH v3] drm: Move radeon and amdgpu Kconfig options into their directories Andrew Davis
2022-10-27 18:11 ` Alex Deucher
2022-10-27 18:11   ` Alex Deucher

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.