From mboxrd@z Thu Jan  1 00:00:00 1970
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH] drm/rcar-du: Add run-time dependencies
Date: Mon, 26 May 2014 14:50:57 +0200
Message-ID: <18685192.JDnqQx3NDs@avalon>
References: <20140526130135.545c40f3@endymion.delvare>
 <5915575.HN1lKbocVo@avalon> <20140526142130.43b12cc9@endymion.delvare>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <dri-devel-bounces@lists.freedesktop.org>
Received: from perceval.ideasonboard.com (perceval.ideasonboard.com
 [95.142.166.194])
 by gabe.freedesktop.org (Postfix) with ESMTP id 8DD866E2E4
 for <dri-devel@lists.freedesktop.org>; Mon, 26 May 2014 05:50:39 -0700 (PDT)
In-Reply-To: <20140526142130.43b12cc9@endymion.delvare>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/dri-devel>,
 <mailto:dri-devel-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/dri-devel>
List-Post: <mailto:dri-devel@lists.freedesktop.org>
List-Help: <mailto:dri-devel-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/dri-devel>,
 <mailto:dri-devel-request@lists.freedesktop.org?subject=subscribe>
Errors-To: dri-devel-bounces@lists.freedesktop.org
Sender: "dri-devel" <dri-devel-bounces@lists.freedesktop.org>
To: Jean Delvare <jdelvare@suse.de>
Cc: dri-devel@lists.freedesktop.org
List-Id: dri-devel@lists.freedesktop.org

Hi Jean,

On Monday 26 May 2014 14:21:30 Jean Delvare wrote:
> On Mon, 26 May 2014 13:36:11 +0200, Laurent Pinchart wrote:
> > On Monday 26 May 2014 13:01:35 Jean Delvare wrote:
> > > The Renesas R-Car Display Unit driver is only useful on shmobile
> > > unless build testing. The LVDS output is useful on an even more
> > > reduced hardware set.
> > > 
> > > Signed-off-by: Jean Delvare <jdelvare@suse.de>
> > > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> > > Cc: David Airlie <airlied@linux.ie>
> > > ---
> > > 
> > >  drivers/gpu/drm/rcar-du/Kconfig |    2 ++
> > >  1 file changed, 2 insertions(+)
> > > 
> > > --- linux-3.15-rc6.orig/drivers/gpu/drm/rcar-du/Kconfig	2014-03-31
> > > 05:40:15.000000000 +0200 +++
> > > linux-3.15-rc6/drivers/gpu/drm/rcar-du/Kconfig	2014-05-23
> > > 18:03:16.440036942 +0200 @@ -1,6 +1,7 @@
> > >  config DRM_RCAR_DU
> > >  	tristate "DRM Support for R-Car Display Unit"
> > >  	depends on DRM && ARM
> > 
> > Have you checked whether ARM is needed for compilation ? If not you could
> > drop it.
> 
> It does depend on ARM indirectly, because it selects DRM_GEM_CMA_HELPER
> which builds code that calls dma_alloc_writecombine(), which in turn is
> only defined on arm (and avr32.) That's why I left the build-time
> dependency on ARM.

In that case,

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> > > +	depends on ARCH_SHMOBILE || COMPILE_TEST
> > >  	select DRM_KMS_HELPER
> > >  	select DRM_KMS_CMA_HELPER
> > >  	select DRM_GEM_CMA_HELPER
> > > @@ -12,6 +13,7 @@ config DRM_RCAR_DU
> > >  config DRM_RCAR_LVDS
> > >  	bool "R-Car DU LVDS Encoder Support"
> > >  	depends on DRM_RCAR_DU
> > > +	depends on ARCH_R8A7790 || ARCH_R8A7791 || COMPILE_TEST
> > >  	help
> > >  	  Enable support the R-Car Display Unit embedded LVDS encoders
> > >  	  (currently only on R8A7790).

-- 
Regards,

Laurent Pinchart