From mboxrd@z Thu Jan 1 00:00:00 1970 From: laurent.pinchart@ideasonboard.com (Laurent Pinchart) Date: Mon, 20 Jan 2014 16:30:21 +0100 Subject: [PATCH 13/15] fbdev: sh-mobile-lcdcfb: Enable driver compilation with COMPILE_TEST In-Reply-To: <52DD06CB.8050606@ti.com> References: <1385515117-23664-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <22423852.GiH1NBjiii@avalon> <52DD06CB.8050606@ti.com> Message-ID: <5189278.VEETF0BgjA@avalon> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Tomi, On Monday 20 January 2014 13:21:47 Tomi Valkeinen wrote: > On 2014-01-19 23:01, Laurent Pinchart wrote: > > Hi Tomi, > > > > The lcdc driver can be compiled without meram support. This is handled by > > conditional compilation in include/video/sh_mobile_meram.h that defines > > the > > meram functions as stubs when meram support isn't selected. > > > > The problem comes from the combination of FB_SH_MOBILE_MERAM=m and > > FB_SH_MOBILE_LCDC=y. The former makes the meram function non-stubs, while > > the later makes the LCDC driver fail to link, as meram support is then > > compiled as a module. > > > > How do you usually handle this ? > > I guess the easiest option is to make FB_SH_MOBILE_MERAM a bool, instead > of tristate. That's easy, but it would prevent meram support from being compiled as a module when lcdc support is compiled as a module as well. -- Regards, Laurent Pinchart -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 490 bytes Desc: This is a digitally signed message part. URL: