linux-kernel - Re: [Linux-fbdev-devel] [PATCH 1/1 try#2] [VIDEO/FRAMEBUFFER]: add BF52x EZkit Display driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Pine.LNX.4.64.0803061906210.20407@anakin>
Date:	Thu, 6 Mar 2008 19:07:34 +0100 (CET)
From:	Geert Uytterhoeven <geert@...ux-m68k.org>
To:	linux-fbdev-devel@...ts.sourceforge.net
cc:	Andrew Morton <akpm@...ux-foundation.org>, bryan.wu@...log.com,
	Randy Dunlap <randy.dunlap@...cle.com>, adaplas@...il.com,
	Michael Hennerich <michael.hennerich@...log.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [Linux-fbdev-devel] [PATCH 1/1 try#2] [VIDEO/FRAMEBUFFER]: add
 BF52x EZkit Display driver

On Thu, 6 Mar 2008, Bryan Wu wrote:
> On Wed, Mar 5, 2008 at 11:02 PM, Andrew Morton
> <akpm@...ux-foundation.org> wrote:
> > On Thu, 31 Jan 2008 00:49:43 +0800 Bryan Wu <bryan.wu@...log.com> wrote:
> >  > +     fbinfo =
> >  > +         framebuffer_alloc(sizeof(struct bfin_t350mcqbfb_info), &pdev->dev);
> >  > +     if (!fbinfo) {
> >  > +             ret = -ENOMEM;
> >  > +             goto out2;
> >  > +     }

    [...]

> >  > +     fbinfo->pseudo_palette = kmalloc(sizeof(u32) * 16, GFP_KERNEL);
> >  > +     if (!fbinfo->pseudo_palette) {
> >  > +             printk(KERN_ERR DRIVER_NAME
> >  > +                    "Fail to allocate pseudo_palette\n");
> >  > +
> >  > +             ret = -ENOMEM;
> >  > +             goto out4;
> >  > +     }
> >  > +
> >  > +     memset(fbinfo->pseudo_palette, 0, sizeof(u32) * 16);
> >
> >  You just invented kzalloc!
> >
> 
> Right

Alternatively, you could embed the pseudo palette inside struct
bfin_t350mcqbfb_info, so it's allocated by the framebuffer_alloc() call
above.

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/