[PULL] http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final

[PULL] http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final

[Devin Heitmueller]

Hello,

Please pull from the following for some basic fixes related to
applications such as tvtime hanging when no video is present, as well
as some quality improvements for analog.

http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final

Please let me know if there are any questions/problems.

Thanks,

Devin

-- 
Devin J. Heitmueller - Kernel Labs
http://www.kernellabs.com

Re: [PULL] http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final

[Devin Heitmueller]

On Sat, Oct 9, 2010 at 2:40 PM, Devin Heitmueller
<dheitmueller@kernellabs.com> wrote:
> Hello,
>
> Please pull from the following for some basic fixes related to
> applications such as tvtime hanging when no video is present, as well
> as some quality improvements for analog.
>
> http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final
>
> Please let me know if there are any questions/problems.

<ping>


-- 
Devin J. Heitmueller - Kernel Labs
http://www.kernellabs.com

Re: [PULL] http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final

[Mauro Carvalho Chehab]

Em 02-11-2010 14:47, Devin Heitmueller escreveu:
> On Sat, Oct 9, 2010 at 2:40 PM, Devin Heitmueller
> <dheitmueller@kernellabs.com> wrote:
>> Hello,
>>
>> Please pull from the following for some basic fixes related to
>> applications such as tvtime hanging when no video is present, as well
>> as some quality improvements for analog.
>>
>> http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final
>>
>> Please let me know if there are any questions/problems.
> 
> <ping>
> 
> 
Hi Devin,

I didn't start to pull any fixes yet. I might eventually have some time
during this week, but it is more likely that I'll handle after my
return back.

Cheers,
Mauro

Re: [PULL] http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final

[Devin Heitmueller]

On Wed, Nov 3, 2010 at 7:10 AM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Hi Devin,
>
> I didn't start to pull any fixes yet. I might eventually have some time
> during this week, but it is more likely that I'll handle after my
> return back.

That's fine.  It's been pending for almost a month so I just wanted to
be sure it didn't get lost.

Regards,

Devin

-- 
Devin J. Heitmueller - Kernel Labs
http://www.kernellabs.com

Re: [PULL] http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final

[Mauro Carvalho Chehab]

Em 02-11-2010 16:47, Devin Heitmueller escreveu:
> On Sat, Oct 9, 2010 at 2:40 PM, Devin Heitmueller
> <dheitmueller@kernellabs.com> wrote:
>> Hello,
>>
>> Please pull from the following for some basic fixes related to
>> applications such as tvtime hanging when no video is present, as well
>> as some quality improvements for analog.
>>
>> http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final
>>
>> Please let me know if there are any questions/problems.

I'm still importing your patches, but, at the very first one, you
forgot to send your Signed-off-by:

Generating hg_15168_djh_merge_vbi_changes.patch
WARNING: please, no space before tabs
#39: FILE: drivers/media/video/au0828/au0828.h:155:
+^Istruct au0828_buffer    ^I*vbi_buf;$

ERROR: Missing Signed-off-by: line(s)

Cheers,
Mauro

Re: [PULL] http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final

[Devin Heitmueller]

On Tue, Nov 9, 2010 at 11:03 AM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Em 02-11-2010 16:47, Devin Heitmueller escreveu:
>> On Sat, Oct 9, 2010 at 2:40 PM, Devin Heitmueller
>> <dheitmueller@kernellabs.com> wrote:
>>> Hello,
>>>
>>> Please pull from the following for some basic fixes related to
>>> applications such as tvtime hanging when no video is present, as well
>>> as some quality improvements for analog.
>>>
>>> http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final
>>>
>>> Please let me know if there are any questions/problems.
>
> I'm still importing your patches, but, at the very first one, you
> forgot to send your Signed-off-by:
>
> Generating hg_15168_djh_merge_vbi_changes.patch
> WARNING: please, no space before tabs
> #39: FILE: drivers/media/video/au0828/au0828.h:155:
> +^Istruct au0828_buffer    ^I*vbi_buf;$
>
> ERROR: Missing Signed-off-by: line(s)
>
> Cheers,
> Mauro
>

"djh - merge vbi changes" was just a rebase against the latest code.
The very first patch in the series is one earlier (047a8c9fa9d5).

Devin

-- 
Devin J. Heitmueller - Kernel Labs
http://www.kernellabs.com

Re: [PULL] http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final

[Mauro Carvalho Chehab]

Em 09-11-2010 14:07, Devin Heitmueller escreveu:
> On Tue, Nov 9, 2010 at 11:03 AM, Mauro Carvalho Chehab
> <mchehab@redhat.com> wrote:
>> Em 02-11-2010 16:47, Devin Heitmueller escreveu:
>>> On Sat, Oct 9, 2010 at 2:40 PM, Devin Heitmueller
>>> <dheitmueller@kernellabs.com> wrote:
>>>> Hello,
>>>>
>>>> Please pull from the following for some basic fixes related to
>>>> applications such as tvtime hanging when no video is present, as well
>>>> as some quality improvements for analog.
>>>>
>>>> http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final
>>>>
>>>> Please let me know if there are any questions/problems.
>>
>> I'm still importing your patches, but, at the very first one, you
>> forgot to send your Signed-off-by:
>>
>> Generating hg_15168_djh_merge_vbi_changes.patch
>> WARNING: please, no space before tabs
>> #39: FILE: drivers/media/video/au0828/au0828.h:155:
>> +^Istruct au0828_buffer    ^I*vbi_buf;$
>>
>> ERROR: Missing Signed-off-by: line(s)
>>
>> Cheers,
>> Mauro
>>
> 
> "djh - merge vbi changes" was just a rebase against the latest code.
> The very first patch in the series is one earlier (047a8c9fa9d5).

Ok.

Hmm... the second patch is also without SOB:

patches/hg_15169_au8522_properly_set_default_brightness.patch
Changeset: 15169
From: Devin Heitmueller  <dheitmueller@kernellabs.com>
Commiter: Devin Heitmueller <dheitmueller@kernellabs.com>
Date: Sun Jun 13 16:31:22 2010 -0400
Subject: au8522: Properly set default brightness

The chip's default value for the brightness didn't match what we were sending
back in the queryctrl ioctl(), so if the application actually set the
brightness to the "default", it would actually end up being way too bright.

This work was sponsored by GetWellNetwork Inc.

Priority: normal

---

diff -upNr oldtree/drivers/media/dvb/frontends/au8522_decoder.c linux/drivers/media/dvb/frontends/au8522_decoder.c
--- oldtree/drivers/media/dvb/frontends/au8522_decoder.c	2010-11-09 14:00:23.000000000 -0200
+++ linux/drivers/media/dvb/frontends/au8522_decoder.c	2010-11-09 14:00:13.000000000 -0200
@@ -623,7 +623,7 @@ static int au8522_queryctrl(struct v4l2_
 		return v4l2_ctrl_query_fill(qc, 0, 255, 1,
 					    AU8522_TVDEC_CONTRAST_REG00BH_CVBS);
 	case V4L2_CID_BRIGHTNESS:
-		return v4l2_ctrl_query_fill(qc, 0, 255, 1, 128);
+		return v4l2_ctrl_query_fill(qc, 0, 255, 1, 109);
 	case V4L2_CID_SATURATION:
 		return v4l2_ctrl_query_fill(qc, 0, 255, 1, 128);
 	case V4L2_CID_HUE:

Re: [PULL] http://www.kernellabs.com/hg/~dheitmueller/v4l-dvb-950q-final

[Devin Heitmueller]

On Tue, Nov 9, 2010 at 1:02 PM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Ok.
>
> Hmm... the second patch is also without SOB:

Doh.

Signed-off-by: Devin Heitmueller <dheitmueller@kernellabs.com>

Thanks,

Devin

-- 
Devin J. Heitmueller - Kernel Labs
http://www.kernellabs.com