On Wed, Aug 13, 2014 at 9:25 AM, David Herrmann <[email protected]> wrote:
>> @@ -225,7 +234,7 @@ static int simplefb_probe(struct platform_device *pdev)
>> framebuffer_release(info);
>> return -ENODEV;
>> }
>> - info->pseudo_palette = (void *)(info + 1);
>> + info->pseudo_palette = (void *) par->palette;
>
> I think coding-style is this (i.e., no whitespace):
> info->pseudo_palette = (void*)par->palette;
<casts-are-evil>
Is this cast even needed?
</casts-are-evil>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
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
--
You received this message because you are subscribed to the Google Groups
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
