Re: Status of the patches under review (29 patches)
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 > 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 > Signed-off-by: Mauro Carvalho Chehab > > commit e151340a2a9e7147eb48114af0381122130266b0 > Author: Jean Delvare > 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 > CC: sta...@kernel.org > Signed-off-by: Mauro Carvalho Chehab > > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Status of the patches under review (29 patches)
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 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Status of the patches under review (29 patches)
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 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 Signed-off-by: Mauro Carvalho Chehab commit e151340a2a9e7147eb48114af0381122130266b0 Author: Jean Delvare 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 CC: sta...@kernel.org Signed-off-by: Mauro Carvalho Chehab 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 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Status of the patches under review (29 patches)
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 >> 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 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 Signed-off-by: Guennadi Liakhovetski Signed-off-by: Mauro Carvalho Chehab 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 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Status of the patches under review (29 patches)
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 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Status of the patches under review (29 patches)
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 > 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/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Status of the patches under review (29 patches)
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 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 review == Feb, 3 2010: radio-si470x-common: -EINVAL overwritten in si470x_vidioc_s_tuner()http://patchwork.kernel.org/patch/76786 == waiting for Jean Delvare 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 == Nov, 1 2009: lgs8gxx: remove firmware for lgs8g75 http://patchwork.kernel.org/patch/56822 == Andy Walls and Aleksandr Piskunov are discussing around the solution == Oct,11 2009: AVerTV MCE 116 Plus radio http://patchwork.kernel.org/patch/52981 == Patches waiting for Patrick Boettcher 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 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 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 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 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 formatshttp://patchwork.kernel.org/patch/79987 == Waiting for Steven Toth 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 pro