Hi On Wed, Aug 13, 2014 at 10:46 AM, Geert Uytterhoeven <[email protected]> wrote: > 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>
"pseudo_palette" is "void*", so not at all. Thanks David -- 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.
