Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751377AbaG3Ts3 (ORCPT ); Wed, 30 Jul 2014 15:48:29 -0400 Received: from mail-ig0-f175.google.com ([209.85.213.175]:65302 "EHLO mail-ig0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750777AbaG3Ts2 (ORCPT ); Wed, 30 Jul 2014 15:48:28 -0400 MIME-Version: 1.0 X-Originating-IP: [84.73.67.144] In-Reply-To: <1406742127-27654-12-git-send-email-benjamin.gaignard@linaro.org> References: <1406742127-27654-1-git-send-email-benjamin.gaignard@linaro.org> <1406742127-27654-12-git-send-email-benjamin.gaignard@linaro.org> Date: Wed, 30 Jul 2014 21:48:27 +0200 X-Google-Sender-Auth: BB90Lu8LNv_h4FDxKaY8YTLY1XE Message-ID: Subject: Re: [PATCH v8 11/11] drm: sti: Add DRM driver itself From: Daniel Vetter To: Benjamin Gaignard Cc: dri-devel , "linaro-mm-sig@lists.linaro.org" , Linux Kernel Mailing List , Dave Airlie Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jul 30, 2014 at 7:42 PM, Benjamin Gaignard wrote: > @@ -87,11 +90,50 @@ static int sti_compositor_bind(struct device *dev, struct device *master, > struct sti_compositor *compo = dev_get_drvdata(dev); > struct drm_device *drm_dev = data; > unsigned int i, crtc = 0, plane = 0; > + struct sti_drm_private *dev_priv = drm_dev->dev_private; > + struct drm_plane *cursor = NULL; > + struct drm_plane *primary = NULL; > + > + dev_priv->compo = compo; > > drm_vblank_init(drm_dev, crtc); This looks strange - you should pass this the total number of crtcs (the same that eventually ends up in dev->mode_config.num_crtc), not 0. And the assignement of cursors to crtcs looks a bit strange on first read-through, but I have no clue about the sti hw. And in any case those pointers really only matter for backwards compat with existing pageflip and cursor ioctls, so doesn't really matter too much. Anyway didn't spot anything else which would need to be upgrade to never kms interfaces, so ack from my side for that. Only looked at that since right now I'm refreshing drm docs in those areas ;-) -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/