[PATCH 3/5] DRM: Armada: Add support for ARGB 32x64 or 64x32 hardware cursors

2013-10-18 Thread Alex Deucher
On Thu, Oct 17, 2013 at 7:58 PM, Rob Clark wrote: > On Mon, Oct 7, 2013 at 8:50 AM, Russell King - ARM Linux > wrote: >> On Mon, Oct 07, 2013 at 03:29:15PM +0300, Siarhei Siamashka wrote: >>> On Mon, 7 Oct 2013 11:32:50 +0100 >>> Russell King - ARM Linux wrote: >>> > However, what you're suggest

[PATCH 3/5] DRM: Armada: Add support for ARGB 32x64 or 64x32 hardware cursors

2013-10-17 Thread Rob Clark
On Mon, Oct 7, 2013 at 8:50 AM, Russell King - ARM Linux wrote: > On Mon, Oct 07, 2013 at 03:29:15PM +0300, Siarhei Siamashka wrote: >> On Mon, 7 Oct 2013 11:32:50 +0100 >> Russell King - ARM Linux wrote: >> > However, what you're suggesting is dangerous. What do you do when you're >> > presente

Re: [PATCH 3/5] DRM: Armada: Add support for ARGB 32x64 or 64x32 hardware cursors

2013-10-07 Thread Russell King - ARM Linux
On Mon, Oct 07, 2013 at 03:29:15PM +0300, Siarhei Siamashka wrote: > On Mon, 7 Oct 2013 11:32:50 +0100 > Russell King - ARM Linux wrote: > > However, what you're suggesting is dangerous. What do you do when you're > > presented with a cursor which is 64x64 ? Do you: > > > > (a) not display it >

Re: [PATCH 3/5] DRM: Armada: Add support for ARGB 32x64 or 64x32 hardware cursors

2013-10-07 Thread Russell King - ARM Linux
On Mon, Oct 07, 2013 at 12:09:22PM +0200, Jean-Francois Moine wrote: > On Mon, 7 Oct 2013 10:40:08 +0100 > Russell King - ARM Linux wrote: > > > > > This patch adds ARGB hardware cursor support to the DRM driver for the > > > > Marvell Armada SoCs. ARGB cursors are supported at either 32x64 or >

Re: [PATCH 3/5] DRM: Armada: Add support for ARGB 32x64 or 64x32 hardware cursors

2013-10-07 Thread Russell King - ARM Linux
On Mon, Oct 07, 2013 at 11:01:41AM +0200, Jean-Francois Moine wrote: > On Sun, 06 Oct 2013 23:09:56 +0100 > Russell King wrote: > > > This patch adds ARGB hardware cursor support to the DRM driver for the > > Marvell Armada SoCs. ARGB cursors are supported at either 32x64 or > > 64x32 resolution

Re: [PATCH 3/5] DRM: Armada: Add support for ARGB 32x64 or 64x32 hardware cursors

2013-10-07 Thread Siarhei Siamashka
On Mon, 7 Oct 2013 11:32:50 +0100 Russell King - ARM Linux wrote: > On Mon, Oct 07, 2013 at 12:09:22PM +0200, Jean-Francois Moine wrote: > > On Mon, 7 Oct 2013 10:40:08 +0100 > > Russell King - ARM Linux wrote: > > > > > > > This patch adds ARGB hardware cursor support to the DRM driver for the

Re: [PATCH 3/5] DRM: Armada: Add support for ARGB 32x64 or 64x32 hardware cursors

2013-10-07 Thread Jean-Francois Moine
On Mon, 7 Oct 2013 10:40:08 +0100 Russell King - ARM Linux wrote: > > > This patch adds ARGB hardware cursor support to the DRM driver for the > > > Marvell Armada SoCs. ARGB cursors are supported at either 32x64 or > > > 64x32 resolutions. > > [snip] > > > > I don't see the interest of s

Re: [PATCH 3/5] DRM: Armada: Add support for ARGB 32x64 or 64x32 hardware cursors

2013-10-07 Thread Jean-Francois Moine
On Sun, 06 Oct 2013 23:09:56 +0100 Russell King wrote: > This patch adds ARGB hardware cursor support to the DRM driver for the > Marvell Armada SoCs. ARGB cursors are supported at either 32x64 or > 64x32 resolutions. [snip] I don't see the interest of such cursors. Actually, most often

[PATCH 3/5] DRM: Armada: Add support for ARGB 32x64 or 64x32 hardware cursors

2013-10-07 Thread Russell King
This patch adds ARGB hardware cursor support to the DRM driver for the Marvell Armada SoCs. ARGB cursors are supported at either 32x64 or 64x32 resolutions. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_510.c |1 + drivers/gpu/drm/armada/armada_crtc.c | 245 +++