Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1556695

Re: [PATCH 2/2] drm: add more MMU dependencies

Path csiph.com!news.mixmin.net!news.unit0.net!news.panservice.it!bofh.it!news.nic.it!robomod
From Daniel Vetter <daniel@ffwll.ch>
Newsgroups linux.kernel
Subject Re: [PATCH 2/2] drm: add more MMU dependencies
Date Wed, 11 Jan 2017 17:30:02 +0100
Message-ID <sYtWG-In-41@gated-at.bofh.it> (permalink)
References <sYria-7uK-5@gated-at.bofh.it> <sYria-7uK-21@gated-at.bofh.it> <sYrLb-7Tv-3@gated-at.bofh.it>
X-Original-To Lucas Stach <l.stach@pengutronix.de>
Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=sender:date:from:to:cc:subject:message-id:mail-followup-to :references:mime-version:content-disposition:in-reply-to:user-agent; bh=SqhVEYOUWruNjV+CxRafXkQ6S2zn2wlTgZt8ksDTqDo=; b=RJyn7L9GphbIBbmo7wvUC1nrIpvpZsDg+xqqu6v42tqPXerVCe0v2ylz1jOYdgp3TG Yq4gJ/haxDgLuKozxsiu2ET3vP70uXEObpY58hF58RmHaP53s+yrHh/dEyKLCR2sOA8H 0zIQDFOAufldPIklR+DQAclF7n1QzH2kZ7s8I=
X-Google-Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :mail-followup-to:references:mime-version:content-disposition :in-reply-to:user-agent; bh=SqhVEYOUWruNjV+CxRafXkQ6S2zn2wlTgZt8ksDTqDo=; b=gxU47leYw8FiVJGFTxa7BiiUVrHxScnktO0CUFUtxgVleFADCd4+K1K2X4NIh+ZXiy fbQtJ0pKeccwzauPGZ1KKGcOS0siOSzOjgcdA6VfQD0k8n8yDDk4dE2zy9pIAj+Xucl/ vDgTtvbTPFIut34EcOC59U6RzPohl6tX9al/QkJeQ0Pfhhu/6OozSEway9vQQ74jf2mb 6LZGMwAnYas9FpTGjoq+27i/Z6jY96iIfWeF1IXnKrcYF4dEAHb2ILPy06Gc684/vryp YWVAi5gyzs6/2PQlIoAc6ms6/dqzH4Utna3qV+OO0PqK7jL1jt9ZpLYR0EV+mENdSye2 OVjQ==
X-Gm-Message-State AIkVDXJOfxdtdKJWahnzvhxJg9Et2/faXaHQ9GxnvhAZzSs8HlGLn+gBFHuf4QYguPZXdA==
X-Received by 10.28.99.69 with SMTP id x66mr3717057wmb.91.1484151926685; Wed, 11 Jan 2017 08:25:26 -0800 (PST)
Mail-Followup-To Lucas Stach <l.stach@pengutronix.de>, Arnd Bergmann <arnd@arndb.de>, Rob Clark <robdclark@gmail.com>, David Airlie <airlied@linux.ie>, linux-arm-msm@vger.kernel.org, etnaviv@lists.freedesktop.org, Christian Gmeiner <christian.gmeiner@gmail.com>, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Russell King <rmk+kernel@armlinux.org.uk>, Benjamin Gaignard <benjamin.gaignard@linaro.org>, freedreno@lists.freedesktop.org
MIME-Version 1.0
Content-Type text/plain; charset=us-ascii
Content-Disposition inline
X-Operating-System Linux phenom 4.8.0-1-amd64
User-Agent NeoMutt/20161126 (1.7.1)
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 73
Organization linux.* mail to news gateway
X-Original-Cc Arnd Bergmann <arnd@arndb.de>, Daniel Vetter <daniel.vetter@ffwll.ch>, Rob Clark <robdclark@gmail.com>, David Airlie <airlied@linux.ie>, linux-arm-msm@vger.kernel.org, etnaviv@lists.freedesktop.org, Christian Gmeiner <christian.gmeiner@gmail.com>, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Russell King <rmk+kernel@armlinux.org.uk>, Benjamin Gaignard <benjamin.gaignard@linaro.org>, freedreno@lists.freedesktop.org
X-Original-Date Wed, 11 Jan 2017 17:25:23 +0100
X-Original-Message-ID <20170111162523.3akgmd4vfury6ua5@phenom.ffwll.local>
X-Original-References <20170111133357.3664191-1-arnd@arndb.de> <20170111133357.3664191-2-arnd@arndb.de> <1484143359.30810.12.camel@pengutronix.de>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1556695

Show key headers only | View raw


On Wed, Jan 11, 2017 at 03:02:39PM +0100, Lucas Stach wrote:
> Am Mittwoch, den 11.01.2017, 14:33 +0100 schrieb Arnd Bergmann:
> > Many DRM drivers only work with an MMU, and after the patch to enable
> > core DRM support without MMU, we already had one fixup for many of them.
> > The etnaviv, armada and msm drivers were missed and have the same problem:
> > 
> > warning: (DRM_ETNAVIV) selects IOMMU_SUPPORT which has unmet direct dependencies (MMU)
> > warning: (DRM_I915 && DRM_MSM && DRM_ETNAVIV) selects SHMEM which has unmet direct dependencies (MMU)
> > drivers/gpu/drm/armada/armada_gem.o: In function `armada_gem_vm_fault':
> > armada_gem.c:(.text.armada_gem_vm_fault+0x14): undefined reference to `vm_insert_pfn'
> > arch/arm/mm/dma-mapping.c: In function '__iommu_alloc_remap':
> > arch/arm/mm/dma-mapping.c:1390:4: error: 'VM_ARM_DMA_CONSISTENT' undeclared (first use in this function)
> > arch/arm/mm/dma-mapping.c:1456:31: error: 'atomic_pool' undeclared (first use in this function); did you mean 'atomic_xor'?
> > 
> > Fixes: 011cda589938 ("drm: fix compilations issues introduced by "drm: allow to use mmuless SoC"")
> > Fixes: 62a0d98a188c ("drm: allow to use mmuless SoC")
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> 
> Acked-by: Lucas Stach <l.stach@pengutronix.de>

Applied to drm-misc, thanks.
-Daniel

> 
> > ---
> >  drivers/gpu/drm/armada/Kconfig  | 2 +-
> >  drivers/gpu/drm/etnaviv/Kconfig | 1 +
> >  drivers/gpu/drm/msm/Kconfig     | 1 +
> >  3 files changed, 3 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/gpu/drm/armada/Kconfig b/drivers/gpu/drm/armada/Kconfig
> > index 15f3ecfb16f1..eafaeeb7b5b1 100644
> > --- a/drivers/gpu/drm/armada/Kconfig
> > +++ b/drivers/gpu/drm/armada/Kconfig
> > @@ -1,6 +1,6 @@
> >  config DRM_ARMADA
> >  	tristate "DRM support for Marvell Armada SoCs"
> > -	depends on DRM && HAVE_CLK && ARM
> > +	depends on DRM && HAVE_CLK && ARM && MMU
> >  	select DRM_KMS_HELPER
> >  	help
> >  	  Support the "LCD" controllers found on the Marvell Armada 510
> > diff --git a/drivers/gpu/drm/etnaviv/Kconfig b/drivers/gpu/drm/etnaviv/Kconfig
> > index 2cde7a5442fb..656c061b439d 100644
> > --- a/drivers/gpu/drm/etnaviv/Kconfig
> > +++ b/drivers/gpu/drm/etnaviv/Kconfig
> > @@ -3,6 +3,7 @@ config DRM_ETNAVIV
> >  	tristate "ETNAVIV (DRM support for Vivante GPU IP cores)"
> >  	depends on DRM
> >  	depends on ARCH_MXC || ARCH_DOVE
> > +	depends on MMU
> >  	select SHMEM
> >  	select TMPFS
> >  	select IOMMU_API
> > diff --git a/drivers/gpu/drm/msm/Kconfig b/drivers/gpu/drm/msm/Kconfig
> > index d96b2b6898a3..7f78da695dff 100644
> > --- a/drivers/gpu/drm/msm/Kconfig
> > +++ b/drivers/gpu/drm/msm/Kconfig
> > @@ -4,6 +4,7 @@ config DRM_MSM
> >  	depends on DRM
> >  	depends on ARCH_QCOM || (ARM && COMPILE_TEST)
> >  	depends on OF && COMMON_CLK
> > +	depends on MMU
> >  	select REGULATOR
> >  	select DRM_KMS_HELPER
> >  	select DRM_PANEL
> 
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 1/2] drm: fix drm_vm for NOMMU builds Arnd Bergmann <arnd@arndb.de> - 2017-01-11 14:40 +0100
  [PATCH 2/2] drm: add more MMU dependencies Arnd Bergmann <arnd@arndb.de> - 2017-01-11 14:40 +0100
    Re: [PATCH 2/2] drm: add more MMU dependencies Lucas Stach <l.stach@pengutronix.de> - 2017-01-11 15:10 +0100
      Re: [PATCH 2/2] drm: add more MMU dependencies Daniel Vetter <daniel@ffwll.ch> - 2017-01-11 17:30 +0100
  Re: [PATCH 1/2] drm: fix drm_vm for NOMMU builds Daniel Vetter <daniel@ffwll.ch> - 2017-01-11 17:30 +0100
    Re: [PATCH 1/2] drm: fix drm_vm for NOMMU builds Arnd Bergmann <arnd@arndb.de> - 2017-01-11 17:40 +0100
      Re: [PATCH 1/2] drm: fix drm_vm for NOMMU builds Daniel Vetter <daniel@ffwll.ch> - 2017-01-11 17:40 +0100
        Re: [PATCH 1/2] drm: fix drm_vm for NOMMU builds Arnd Bergmann <arnd@arndb.de> - 2017-01-11 18:10 +0100
          Re: [PATCH 1/2] drm: fix drm_vm for NOMMU builds Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2017-01-11 18:20 +0100

csiph-web