diff options
author | Florian Tobias Schandinat <FlorianSchandinat@gmx.de> | 2009-09-22 16:47:10 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-09-23 07:39:51 -0700 |
commit | 0e3ca33ac7aa88ac2f28d4ec99f0bfeaf2a2318d (patch) | |
tree | 1efc7c9649d0ffbf63ec42eac8455ff06c0ea487 /drivers/video/via/hw.c | |
parent | 88017bda96a5fd568a982b01546c8fb1782dda62 (diff) |
viafb: remove duplicated CX700 register init
The current code initializes the register for CX700 chips 2 times due to a
missing break as discovered by Harald Welte.
As CX700 and VX800 have exactly the same register initialization we can
use one for both to avoid duplicated code.
As this is a pure code cleanup no measurable runtime effects are expected.
Signed-off-by: Florian 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: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/video/via/hw.c')
-rw-r--r-- | drivers/video/via/hw.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/video/via/hw.c b/drivers/video/via/hw.c index c8960003f47..64a820c6d49 100644 --- a/drivers/video/via/hw.c +++ b/drivers/video/via/hw.c @@ -2271,11 +2271,8 @@ int viafb_setmode(int vmode_index, int hor_res, int ver_res, int video_bpp, break; case UNICHROME_CX700: - viafb_write_regx(CX700_ModeXregs, NUM_TOTAL_CX700_ModeXregs); - case UNICHROME_VX800: - viafb_write_regx(VX800_ModeXregs, NUM_TOTAL_VX800_ModeXregs); - + viafb_write_regx(CX700_ModeXregs, NUM_TOTAL_CX700_ModeXregs); break; } |