Re: [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
[off-list ref] wrote:

quoted

On Thu, 31 Jan 2008 00:49:43 +0800 Bryan Wu [off-list ref] wrote:
 > +     fbinfo =
 > +         framebuffer_alloc(sizeof(struct bfin_t350mcqbfb_info), &pdev->dev);
 > +     if (!fbinfo) {
 > +             ret = -ENOMEM;
 > +             goto out2;
 > +     }

    [...]

quoted

 > +     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@linux-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

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/