* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 12:34 Status of the patches under review at LMML (28 patches) Mauro Carvalho Chehab
@ 2011-09-24 14:52 ` Andy Walls
2011-09-24 15:16 ` Mauro Carvalho Chehab
2011-09-24 18:11 ` Manu Abraham
` (8 subsequent siblings)
9 siblings, 1 reply; 14+ messages in thread
From: Andy Walls @ 2011-09-24 14:52 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: LMML, Pawel Osiak, Morimoto Kuninori, Manu Abraham, Jarod Wilson,
Eddi De Pieri, Hans de Goede, Laurent Pinchart,
Guennadi Liakhovetski, Dmitri Belimov, Michael Krufky
On Sat, 2011-09-24 at 09:34 -0300, Mauro Carvalho Chehab wrote:
> Everything at patchwork were reviewed by me, and I've applied all patches
> that I didn't notice any review by the drivers maintainers.
>
> Driver maintainers:
> Please review the remaining patches.
> == Patches for Andy Walls <Andy Walls <awalls@md.metrocast.net>> review ==
>
> May,25 2011: ivtv: use display information in info not in var for panning http://patchwork.linuxtv.org/patch/6706 Laurent Pinchart <laurent.pinchart@ideasonboard.com>
You committed this one from one of my pull requests. The subject line
changed a little (ivtv: -> ivtvfb:)
http://www.gossamer-threads.com/lists/ivtv/devel/41425
http://git.linuxtv.org/media_tree.git?a=commitdiff;h=5d9c08dea0c13c09408f97fe61d34c198c4f3277
http://lkml.org/lkml/2011/6/7/311
> == Waiting for Andy Walls <awalls@md.metrocast.net> double-check ==
>
> Dec,19 2010: [RESEND, for, 2.6.37] cx23885, cx25840: Provide IR Rx timeout event re http://patchwork.linuxtv.org/patch/5133 Andy Walls <awalls@md.metrocast.net>
By inspection, this one is still OK. It is still relevant and looks
like it still should apply cleanly. I have not compile tested it
recently.
> == waiting for Michael Krufky <mkrufky@linuxtv.org> review ==
>
> Sep, 4 2011: Medion 95700 analog video support http://patchwork.linuxtv.org/patch/7767 Maciej Szmigiero <mhej@o2.pl>
I need to comment on the cx25840 portion of this patch. I plan to make
time to do that in a few hours.
Regards,
Andy
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 14:52 ` Andy Walls
@ 2011-09-24 15:16 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 14+ messages in thread
From: Mauro Carvalho Chehab @ 2011-09-24 15:16 UTC (permalink / raw)
To: Andy Walls
Cc: LMML, Pawel Osiak, Morimoto Kuninori, Manu Abraham, Jarod Wilson,
Eddi De Pieri, Hans de Goede, Laurent Pinchart,
Guennadi Liakhovetski, Dmitri Belimov, Michael Krufky
Em 24-09-2011 11:52, Andy Walls escreveu:
> On Sat, 2011-09-24 at 09:34 -0300, Mauro Carvalho Chehab wrote:
>> Everything at patchwork were reviewed by me, and I've applied all patches
>> that I didn't notice any review by the drivers maintainers.
>>
>> Driver maintainers:
>> Please review the remaining patches.
>
>> == Patches for Andy Walls <Andy Walls <awalls@md.metrocast.net>> review ==
>>
>> May,25 2011: ivtv: use display information in info not in var for panning http://patchwork.linuxtv.org/patch/6706 Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
> You committed this one from one of my pull requests. The subject line
> changed a little (ivtv: -> ivtvfb:)
>
> http://www.gossamer-threads.com/lists/ivtv/devel/41425
> http://git.linuxtv.org/media_tree.git?a=commitdiff;h=5d9c08dea0c13c09408f97fe61d34c198c4f3277
> http://lkml.org/lkml/2011/6/7/311
Ok. Patch updated.
>
>> == Waiting for Andy Walls <awalls@md.metrocast.net> double-check ==
>>
>> Dec,19 2010: [RESEND, for, 2.6.37] cx23885, cx25840: Provide IR Rx timeout event re http://patchwork.linuxtv.org/patch/5133 Andy Walls <awalls@md.metrocast.net>
>
> By inspection, this one is still OK. It is still relevant and looks
> like it still should apply cleanly. I have not compile tested it
> recently.
Thanks! Applied.
>
>
>> == waiting for Michael Krufky <mkrufky@linuxtv.org> review ==
>>
>> Sep, 4 2011: Medion 95700 analog video support http://patchwork.linuxtv.org/patch/7767 Maciej Szmigiero <mhej@o2.pl>
>
> I need to comment on the cx25840 portion of this patch. I plan to make
> time to do that in a few hours.
Thank you!
Mauro
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 12:34 Status of the patches under review at LMML (28 patches) Mauro Carvalho Chehab
2011-09-24 14:52 ` Andy Walls
@ 2011-09-24 18:11 ` Manu Abraham
2011-09-25 11:42 ` Mauro Carvalho Chehab
2011-09-24 18:15 ` Manu Abraham
` (7 subsequent siblings)
9 siblings, 1 reply; 14+ messages in thread
From: Manu Abraham @ 2011-09-24 18:11 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: LMML
On Sat, Sep 24, 2011 at 6:04 PM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Everything at patchwork were reviewed by me, and I've applied all patches
> that I didn't notice any review by the drivers maintainers.
>
> Driver maintainers:
> Please review the remaining patches.
>
> == Patches for Manu Abraham <abraham.manu@gmail.com> review ==
>
> Jun,11 2010: stb0899: Removed an extra byte sent at init on DiSEqC bus http://patchwork.linuxtv.org/patch/3639 Florent AUDEBERT <florent.audebert@anevia.com>
A single byte doesn't make much of a difference, but well it is still
a difference. The DiSEqC has some known issues and has some tricky
workarounds for different Silicon cuts. This patch *might* be good on
some chips while have an adverse effect. But that said I have not
tested this patch.
If general users would like to have this patch and is proven good on
different versions, I have no objection for this patch to go in.
In which case; Acked-by: Manu Abraham <manu@linuxtv.org>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 18:11 ` Manu Abraham
@ 2011-09-25 11:42 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 14+ messages in thread
From: Mauro Carvalho Chehab @ 2011-09-25 11:42 UTC (permalink / raw)
To: Manu Abraham; +Cc: LMML
Em 24-09-2011 15:11, Manu Abraham escreveu:
> On Sat, Sep 24, 2011 at 6:04 PM, Mauro Carvalho Chehab
> <mchehab@redhat.com> wrote:
>> Everything at patchwork were reviewed by me, and I've applied all patches
>> that I didn't notice any review by the drivers maintainers.
>>
>> Driver maintainers:
>> Please review the remaining patches.
>>
>> == Patches for Manu Abraham <abraham.manu@gmail.com> review ==
>>
>> Jun,11 2010: stb0899: Removed an extra byte sent at init on DiSEqC bus http://patchwork.linuxtv.org/patch/3639 Florent AUDEBERT <florent.audebert@anevia.com>
>
> A single byte doesn't make much of a difference, but well it is still
> a difference. The DiSEqC has some known issues and has some tricky
> workarounds for different Silicon cuts. This patch *might* be good on
> some chips while have an adverse effect. But that said I have not
> tested this patch.
>
> If general users would like to have this patch and is proven good on
> different versions, I have no objection for this patch to go in.
>
> In which case; Acked-by: Manu Abraham <manu@linuxtv.org>
Ok. Well, let's apply it and see the feedbacks if any.
Thanks for reviewing it!
Mauro
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 12:34 Status of the patches under review at LMML (28 patches) Mauro Carvalho Chehab
2011-09-24 14:52 ` Andy Walls
2011-09-24 18:11 ` Manu Abraham
@ 2011-09-24 18:15 ` Manu Abraham
2011-09-24 18:24 ` Manu Abraham
` (6 subsequent siblings)
9 siblings, 0 replies; 14+ messages in thread
From: Manu Abraham @ 2011-09-24 18:15 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: LMML
On Sat, Sep 24, 2011 at 6:04 PM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Everything at patchwork were reviewed by me, and I've applied all patches
> that I didn't notice any review by the drivers maintainers.
>
> Driver maintainers:
> Please review the remaining patches.
>
> == Patches for Manu Abraham <abraham.manu@gmail.com> review ==
>
> Aug, 7 2010: Refactor Mantis DMA transfer to deliver 16Kb TS data per interrupt http://patchwork.linuxtv.org/patch/4104 Marko Ristola <marko.ristola@kolumbus.fi>
I did test this patch. It doesn't bring in any side effects in
general. It increases the latency, which it is expected. On some
PCI-PCI chipset the patch makes the TS handling worser. But in other
cases, it looks okay.
Reviewed-by: Manu Abraham <manu@linuxtv.org>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 12:34 Status of the patches under review at LMML (28 patches) Mauro Carvalho Chehab
` (2 preceding siblings ...)
2011-09-24 18:15 ` Manu Abraham
@ 2011-09-24 18:24 ` Manu Abraham
2011-09-24 18:27 ` Manu Abraham
` (5 subsequent siblings)
9 siblings, 0 replies; 14+ messages in thread
From: Manu Abraham @ 2011-09-24 18:24 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: LMML
On Sat, Sep 24, 2011 at 6:04 PM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Everything at patchwork were reviewed by me, and I've applied all patches
> that I didn't notice any review by the drivers maintainers.
>
> Driver maintainers:
> Please review the remaining patches.
>
> == Patches for Manu Abraham <abraham.manu@gmail.com> review ==
>
> May,21 2011: Disable dynamic current limit for ttpci budget cards http://patchwork.linuxtv.org/patch/6669 Guy Martin <gmsoft@tuxicoman.be>
Strictly, the patch is incorrect.
- Dynamic Current Limiting is nothing but a PWM operation where Ton =
20mS, Toff = 900mS.
In fact, DCL is much more preferred, since it can protect the Power
tracks to the PCI slot in case the fuse on the card doesn't blow out.
This *might* prove problematic with some DiSEqC switches. Only in such
a case, it might be wise to disable DCL. But I haven't yet seen anyone
complain on the same, otherwise incorrect DiSEqC commands ?
Regards,
Manu
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 12:34 Status of the patches under review at LMML (28 patches) Mauro Carvalho Chehab
` (3 preceding siblings ...)
2011-09-24 18:24 ` Manu Abraham
@ 2011-09-24 18:27 ` Manu Abraham
2011-09-24 18:29 ` Manu Abraham
` (4 subsequent siblings)
9 siblings, 0 replies; 14+ messages in thread
From: Manu Abraham @ 2011-09-24 18:27 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: LMML
On Sat, Sep 24, 2011 at 6:04 PM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Everything at patchwork were reviewed by me, and I've applied all patches
> that I didn't notice any review by the drivers maintainers.
>
> Driver maintainers:
> Please review the remaining patches.
>
> == Patches for Manu Abraham <abraham.manu@gmail.com> review ==
>
> May,23 2011: Increase a timeout, so that bad scheduling does not accidentially caus http://patchwork.linuxtv.org/patch/7178 Hans Petter Selasky <hselasky@c2i.net>
Haven't heard timeout issues recently though ... but still;
Acked-by: Manu Abraham <manu@linuxtv.org>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 12:34 Status of the patches under review at LMML (28 patches) Mauro Carvalho Chehab
` (4 preceding siblings ...)
2011-09-24 18:27 ` Manu Abraham
@ 2011-09-24 18:29 ` Manu Abraham
2011-09-25 12:44 ` Mauro Carvalho Chehab
2011-09-24 18:31 ` Manu Abraham
` (3 subsequent siblings)
9 siblings, 1 reply; 14+ messages in thread
From: Manu Abraham @ 2011-09-24 18:29 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: LMML
On Sat, Sep 24, 2011 at 6:04 PM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Everything at patchwork were reviewed by me, and I've applied all patches
> that I didn't notice any review by the drivers maintainers.
>
> Driver maintainers:
> Please review the remaining patches.
>
> == Patches for Manu Abraham <abraham.manu@gmail.com> review ==
>
> Jun, 8 2011: Add remote control support for mantis http://patchwork.linuxtv.org/patch/7217 Christoph Pinkl <christoph.pinkl@gmail.com>
Haven't tested this patch. Please hold on with this patch.
Best Regards,
Manu
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 18:29 ` Manu Abraham
@ 2011-09-25 12:44 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 14+ messages in thread
From: Mauro Carvalho Chehab @ 2011-09-25 12:44 UTC (permalink / raw)
To: Manu Abraham; +Cc: LMML
Em 24-09-2011 15:29, Manu Abraham escreveu:
> On Sat, Sep 24, 2011 at 6:04 PM, Mauro Carvalho Chehab
> <mchehab@redhat.com> wrote:
>> Everything at patchwork were reviewed by me, and I've applied all patches
>> that I didn't notice any review by the drivers maintainers.
>>
>> Driver maintainers:
>> Please review the remaining patches.
>>
>> == Patches for Manu Abraham <abraham.manu@gmail.com> review ==
>>
Ok, status updated at all patches, and added the ones that were acked/reviewed.
>> Jun, 8 2011: Add remote control support for mantis http://patchwork.linuxtv.org/patch/7217 Christoph Pinkl <christoph.pinkl@gmail.com>
>
> Haven't tested this patch. Please hold on with this patch.
Ok.
FYI, this patch looks correct on my eyes, with respect to RC core usage.
Thanks,
Mauro
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 12:34 Status of the patches under review at LMML (28 patches) Mauro Carvalho Chehab
` (5 preceding siblings ...)
2011-09-24 18:29 ` Manu Abraham
@ 2011-09-24 18:31 ` Manu Abraham
2011-09-24 18:36 ` Manu Abraham
` (2 subsequent siblings)
9 siblings, 0 replies; 14+ messages in thread
From: Manu Abraham @ 2011-09-24 18:31 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: LMML
On Sat, Sep 24, 2011 at 6:04 PM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Everything at patchwork were reviewed by me, and I've applied all patches
> that I didn't notice any review by the drivers maintainers.
>
> Driver maintainers:
> Please review the remaining patches.
>
> == Patches for Manu Abraham <abraham.manu@gmail.com> review ==
>
> May,24 2011: Fix the derot zig-zag to work with TT-USB2.0 TechnoTrend. http://patchwork.linuxtv.org/patch/6777 Hans Petter Selasky <hselasky@c2i.net>
Patch doesn't make any sense.
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 12:34 Status of the patches under review at LMML (28 patches) Mauro Carvalho Chehab
` (6 preceding siblings ...)
2011-09-24 18:31 ` Manu Abraham
@ 2011-09-24 18:36 ` Manu Abraham
2011-09-25 21:54 ` Pawel Osciak
2011-09-27 12:03 ` Laurent Pinchart
9 siblings, 0 replies; 14+ messages in thread
From: Manu Abraham @ 2011-09-24 18:36 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: LMML
On Sat, Sep 24, 2011 at 6:04 PM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Everything at patchwork were reviewed by me, and I've applied all patches
> that I didn't notice any review by the drivers maintainers.
>
> Driver maintainers:
> Please review the remaining patches.
>
> == Patches for Manu Abraham <abraham.manu@gmail.com> review ==
>
> Jun, 1 2011: stv090x: set status bits when there is no lock http://patchwork.linuxtv.org/patch/6804 Guy Martin <gmsoft@tuxicoman.be>
Patch looks okay, but untested at my side.
Acked-by: Manu Abraham <manu@linuxtv.org>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 12:34 Status of the patches under review at LMML (28 patches) Mauro Carvalho Chehab
` (7 preceding siblings ...)
2011-09-24 18:36 ` Manu Abraham
@ 2011-09-25 21:54 ` Pawel Osciak
2011-09-27 12:03 ` Laurent Pinchart
9 siblings, 0 replies; 14+ messages in thread
From: Pawel Osciak @ 2011-09-25 21:54 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: LMML, Morimoto Kuninori, Manu Abraham, Jarod Wilson,
Eddi De Pieri, Hans de Goede, Andy Walls, Laurent Pinchart,
Guennadi Liakhovetski, Dmitri Belimov, Michael Krufky
Hi Mauro,
On Sat, Sep 24, 2011 at 05:34, Mauro Carvalho Chehab <mchehab@redhat.com> wrote:
> Everything at patchwork were reviewed by me, and I've applied all patches
> that I didn't notice any review by the drivers maintainers.
>
> Driver maintainers:
> Please review the remaining patches.
(...)
> == Waiting for Pawel Osiak <pawel@osciak.com> review ==
>
> Jul,12 2011: v4l: mem2mem: add wait_{prepare,finish} ops to m2m_testdev http://patchwork.linuxtv.org/patch/7431 Michael Olbrich <m.olbrich@pengutronix.de>
>
This is an ack from me.
--
Best regards,
Pawel Osciak
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: Status of the patches under review at LMML (28 patches)
2011-09-24 12:34 Status of the patches under review at LMML (28 patches) Mauro Carvalho Chehab
` (8 preceding siblings ...)
2011-09-25 21:54 ` Pawel Osciak
@ 2011-09-27 12:03 ` Laurent Pinchart
9 siblings, 0 replies; 14+ messages in thread
From: Laurent Pinchart @ 2011-09-27 12:03 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: LMML, Pawel Osiak, Morimoto Kuninori, Manu Abraham, Jarod Wilson,
Eddi De Pieri, Hans de Goede, Andy Walls, Guennadi Liakhovetski,
Dmitri Belimov, Michael Krufky
Hi Mauro,
On Saturday 24 September 2011 14:34:57 Mauro Carvalho Chehab wrote:
> Everything at patchwork were reviewed by me, and I've applied all patches
> that I didn't notice any review by the drivers maintainers.
>
> Driver maintainers:
> Please review the remaining patches.
>
> == Patches waiting for Laurent Pinchart
> <laurent.pinchart@ideasonboard.com> review ==
>
> Jun,22 2011: Improve UVC buffering with regard to USB. Add checks to avoid
> division
> http://patchwork.linuxtv.org/patch/7290
> Hans Petter Selasky <hselasky@c2i.net>
On my TODO list.
> Jul,11 2011: Error routes through omap3isp ccdc.
> http://patchwork.linuxtv.org/patch/7428
> Jonathan Cameron <jic23@cam.ac.uk>
This has been superseded by "[media] omap3isp: Don't accept pipelines with no
video source as valid" which is already in your tree.
> Jul,14 2011: uvcvideo: add fix suspend/resume quirk for Microdia camera
> http://patchwork.linuxtv.org/patch/186
> Ming Lei <tom.leiming@gmail.com>
This has been superseded by "uvcvideo: Set alternate setting 0 on resume if
the bus has been reset" which is hopefully in your tree on its way to v3.1 :-)
> Jul,13 2011: [RFC, v1] mt9v113: VGA camera sensor driver and support for
> BeagleBoar
> http://patchwork.linuxtv.org/patch/184
> Joel A Fernandes <agnel.joel@gmail.com>
I've reviewed the patch, waiting for the next version.
> Sep, 6 2011: mt9p031: Do not use PLL if external frequency is the same as
> target fr
> http://patchwork.linuxtv.org/patch/7783
> Javier Martin <javier.martin@vista-silicon.com>
I've reviewed the patch, I think we should implement proper PLL support.
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 14+ messages in thread