Re: [PATCH RFC 8/9] drm/omap: move omapdss & displays under omapdrm

From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH RFC 8/9] drm/omap: move omapdss & displays under omapdrm
Date: Mon, 14 Dec 2015 07:12:56 +0000	[thread overview]
Message-ID: <566E6BF8.40707@ti.com> (raw)
In-Reply-To: <13812119.EOZzmrY4TL@avalon>

[-- Attachment #1: Type: text/plain, Size: 842 bytes --]

On 13/12/15 21:08, Laurent Pinchart wrote:
> Hi Tomi,
> 
> Thank you for the patch.
> 
> On Thursday 10 December 2015 16:25:34 Tomi Valkeinen wrote:
>> Now that omapfb has its own copy of omapdss and display drivers, we can
>> move omapdss and display drivers which omapdrm uses to omapdrm's
>> directory.
>>
>> We also need to change the main drm Makefile so that omapdrm directory
>> is always entered, because omapdss has a file that always needs to be
>> built-in.
> 
> Which file is that ? omapdss-boot-init.c ? I would say "that can't be built as 
> a module' instead of 'that always needs to be built-in' as the later implies 
> (at least for me) that the file always have to be built-in regardless of 
> whether omapdss support is enabled or not.

Yes, ompadss-boot-init.c. I'll change the wording.

 Tomi

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]