From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753875AbeBULXr convert rfc822-to-8bit (ORCPT ); Wed, 21 Feb 2018 06:23:47 -0500 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:53086 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752892AbeBULXp (ORCPT ); Wed, 21 Feb 2018 06:23:45 -0500 Date: Wed, 21 Feb 2018 12:23:40 +0100 From: Cornelia Huck To: Thomas Huth Cc: Christian Borntraeger , Farhan Ali , linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org, linux-fbdev@vger.kernel.org, geert@linux-m68k.org, tomi.valkeinen@ti.com, b.zolnierkie@samsung.com Subject: Re: [PATCH v3 1/3] Kconfig : Remove HAS_IOMEM dependency for Graphics support Message-ID: <20180221122340.36f4c1ed.cohuck@redhat.com> In-Reply-To: References: <2beec7645bac82e7bdef4f6df0b9f4fe28696575.1519053920.git.alifm@linux.vnet.ibm.com> <6cae85dc-76d5-1b92-ab13-a5e441eb84a0@linux.vnet.ibm.com> <20180221113217.11bb1c69.cohuck@redhat.com> Organization: Red Hat GmbH MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 21 Feb 2018 12:14:36 +0100 Thomas Huth wrote: > On 21.02.2018 12:09, Christian Borntraeger wrote: > > > > > > On 02/21/2018 11:32 AM, Cornelia Huck wrote: > >> On Wed, 21 Feb 2018 11:22:38 +0100 > >> Christian Borntraeger wrote: > >> > >>> On 02/21/2018 11:05 AM, Christian Borntraeger wrote: > >>>> > >>>> > >>>> On 02/19/2018 05:38 PM, Farhan Ali wrote: > >>>>> > >>>>> > >>>>> On 02/19/2018 11:25 AM, Thomas Huth wrote: > >>>>>> On 19.02.2018 16:47, Farhan Ali wrote: > >>>>>>> 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 > >>>>>>> --- > >>>>>>>   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? > > > > It is a ccw virtio-gpu. But s390 has no IOMEM without CONFIG_PCI, so you cannot > > select VIRTIO_GPU, which needs DRM, which need IOMEM. > > So the 'source "drivers/gpu/drm/Kconfig"' should maybe rather reside > outside of the "if HAS_IOMEM" path? Or does it not compile anymore that way? Alternatively, some of the IOMEM stuff could be moved out from under the pci dependency on s390 (not sure what is needed and what actually makes sense.) This is good enough for now, but it would be nice to get this sorted out later. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cornelia Huck Date: Wed, 21 Feb 2018 11:23:40 +0000 Subject: Re: [PATCH v3 1/3] Kconfig : Remove HAS_IOMEM dependency for Graphics support Message-Id: <20180221122340.36f4c1ed.cohuck@redhat.com> References: <2beec7645bac82e7bdef4f6df0b9f4fe28696575.1519053920.git.alifm@linux.vnet.ibm.com> <6cae85dc-76d5-1b92-ab13-a5e441eb84a0@linux.vnet.ibm.com> <20180221113217.11bb1c69.cohuck@redhat.com> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit List-Archive: List-Post: To: Thomas Huth Cc: Christian Borntraeger , Farhan Ali , linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org, linux-fbdev@vger.kernel.org, geert@linux-m68k.org, tomi.valkeinen@ti.com, b.zolnierkie@samsung.com List-ID: On Wed, 21 Feb 2018 12:14:36 +0100 Thomas Huth wrote: > On 21.02.2018 12:09, Christian Borntraeger wrote: > > > > > > On 02/21/2018 11:32 AM, Cornelia Huck wrote: > >> On Wed, 21 Feb 2018 11:22:38 +0100 > >> Christian Borntraeger wrote: > >> > >>> On 02/21/2018 11:05 AM, Christian Borntraeger wrote: > >>>> > >>>> > >>>> On 02/19/2018 05:38 PM, Farhan Ali wrote: > >>>>> > >>>>> > >>>>> On 02/19/2018 11:25 AM, Thomas Huth wrote: > >>>>>> On 19.02.2018 16:47, Farhan Ali wrote: > >>>>>>> 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 > >>>>>>> --- > >>>>>>>   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? > > > > It is a ccw virtio-gpu. But s390 has no IOMEM without CONFIG_PCI, so you cannot > > select VIRTIO_GPU, which needs DRM, which need IOMEM. > > So the 'source "drivers/gpu/drm/Kconfig"' should maybe rather reside > outside of the "if HAS_IOMEM" path? Or does it not compile anymore that way? Alternatively, some of the IOMEM stuff could be moved out from under the pci dependency on s390 (not sure what is needed and what actually makes sense.) This is good enough for now, but it would be nice to get this sorted out later.