summaryrefslogtreecommitdiffstats
path: root/drivers/video
diff options
context:
space:
mode:
authorFlorian Tobias Schandinat <FlorianSchandinat@gmx.de>2010-07-23 11:32:25 +0000
committerFlorian Tobias Schandinat <FlorianSchandinat@gmx.de>2010-07-23 14:46:11 +0000
commitaf29a5b1781bf642a29b5e9bb6162dccfcd626df (patch)
tree71620cfea61cefeb502149a6c61e1937b4d31dfc /drivers/video
parentcd5899551c48dff5fe05914fc187932111091953 (diff)
downloadop-kernel-dev-af29a5b1781bf642a29b5e9bb6162dccfcd626df.zip
op-kernel-dev-af29a5b1781bf642a29b5e9bb6162dccfcd626df.tar.gz
viafb: fix accel_flags check_var bug
viafb: fix accel_flags check_var bug In check_var we should check and modify the var given and not the one which is currently active. So this code was obviously wrong. Probably this was doing no harm because all acceleration functions also check whether acceleration is possible. (otherwise I would expect this to lead to a null pointer dereference) Signed-off-by: Florian Tobias Schandinat <FlorianSchandinat@gmx.de> Cc: Joseph Chan <JosephChan@via.com.tw>
Diffstat (limited to 'drivers/video')
-rw-r--r--drivers/video/via/viafbdev.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/video/via/viafbdev.c b/drivers/video/via/viafbdev.c
index 0f56eb2..bdd0e41 100644
--- a/drivers/video/via/viafbdev.c
+++ b/drivers/video/via/viafbdev.c
@@ -216,9 +216,9 @@ static int viafb_check_var(struct fb_var_screeninfo *var,
/* Adjust var according to our driver's own table */
viafb_fill_var_timing_info(var, viafb_refresh, vmode_entry);
- if (info->var.accel_flags & FB_ACCELF_TEXT &&
+ if (var->accel_flags & FB_ACCELF_TEXT &&
!ppar->shared->vdev->engine_mmio)
- info->var.accel_flags = 0;
+ var->accel_flags = 0;
return 0;
}
OpenPOWER on IntegriCloud