From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Jon Medhurst (Tixy)" Date: Fri, 27 Sep 2013 10:43:02 +0000 Subject: Re: [PATCH v4 1/2] video: ARM CLCD: Add DT support Message-Id: <1380278582.3433.10.camel@linaro1.home> List-Id: References: <1379686800-27269-1-git-send-email-pawel.moll@arm.com> In-Reply-To: <1379686800-27269-1-git-send-email-pawel.moll@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org On Fri, 2013-09-20 at 15:19 +0100, Pawel Moll wrote: > This patch adds basic DT bindings for the PL11x CLCD cells > and make their fbdev driver use them. > > Signed-off-by: Pawel Moll > --- [...] > diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig > index 84b685f..b7d8c23 100644 > --- a/drivers/video/Kconfig > +++ b/drivers/video/Kconfig > @@ -316,6 +316,7 @@ config FB_ARMCLCD > select FB_CFB_FILLRECT > select FB_CFB_COPYAREA > select FB_CFB_IMAGEBLIT To avoid undefined reference to of_get_fb_videomode I needed to also add: select FB_MODE_HELPERS > + select VIDEOMODE_HELPERS if OF > help > This framebuffer device driver is for the ARM PrimeCell PL110 > Colour LCD controller. ARM PrimeCells provide the building [...] With the config change, for both this patch and patch 2/2 Tested-by: Jon Medhurst (I tested both DT and ATAGs booting on Versatile Express with CA9x4 CoreTile, and DT boot with CA15_A7 a.k.a TC2 :-) -- Tixy