lkml.org 
[lkml]   [2008]   [Jun]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 13/13] viafb: via_utility.c, via_utility.h, vt1636.c, vt1636.h
> +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 <alan@redhat.com> - for the patch series, subject to
further tidyups getting done after it is all merged.

Alan


\
 
 \ /
  Last update: 2008-06-30 12:05    [W:0.100 / U:0.752 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site