diff options
author | Florian Tobias Schandinat <FlorianSchandinat@gmx.de> | 2009-09-22 19:47:24 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-09-23 10:39:52 -0400 |
commit | c4df5489e40e55f2962b9e8100ebc0d4d1374415 (patch) | |
tree | caf6538a909b8578dd45f6dcbd45ed4a7281ef21 /drivers/video/via/dvi.c | |
parent | 68fa92082ffda84adcbae06fdd307fca53469c25 (diff) |
viafb: improve viafb_par
This patch introduces viafb_shared and is the beginning of a smooth
transition to use it.
viafb_shared should contain all general, non-surface specific data that
should be shared along all viafb framebuffers while viafb_par should only
contain things that are specific to each surface or in other words extend
fb_info. This change is intended to clean the dual/multi framebuffer
handling up.
This removes the annoyance that viafbinfo1->par points to a different
structure than viaparinfo1.
As the last change is fundamental it is difficult to ensure that all parts
of the driver do not depend on the previous brokenness but the chance of
regressions is very low.
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/dvi.c')
-rw-r--r-- | drivers/video/via/dvi.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/video/via/dvi.c b/drivers/video/via/dvi.c index d6965447ca69..c5c32b6b6e6c 100644 --- a/drivers/video/via/dvi.c +++ b/drivers/video/via/dvi.c | |||
@@ -160,7 +160,7 @@ int viafb_tmds_trasmitter_identify(void) | |||
160 | 160 | ||
161 | static void tmds_register_write(int index, u8 data) | 161 | static void tmds_register_write(int index, u8 data) |
162 | { | 162 | { |
163 | viaparinfo->i2c_stuff.i2c_port = | 163 | viaparinfo->shared->i2c_stuff.i2c_port = |
164 | viaparinfo->chip_info->tmds_chip_info.i2c_port; | 164 | viaparinfo->chip_info->tmds_chip_info.i2c_port; |
165 | 165 | ||
166 | viafb_i2c_writebyte(viaparinfo->chip_info->tmds_chip_info. | 166 | viafb_i2c_writebyte(viaparinfo->chip_info->tmds_chip_info. |
@@ -172,7 +172,7 @@ static int tmds_register_read(int index) | |||
172 | { | 172 | { |
173 | u8 data; | 173 | u8 data; |
174 | 174 | ||
175 | viaparinfo->i2c_stuff.i2c_port = | 175 | viaparinfo->shared->i2c_stuff.i2c_port = |
176 | viaparinfo->chip_info->tmds_chip_info.i2c_port; | 176 | viaparinfo->chip_info->tmds_chip_info.i2c_port; |
177 | viafb_i2c_readbyte((u8) viaparinfo->chip_info-> | 177 | viafb_i2c_readbyte((u8) viaparinfo->chip_info-> |
178 | tmds_chip_info.tmds_chip_slave_addr, | 178 | tmds_chip_info.tmds_chip_slave_addr, |
@@ -182,7 +182,7 @@ static int tmds_register_read(int index) | |||
182 | 182 | ||
183 | static int tmds_register_read_bytes(int index, u8 *buff, int buff_len) | 183 | static int tmds_register_read_bytes(int index, u8 *buff, int buff_len) |
184 | { | 184 | { |
185 | viaparinfo->i2c_stuff.i2c_port = | 185 | viaparinfo->shared->i2c_stuff.i2c_port = |
186 | viaparinfo->chip_info->tmds_chip_info.i2c_port; | 186 | viaparinfo->chip_info->tmds_chip_info.i2c_port; |
187 | viafb_i2c_readbytes((u8) viaparinfo->chip_info->tmds_chip_info. | 187 | viafb_i2c_readbytes((u8) viaparinfo->chip_info->tmds_chip_info. |
188 | tmds_chip_slave_addr, (u8) index, buff, buff_len); | 188 | tmds_chip_slave_addr, (u8) index, buff, buff_len); |