[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