Status of the patches under review (29 patches)

Status of the patches under review (29 patches)

[Mauro Carvalho Chehab]

Hi,

I suspect that Linus should be releasing the 2.6.33 kernel any time soon,
so the next merge window is about to open. I've handled already everything
on my pending queues. However, I missed some emails due to a crash on my 
exim filter. Also, patchwork.kernel.org missed some emails due to some
trouble there. So, maybe there are still some unnoticed pending stuff.

If you still have any pending work for 2.6.34 that aren't merged nor
are under review, please submit it ASAP, as patches received after the 
release of 2.6.33 will likely be postponed to 2.6.35.

Cheers,
Mauro

--

This is the summary of the patches that are currently under review.
Each patch is represented by its submission date, the subject (up to 70 
chars) and the patchwork link (if submitted via email).

		== Videobuf patch - Need more tests before committing it - Volunteers? == 

Jan,19 2010: [v1,1/1] V4L: Add sync before a hardware operation to videobuf.        http://patchwork.kernel.org/patch/73896

		== Waiting for Muralidharan Karicheri <m-karicheri2@ti.com> review == 

Feb,19 2010: video_device: don't free_irq() an element past array vpif_obj.dev[]    http://patchwork.kernel.org/patch/80845

		== Waiting for Tobias Lorenz <tobias.lorenz@gmx.net> review == 

Feb, 3 2010: radio-si470x-common: -EINVAL overwritten in si470x_vidioc_s_tuner()    http://patchwork.kernel.org/patch/76786

		== waiting for Jean Delvare <khali@linux-fr.org> final submission after tests == 

Jan,30 2010: saa7134: Fix IR support of some ASUS TV-FM 7135 variants               http://patchwork.kernel.org/patch/75883

		== Waiting for its addition at linux-firmware and driver test by David Wong <davidtlwong@gmail.com>  == 

Nov, 1 2009: lgs8gxx: remove firmware for lgs8g75                                   http://patchwork.kernel.org/patch/56822

		== Andy Walls <awalls@radix.net> and Aleksandr Piskunov <aleksandr.v.piskunov@gmail.com> are discussing around the solution == 

Oct,11 2009: AVerTV MCE 116 Plus radio                                              http://patchwork.kernel.org/patch/52981

		== Patches waiting for Patrick Boettcher <pboettcher@dibcom.fr> review == 

Jan,15 2010: remove obsolete conditionalizing on DVB_DIBCOM_DEBUG                   http://patchwork.kernel.org/patch/73147
Feb, 8 2010: dib7000p: reduce large stack usage                                     http://patchwork.kernel.org/patch/77891
Feb, 8 2010: dib3000mc: reduce large stack usage                                    http://patchwork.kernel.org/patch/77892

		== IR driver for IMON - Will likely go via drivers/input tree == 

Dec,31 2009: [v2] input: imon driver for SoundGraph iMON/Antec Veris IR devices     http://patchwork.kernel.org/patch/70348

		== Waiting Helmut Auer <helmut@helmutauer.de> to get the Signed-off-by from Dirk Herrendoerfer == 

Feb,11 2010: Add support for SMT7020 to cx88                                        http://patchwork.kernel.org/patch/78823

		== Gspca patches - Waiting Hans de Goede <hdegoede@redhat.com> submission/review == 

Jan,29 2010: [gspca_jf,tree] gspca zc3xx: signal when unknown packet received       http://patchwork.kernel.org/patch/75837

		== Gspca patches - Waiting Jean-Francois Moine <moinejf@free.fr> submission/review == 

Jan,31 2010: [RFC] gspca pac7302: add USB PID range based on heuristics             http://patchwork.kernel.org/patch/75960
Jan,14 2010: Problem with gspca and zc3xx                                           http://patchwork.kernel.org/patch/72895

		== soc_camera patches - Waiting Guennadi <g.liakhovetski@gmx.de> submission/review == 

Feb, 9 2010: mt9t031: use runtime pm support to restore ADDRESS_MODE registers      http://patchwork.kernel.org/patch/77997
Feb,19 2010: v4l: soc_camera: fix bound checking of mbus_fmt[] index                http://patchwork.kernel.org/patch/80757
Feb, 2 2010: [1/3] soc-camera: mt9t112: modify exiting conditions from standby mode http://patchwork.kernel.org/patch/76212
Feb, 2 2010: [2/3] soc-camera: mt9t112: modify delay time after initialize          http://patchwork.kernel.org/patch/76213
Feb, 2 2010: [3/3] soc-camera: mt9t112: The flag which control camera-init is       http://patchwork.kernel.org/patch/76214

		== Waiting for my fixes on Docbook == 

Feb,17 2010: [1/4] DocBook/Makefile: Make it less verbose                           http://patchwork.kernel.org/patch/79981
Feb,17 2010: [2/4] DocBook: Add rules to auto-generate some media docbooks          http://patchwork.kernel.org/patch/79983
Feb,17 2010: [3/4] DocBook/v4l/pixfmt.xml: Add missing formats for gspca cpia1 and  http://patchwork.kernel.org/patch/79985
Feb,17 2010: [4/4] V4L/DVB: v4l: document new Bayer and monochrome pixel formats    http://patchwork.kernel.org/patch/79987

		== Waiting for Steven Toth <stoth@kernellabs.com> comments on it == 

Feb, 6 2010: cx23885: Enable Message Signaled Interrupts(MSI).                      http://patchwork.kernel.org/patch/77492

		== Patch(broken) Lock fixes. Patch is wrong but Siano has lock problems - Waiting for developers with time to fix it == 

Nov,12 2009: Locking in Siano driver (untested)                                     http://patchwork.kernel.org/patch/59590

		== Patch(broken) - Waiting Steven Toth <stoth@kernellabs.com> comments on it == 

Aug,19 2009: cx23885: fix support for TBS 6920 card                                 http://patchwork.kernel.org/patch/42777

		== Patch(broken) - DVB Stats API - Got a few more comments in priv, but better wait for a while == 

Dec, 7 2009: Details about DVB frontend API                                         http://patchwork.kernel.org/patch/65583
Dec, 9 2009: New DVB-Statistics API                                                 http://patchwork.kernel.org/patch/66133

		== Patch(broken) - waiting for Manu Abraham <abraham.manu@gmail.com> submission == 

Feb,16 2010: Twinhan 1027 + IR Port support                                         http://patchwork.kernel.org/patch/79753

Cheers,
Mauro

---

All patches sent to linux-media@vger.kernel.org are automatically stored at
Patchwork. The driver maintainer then analyzes it it and apply or reply
with comments. Others also may comment about it.

When the patch is is too trivial, or touches on a driver without a specific
maintainer, I analyze and apply it directly. Otherwise, it will be taged
as under review and the proper maintainer will be warned.

If you submitted or are waiting for the review of some patchset that weren't
applied at the git://git.kernel.org/v4l-dvb.git nor it is at the list bellow,
please check the status of the patch at the patchwork:
	http://patchwork.kernel.org/project/linux-media/list/

To check for a patch that is not New or Under Review, you'll need to click at
the "Filters" and pass some parameters to find the patch. Don't forget to
change the "State" to "any".

Eventually, your patch were rejected for some reason, or someone asked for
changes on it. If the patch is there, with a status different than
New/Under Review, double check for the received comments at the mailing list.
If it got Rejected, marked as RFC or as Changes Requested, please review it
according with the requests and send it again. In the very unlikely case
that the patch were wrongly tagged, please ping me.

If you discover any patch submitted via email that weren't caught by Patchwork,
this means that the patch got mangled by your emailer. The more likely
cause is that the emailer converted tabs into spaces or broke long lines.

If you're using Thunderbird, the solution is to install Asalted Patches
extension, available at:
	https://hg.mozilla.org/users/clarkbw_gnome.org/asalted-patches/
Other emailers will need you to disable the wrapping long lines feature.

Re: Status of the patches under review (29 patches)

[Guennadi Liakhovetski]

Hi Mauro

On Wed, 24 Feb 2010, Mauro Carvalho Chehab wrote:

> Hi,
> 
> I suspect that Linus should be releasing the 2.6.33 kernel any time soon,
> so the next merge window is about to open. I've handled already everything
> on my pending queues. However, I missed some emails due to a crash on my 
> exim filter. Also, patchwork.kernel.org missed some emails due to some
> trouble there. So, maybe there are still some unnoticed pending stuff.

Looks like you missed about two of my pull requests.

> If you still have any pending work for 2.6.34 that aren't merged nor
> are under review, please submit it ASAP, as patches received after the 
> release of 2.6.33 will likely be postponed to 2.6.35.

Namely:

> 		== soc_camera patches - Waiting Guennadi <g.liakhovetski@gmx.de> submission/review == 
> 
> Feb, 9 2010: mt9t031: use runtime pm support to restore ADDRESS_MODE registers      http://patchwork.kernel.org/patch/77997

This one depends on my runtime-pm patch (which is not listed here btw), 
which we didn't come to a consensus about, so, I think, I'll just push 
both of them and let you decide whether or not to pull.

> Feb,19 2010: v4l: soc_camera: fix bound checking of mbus_fmt[] index                http://patchwork.kernel.org/patch/80757

This one 
(http://article.gmane.org/gmane.linux.drivers.video-input-infrastructure/16068/match=pull+soc+camera) 
and

> Feb, 2 2010: [1/3] soc-camera: mt9t112: modify exiting conditions from standby mode http://patchwork.kernel.org/patch/76212

this one are in fixes branch of 
http://git.linuxtv.org/gliakhovetski/soc-camera.git?a=shortlog;h=refs/heads/fixes 
which I asked you to pull in 
http://www.spinics.net/lists/linux-media/msg16281.html

> Feb, 2 2010: [2/3] soc-camera: mt9t112: modify delay time after initialize          http://patchwork.kernel.org/patch/76213
> Feb, 2 2010: [3/3] soc-camera: mt9t112: The flag which control camera-init is       http://patchwork.kernel.org/patch/76214

These two have been put on hold by Morimoto-san.

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/

Re: Status of the patches under review (29 patches)

[Mauro Carvalho Chehab]

Guennadi Liakhovetski wrote:
> Hi Mauro
> 
> On Wed, 24 Feb 2010, Mauro Carvalho Chehab wrote:
> 
>> Hi,
>>
>> I suspect that Linus should be releasing the 2.6.33 kernel any time soon,
>> so the next merge window is about to open. I've handled already everything
>> on my pending queues. However, I missed some emails due to a crash on my 
>> exim filter. Also, patchwork.kernel.org missed some emails due to some
>> trouble there. So, maybe there are still some unnoticed pending stuff.
> 
> Looks like you missed about two of my pull requests.
> 
>> If you still have any pending work for 2.6.34 that aren't merged nor
>> are under review, please submit it ASAP, as patches received after the 
>> release of 2.6.33 will likely be postponed to 2.6.35.
> 
> Namely:
> 
>> 		== soc_camera patches - Waiting Guennadi <g.liakhovetski@gmx.de> submission/review == 
>>
>> Feb, 9 2010: mt9t031: use runtime pm support to restore ADDRESS_MODE registers      http://patchwork.kernel.org/patch/77997
> 
> This one depends on my runtime-pm patch (which is not listed here btw), 
> which we didn't come to a consensus about, so, I think, I'll just push 
> both of them and let you decide whether or not to pull.

Yes, please do it. I'll keep it as "under review" for now.

> 
>> Feb,19 2010: v4l: soc_camera: fix bound checking of mbus_fmt[] index                http://patchwork.kernel.org/patch/80757
> 
> This one 
> (http://article.gmane.org/gmane.linux.drivers.video-input-infrastructure/16068/match=pull+soc+camera) 

This one I missed. Applying it.

> and
> 
>> Feb, 2 2010: [1/3] soc-camera: mt9t112: modify exiting conditions from standby mode http://patchwork.kernel.org/patch/76212

This patch were applied only to the fixes.git tree:

commit 2b59125b1b5f8c9bb0524b8a0bdad4b780a239ac
Author: Kuninori Morimoto <morimoto.kuninori@renesas.com>
Date:   Tue Feb 2 13:17:54 2010 +0900

    soc-camera: mt9t112: modify exiting conditions from standby mode
    
    This polling is needed if camera is in standby mode, but current exiting
    condition is inverted.
    
    Signed-off-by: Kuninori Morimoto <morimoto.kuninori@renesas.com>
    Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
    Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

I'll merge it back to v4l-dvb.git when merging 2.6.33, after its release.

Removed it from my queue, since it should be upstream already.

> this one are in fixes branch of 
> http://git.linuxtv.org/gliakhovetski/soc-camera.git?a=shortlog;h=refs/heads/fixes 
> which I asked you to pull in 
> http://www.spinics.net/lists/linux-media/msg16281.html
> 
>> Feb, 2 2010: [2/3] soc-camera: mt9t112: modify delay time after initialize          http://patchwork.kernel.org/patch/76213
>> Feb, 2 2010: [3/3] soc-camera: mt9t112: The flag which control camera-init is       http://patchwork.kernel.org/patch/76214
> 
> These two have been put on hold by Morimoto-san.

Ok, I think I'll keep them at the "Under Review" status.

-- 

Cheers,
Mauro

Re: Status of the patches under review (29 patches)

[Jean Delvare]

Hi Mauro,

On Wed, 24 Feb 2010 02:40:00 -0300, Mauro Carvalho Chehab wrote:
> Hi,
> 
> I suspect that Linus should be releasing the 2.6.33 kernel any time soon,
> so the next merge window is about to open. I've handled already everything
> on my pending queues. However, I missed some emails due to a crash on my 
> exim filter. Also, patchwork.kernel.org missed some emails due to some
> trouble there. So, maybe there are still some unnoticed pending stuff.
> 
> If you still have any pending work for 2.6.34 that aren't merged nor
> are under review, please submit it ASAP, as patches received after the 
> release of 2.6.33 will likely be postponed to 2.6.35.

I have 3 patches pending which aren't in your list. I can see them in
patchwork:

http://patchwork.kernel.org/patch/79755/
http://patchwork.kernel.org/patch/79754/
http://patchwork.kernel.org/patch/77349/

The former two are in "Accepted" state, and actually I received an
e-mail telling me they had been accepted, however I can't see them in
the hg repository. So where are they?

The latter is in "Not Applicable" state, and I have no idea what it
means. The patch is really simple and I see no formatting issue. Should
I just resend it?

Thanks,
-- 
Jean Delvare

Re: Status of the patches under review (29 patches)

[Mauro Carvalho Chehab]

Hi Jean,

Jean Delvare wrote:
 
> I have 3 patches pending which aren't in your list. I can see them in
> patchwork:
> 
> http://patchwork.kernel.org/patch/79755/
> http://patchwork.kernel.org/patch/79754/
> http://patchwork.kernel.org/patch/77349/
> 
> The former two are in "Accepted" state, and actually I received an
> e-mail telling me they had been accepted, however I can't see them in
> the hg repository. So where are they?

They are already on the git tree:

commit 2887117b31b77ebe5fb42f95ea8d77a3716b405b
Author: Jean Delvare <khali@linux-fr.org>
Date:   Tue Feb 16 14:22:37 2010 -0300

    V4L/DVB: bttv: Let the user disable IR support
    
    Add a new module parameter "disable_ir" to disable IR support. Several
    other drivers do that already, and this can be very handy for
    debugging purposes.
    
    Signed-off-by: Jean Delvare <khali@linux-fr.org>
    Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

commit e151340a2a9e7147eb48114af0381122130266b0
Author: Jean Delvare <khali@linux-fr.org>
Date:   Fri Feb 19 00:18:41 2010 -0300

    V4L/DVB: bttv: Move I2C IR initialization
    
    Move I2C IR initialization from just after I2C bus setup to right
    before non-I2C IR initialization. This avoids the case where an I2C IR
    device is blocking audio support (at least the PV951 suffers from
    this). It is also more logical to group IR support together,
    regardless of the connectivity.
    
    This fixes bug #15184:
    http://bugzilla.kernel.org/show_bug.cgi?id=15184
    
    Signed-off-by: Jean Delvare <khali@linux-fr.org>
    CC: stable@kernel.org
    Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

As patches in -hg are manually backported, maybe Douglas
haven't backported it yet or he simply missed.

Douglas, could you please check this?

> The latter is in "Not Applicable" state, and I have no idea what it
> means. The patch is really simple and I see no formatting issue. Should
> I just resend it?

This means that this patch is not applicable on -git. There's no versions.txt
upstream. All patches that don't have upstream code are marked as such on
patchwork. I generally ping Douglas on such cases, for him to double check on
-hg.

Anyway, the better is to c/c to Douglas on all patches that are meant only
to the building system.

Douglas, could you please check if you've applied this patch?

-- 

Cheers,
Mauro

Re: Status of the patches under review (29 patches)

[Jean Delvare]

On Wed, 24 Feb 2010 10:10:16 -0300, Mauro Carvalho Chehab wrote:
> Hi Jean,
> 
> Jean Delvare wrote:
>  
> > I have 3 patches pending which aren't in your list. I can see them in
> > patchwork:
> > 
> > http://patchwork.kernel.org/patch/79755/
> > http://patchwork.kernel.org/patch/79754/
> > http://patchwork.kernel.org/patch/77349/
> > 
> > The former two are in "Accepted" state, and actually I received an
> > e-mail telling me they had been accepted, however I can't see them in
> > the hg repository. So where are they?
> 
> They are already on the git tree:
> 
> commit 2887117b31b77ebe5fb42f95ea8d77a3716b405b
> Author: Jean Delvare <khali@linux-fr.org>
> Date:   Tue Feb 16 14:22:37 2010 -0300
> 
>     V4L/DVB: bttv: Let the user disable IR support
>     
>     Add a new module parameter "disable_ir" to disable IR support. Several
>     other drivers do that already, and this can be very handy for
>     debugging purposes.
>     
>     Signed-off-by: Jean Delvare <khali@linux-fr.org>
>     Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
> 
> commit e151340a2a9e7147eb48114af0381122130266b0
> Author: Jean Delvare <khali@linux-fr.org>
> Date:   Fri Feb 19 00:18:41 2010 -0300
> 
>     V4L/DVB: bttv: Move I2C IR initialization
>     
>     Move I2C IR initialization from just after I2C bus setup to right
>     before non-I2C IR initialization. This avoids the case where an I2C IR
>     device is blocking audio support (at least the PV951 suffers from
>     this). It is also more logical to group IR support together,
>     regardless of the connectivity.
>     
>     This fixes bug #15184:
>     http://bugzilla.kernel.org/show_bug.cgi?id=15184
>     
>     Signed-off-by: Jean Delvare <khali@linux-fr.org>
>     CC: stable@kernel.org
>     Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
> 
> As patches in -hg are manually backported, maybe Douglas
> haven't backported it yet or he simply missed.
> 
> Douglas, could you please check this?

Ah, my bad. I totally missed that you had moved from hg to git for the
main development tree. I was still pulling from hg and basing my
patches on it. I will clone the git tree now, sorry for the confusion.

> > The latter is in "Not Applicable" state, and I have no idea what it
> > means. The patch is really simple and I see no formatting issue. Should
> > I just resend it?
> 
> This means that this patch is not applicable on -git. There's no versions.txt
> upstream. All patches that don't have upstream code are marked as such on
> patchwork. I generally ping Douglas on such cases, for him to double check on
> -hg.
> 
> Anyway, the better is to c/c to Douglas on all patches that are meant only
> to the building system.
> 
> Douglas, could you please check if you've applied this patch?

Thanks Douglas.

-- 
Jean Delvare

Re: Status of the patches under review (29 patches)

[Andy Walls]

On Wed, 2010-02-24 at 02:40 -0300, Mauro Carvalho Chehab wrote:
> Hi,
> 
> I suspect that Linus should be releasing the 2.6.33 kernel any time soon,
> so the next merge window is about to open. I've handled already everything
> on my pending queues. However, I missed some emails due to a crash on my 
> exim filter. Also, patchwork.kernel.org missed some emails due to some
> trouble there. So, maybe there are still some unnoticed pending stuff.

I had emailed some cx18 component video patches to the list.  They are
not critical for 2.6.34 but you can find them as the two most recent
patches here:

http://linuxtv.org/hg/~awalls/cx18-pvr2100-component/

Regards,
Andy

> If you still have any pending work for 2.6.34 that aren't merged nor
> are under review, please submit it ASAP, as patches received after the 
> release of 2.6.33 will likely be postponed to 2.6.35.
> 
> Cheers,
> Mauro