Re: [PATCH weston] compositor-drm: support RGB565 with pixman renderer

2016-06-22 Thread Daniel Stone
Hi, On 21 June 2016 at 16:28, Tomi Valkeinen wrote: > On 21/06/16 04:32, Daniel Stone wrote: >> 16bpp in 2016? Ouch. :( > > Yeah... Well, it cuts the mem bandwidth usage to half, so it's quite a > nice bonus on low end SoCs. Pushed with mine & Derek's review: c7d88fa..f8da0c2 upstream -> mas

Re: [PATCH weston] compositor-drm: support RGB565 with pixman renderer

2016-06-22 Thread Derek Foreman
On 20/06/16 06:18 AM, Tomi Valkeinen wrote: > From: Tomi Valkeinen > > At the moment only XRGB is supported when using pixman renderer. > This patch adds support also for RGB565. > > Signed-off-by: Tomi Valkeinen Looks fine to me, Reviewed-by: Derek Foreman > --- > src/compositor-drm.c

Re: [PATCH weston] compositor-drm: support RGB565 with pixman renderer

2016-06-20 Thread Tomi Valkeinen
On 21/06/16 04:32, Daniel Stone wrote: > On 20 June 2016 at 21:18, Tomi Valkeinen wrote: >> At the moment only XRGB is supported when using pixman renderer. >> This patch adds support also for RGB565. > > 16bpp in 2016? Ouch. :( Yeah... Well, it cuts the mem bandwidth usage to half, so it's

Re: [PATCH weston] compositor-drm: support RGB565 with pixman renderer

2016-06-20 Thread Daniel Stone
On 20 June 2016 at 21:18, Tomi Valkeinen wrote: > At the moment only XRGB is supported when using pixman renderer. > This patch adds support also for RGB565. 16bpp in 2016? Ouch. :( Reviewed-by: Daniel Stone Cheers, Daniel ___ wayland-devel maili

[PATCH weston] compositor-drm: support RGB565 with pixman renderer

2016-06-20 Thread Tomi Valkeinen
From: Tomi Valkeinen At the moment only XRGB is supported when using pixman renderer. This patch adds support also for RGB565. Signed-off-by: Tomi Valkeinen --- src/compositor-drm.c | 62 ++-- 1 file changed, 55 insertions(+), 7 deletions(-)