This patch fixes the smscufx driver on big endian platforms, by
ensuring the framebuffer is correctly byte-swapped before sending
to the device.
Register and control words were already correctly swapped, so
without this patch the device "works" but with obviously incorrect
RGB values displayed on the output device.
This is implemented as an ifdef so platforms that don't require
byte swapping can use the faster memcpy method.
Signed-off-by: Steve Glendinning <steve.glendinning@xxxxxxxxxxx>
---
drivers/video/smscufx.c | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/drivers/video/smscufx.c b/drivers/video/smscufx.c
index 97bd662..8009400 100644
--- a/drivers/video/smscufx.c
+++ b/drivers/video/smscufx.c
@@ -840,8 +840,16 @@ static void ufx_raw_rect(struct ufx_data *dev, u16 *cmd, int x, int y,
for (line = 0; line < height; line++) {
const int line_offset = dev->info->fix.line_length * (y + line);
const int byte_offset = line_offset + (x * BPP);
- memcpy(&cmd[(24 + (packed_line_len * line)) / 2],
- (char *)dev->info->fix.smem_start + byte_offset, width * BPP);
+ const int cmd_base = (24 + (packed_line_len * line)) / 2;
+ const u16 *source = (u16 *)((char *)dev->info->fix.smem_start + byte_offset);
+ u16 *dest = (u16 *)(&cmd[cmd_base]);
+#ifdef __BIG_ENDIAN
+ int pixel;
+ for (pixel = 0; pixel < width; pixel++)
+ dest[pixel] = cpu_to_le16(source[pixel]);
+#else /* __BIG_ENDIAN */
+ memcpy(dest, source, width * BPP);
+#endif /* __BIG_ENDIAN */
}
}
--
1.7.10.4
--
To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
