From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ryan Mallon Subject: [RFC PATCH] vmwgfx: Fix assignment in vmw_framebuffer_create_handle Date: Fri, 27 Jan 2012 17:25:12 +1100 Message-ID: <4F224348.8050607@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Sender: linux-kernel-owner@vger.kernel.org To: airlied@linux.ie, airlied@redhat.com, thellstrom@vmware.com, jakob@vmware.com Cc: dri-devel@lists.freedesktop.org, "linux-kernel@vger.kernel.org" , joe Perches List-Id: dri-devel@lists.freedesktop.org The assignment of handle in vmw_framebuffer_create_handle doesn't actually do anything useful and is incorrectly assigning an integer value to a pointer argument. It appears that this is a typo and should be dereferencing handle rather than assigning to it directly. Signed-off-by: Ryan Mallon --- diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c index 0af6ebd..b66ef0e 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c @@ -378,7 +378,7 @@ int vmw_framebuffer_create_handle(struct drm_framebuffer *fb, unsigned int *handle) { if (handle) - handle = 0; + *handle = 0; return 0; }