Re: [DRM] Patch for Compositor Drm

2013-05-01 Thread Kristian Høgsberg
On Mon, Apr 22, 2013 at 05:12:58PM +0100, Christopher Michael wrote: > From acb79e4a5921525b35e07e48f7f903e42a08fb7c Mon Sep 17 00:00:00 2001 > From: Chris Michael > Date: Mon, 22 Apr 2013 15:22:48 +0100 > Subject: [PATCH] Fix not checking return value of drmIoctl function call to > map dumb buff

Re: [DRM] Patch for Compositor Drm

2013-04-23 Thread Ander Conselvan de Oliveira
Hi, Thanks for fixing this. Just a small nitpick about the commit message. We keep the summary line below 76 characters because of git log, and we also prefix it with the component to which the change was made. In this particular case, I would go with something like compositor-drm: Check re

Re: [DRM] Patch for Compositor Drm

2013-04-22 Thread Emilio Pozuelo Monfort
Hi, This is obviously correct. Thanks for the fix! Emilio On 04/22/2013 06:12 PM, Christopher Michael wrote: > From acb79e4a5921525b35e07e48f7f903e42a08fb7c Mon Sep 17 00:00:00 2001 > From: Chris Michael > Date: Mon, 22 Apr 2013 15:22:48 +0100 > Subject: [PATCH] Fix not checking return value of

[DRM] Patch for Compositor Drm

2013-04-22 Thread Christopher Michael
From acb79e4a5921525b35e07e48f7f903e42a08fb7c Mon Sep 17 00:00:00 2001 From: Chris Michael Date: Mon, 22 Apr 2013 15:22:48 +0100 Subject: [PATCH] Fix not checking return value of drmIoctl function call to map dumb buffer. in drm_fb_create_dumb, the return value of the drmIoctl function call to