Re: [PATCH 13/15] fbdev: sh-mobile-lcdcfb: Enable driver compilation with COMPILE_TEST

From
Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date
2014-01-20 15:30:21
Also in
linux-arm-kernel, linux-sh
Reply to
Tomi Valkeinen, 2014-01-20
Hi Tomi,

On Monday 20 January 2014 13:21:47 Tomi Valkeinen wrote:
On 2014-01-19 23:01, Laurent Pinchart wrote:
quoted
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

Attachments

Thread (8 messages) 8 messages, 3 authors, last reply 2014-01-20
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help