[PATCH 1/4] simplefb: formalize pseudo palette handling
David Herrmann
dh.herrmann at gmail.com
Wed Aug 13 01:50:51 PDT 2014
Hi
On Wed, Aug 13, 2014 at 10:46 AM, Geert Uytterhoeven
<geert at linux-m68k.org> wrote:
> On Wed, Aug 13, 2014 at 9:25 AM, David Herrmann <dh.herrmann at gmail.com> 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>
"pseudo_palette" is "void*", so not at all.
Thanks
David
More information about the linux-arm-kernel
mailing list