[GIT PULL] Re: [PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-11-16 Thread Russell King - ARM Linux
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 +, Russell King

[GIT PULL] Re: [PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-11-15 Thread Russell King - ARM Linux
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 +, 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. Anywa

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-11-08 Thread Daniel Vetter
On Tue, Nov 08, 2016 at 10:59:43AM +, 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 > >

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-11-08 Thread Russell King - ARM Linux
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

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-11-08 Thread Daniel Vetter
On Mon, Oct 31, 2016 at 08:58:43AM +, Russell King - ARM Linux wrote: > On Mon, Oct 24, 2016 at 10:24:42PM +0200, Daniel Vetter wrote: > > On Mon, Oct 24, 2016 at 4:52 PM, Brian Starkey > > wrote: > > >> > > >>> diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c > > >>> b/drivers/gpu/drm/i2c/tda9

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-10-31 Thread Russell King - ARM Linux
On Mon, Oct 31, 2016 at 09:00:08AM +, Russell King - ARM Linux wrote: > I need the patch against v4.8. I'm happy to pick it up and add it > to my drm-tda998x-devel branch, which you can then merge into > drm-misc if you wish. ... or if Brian wants to send a git pull request to us with the pat

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-10-31 Thread Russell King - ARM Linux
On Tue, Oct 25, 2016 at 12:19:19PM +0200, Daniel Vetter wrote: > On Tue, Oct 25, 2016 at 10:52:33AM +0100, Brian Starkey wrote: > > Hi Daniel, > > > > On Mon, Oct 24, 2016 at 10:24:42PM +0200, Daniel Vetter wrote: > > > On Mon, Oct 24, 2016 at 4:52 PM, Brian Starkey > > > wrote: > > > > > > > >

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-10-31 Thread Russell King - ARM Linux
On Mon, Oct 24, 2016 at 10:24:42PM +0200, Daniel Vetter wrote: > On Mon, Oct 24, 2016 at 4:52 PM, Brian Starkey > wrote: > >> > >>> diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c > >>> b/drivers/gpu/drm/i2c/tda998x_drv.c > >>> index f4315bc..6e6fca2 100644 > >>> --- a/drivers/gpu/drm/i2c/tda998x_

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-10-25 Thread Daniel Vetter
On Tue, Oct 25, 2016 at 10:52:33AM +0100, Brian Starkey wrote: > Hi Daniel, > > On Mon, Oct 24, 2016 at 10:24:42PM +0200, Daniel Vetter wrote: > > On Mon, Oct 24, 2016 at 4:52 PM, Brian Starkey > > wrote: > > > > > > > > > diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c > > > > > b/drivers/gpu/d

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-10-25 Thread Brian Starkey
On Tue, Oct 25, 2016 at 12:19:19PM +0200, Daniel Vetter wrote: >On Tue, Oct 25, 2016 at 10:52:33AM +0100, Brian Starkey wrote: > >Ah, makes sense. Should I apply this to drm-misc so it's in a shared tree? Honestly, I don't know. I didn't entirely follow what it was Russell wanted in terms of getti

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-10-25 Thread Brian Starkey
Hi Daniel, On Mon, Oct 24, 2016 at 10:24:42PM +0200, Daniel Vetter wrote: >On Mon, Oct 24, 2016 at 4:52 PM, Brian Starkey >wrote: >>> diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c index f4315bc..6e6fca2 100644 --- a/drivers/gpu/drm/i2c/tda

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-10-24 Thread Daniel Vetter
On Mon, Oct 24, 2016 at 4:52 PM, Brian Starkey wrote: >> >>> diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c >>> b/drivers/gpu/drm/i2c/tda998x_drv.c >>> index f4315bc..6e6fca2 100644 >>> --- a/drivers/gpu/drm/i2c/tda998x_drv.c >>> +++ b/drivers/gpu/drm/i2c/tda998x_drv.c >>> @@ -1369,7 +1369,6 @@ co

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-10-24 Thread Daniel Vetter
On Mon, Oct 24, 2016 at 03:27:59PM +0100, Brian Starkey wrote: > Connectors shouldn't be registered until the rest of the whole device > is set up, so that consistent state is presented to userspace. > > As such, remove the calls to drm_connector_register() and > drm_connector_unregister() from td

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-10-24 Thread Brian Starkey
Hi Daniel, On Mon, Oct 24, 2016 at 04:36:27PM +0200, Daniel Vetter wrote: >On Mon, Oct 24, 2016 at 03:27:59PM +0100, Brian Starkey wrote: >> Connectors shouldn't be registered until the rest of the whole device >> is set up, so that consistent state is presented to userspace. >> >> As such, remove

[PATCH] drm: tda998x: mali-dp: hdlcd: refactor connector registration

2016-10-24 Thread Brian Starkey
Connectors shouldn't be registered until the rest of the whole device is set up, so that consistent state is presented to userspace. As such, remove the calls to drm_connector_register() and drm_connector_unregister() from tda998x, as these are now handled by drm_dev_(un)register() itself. To wor