Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752467AbaG3VEK (ORCPT ); Wed, 30 Jul 2014 17:04:10 -0400 Received: from mail-ie0-f170.google.com ([209.85.223.170]:34755 "EHLO mail-ie0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751230AbaG3VEJ (ORCPT ); Wed, 30 Jul 2014 17:04:09 -0400 MIME-Version: 1.0 In-Reply-To: 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 17:04:07 -0400 Message-ID: Subject: Re: [PATCH v8 11/11] drm: sti: Add DRM driver itself From: Rob Clark To: Daniel Vetter Cc: Benjamin Gaignard , "linaro-mm-sig@lists.linaro.org" , Linux Kernel Mailing List , dri-devel 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 3:48 PM, Daniel Vetter wrote: > 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 hmm, Benjamin probably should try modetest w/ -v arg.. it does look a bit like something is missing here.. BR, -R > 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 > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel -- 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/