Re: [Linux-fbdev-devel] [PATCH 13/13] viafb: via_utility.c, via_utility.h, vt1636.c, vt1636.h

[Alan C. <al...@lx...>]

> +void get_gamma_table(unsigned int *gamma_table)

Still more globals with names asking to cause clashes

> +{
> +	unsigned char color_r[256];
> +	unsigned char color_g[256];
> +	unsigned char color_b[256];
> +	unsigned char sr1a = 0;

and possibly excesive stack usage


I think at this point however the code is in a state where it would be
more productive to merge it and then fix up the remaining smaller details
than keep going round back to VIA to change it internally and return.

So

Acked-by: Alan Cox <al...@re...>  - for the patch series, subject to
further tidyups getting done after it is all merged.

Alan