Thread (24 messages) 24 messages, 5 authors, 2018-02-22

Re: [PATCH v3 1/3] Kconfig : Remove HAS_IOMEM dependency for Graphics support

From: Cornelia Huck <cohuck@redhat.com>
Date: 2018-02-21 10:32:23
Also in: linux-s390, lkml

On Wed, 21 Feb 2018 11:22:38 +0100
Christian Borntraeger [off-list ref] wrote:
On 02/21/2018 11:05 AM, Christian Borntraeger wrote:
quoted

On 02/19/2018 05:38 PM, Farhan Ali wrote:  
quoted

On 02/19/2018 11:25 AM, Thomas Huth wrote:  
quoted
On 19.02.2018 16:47, Farhan Ali wrote:  
quoted
The 'commit e25df1205f37 ("[S390] Kconfig: menus with depends on HAS_IOMEM.")'
added the HAS_IOMEM dependecy for "Graphics support". This disabled the
"Graphics support" menu for S390. But if we enable VT layer for S390,
we would also need to enable the dummy console. So let's remove the
HAS_IOMEM dependency.

Move this dependency to sub menu items and console drivers that use
io memory.

Signed-off-by: Farhan Ali <redacted>
---
  drivers/video/Kconfig         | 21 +++++++++++----------
  drivers/video/console/Kconfig |  4 ++--
  2 files changed, 13 insertions(+), 12 deletions(-)
diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
index 3c20af9..8f10915 100644
--- a/drivers/video/Kconfig
+++ b/drivers/video/Kconfig
@@ -3,7 +3,6 @@
  #
    menu "Graphics support"
-    depends on HAS_IOMEM
    config HAVE_FB_ATMEL
      bool
@@ -11,20 +10,22 @@ config HAVE_FB_ATMEL
  config SH_LCD_MIPI_DSI
      bool
  -source "drivers/char/agp/Kconfig"
+if HAS_IOMEM
+    source "drivers/char/agp/Kconfig"
  -source "drivers/gpu/vga/Kconfig"
+    source "drivers/gpu/vga/Kconfig"
  -source "drivers/gpu/host1x/Kconfig"
-source "drivers/gpu/ipu-v3/Kconfig"
+    source "drivers/gpu/host1x/Kconfig"
+    source "drivers/gpu/ipu-v3/Kconfig"
  -source "drivers/gpu/drm/Kconfig"
+    source "drivers/gpu/drm/Kconfig"  

Hmmm, looks like that this makes it impossible to select VIRTIO_GPU - need still more
work.
  
Sorry my fault. I had CONFIG_PCI disabled.
That smells like the s390 HAS_IOMEM stuff needs more work -- I guess
that you want to enable a ccw virtio-gpu device, not a pci one, right?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help