• Florian Tobias Schandinat's avatar
    viafb: cleanup viafb_cursor · 5016af53
    Florian Tobias Schandinat authored
    Clean the hardware cursor handling up.
    
    The most notable change is that it no longer buffers the values in
    viacursor but uses the ones in cursor instead as they are guaranteed to be
    always valid.
    
    Furthermore it uses local instead global variables where possible, moves
    the cursor variable in shared as only one hardware cursor is supported and
    returns an error if memory allocation fails.  Last but not least it fixes
    a too small buffer (as u32 has only 4 and not 32 bytes) but this did not
    produce any known problems.
    
    This is mostly a code cleanup, no negative runtime changes are expected.
    Signed-off-by: default avatarFlorian Tobias Schandinat <FlorianSchandinat@gmx.de>
    Cc: Scott Fang <ScottFang@viatech.com.cn>
    Cc: Joseph Chan <JosephChan@via.com.tw>
    Cc: Harald Welte <laforge@gnumonks.org>
    Cc: Jonathan Corbet <corbet@lwn.net>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
    5016af53
global.h 2.27 KB