diff options
author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2016-08-25 13:44:12 -0400 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2016-09-27 04:43:24 -0400 |
commit | 59921b239056fb6389a865083284e00ce0518db6 (patch) | |
tree | 485d9c910c5a7217bf6ddd41d74063e5b7d4bc5e | |
parent | d8ed9e87e6016433b8d178728a5391ea68101bca (diff) |
matroxfb: fix size of memcpy
hw->DACreg has a size of 80 bytes and MGADACbpp32 has 21. So when
memcpy copies MGADACbpp32 to hw->DACreg it copies 80 bytes but
only 21 bytes are valid.
Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
-rw-r--r-- | drivers/video/fbdev/matrox/matroxfb_Ti3026.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/video/fbdev/matrox/matroxfb_Ti3026.c b/drivers/video/fbdev/matrox/matroxfb_Ti3026.c index 195ad7cac1ba..68fa037d8cbc 100644 --- a/drivers/video/fbdev/matrox/matroxfb_Ti3026.c +++ b/drivers/video/fbdev/matrox/matroxfb_Ti3026.c | |||
@@ -372,7 +372,7 @@ static int Ti3026_init(struct matrox_fb_info *minfo, struct my_timming *m) | |||
372 | 372 | ||
373 | DBG(__func__) | 373 | DBG(__func__) |
374 | 374 | ||
375 | memcpy(hw->DACreg, MGADACbpp32, sizeof(hw->DACreg)); | 375 | memcpy(hw->DACreg, MGADACbpp32, sizeof(MGADACbpp32)); |
376 | switch (minfo->fbcon.var.bits_per_pixel) { | 376 | switch (minfo->fbcon.var.bits_per_pixel) { |
377 | case 4: hw->DACreg[POS3026_XLATCHCTRL] = TVP3026_XLATCHCTRL_16_1; /* or _8_1, they are same */ | 377 | case 4: hw->DACreg[POS3026_XLATCHCTRL] = TVP3026_XLATCHCTRL_16_1; /* or _8_1, they are same */ |
378 | hw->DACreg[POS3026_XTRUECOLORCTRL] = TVP3026_XTRUECOLORCTRL_PSEUDOCOLOR; | 378 | hw->DACreg[POS3026_XTRUECOLORCTRL] = TVP3026_XTRUECOLORCTRL_PSEUDOCOLOR; |