Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935813AbcKPVcN (ORCPT ); Wed, 16 Nov 2016 16:32:13 -0500 Received: from pandora.armlinux.org.uk ([78.32.30.218]:36528 "EHLO pandora.armlinux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752278AbcKPVcK (ORCPT ); Wed, 16 Nov 2016 16:32:10 -0500 Date: Wed, 16 Nov 2016 21:31:55 +0000 From: Russell King - ARM Linux To: Daniel Vetter , David Airlie Cc: Brian Starkey , dri-devel , Linux Kernel Mailing List Subject: Re: [GIT PULL] Re: [PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration Message-ID: <20161116213154.GA13694@n2100.armlinux.org.uk> References: <20161024142312.GA1988@e106950-lin.cambridge.arm.com> <1477319279-21726-1-git-send-email-brian.starkey@arm.com> <20161024143627.GT20761@phenom.ffwll.local> <20161024145202.GB1988@e106950-lin.cambridge.arm.com> <20161031085843.GU1041@n2100.armlinux.org.uk> <20161108092552.6qnum32uhndnsdou@phenom.ffwll.local> <20161108105943.GN1041@n2100.armlinux.org.uk> <20161115094631.GA6680@n2100.armlinux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161115094631.GA6680@n2100.armlinux.org.uk> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4075 Lines: 92 Okay, I've queued this into my own for-next branch, along with the now reviewed and tested set of tda998x patches that I sent out for comment and testing. I'm still hopeful that Dave's going to pull the initial patch at some point... please? On Tue, Nov 15, 2016 at 09:46:31AM +0000, Russell King - ARM Linux wrote: > I guess Dave must have missed this as I can't see it in drm-next, so > I'm resending the pull request. > > On Tue, Nov 08, 2016 at 10:59:43AM +0000, Russell King - ARM Linux wrote: > > On Tue, Nov 08, 2016 at 10:25:52AM +0100, Daniel Vetter wrote: > > > Hm, I entirely missed that part of the troubles. Anyway, if you all agree > > > on a patch I certainly won't block it, feel free to merge through suitable > > > trees (or I can smash it into drm-misc if that's wanted). > > > > I think those who are interested in seeing the drm_connector_register() > > call disappear from tda998x only care about that happening, but not how > > it happens. > > > > We have agreement between myself, Brian and Liviu on this approach, and > > I think everyone else is waiting for me to push out the commit so it can > > be used as the basis for their work. I think everyone else is waiting > > for me to push something out which gets us past this log-jam. > > > > I don't understand the connectivity between drm-misc and David's drm > > tree - so I'm going to let you make the decision on whether to merge > > this into drm-misc. I normally send my pull requests for Armada and > > TDA998x changes to David, which means when I send my other TDA998x > > changes, the mali/tda998x commit will be included in that pull > > request too. So I'm wondering whether it would make more sense for > > me to send it to David instead, or whether I need to send my other > > changes through drm-misc instead. I find the whole drm vs drm-misc > > thing rather confusing. > > > > I think we should get this accepted into drm trees before anyone bases > > their work on this commit (which is why I've been holding off during > > the last week, waiting for DRM folk to get back from Santa Fe and > > readjust to the higher atmospheric pressure!) > > > > Anyway, here is my pull request for the mali/hdlcd/tda998x commit which > > I'd normally send to David - I don't mind which tree it goes into as > > long as things work out nicely. > > > > 8<=== > > > > David, > > > > Please incorporate the latest TDA998x I2C driver (drm-tda998x-mali > > branch), which can be found at: > > > > git://git.armlinux.org.uk/~rmk/linux-arm.git drm-tda998x-mali > > > > with SHA1 90731c24d2db7ec04df43ddbcee9605183d05187. > > > > This change removes the call to drm_connector_register() which has been > > blocking the proper de-midlayer conversion of other DRM drivers. > > Unfortunately, hdlcd and mali have intimate dependencies on this change, > > which is why these drivers need to be fixed up in the same commit - they > > can't be separate commits without these drivers breaking. All other > > DRM drivers which make use of tda998x (to my knowledge - armada, tilcdc) > > cope with this change. > > > > This will update the following files: > > > > drivers/gpu/drm/arm/hdlcd_drv.c | 19 +++++++++++-------- > > drivers/gpu/drm/arm/malidp_drv.c | 18 +++++++++++------- > > drivers/gpu/drm/i2c/tda998x_drv.c | 8 -------- > > 3 files changed, 22 insertions(+), 23 deletions(-) > > > > through these changes: > > > > Brian Starkey (1): > > drm/i2c: tda998x: mali-dp: hdlcd: refactor connector registration > > > > Many thanks. > > > > -- > > 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. > > -- > 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. -- 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.