[GIT PULL FOR v3.20] tda998x enhancement

[GIT PULL FOR v3.20] tda998x enhancement

[Laurent Pinchart]

Hi Dave,

The following changes since commit adc31849b27fefeca6c225d3895143a2ec6970fa:

  Merge tag 'drm-intel-next-2014-12-19' of git://anongit.freedesktop.org/drm-
intel into drm-next (2015-01-10 08:46:24 +1000)

are available in the git repository at:

  git://linuxtv.org/pinchartl/fbdev.git drm/next/tda998x

for you to fetch changes up to b54a19545ac06b9dfa2e8a77df4b229e96ab04db:

  drm: tda998x: Use drm_do_get_edid() (2015-01-16 17:38:21 +0200)

----------------------------------------------------------------
Laurent Pinchart (1):
      drm: tda998x: Use drm_do_get_edid()

 drivers/gpu/drm/i2c/tda998x_drv.c | 86 ++++++++------------------------------
 1 file changed, 18 insertions(+), 68 deletions(-)

-- 
Regards,

Laurent Pinchart

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [GIT PULL FOR v3.20] tda998x enhancement

[Russell King - ARM Linux]

On Fri, Jan 16, 2015 at 05:44:52PM +0200, Laurent Pinchart wrote:
> Hi Dave,
> 
> The following changes since commit adc31849b27fefeca6c225d3895143a2ec6970fa:
> 
>   Merge tag 'drm-intel-next-2014-12-19' of git://anongit.freedesktop.org/drm-
> intel into drm-next (2015-01-10 08:46:24 +1000)
> 
> are available in the git repository at:
> 
>   git://linuxtv.org/pinchartl/fbdev.git drm/next/tda998x

Err... please send to me instead so I can merge them with the other patches
I have queued.

-- 
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [GIT PULL FOR v3.20] tda998x enhancement

[Laurent Pinchart]

Hi Russell,

On Friday 16 January 2015 15:46:28 Russell King - ARM Linux wrote:
> On Fri, Jan 16, 2015 at 05:44:52PM +0200, Laurent Pinchart wrote:
> > Hi Dave,
> > 
> > The following changes since commit 
adc31849b27fefeca6c225d3895143a2ec6970fa:
> >   Merge tag 'drm-intel-next-2014-12-19' of
> >   git://anongit.freedesktop.org/drm-
> > 
> > intel into drm-next (2015-01-10 08:46:24 +1000)
> > 
> > are available in the git repository at:
> >   git://linuxtv.org/pinchartl/fbdev.git drm/next/tda998x
> 
> Err... please send to me instead so I can merge them with the other patches
> I have queued.

I've sent you an e-mail a couple of weeks ago to ask whether you wanted to 
take the patch in your tree (see 
http://comments.gmane.org/gmane.comp.video.dri.devel/119214). I'm certainly 
not in a good position to complain when mails slip through the cracks ;-)

Could you please take this patch in your tree and push it for v3.20 ?

-- 
Regards,

Laurent Pinchart

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [GIT PULL FOR v3.20] tda998x enhancement

[Russell King - ARM Linux]

On Fri, Jan 16, 2015 at 06:17:39PM +0200, Laurent Pinchart wrote:
> Hi Russell,
> 
> On Friday 16 January 2015 15:46:28 Russell King - ARM Linux wrote:
> > On Fri, Jan 16, 2015 at 05:44:52PM +0200, Laurent Pinchart wrote:
> > > Hi Dave,
> > > 
> > > The following changes since commit 
> adc31849b27fefeca6c225d3895143a2ec6970fa:
> > >   Merge tag 'drm-intel-next-2014-12-19' of
> > >   git://anongit.freedesktop.org/drm-
> > > 
> > > intel into drm-next (2015-01-10 08:46:24 +1000)
> > > 
> > > are available in the git repository at:
> > >   git://linuxtv.org/pinchartl/fbdev.git drm/next/tda998x
> > 
> > Err... please send to me instead so I can merge them with the other patches
> > I have queued.
> 
> I've sent you an e-mail a couple of weeks ago to ask whether you wanted to 
> take the patch in your tree (see 
> http://comments.gmane.org/gmane.comp.video.dri.devel/119214).

Yes, I found it, you sent it on the 23rd December.  Do you not think people
would be busy around Christmas time, and possibly would not bother reading
email?

Personally, I decided to stop on the weekend before Christmas, and only
resume on the 5th Jan.  However, I was ill with the flu, and I'm still
recovering from that three weeks on, so I didn't resume on the 5th -
that meant a few hundred extra messages, and I ended up deciding just
to ignore all that email (which included your message), rather than try
to catch up.

However, upon reading your email, and the original, you indicated that it
depends on another patch which I have no visibility of:

| This patch has been compile-tested only as I lack test hardware. It depends
| on "[PATCH v4 6/9] drm: Decouple EDID parsing from I2C adapter" previously
| posted to the dri-devel mailing list.

How can I merge your patch when you say that it depends on another patch
which I have no access to?  What is the status of this patch?  Is it
merged into mainline, or is it a patch in Dave's DRM tree?

> I'm certainly 
> not in a good position to complain when mails slip through the cracks ;-)
> 
> Could you please take this patch in your tree and push it for v3.20 ?

I'm not entirely thrilled by the thought of merging Dave's tree into mine.
Would it be possible to base it on some -rc release please?  What about
its dependent patch, which presumably this patch breaks without?

-- 
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [GIT PULL FOR v3.20] tda998x enhancement

[Laurent Pinchart]

Hi Russell,

On Friday 16 January 2015 16:26:00 Russell King - ARM Linux wrote:
> On Fri, Jan 16, 2015 at 06:17:39PM +0200, Laurent Pinchart wrote:
> > On Friday 16 January 2015 15:46:28 Russell King - ARM Linux wrote:
> >> On Fri, Jan 16, 2015 at 05:44:52PM +0200, Laurent Pinchart wrote:
> >>> Hi Dave,
> >>> 
> >>> The following changes since commit
> >>> adc31849b27fefeca6c225d3895143a2ec6970fa:
> >>>   Merge tag 'drm-intel-next-2014-12-19' of
> >>>   git://anongit.freedesktop.org/drm-
> >>> 
> >>> intel into drm-next (2015-01-10 08:46:24 +1000)
> >>> 
> >>> are available in the git repository at:
> >>>   git://linuxtv.org/pinchartl/fbdev.git drm/next/tda998x
> >> 
> >> Err... please send to me instead so I can merge them with the other
> >> patches I have queued.
> > 
> > I've sent you an e-mail a couple of weeks ago to ask whether you wanted to
> > take the patch in your tree (see
> > http://comments.gmane.org/gmane.comp.video.dri.devel/119214).
> 
> Yes, I found it, you sent it on the 23rd December.  Do you not think people
> would be busy around Christmas time, and possibly would not bother reading
> email?
> 
> Personally, I decided to stop on the weekend before Christmas, and only
> resume on the 5th Jan.  However, I was ill with the flu, and I'm still
> recovering from that three weeks on, so I didn't resume on the 5th -
> that meant a few hundred extra messages, and I ended up deciding just
> to ignore all that email (which included your message), rather than try
> to catch up.

If it wasn't clear from my previous mail, no blame meant at all.

> However, upon reading your email, and the original, you indicated that it
> depends on another patch which I have no visibility of:
>
> | This patch has been compile-tested only as I lack test hardware. It
> | depends on "[PATCH v4 6/9] drm: Decouple EDID parsing from I2C adapter"
> | previously posted to the dri-devel mailing list.
> 
> How can I merge your patch when you say that it depends on another patch
> which I have no access to?  What is the status of this patch?  Is it
> merged into mainline, or is it a patch in Dave's DRM tree?

The patch has been merged in v3.19-rc1 as 
18df89fef2d5c7cfd15ab898b17a292d00222b0c.

> > I'm certainly not in a good position to complain when mails slip through
> > the cracks ;-)
> > 
> > Could you please take this patch in your tree and push it for v3.20 ?
> 
> I'm not entirely thrilled by the thought of merging Dave's tree into mine.
> Would it be possible to base it on some -rc release please?  What about
> its dependent patch, which presumably this patch breaks without?

The patch applies on v3.19-rc1 without any issue. For your convenience I've 
just resent it with the Reviewed-by and Tested-by lines.

-- 
Regards,

Laurent Pinchart

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel