[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.62.0707130855360.15603@pademelon.sonytel.be>
Date: Fri, 13 Jul 2007 08:56:24 +0200 (CEST)
From: Geert Uytterhoeven <Geert.Uytterhoeven@...ycom.com>
To: "Antonino A. Daplas" <adaplas@...il.com>
cc: Andrew Morton <akpm@...ux-foundation.org>,
Arnd Bergmann <arnd@...db.de>,
James Simmons <jsimmons@...radead.org>,
linux-fbdev-devel@...ts.sourceforge.net, cbe-oss-dev@...abs.org,
linux-kernel@...r.kernel.org,
Geoff Levand <geoffrey.levand@...sony.com>
Subject: Re: [patch 2/4] fbdev: Add fb_append_extra_logo()
On Fri, 13 Jul 2007, Antonino A. Daplas wrote:
> On Tue, 2007-07-10 at 14:27 +0200, Geert Uytterhoeven wrote:
> > --- a/drivers/video/fbmem.c
> > +++ b/drivers/video/fbmem.c
> > @@ -318,6 +318,13 @@ static struct logo_data {
> > const struct linux_logo *logo;
> > } fb_logo __read_mostly;
> >
> > +#define FB_LOGO_EX_NUM_MAX 10
> > +static struct logo_data_extra {
> > + const struct linux_logo *logo;
> > + unsigned int n;
> > +} fb_logo_ex[FB_LOGO_EX_NUM_MAX];
> > +static unsigned int fb_logo_ex_num;
> > +
> > static void fb_rotate_logo_ud(const u8 *in, u8 *out, u32 width, u32 height)
> > {
> > u32 size = width * height, i;
> > @@ -411,10 +418,22 @@ static void fb_do_show_logo(struct fb_in
> > }
> > }
> >
> > +#ifdef CONFIG_FB
>
> The #ifdef is redundant.
IIRC, it's needed for the CONFIG_FB=m case.
With kind regards,
Geert Uytterhoeven
Software Architect
Sony Network and Software Technology Center Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium
Phone: +32 (0)2 700 8453
Fax: +32 (0)2 700 8622
E-mail: Geert.Uytterhoeven@...ycom.com
Internet: http://www.sony-europe.com/
Sony Network and Software Technology Center Europe
A division of Sony Service Centre (Europe) N.V.
Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium
VAT BE 0413.825.160 · RPR Brussels
Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
Powered by blists - more mailing lists