From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [PATCH] drm/i2c: tda998x: don't register the connector Date: Mon, 25 Jul 2016 13:25:04 +0100 Message-ID: <20160725122504.GS1041@n2100.armlinux.org.uk> References: <1469444148-19003-1-git-send-email-brian.starkey@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1469444148-19003-1-git-send-email-brian.starkey@arm.com> Sender: linux-kernel-owner@vger.kernel.org To: Brian Starkey Cc: dri-devel@lists.freedesktop.org, airlied@linux.ie, linux-kernel@vger.kernel.org, Liviu.Dudau@arm.com List-Id: dri-devel@lists.freedesktop.org On Mon, Jul 25, 2016 at 11:55:48AM +0100, Brian Starkey wrote: > The connector shouldn't be registered until the rest of the whole device > is set up, so that consistent state is presented to userspace. > > As drm_dev_register() now registers all of the connectors anyway, > there's no need to explicitly do it in individual drivers so remove > the calls to drm_connector_register()/drm_connector_unregister(). > > This allows componentised drivers to use tda998x without having racy > initialisation. Is there a corresponding patch for armada-drm so that the cubox doesn't regress? Has it already been merged? -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.