[PATCH] drm: hdlcd: Work properly in big-endian mode

2016-12-08 Thread Daniel Vetter
On Wed, Dec 07, 2016 at 07:16:30PM -0500, Ilia Mirkin wrote: > On Wed, Dec 7, 2016 at 11:42 AM, Robin Murphy wrote: > > By comparison, the same use-cases (fbcon and fb-test) under the same > > big-endian kernel do *not* show the same problem with nouveau driving a > > PCIe 7600GT card, which led m

[PATCH] drm: hdlcd: Work properly in big-endian mode

2016-12-08 Thread Daniel Vetter
On Wed, Dec 07, 2016 at 04:50:31PM +, Robin Murphy wrote: > On 07/12/16 16:42, Robin Murphy wrote: > > On 07/12/16 15:57, Daniel Vetter wrote: > >> On Wed, Dec 07, 2016 at 03:31:40PM +, Robin Murphy wrote: > >>> Under a big-endian kernel, colours on the framebuffer all come out a > >>> deli

[PATCH] drm: hdlcd: Work properly in big-endian mode

2016-12-07 Thread Ilia Mirkin
On Wed, Dec 7, 2016 at 11:42 AM, Robin Murphy wrote: > By comparison, the same use-cases (fbcon and fb-test) under the same > big-endian kernel do *not* show the same problem with nouveau driving a > PCIe 7600GT card, which led me to believe it was HDLCD-specific. Just to randomly insert info her

[PATCH] drm: hdlcd: Work properly in big-endian mode

2016-12-07 Thread Ville Syrjälä
On Wed, Dec 07, 2016 at 04:57:14PM +0100, Daniel Vetter wrote: > On Wed, Dec 07, 2016 at 03:31:40PM +, Robin Murphy wrote: > > Under a big-endian kernel, colours on the framebuffer all come out a > > delightful shade of wrong, since we fail to take the reversed byte > > order into account. Fort

[PATCH] drm: hdlcd: Work properly in big-endian mode

2016-12-07 Thread Daniel Vetter
On Wed, Dec 07, 2016 at 03:31:40PM +, Robin Murphy wrote: > Under a big-endian kernel, colours on the framebuffer all come out a > delightful shade of wrong, since we fail to take the reversed byte > order into account. Fortunately, the HDLCD has a control bit to make it > automatically byteswa

[PATCH] drm: hdlcd: Work properly in big-endian mode

2016-12-07 Thread Robin Murphy
On 07/12/16 16:42, Robin Murphy wrote: > On 07/12/16 15:57, Daniel Vetter wrote: >> On Wed, Dec 07, 2016 at 03:31:40PM +, Robin Murphy wrote: >>> Under a big-endian kernel, colours on the framebuffer all come out a >>> delightful shade of wrong, since we fail to take the reversed byte >>> order

[PATCH] drm: hdlcd: Work properly in big-endian mode

2016-12-07 Thread Robin Murphy
On 07/12/16 15:57, Daniel Vetter wrote: > On Wed, Dec 07, 2016 at 03:31:40PM +, Robin Murphy wrote: >> Under a big-endian kernel, colours on the framebuffer all come out a >> delightful shade of wrong, since we fail to take the reversed byte >> order into account. Fortunately, the HDLCD has a c

[PATCH] drm: hdlcd: Work properly in big-endian mode

2016-12-07 Thread liviu.du...@arm.com
On Wed, Dec 07, 2016 at 03:31:40PM +, Robin Murphy wrote: > Under a big-endian kernel, colours on the framebuffer all come out a > delightful shade of wrong, So you are saying that wrong is only a 1 bit value? > since we fail to take the reversed byte > order into account. Fortunately, the

[PATCH] drm: hdlcd: Work properly in big-endian mode

2016-12-07 Thread Robin Murphy
Under a big-endian kernel, colours on the framebuffer all come out a delightful shade of wrong, since we fail to take the reversed byte order into account. Fortunately, the HDLCD has a control bit to make it automatically byteswap big-endian data; let's use it as appropriate. Signed-off-by: Robin