public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
* Old patches sent via the Mailing list
@ 2010-10-17 21:20 Mauro Carvalho Chehab
  2010-10-17 22:03 ` Guennadi Liakhovetski
                   ` (6 more replies)
  0 siblings, 7 replies; 22+ messages in thread
From: Mauro Carvalho Chehab @ 2010-10-17 21:20 UTC (permalink / raw)
  To: LMML
  Cc: Igor M. Liplianin, Manu Abraham, Jean-Francois Moine,
	Jarod Wilson, Richard Zidlicky, Antti Palosaari, Sven Barth,
	Patrick Boettcher, Mauro Carvalho Chehab, Guennadi Liakhovetski,
	Henrik Kurelid, Hans de Goede

Hi,

I did a large effort during this weekend to handle the maximum amount of patches, in order to have them
ready for 2.6.37. While there are still some patches marked as NEW at patchwork, and a few pending pull
requests (mostly related to more kABI changes), there are still a list of patches that are marked as
Under review. Except for 4 patches from me, related to Doc (that I'm keeping in this list just to remind
me that I'll need to fix them when I have some time - just some automation stuff at DocBook), all other
patches marked as Under review are stuff that I basically depend on others.

The last time I sent this list, I was about to travel, and I may have missed some comments, or maybe I
may just forgot to update. But I suspect that, for the list bellow, most of them are stuff where the
driver maintainer just forgot at limbo.

>From the list of patches under review, we have:

Waiting for new patch, signed, from Sven Barth <pascaldragon@googlemail.com>
  Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400                       http://patchwork.kernel.org/patch/94960   Sven Barth <pascaldragon@googlemail.com>

There are some patches where i want to have more tests/acks before applying:
  Aug, 7 2010: Avoid unnecessary data copying inside dvb_dmx_swfilter_204() function  http://patchwork.kernel.org/patch/118147  Marko Ristola <marko.ristola@kolumbus.fi>
  Mar, 1 2010: firedtv: add parameter to fake ca_system_ids in CA_INFO                http://patchwork.kernel.org/patch/82912   Henrik Kurelid <henrik@kurelid.se>
(I'm not sure about this one, Henrik's agument didn't convince me that we should add a workaround for MythTV)

I think that the better is to wait for videobuf2, instead of applying those patches:
  Mar,17 2010: [2/2] V4L/DVB: buf-dma-sg.c: support non-pageable user-allocated memor http://patchwork.kernel.org/patch/97263   Arnout Vandecappelle <arnout@mind.be>
  Jul,27 2010: videobuf_dma_sg: a new implementation for mmap                         http://patchwork.kernel.org/patch/114520  Figo.zhang <figo1802@gmail.com>
  Jul,28 2010: [v2] videobuf_dma_sg: a new implementation for mmap                    http://patchwork.kernel.org/patch/114760  Figo.zhang <figo1802@gmail.com>
  Jul,30 2010: [v2] Resend:videobuf_dma_sg: a new implementation for mmap             http://patchwork.kernel.org/patch/115348  Figo.zhang <figo1802@gmail.com>


Other than that, we have a pending list of older patches, submitted by someone, that I'm still
waiting for the maintainer to take care. If I don't have any return from the maintainer for
a patch older than, let's say, a month, is probably because it is safe for me to apply.

		== mantis patches - Waiting for Manu Abraham <abraham.manu@gmail.com> == 

Apr,15 2010: [5/8] ir-core: convert mantis from ir-functions.c                      http://patchwork.kernel.org/patch/92961   David Härdeman <david@hardeman.nu>
Jun,20 2010: Mantis DMA transfer cleanup, fixes data corruption and a race, improve http://patchwork.kernel.org/patch/107036  Marko Ristola <marko.ristola@kolumbus.fi>
Jun,20 2010: [2/2] DVB/V4L: mantis: remove unused files                             http://patchwork.kernel.org/patch/107062  Bjørn Mork <bjorn@mork.no>
Jun,20 2010: mantis: use dvb_attach to avoid double dereferencing on module removal http://patchwork.kernel.org/patch/107063  Bjørn Mork <bjorn@mork.no>
Jun,21 2010: Mantis, hopper: use MODULE_DEVICE_TABLE use the macro to make modules  http://patchwork.kernel.org/patch/107147  Manu Abraham <abraham.manu@gmail.com>
Jul, 3 2010: mantis: Rename gpio_set_bits to mantis_gpio_set_bits                   http://patchwork.kernel.org/patch/109972  Ben Hutchings <ben@decadent.org.uk>
Jul, 8 2010: Mantis DMA transfer cleanup, fixes data corruption and a race, improve http://patchwork.kernel.org/patch/110909  Marko Ristola <marko.ristola@kolumbus.fi>
Jul, 9 2010: Mantis: append tasklet maintenance for DVB stream delivery             http://patchwork.kernel.org/patch/111090  Marko Ristola <marko.ristola@kolumbus.fi>
Jul,10 2010: Mantis driver patch: use interrupt for I2C traffic instead of busy reg http://patchwork.kernel.org/patch/111245  Marko Ristola <marko.ristola@kolumbus.fi>
Jul,19 2010: Twinhan DTV Ter-CI (3030 mantis)                                       http://patchwork.kernel.org/patch/112708  Niklas Claesson <nicke.claesson@gmail.com>
Aug, 7 2010: Refactor Mantis DMA transfer to deliver 16Kb TS data per interrupt     http://patchwork.kernel.org/patch/118173  Marko Ristola <marko.ristola@kolumbus.fi>
Oct,10 2010: [v2] V4L/DVB: faster DVB-S lock for mantis cards using stb0899 demod   http://patchwork.kernel.org/patch/244201  Tuxoholic <tuxoholic@hotmail.de>

		== Soc_camera waiting for Guennadi Liakhovetski <g.liakhovetski@gmx.de> review == 

(Guennadi gave me an update about those patches last time. I'm not sure about the updated status)

Jul, 5 2010: soc-camera: module_put() fix                                           http://patchwork.kernel.org/patch/110202  Magnus Damm <damm@opensource.se>
Jul,27 2010: [1/4] mx2_camera: fix a race causing NULL dereference                  http://patchwork.kernel.org/patch/114515  Baruch Siach <baruch@tkos.co.il>
Jul,27 2010: [2/4] mx2_camera: return IRQ_NONE when doing nothing                   http://patchwork.kernel.org/patch/114517  Baruch Siach <baruch@tkos.co.il>
Jul,27 2010: [4/4] mx2_camera: implement forced termination of active buffer for mx http://patchwork.kernel.org/patch/114518  Baruch Siach <baruch@tkos.co.il>
Aug, 3 2010: [2/5] mx2_camera: remove emma limitation for RGB565                    http://patchwork.kernel.org/patch/116703  Michael Grzeschik <m.grzeschik@pengutronix.de>
Aug, 3 2010: [3/5] mx2_camera: fix for list bufnum in frame_done_emma               http://patchwork.kernel.org/patch/116705  Michael Grzeschik <m.grzeschik@pengutronix.de>
Aug, 3 2010: [4/5] mx2_camera: add rising edge for pixclock                         http://patchwork.kernel.org/patch/116704  Michael Grzeschik <m.grzeschik@pengutronix.de>
Aug, 3 2010: [5/5] mx2_camera: add informative camera clock frequency printout      http://patchwork.kernel.org/patch/116707  Michael Grzeschik <m.grzeschik@pengutronix.de>
Aug, 3 2010: [04/11] mt9m111: added new bit offset defines                          http://patchwork.kernel.org/patch/116721  Michael Grzeschik <m.grzeschik@pengutronix.de>
Aug, 3 2010: [06/11] mt9m111: cropcap and s_crop check if type is VIDEO_CAPTURE     http://patchwork.kernel.org/patch/116726  Michael Grzeschik <m.grzeschik@pengutronix.de>
Aug, 3 2010: [07/11] mt9m111: added current colorspace at g_fmt                     http://patchwork.kernel.org/patch/116724  Michael Grzeschik <m.grzeschik@pengutronix.de>
Aug, 3 2010: [08/11] mt9m111: added reg_mask function                               http://patchwork.kernel.org/patch/116722  Michael Grzeschik <m.grzeschik@pengutronix.de>
Aug, 3 2010: [v2,10/11] mt9m111: rewrite set_pixfmt                                 http://patchwork.kernel.org/patch/116728  Michael Grzeschik <m.grzeschik@pengutronix.de>
Aug, 3 2010: [v2,11/11] mt9m111: make use of testpattern                            http://patchwork.kernel.org/patch/116730  Michael Grzeschik <m.grzeschik@pengutronix.de>

		== Waiting for Antti Palosaari <crope@iki.fi> review == 

Mar,21 2010: af9015 : more robust eeprom parsing                                    http://patchwork.kernel.org/patch/87243   matthieu castet <castet.matthieu@free.fr>

		== Waiting for Igor M. Liplianin <liplianin@tut.by> review == 

Mar,10 2010: DM1105: could not attach frontend 195d:1105                            http://patchwork.kernel.org/patch/84549   Hendrik Skarpeid <skarp@online.no>

		== Waiting for Jarod Wilson <jarod@redhat.com> review/ack == 

Jun,20 2010: drivers/media/IR/imon.c: Use pr_err instead of err                     http://patchwork.kernel.org/patch/107025  Joe Perches <joe@perches.com>

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

Jul, 8 2010: video:gspca.c Fix warning: case value '7' not in enumerated type 'enum http://patchwork.kernel.org/patch/110779  Jean-Francois Moine <moinejf@free.fr>

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

Jan,29 2010: [gspca_jf,tree] gspca zc3xx: signal when unknown packet received       http://patchwork.kernel.org/patch/75837   Németh Márton <nm127@freemail.hu>

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

May,25 2010: Adding support to the Geniatech/MyGica SBTVD Stick S870 remote control http://patchwork.kernel.org/patch/102314  Hernán Ordiales <h.ordiales@gmail.com>
Jul,14 2010: [1/4] drivers/media/dvb: Remove dead Configs                           http://patchwork.kernel.org/patch/111972  Christian Dietrich <qy03fugy@stud.informatik.uni-erlangen.de>
Jul,14 2010: [2/4] drivers/media/dvb: Remove undead configs                         http://patchwork.kernel.org/patch/111973  Christian Dietrich <qy03fugy@stud.informatik.uni-erlangen.de>

Cheers,
Mauro

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-17 21:20 Old patches sent via the Mailing list Mauro Carvalho Chehab
@ 2010-10-17 22:03 ` Guennadi Liakhovetski
  2010-10-17 23:08   ` Mauro Carvalho Chehab
  2010-10-17 22:27 ` Antti Palosaari
                   ` (5 subsequent siblings)
  6 siblings, 1 reply; 22+ messages in thread
From: Guennadi Liakhovetski @ 2010-10-17 22:03 UTC (permalink / raw)
  To: Mauro Carvalho Chehab
  Cc: LMML, Igor M. Liplianin, Manu Abraham, Jean-Francois Moine,
	Jarod Wilson, Richard Zidlicky, Antti Palosaari, Sven Barth,
	Patrick Boettcher, Henrik Kurelid, Hans de Goede

Hi Mauro

On Sun, 17 Oct 2010, Mauro Carvalho Chehab wrote:

> 		== Soc_camera waiting for Guennadi Liakhovetski <g.liakhovetski@gmx.de> review == 
> 
> (Guennadi gave me an update about those patches last time. I'm not sure about the updated status)
> 
> Jul, 5 2010: soc-camera: module_put() fix                                           http://patchwork.kernel.org/patch/110202  Magnus Damm <damm@opensource.se>

Please, mark as dropped

> Jul,27 2010: [1/4] mx2_camera: fix a race causing NULL dereference                  http://patchwork.kernel.org/patch/114515  Baruch Siach <baruch@tkos.co.il>

is upstream 
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=5384a12b23160e11ff949a94172051476d308b66

> Jul,27 2010: [2/4] mx2_camera: return IRQ_NONE when doing nothing                   http://patchwork.kernel.org/patch/114517  Baruch Siach <baruch@tkos.co.il>

mark as dropped

> Jul,27 2010: [4/4] mx2_camera: implement forced termination of active buffer for mx http://patchwork.kernel.org/patch/114518  Baruch Siach <baruch@tkos.co.il>

is in next 
http://git.kernel.org/?p=linux/kernel/git/next/linux-next.git;a=commit;h=9cf6ddf5eeedaffd989f9b93df1b7ea8d459786b

> Aug, 3 2010: [2/5] mx2_camera: remove emma limitation for RGB565                    http://patchwork.kernel.org/patch/116703  Michael Grzeschik <m.grzeschik@pengutronix.de>

is in next 
http://git.kernel.org/?p=linux/kernel/git/next/linux-next.git;a=commit;h=2b262a18b79768a2b7a62ba187f8830802790b9a

> Aug, 3 2010: [3/5] mx2_camera: fix for list bufnum in frame_done_emma               http://patchwork.kernel.org/patch/116705  Michael Grzeschik <m.grzeschik@pengutronix.de>

is upstream 
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=cd9ebdbc0541b4e8ee145c81642d68332f79b932

> Aug, 3 2010: [4/5] mx2_camera: add rising edge for pixclock                         http://patchwork.kernel.org/patch/116704  Michael Grzeschik <m.grzeschik@pengutronix.de>

upstream 
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=d86097e19cef2f13a29fc37db0dad17b99b6d5f8

> Aug, 3 2010: [5/5] mx2_camera: add informative camera clock frequency printout      http://patchwork.kernel.org/patch/116707  Michael Grzeschik <m.grzeschik@pengutronix.de>

in next 
http://git.kernel.org/?p=linux/kernel/git/next/linux-next.git;a=commit;h=e7d317b5b210a2f9486faa335e4eff81e5f6210d

> Aug, 3 2010: [04/11] mt9m111: added new bit offset defines                          http://patchwork.kernel.org/patch/116721  Michael Grzeschik <m.grzeschik@pengutronix.de>

waiting an update (possible merge with 08/11 and 10/11 below)

> Aug, 3 2010: [06/11] mt9m111: cropcap and s_crop check if type is VIDEO_CAPTURE     http://patchwork.kernel.org/patch/116726  Michael Grzeschik <m.grzeschik@pengutronix.de>

upstream 
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=6b6d33c746ace7bd0dbbdde674d3fb1100ab081d

> Aug, 3 2010: [07/11] mt9m111: added current colorspace at g_fmt                     http://patchwork.kernel.org/patch/116724  Michael Grzeschik <m.grzeschik@pengutronix.de>

upstream 
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=01f5a394eac48b74c84434e95e74cd172b0682c3

> Aug, 3 2010: [08/11] mt9m111: added reg_mask function                               http://patchwork.kernel.org/patch/116722  Michael Grzeschik <m.grzeschik@pengutronix.de>

waiting for update (see 04/11 above)

> Aug, 3 2010: [v2,10/11] mt9m111: rewrite set_pixfmt                                 http://patchwork.kernel.org/patch/116728  Michael Grzeschik <m.grzeschik@pengutronix.de>

waiting for update (see 04/11 above)

> Aug, 3 2010: [v2,11/11] mt9m111: make use of testpattern                            http://patchwork.kernel.org/patch/116730  Michael Grzeschik <m.grzeschik@pengutronix.de>

waiting for a new version, using additional inputs to provide test 
patterns.

Somehow, looks like patches, that I push to you, don't (automatically) get 
updated in patchwork, is there anything, that I'm doing wrongly, why this 
is happening?

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

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-17 21:20 Old patches sent via the Mailing list Mauro Carvalho Chehab
  2010-10-17 22:03 ` Guennadi Liakhovetski
@ 2010-10-17 22:27 ` Antti Palosaari
  2010-10-17 22:58   ` Mauro Carvalho Chehab
  2010-10-17 23:36 ` Andy Walls
                   ` (4 subsequent siblings)
  6 siblings, 1 reply; 22+ messages in thread
From: Antti Palosaari @ 2010-10-17 22:27 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: LMML

On 10/18/2010 12:20 AM, Mauro Carvalho Chehab wrote:
> Hi,
>
> I did a large effort during this weekend to handle the maximum amount of patches, in order to have them
> ready for 2.6.37. While there are still some patches marked as NEW at patchwork, and a few pending pull
> requests (mostly related to more kABI changes), there are still a list of patches that are marked as
> Under review. Except for 4 patches from me, related to Doc (that I'm keeping in this list just to remind
> me that I'll need to fix them when I have some time - just some automation stuff at DocBook), all other
> patches marked as Under review are stuff that I basically depend on others.
>
> The last time I sent this list, I was about to travel, and I may have missed some comments, or maybe I
> may just forgot to update. But I suspect that, for the list bellow, most of them are stuff where the
> driver maintainer just forgot at limbo.
>
>> From the list of patches under review, we have:
>

> 		== Waiting for Antti Palosaari<crope@iki.fi>  review ==
>
> Mar,21 2010: af9015 : more robust eeprom parsing                                    http://patchwork.kernel.org/patch/87243   matthieu castet<castet.matthieu@free.fr>


Mark as dropped.

Antti

-- 
http://palosaari.fi/

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-17 22:27 ` Antti Palosaari
@ 2010-10-17 22:58   ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 22+ messages in thread
From: Mauro Carvalho Chehab @ 2010-10-17 22:58 UTC (permalink / raw)
  To: Antti Palosaari; +Cc: LMML

Em 17-10-2010 20:27, Antti Palosaari escreveu:
> On 10/18/2010 12:20 AM, Mauro Carvalho Chehab wrote:
>> Hi,
>>
>> I did a large effort during this weekend to handle the maximum amount of patches, in order to have them
>> ready for 2.6.37. While there are still some patches marked as NEW at patchwork, and a few pending pull
>> requests (mostly related to more kABI changes), there are still a list of patches that are marked as
>> Under review. Except for 4 patches from me, related to Doc (that I'm keeping in this list just to remind
>> me that I'll need to fix them when I have some time - just some automation stuff at DocBook), all other
>> patches marked as Under review are stuff that I basically depend on others.
>>
>> The last time I sent this list, I was about to travel, and I may have missed some comments, or maybe I
>> may just forgot to update. But I suspect that, for the list bellow, most of them are stuff where the
>> driver maintainer just forgot at limbo.
>>
>>> From the list of patches under review, we have:
>>
> 
>>         == Waiting for Antti Palosaari<crope@iki.fi>  review ==
>>
>> Mar,21 2010: af9015 : more robust eeprom parsing                                    http://patchwork.kernel.org/patch/87243   matthieu castet<castet.matthieu@free.fr>
> 
> 
> Mark as dropped.

Done. Thanks!

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-17 22:03 ` Guennadi Liakhovetski
@ 2010-10-17 23:08   ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 22+ messages in thread
From: Mauro Carvalho Chehab @ 2010-10-17 23:08 UTC (permalink / raw)
  To: Guennadi Liakhovetski
  Cc: LMML, Igor M. Liplianin, Manu Abraham, Jean-Francois Moine,
	Jarod Wilson, Richard Zidlicky, Antti Palosaari, Sven Barth,
	Patrick Boettcher, Henrik Kurelid, Hans de Goede

Em 17-10-2010 20:03, Guennadi Liakhovetski escreveu:
> Hi Mauro
> 
> On Sun, 17 Oct 2010, Mauro Carvalho Chehab wrote:
> 
>> 		== Soc_camera waiting for Guennadi Liakhovetski <g.liakhovetski@gmx.de> review == 
>>
>> (Guennadi gave me an update about those patches last time. I'm not sure about the updated status)
>>
>> Jul, 5 2010: soc-camera: module_put() fix                                           http://patchwork.kernel.org/patch/110202  Magnus Damm <damm@opensource.se>
> 
> Please, mark as dropped
> 
>> Jul,27 2010: [1/4] mx2_camera: fix a race causing NULL dereference                  http://patchwork.kernel.org/patch/114515  Baruch Siach <baruch@tkos.co.il>
> 
> is upstream 
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=5384a12b23160e11ff949a94172051476d308b66
> 
>> Jul,27 2010: [2/4] mx2_camera: return IRQ_NONE when doing nothing                   http://patchwork.kernel.org/patch/114517  Baruch Siach <baruch@tkos.co.il>
> 
> mark as dropped
> 
>> Jul,27 2010: [4/4] mx2_camera: implement forced termination of active buffer for mx http://patchwork.kernel.org/patch/114518  Baruch Siach <baruch@tkos.co.il>
> 
> is in next 
> http://git.kernel.org/?p=linux/kernel/git/next/linux-next.git;a=commit;h=9cf6ddf5eeedaffd989f9b93df1b7ea8d459786b
> 
>> Aug, 3 2010: [2/5] mx2_camera: remove emma limitation for RGB565                    http://patchwork.kernel.org/patch/116703  Michael Grzeschik <m.grzeschik@pengutronix.de>
> 
> is in next 
> http://git.kernel.org/?p=linux/kernel/git/next/linux-next.git;a=commit;h=2b262a18b79768a2b7a62ba187f8830802790b9a
> 
>> Aug, 3 2010: [3/5] mx2_camera: fix for list bufnum in frame_done_emma               http://patchwork.kernel.org/patch/116705  Michael Grzeschik <m.grzeschik@pengutronix.de>
> 
> is upstream 
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=cd9ebdbc0541b4e8ee145c81642d68332f79b932
> 
>> Aug, 3 2010: [4/5] mx2_camera: add rising edge for pixclock                         http://patchwork.kernel.org/patch/116704  Michael Grzeschik <m.grzeschik@pengutronix.de>
> 
> upstream 
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=d86097e19cef2f13a29fc37db0dad17b99b6d5f8
> 
>> Aug, 3 2010: [5/5] mx2_camera: add informative camera clock frequency printout      http://patchwork.kernel.org/patch/116707  Michael Grzeschik <m.grzeschik@pengutronix.de>
> 
> in next 
> http://git.kernel.org/?p=linux/kernel/git/next/linux-next.git;a=commit;h=e7d317b5b210a2f9486faa335e4eff81e5f6210d
> 
>> Aug, 3 2010: [04/11] mt9m111: added new bit offset defines                          http://patchwork.kernel.org/patch/116721  Michael Grzeschik <m.grzeschik@pengutronix.de>
> 
> waiting an update (possible merge with 08/11 and 10/11 below)
> 
>> Aug, 3 2010: [06/11] mt9m111: cropcap and s_crop check if type is VIDEO_CAPTURE     http://patchwork.kernel.org/patch/116726  Michael Grzeschik <m.grzeschik@pengutronix.de>
> 
> upstream 
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=6b6d33c746ace7bd0dbbdde674d3fb1100ab081d
> 
>> Aug, 3 2010: [07/11] mt9m111: added current colorspace at g_fmt                     http://patchwork.kernel.org/patch/116724  Michael Grzeschik <m.grzeschik@pengutronix.de>
> 
> upstream 
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=01f5a394eac48b74c84434e95e74cd172b0682c3
> 
>> Aug, 3 2010: [08/11] mt9m111: added reg_mask function                               http://patchwork.kernel.org/patch/116722  Michael Grzeschik <m.grzeschik@pengutronix.de>
> 
> waiting for update (see 04/11 above)
> 
>> Aug, 3 2010: [v2,10/11] mt9m111: rewrite set_pixfmt                                 http://patchwork.kernel.org/patch/116728  Michael Grzeschik <m.grzeschik@pengutronix.de>
> 
> waiting for update (see 04/11 above)
> 
>> Aug, 3 2010: [v2,11/11] mt9m111: make use of testpattern                            http://patchwork.kernel.org/patch/116730  Michael Grzeschik <m.grzeschik@pengutronix.de>
> 
> waiting for a new version, using additional inputs to provide test 
> patterns.

Updated, thanks!
> 
> Somehow, looks like patches, that I push to you, don't (automatically) get 
> updated in patchwork, is there anything, that I'm doing wrongly, why this 
> is happening?

Unfortunately, they're not updated automatically. There's no easy way to
discover when a patch at a pull request were also sent to the ML. I need
to do it manually.

In fact, there's an easy way to allow me to automatically update the patchwork ID.
All you need is to add:

X-Patchwork-Id: <ID>

in the body of the email. My scripts use this tag to mark a patch as accepted/rejected
when I accept/reject a patch, as this tag is generated by patchwork.

Thanks,
Mauro

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-17 21:20 Old patches sent via the Mailing list Mauro Carvalho Chehab
  2010-10-17 22:03 ` Guennadi Liakhovetski
  2010-10-17 22:27 ` Antti Palosaari
@ 2010-10-17 23:36 ` Andy Walls
  2010-10-18  6:15   ` Mauro Carvalho Chehab
  2010-10-18  4:27 ` Németh Márton
                   ` (3 subsequent siblings)
  6 siblings, 1 reply; 22+ messages in thread
From: Andy Walls @ 2010-10-17 23:36 UTC (permalink / raw)
  To: Mauro Carvalho Chehab
  Cc: LMML, Igor M. Liplianin, Manu Abraham, Jean-Francois Moine,
	Jarod Wilson, Richard Zidlicky, Antti Palosaari, Sven Barth,
	Patrick Boettcher, Guennadi Liakhovetski, Henrik Kurelid,
	Hans de Goede, Sven Barth

On Sun, 2010-10-17 at 19:20 -0200, Mauro Carvalho Chehab wrote:
> Hi,
> 
> I did a large effort during this weekend to handle the maximum amount of patches, in order to have them
> ready for 2.6.37. While there are still some patches marked as NEW at patchwork, and a few pending pull
> requests (mostly related to more kABI changes), there are still a list of patches that are marked as
> Under review. Except for 4 patches from me, related to Doc (that I'm keeping in this list just to remind
> me that I'll need to fix them when I have some time - just some automation stuff at DocBook), all other
> patches marked as Under review are stuff that I basically depend on others.
> 
> The last time I sent this list, I was about to travel, and I may have missed some comments, or maybe I
> may just forgot to update. But I suspect that, for the list bellow, most of them are stuff where the
> driver maintainer just forgot at limbo.
> 
> >From the list of patches under review, we have:
> 
> Waiting for new patch, signed, from Sven Barth <pascaldragon@googlemail.com>
>   Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400                       http://patchwork.kernel.org/patch/94960   Sven Barth <pascaldragon@googlemail.com>

Sven,

We need a "Signed-off-by: " for your submitted patch:

http://www.linuxtv.org/wiki/index.php/Development:_Submitting_Patches#Sign_your_work

Note, your patch has an obvious, unintentional white space change for
"if (std == V4L2_STD_NTSC_M_JP)", so could you fix that up and send a
new signed off version?


Mauro,

This patch makes obvious sense to me: don't perform audio register
updates on a chip that doesn't have an audio processing block.  Sven's
approach was based on my recommended approach, after his initial
discovery on how to get his audio working.

Do we really need an S.O.B for something that appears to be common
sense, and wouldn't have been implemented any other way, even if I had
implemented it?

Regards,
Andy



^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-17 21:20 Old patches sent via the Mailing list Mauro Carvalho Chehab
                   ` (2 preceding siblings ...)
  2010-10-17 23:36 ` Andy Walls
@ 2010-10-18  4:27 ` Németh Márton
  2010-10-18  5:28   ` Mauro Carvalho Chehab
  2010-10-18  6:34 ` Jean-Francois Moine
                   ` (2 subsequent siblings)
  6 siblings, 1 reply; 22+ messages in thread
From: Németh Márton @ 2010-10-18  4:27 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: LMML, Hans de Goede

Mauro Carvalho Chehab wrote:
> 		== Gspca patches - Waiting Hans de Goede <hdegoede@redhat.com> review == 
> 
> Jan,29 2010: [gspca_jf,tree] gspca zc3xx: signal when unknown packet received       http://patchwork.kernel.org/patch/75837   Németh Márton <nm127@freemail.hu>

This was NACKed by Hans de Goede, as the pachwork link shows.
Please drop it.

	Márton Németh

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-18  4:27 ` Németh Márton
@ 2010-10-18  5:28   ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 22+ messages in thread
From: Mauro Carvalho Chehab @ 2010-10-18  5:28 UTC (permalink / raw)
  To: Németh Márton; +Cc: LMML, Hans de Goede

Em 18-10-2010 02:27, Németh Márton escreveu:
> Mauro Carvalho Chehab wrote:
>> 		== Gspca patches - Waiting Hans de Goede <hdegoede@redhat.com> review == 
>>
>> Jan,29 2010: [gspca_jf,tree] gspca zc3xx: signal when unknown packet received       http://patchwork.kernel.org/patch/75837   Németh Márton <nm127@freemail.hu>
> 
> This was NACKed by Hans de Goede, as the pachwork link shows.
> Please drop it.
> 
> 	Márton Németh

Updated,
thanks!

Cheers,
Mauro

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-17 23:36 ` Andy Walls
@ 2010-10-18  6:15   ` Mauro Carvalho Chehab
  2010-10-20  5:19     ` Sven Barth
  0 siblings, 1 reply; 22+ messages in thread
From: Mauro Carvalho Chehab @ 2010-10-18  6:15 UTC (permalink / raw)
  To: Andy Walls
  Cc: LMML, Igor M. Liplianin, Manu Abraham, Jean-Francois Moine,
	Jarod Wilson, Richard Zidlicky, Antti Palosaari, Sven Barth,
	Patrick Boettcher, Guennadi Liakhovetski, Henrik Kurelid,
	Hans de Goede

Em 17-10-2010 21:36, Andy Walls escreveu:
> On Sun, 2010-10-17 at 19:20 -0200, Mauro Carvalho Chehab wrote:
>> Hi,
>>
>> I did a large effort during this weekend to handle the maximum amount of patches, in order to have them
>> ready for 2.6.37. While there are still some patches marked as NEW at patchwork, and a few pending pull
>> requests (mostly related to more kABI changes), there are still a list of patches that are marked as
>> Under review. Except for 4 patches from me, related to Doc (that I'm keeping in this list just to remind
>> me that I'll need to fix them when I have some time - just some automation stuff at DocBook), all other
>> patches marked as Under review are stuff that I basically depend on others.
>>
>> The last time I sent this list, I was about to travel, and I may have missed some comments, or maybe I
>> may just forgot to update. But I suspect that, for the list bellow, most of them are stuff where the
>> driver maintainer just forgot at limbo.
>>
>> >From the list of patches under review, we have:
>>
>> Waiting for new patch, signed, from Sven Barth <pascaldragon@googlemail.com>
>>   Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400                       http://patchwork.kernel.org/patch/94960   Sven Barth <pascaldragon@googlemail.com>
> 
> Sven,
> 
> We need a "Signed-off-by: " for your submitted patch:
> 
> http://www.linuxtv.org/wiki/index.php/Development:_Submitting_Patches#Sign_your_work
> 
> Note, your patch has an obvious, unintentional white space change for
> "if (std == V4L2_STD_NTSC_M_JP)", so could you fix that up and send a
> new signed off version?
> 
> 
> Mauro,
> 
> This patch makes obvious sense to me: don't perform audio register
> updates on a chip that doesn't have an audio processing block.  Sven's
> approach was based on my recommended approach, after his initial
> discovery on how to get his audio working.
> 
> Do we really need an S.O.B for something that appears to be common
> sense, and wouldn't have been implemented any other way, even if I had
> implemented it?

The original patch were in the middle of a discussion, no proper description,
bad whitespacing, etc. It is better to let the patch author to fix those issues,
as they learn more about how to submit a patch.

Anyway, I agree with you, the patch is obvious, and can proceed without the SOB.
I did the usual CodingStyle fixups, put part of your above comment as the patch
description, together with your ack and moved it forward. One patch less on my queue ;)

Cheers,
Mauro

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-17 21:20 Old patches sent via the Mailing list Mauro Carvalho Chehab
                   ` (3 preceding siblings ...)
  2010-10-18  4:27 ` Németh Márton
@ 2010-10-18  6:34 ` Jean-Francois Moine
  2010-10-18 12:22   ` Mauro Carvalho Chehab
  2010-10-19 18:00 ` Jarod Wilson
  2010-11-12 14:43 ` [GIT PATCHES FOR 2.6.38] mantis for_2.6.38 Bjørn Mork
  6 siblings, 1 reply; 22+ messages in thread
From: Jean-Francois Moine @ 2010-10-18  6:34 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: LMML

On Sun, 17 Oct 2010 19:20:31 -0200
Mauro Carvalho Chehab <mchehab@redhat.com> wrote:

> 		== Gspca patches - Waiting Jean-Francois Moine
> <moinejf@free.fr> review == 
> 
> Jul, 8 2010: video:gspca.c Fix warning: case value '7' not in
> enumerated type 'enum http://patchwork.kernel.org/patch/110779
> Jean-Francois Moine <moinejf@free.fr>

Hi Mauro,

This patch is already applied.

Cheers.

-- 
Ken ar c'hentañ	|	      ** Breizh ha Linux atav! **
Jef		|		http://moinejf.free.fr/

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-18  6:34 ` Jean-Francois Moine
@ 2010-10-18 12:22   ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 22+ messages in thread
From: Mauro Carvalho Chehab @ 2010-10-18 12:22 UTC (permalink / raw)
  To: Jean-Francois Moine; +Cc: LMML

Em 18-10-2010 04:34, Jean-Francois Moine escreveu:
> On Sun, 17 Oct 2010 19:20:31 -0200
> Mauro Carvalho Chehab <mchehab@redhat.com> wrote:
> 
>> 		== Gspca patches - Waiting Jean-Francois Moine
>> <moinejf@free.fr> review == 
>>
>> Jul, 8 2010: video:gspca.c Fix warning: case value '7' not in
>> enumerated type 'enum http://patchwork.kernel.org/patch/110779
>> Jean-Francois Moine <moinejf@free.fr>
> 
> Hi Mauro,
> 
> This patch is already applied.

Updated, thanks!
> 
> Cheers.
> 


^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-17 21:20 Old patches sent via the Mailing list Mauro Carvalho Chehab
                   ` (4 preceding siblings ...)
  2010-10-18  6:34 ` Jean-Francois Moine
@ 2010-10-19 18:00 ` Jarod Wilson
  2010-11-12 14:43 ` [GIT PATCHES FOR 2.6.38] mantis for_2.6.38 Bjørn Mork
  6 siblings, 0 replies; 22+ messages in thread
From: Jarod Wilson @ 2010-10-19 18:00 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: LMML

On Sun, Oct 17, 2010 at 07:20:31PM -0200, Mauro Carvalho Chehab wrote:
> Hi,
> 
> I did a large effort during this weekend to handle the maximum amount of patches, in order to have them
> ready for 2.6.37. While there are still some patches marked as NEW at patchwork, and a few pending pull
> requests (mostly related to more kABI changes), there are still a list of patches that are marked as
> Under review. Except for 4 patches from me, related to Doc (that I'm keeping in this list just to remind
> me that I'll need to fix them when I have some time - just some automation stuff at DocBook), all other
> patches marked as Under review are stuff that I basically depend on others.
> 
> The last time I sent this list, I was about to travel, and I may have missed some comments, or maybe I
> may just forgot to update. But I suspect that, for the list bellow, most of them are stuff where the
> driver maintainer just forgot at limbo.
> 
> From the list of patches under review, we have:
...
> 		== Waiting for Jarod Wilson <jarod@redhat.com> review/ack == 
> 
> Jun,20 2010: drivers/media/IR/imon.c: Use pr_err instead of err                     http://patchwork.kernel.org/patch/107025  Joe Perches <joe@perches.com>

Acked.

-- 
Jarod Wilson
jarod@redhat.com


^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-18  6:15   ` Mauro Carvalho Chehab
@ 2010-10-20  5:19     ` Sven Barth
  2010-10-20 12:00       ` Andy Walls
  0 siblings, 1 reply; 22+ messages in thread
From: Sven Barth @ 2010-10-20  5:19 UTC (permalink / raw)
  To: Mauro Carvalho Chehab
  Cc: Andy Walls, LMML, Igor M. Liplianin, Manu Abraham,
	Jean-Francois Moine, Jarod Wilson, Richard Zidlicky,
	Antti Palosaari, Patrick Boettcher, Guennadi Liakhovetski,
	Henrik Kurelid, Hans de Goede

Am 18.10.2010 08:15, schrieb Mauro Carvalho Chehab:
> Em 17-10-2010 21:36, Andy Walls escreveu:
>> On Sun, 2010-10-17 at 19:20 -0200, Mauro Carvalho Chehab wrote:
>>> Hi,
>>>
>>> I did a large effort during this weekend to handle the maximum amount of patches, in order to have them
>>> ready for 2.6.37. While there are still some patches marked as NEW at patchwork, and a few pending pull
>>> requests (mostly related to more kABI changes), there are still a list of patches that are marked as
>>> Under review. Except for 4 patches from me, related to Doc (that I'm keeping in this list just to remind
>>> me that I'll need to fix them when I have some time - just some automation stuff at DocBook), all other
>>> patches marked as Under review are stuff that I basically depend on others.
>>>
>>> The last time I sent this list, I was about to travel, and I may have missed some comments, or maybe I
>>> may just forgot to update. But I suspect that, for the list bellow, most of them are stuff where the
>>> driver maintainer just forgot at limbo.
>>>
>>> > From the list of patches under review, we have:
>>>
>>> Waiting for new patch, signed, from Sven Barth<pascaldragon@googlemail.com>
>>>    Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400                       http://patchwork.kernel.org/patch/94960   Sven Barth<pascaldragon@googlemail.com>
>>
>> Sven,
>>
>> We need a "Signed-off-by: " for your submitted patch:
>>
>> http://www.linuxtv.org/wiki/index.php/Development:_Submitting_Patches#Sign_your_work
>>
>> Note, your patch has an obvious, unintentional white space change for
>> "if (std == V4L2_STD_NTSC_M_JP)", so could you fix that up and send a
>> new signed off version?
>>
>>
>> Mauro,
>>
>> This patch makes obvious sense to me: don't perform audio register
>> updates on a chip that doesn't have an audio processing block.  Sven's
>> approach was based on my recommended approach, after his initial
>> discovery on how to get his audio working.
>>
>> Do we really need an S.O.B for something that appears to be common
>> sense, and wouldn't have been implemented any other way, even if I had
>> implemented it?
>
> The original patch were in the middle of a discussion, no proper description,
> bad whitespacing, etc. It is better to let the patch author to fix those issues,
> as they learn more about how to submit a patch.
>
> Anyway, I agree with you, the patch is obvious, and can proceed without the SOB.
> I did the usual CodingStyle fixups, put part of your above comment as the patch
> description, together with your ack and moved it forward. One patch less on my queue ;)
>
> Cheers,
> Mauro

Eh... I thought I had superseeded it with the patch from 10th July (mail 
title: [PATCH] Add support for AUX_PLL on cx2583x chips). It included a 
"Signed-of by" from me as well as "Acked by" from Mike and Andy and I 
also excluded the whitespace change ^^

Regards,
Sven

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-20  5:19     ` Sven Barth
@ 2010-10-20 12:00       ` Andy Walls
  2010-10-20 18:01         ` Sven Barth
  0 siblings, 1 reply; 22+ messages in thread
From: Andy Walls @ 2010-10-20 12:00 UTC (permalink / raw)
  To: Sven Barth
  Cc: Mauro Carvalho Chehab, LMML, Igor M. Liplianin, Manu Abraham,
	Jean-Francois Moine, Jarod Wilson, Richard Zidlicky,
	Antti Palosaari, Patrick Boettcher, Guennadi Liakhovetski,
	Henrik Kurelid, Hans de Goede

On Wed, 2010-10-20 at 07:19 +0200, Sven Barth wrote:
> Am 18.10.2010 08:15, schrieb Mauro Carvalho Chehab:
> > Em 17-10-2010 21:36, Andy Walls escreveu:

> >>> The last time I sent this list, I was about to travel, and I may have missed some comments, or maybe I
> >>> may just forgot to update. But I suspect that, for the list bellow, most of them are stuff where the
> >>> driver maintainer just forgot at limbo.
> >>>
> >>> > From the list of patches under review, we have:
> >>>
> >>> Waiting for new patch, signed, from Sven Barth<pascaldragon@googlemail.com>
> >>>    Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400                       http://patchwork.kernel.org/patch/94960   Sven Barth<pascaldragon@googlemail.com>
> >>
> >> Sven,
> >>
> >> We need a "Signed-off-by: " for your submitted patch:
> >>
> >> http://www.linuxtv.org/wiki/index.php/Development:_Submitting_Patches#Sign_your_work
> >>
> >> Note, your patch has an obvious, unintentional white space change for
> >> "if (std == V4L2_STD_NTSC_M_JP)", so could you fix that up and send a
> >> new signed off version?

> 
> Eh... I thought I had superseeded it with the patch from 10th July (mail 
> title: [PATCH] Add support for AUX_PLL on cx2583x chips). It included a 
> "Signed-of by" from me as well as "Acked by" from Mike and Andy and I 
> also excluded the whitespace change ^^

Hi Sven,

http://www.mail-archive.com/linux-media@vger.kernel.org/msg20296.html

So you have.  How embarrassing. <:}

Thanks,
Andy




^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-20 12:00       ` Andy Walls
@ 2010-10-20 18:01         ` Sven Barth
  2010-10-21 11:44           ` Mauro Carvalho Chehab
  0 siblings, 1 reply; 22+ messages in thread
From: Sven Barth @ 2010-10-20 18:01 UTC (permalink / raw)
  To: Andy Walls; +Cc: Mauro Carvalho Chehab, LMML

On 20.10.2010 14:00, Andy Walls wrote:
> On Wed, 2010-10-20 at 07:19 +0200, Sven Barth wrote:
>> Am 18.10.2010 08:15, schrieb Mauro Carvalho Chehab:
>>> Em 17-10-2010 21:36, Andy Walls escreveu:
>
>>>>> The last time I sent this list, I was about to travel, and I may have missed some comments, or maybe I
>>>>> may just forgot to update. But I suspect that, for the list bellow, most of them are stuff where the
>>>>> driver maintainer just forgot at limbo.
>>>>>
>>>>>>  From the list of patches under review, we have:
>>>>>
>>>>> Waiting for new patch, signed, from Sven Barth<pascaldragon@googlemail.com>
>>>>>     Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400                       http://patchwork.kernel.org/patch/94960   Sven Barth<pascaldragon@googlemail.com>
>>>>
>>>> Sven,
>>>>
>>>> We need a "Signed-off-by: " for your submitted patch:
>>>>
>>>> http://www.linuxtv.org/wiki/index.php/Development:_Submitting_Patches#Sign_your_work
>>>>
>>>> Note, your patch has an obvious, unintentional white space change for
>>>> "if (std == V4L2_STD_NTSC_M_JP)", so could you fix that up and send a
>>>> new signed off version?
>
>>
>> Eh... I thought I had superseeded it with the patch from 10th July (mail
>> title: [PATCH] Add support for AUX_PLL on cx2583x chips). It included a
>> "Signed-of by" from me as well as "Acked by" from Mike and Andy and I
>> also excluded the whitespace change ^^
>
> Hi Sven,
>
> http://www.mail-archive.com/linux-media@vger.kernel.org/msg20296.html
>
> So you have.  How embarrassing.<:}

Well... it's a bit hard to keep the overview in this list. ;) I only saw 
this thread about "old patches" by pure luck.

And thank you for digging up the link, I only had the mail version lying 
around.

[And finally I won't have to patch v4l manually anymore... yippieh! I'm 
looking forward to 2.6.37 :D (Good that I use a distro (ArchLinux) that 
has a rolling release style ^^) ]

Regards,
Sven

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: Old patches sent via the Mailing list
  2010-10-20 18:01         ` Sven Barth
@ 2010-10-21 11:44           ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 22+ messages in thread
From: Mauro Carvalho Chehab @ 2010-10-21 11:44 UTC (permalink / raw)
  To: Sven Barth; +Cc: Andy Walls, LMML

Em 20-10-2010 16:01, Sven Barth escreveu:
> On 20.10.2010 14:00, Andy Walls wrote:
>> On Wed, 2010-10-20 at 07:19 +0200, Sven Barth wrote:
>>> Am 18.10.2010 08:15, schrieb Mauro Carvalho Chehab:
>>>> Em 17-10-2010 21:36, Andy Walls escreveu:
>>
>>>>>> The last time I sent this list, I was about to travel, and I may have missed some comments, or maybe I
>>>>>> may just forgot to update. But I suspect that, for the list bellow, most of them are stuff where the
>>>>>> driver maintainer just forgot at limbo.
>>>>>>
>>>>>>>  From the list of patches under review, we have:
>>>>>>
>>>>>> Waiting for new patch, signed, from Sven Barth<pascaldragon@googlemail.com>
>>>>>>     Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400                       http://patchwork.kernel.org/patch/94960   Sven Barth<pascaldragon@googlemail.com>
>>>>>
>>>>> Sven,
>>>>>
>>>>> We need a "Signed-off-by: " for your submitted patch:
>>>>>
>>>>> http://www.linuxtv.org/wiki/index.php/Development:_Submitting_Patches#Sign_your_work
>>>>>
>>>>> Note, your patch has an obvious, unintentional white space change for
>>>>> "if (std == V4L2_STD_NTSC_M_JP)", so could you fix that up and send a
>>>>> new signed off version?
>>
>>>
>>> Eh... I thought I had superseeded it with the patch from 10th July (mail
>>> title: [PATCH] Add support for AUX_PLL on cx2583x chips). It included a
>>> "Signed-of by" from me as well as "Acked by" from Mike and Andy and I
>>> also excluded the whitespace change ^^
>>
>> Hi Sven,
>>
>> http://www.mail-archive.com/linux-media@vger.kernel.org/msg20296.html
>>
>> So you have.  How embarrassing.<:}
> 
> Well... it's a bit hard to keep the overview in this list. ;) I only saw this thread about "old patches" by pure luck.
> 
> And thank you for digging up the link, I only had the mail version lying around.
> 
> [And finally I won't have to patch v4l manually anymore... yippieh! I'm looking forward to 2.6.37 :D (Good that I use a distro (ArchLinux) that has a rolling release style ^^) ]
> 

OK, I've replaced the non-signed patch to the signed one, at the new branch I've
created for the patches that I'll send during the merge window (staging/v2.6.37-rc1).

The reason why the new patch were not catched is that the emailer broke long lines on
your patch, so, patchwork didn't catch it.

Please, next time, be sure that you'll submit your patch with an emailer that don't break
long lines.

Thanks,
Mauro

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [GIT PATCHES FOR 2.6.38] mantis for_2.6.38
  2010-10-17 21:20 Old patches sent via the Mailing list Mauro Carvalho Chehab
                   ` (5 preceding siblings ...)
  2010-10-19 18:00 ` Jarod Wilson
@ 2010-11-12 14:43 ` Bjørn Mork
  2010-11-13 14:26   ` Mauro Carvalho Chehab
  6 siblings, 1 reply; 22+ messages in thread
From: Bjørn Mork @ 2010-11-12 14:43 UTC (permalink / raw)
  To: linux-media
  Cc: Mauro Carvalho Chehab, Marko Ristola, Manu Abraham, Ben Hutchings,
	Niklas Claesson, Tuxoholic

Hello, 

I've been waiting for this list of patchwork patches to be included for
quite a while, and have now taken the liberty to clean them up as
necessary and add them to a git tree, based on the current media_tree
for_v2.6.38 branch, with exceptions as noted below:

> 		== mantis patches - Waiting for Manu Abraham <abraham.manu@gmail.com> == 
>
> Apr,15 2010: [5/8] ir-core: convert mantis from ir-functions.c                      http://patchwork.kernel.org/patch/92961   David Härdeman <david@hardeman.nu>

already applied as commit f0bdee26a2dc904c463bae1c2ae9ad06f97f100d

> Jun,20 2010: Mantis DMA transfer cleanup, fixes data corruption and a race, improve http://patchwork.kernel.org/patch/107036  Marko Ristola <marko.ristola@kolumbus.fi>

duplicate of http://patchwork.kernel.org/patch/118173

> Jun,20 2010: [2/2] DVB/V4L: mantis: remove unused files                             http://patchwork.kernel.org/patch/107062  Bjørn Mork <bjorn@mork.no>
> Jun,20 2010: mantis: use dvb_attach to avoid double dereferencing on module removal http://patchwork.kernel.org/patch/107063  Bjørn Mork <bjorn@mork.no>
> Jun,21 2010: Mantis, hopper: use MODULE_DEVICE_TABLE use the macro to make modules  http://patchwork.kernel.org/patch/107147  Manu Abraham <abraham.manu@gmail.com>
> Jul, 3 2010: mantis: Rename gpio_set_bits to mantis_gpio_set_bits                   http://patchwork.kernel.org/patch/109972  Ben Hutchings <ben@decadent.org.uk>
> Jul, 8 2010: Mantis DMA transfer cleanup, fixes data corruption and a race, improve http://patchwork.kernel.org/patch/110909  Marko Ristola <marko.ristola@kolumbus.fi>

another duplicate of http://patchwork.kernel.org/patch/118173

> Jul, 9 2010: Mantis: append tasklet maintenance for DVB stream delivery             http://patchwork.kernel.org/patch/111090  Marko Ristola <marko.ristola@kolumbus.fi>
> Jul,10 2010: Mantis driver patch: use interrupt for I2C traffic instead of busy reg http://patchwork.kernel.org/patch/111245  Marko Ristola <marko.ristola@kolumbus.fi>
> Jul,19 2010: Twinhan DTV Ter-CI (3030 mantis)                                       http://patchwork.kernel.org/patch/112708  Niklas Claesson <nicke.claesson@gmail.com>

Missing Signed-off-by, and I'm also a bit confused wrt what the patch
actually is.  Needs further cleanup.

> Aug, 7 2010: Refactor Mantis DMA transfer to deliver 16Kb TS data per interrupt     http://patchwork.kernel.org/patch/118173  Marko Ristola <marko.ristola@kolumbus.fi>
> Oct,10 2010: [v2] V4L/DVB: faster DVB-S lock for mantis cards using stb0899 demod   http://patchwork.kernel.org/patch/244201  Tuxoholic <tuxoholic@hotmail.de>



The following changes since commit 

af9f14f7fc31f0d7b7cdf8f7f7f15a3c3794aea3    [media] IR: add tv power scancode to rc6 mce keymap

are available in the git repository at:

  git://git.mork.no/mantis.git for_2.6.38

Ben Hutchings (1):
      V4L/DVB: mantis: Rename gpio_set_bits to mantis_gpio_set_bits

Bjørn Mork (2):
      V4L/DVB: mantis: use dvb_attach to avoid double dereferencing on module removal
      V4L/DVB/V4L: mantis: remove unused files

Manu Abraham (1):
      Mantis, hopper: use MODULE_DEVICE_TABLE use the macro to make modules auto-loadable

Marko Ristola (3):
      V4L/DVB: mantis: append tasklet maintenance for DVB stream delivery
      Refactor Mantis DMA transfer to deliver 16Kb TS data per interrupt
      media: mantis: use interrupt for I2C traffic instead of busy registry polling

tuxoholic@hotmail.de (1):
      V4L/DVB: faster DVB-S lock for mantis cards using stb0899 demod

 drivers/media/dvb/frontends/stb0899_algo.c |   33 +++--
 drivers/media/dvb/mantis/hopper_cards.c    |    4 +-
 drivers/media/dvb/mantis/hopper_vp3028.c   |    6 +-
 drivers/media/dvb/mantis/mantis_cards.c    |   18 ++-
 drivers/media/dvb/mantis/mantis_common.h   |    9 +-
 drivers/media/dvb/mantis/mantis_core.c     |  235 ----------------------------
 drivers/media/dvb/mantis/mantis_core.h     |   57 -------
 drivers/media/dvb/mantis/mantis_dma.c      |   92 ++++-------
 drivers/media/dvb/mantis/mantis_dvb.c      |   17 ++-
 drivers/media/dvb/mantis/mantis_i2c.c      |  128 ++++++++++++----
 drivers/media/dvb/mantis/mantis_ioc.c      |    4 +-
 drivers/media/dvb/mantis/mantis_ioc.h      |    2 +-
 drivers/media/dvb/mantis/mantis_vp1033.c   |    2 +-
 drivers/media/dvb/mantis/mantis_vp1034.c   |   10 +-
 drivers/media/dvb/mantis/mantis_vp1041.c   |    6 +-
 drivers/media/dvb/mantis/mantis_vp2033.c   |    5 +-
 drivers/media/dvb/mantis/mantis_vp2040.c   |    4 +-
 drivers/media/dvb/mantis/mantis_vp3030.c   |    8 +-
 18 files changed, 208 insertions(+), 432 deletions(-)


Note that some of these patches will trigger checkpatch long line
warnings due to deliberate choices. I have no strong feelings about
reformatting them, but I believe the review is easier the less I have
changed the original patchworks patch...


I sincerely hope this will make your job easier.  Thanks for reviewing,


Bjørn


^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [GIT PATCHES FOR 2.6.38] mantis for_2.6.38
  2010-11-12 14:43 ` [GIT PATCHES FOR 2.6.38] mantis for_2.6.38 Bjørn Mork
@ 2010-11-13 14:26   ` Mauro Carvalho Chehab
  2010-11-13 14:45     ` Bjørn Mork
  2010-11-17 19:43     ` Marko Ristola
  0 siblings, 2 replies; 22+ messages in thread
From: Mauro Carvalho Chehab @ 2010-11-13 14:26 UTC (permalink / raw)
  To: Bjørn Mork
  Cc: linux-media, Marko Ristola, Manu Abraham, Ben Hutchings,
	Niklas Claesson, Tuxoholic

Em 12-11-2010 12:43, Bjørn Mork escreveu:
> Hello, 
> 
> I've been waiting for this list of patchwork patches to be included for
> quite a while, and have now taken the liberty to clean them up as
> necessary and add them to a git tree, based on the current media_tree
> for_v2.6.38 branch, with exceptions as noted below:
> 
>> 		== mantis patches - Waiting for Manu Abraham <abraham.manu@gmail.com> == 
>>
>> Apr,15 2010: [5/8] ir-core: convert mantis from ir-functions.c                      http://patchwork.kernel.org/patch/92961   David Härdeman <david@hardeman.nu>
> 
> already applied as commit f0bdee26a2dc904c463bae1c2ae9ad06f97f100d
> 
>> Jun,20 2010: Mantis DMA transfer cleanup, fixes data corruption and a race, improve http://patchwork.kernel.org/patch/107036  Marko Ristola <marko.ristola@kolumbus.fi>
> 
> duplicate of http://patchwork.kernel.org/patch/118173
> 
>> Jun,20 2010: [2/2] DVB/V4L: mantis: remove unused files                             http://patchwork.kernel.org/patch/107062  Bjørn Mork <bjorn@mork.no>
>> Jun,20 2010: mantis: use dvb_attach to avoid double dereferencing on module removal http://patchwork.kernel.org/patch/107063  Bjørn Mork <bjorn@mork.no>
>> Jun,21 2010: Mantis, hopper: use MODULE_DEVICE_TABLE use the macro to make modules  http://patchwork.kernel.org/patch/107147  Manu Abraham <abraham.manu@gmail.com>
>> Jul, 3 2010: mantis: Rename gpio_set_bits to mantis_gpio_set_bits                   http://patchwork.kernel.org/patch/109972  Ben Hutchings <ben@decadent.org.uk>
>> Jul, 8 2010: Mantis DMA transfer cleanup, fixes data corruption and a race, improve http://patchwork.kernel.org/patch/110909  Marko Ristola <marko.ristola@kolumbus.fi>
> 
> another duplicate of http://patchwork.kernel.org/patch/118173
> 
>> Jul, 9 2010: Mantis: append tasklet maintenance for DVB stream delivery             http://patchwork.kernel.org/patch/111090  Marko Ristola <marko.ristola@kolumbus.fi>
>> Jul,10 2010: Mantis driver patch: use interrupt for I2C traffic instead of busy reg http://patchwork.kernel.org/patch/111245  Marko Ristola <marko.ristola@kolumbus.fi>
>> Jul,19 2010: Twinhan DTV Ter-CI (3030 mantis)                                       http://patchwork.kernel.org/patch/112708  Niklas Claesson <nicke.claesson@gmail.com>
> 
> Missing Signed-off-by, and I'm also a bit confused wrt what the patch
> actually is.  Needs further cleanup.
> 
>> Aug, 7 2010: Refactor Mantis DMA transfer to deliver 16Kb TS data per interrupt     http://patchwork.kernel.org/patch/118173  Marko Ristola <marko.ristola@kolumbus.fi>
>> Oct,10 2010: [v2] V4L/DVB: faster DVB-S lock for mantis cards using stb0899 demod   http://patchwork.kernel.org/patch/244201  Tuxoholic <tuxoholic@hotmail.de>
> 

Patchwork updated. Thanks for checking those duplicated stuff.
> 
> 
> The following changes since commit 
> 
> af9f14f7fc31f0d7b7cdf8f7f7f15a3c3794aea3    [media] IR: add tv power scancode to rc6 mce keymap
> 
> are available in the git repository at:
> 
>   git://git.mork.no/mantis.git for_2.6.38

Didn't work:

git pull git://git.mork.no/mantis.git for_2.6.38
fatal: Couldn't find remote ref for_2.6.38

Cheers,
Mauro

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [GIT PATCHES FOR 2.6.38] mantis for_2.6.38
  2010-11-13 14:26   ` Mauro Carvalho Chehab
@ 2010-11-13 14:45     ` Bjørn Mork
  2010-11-17  2:31       ` Mauro Carvalho Chehab
  2010-11-17 19:43     ` Marko Ristola
  1 sibling, 1 reply; 22+ messages in thread
From: Bjørn Mork @ 2010-11-13 14:45 UTC (permalink / raw)
  To: Mauro Carvalho Chehab
  Cc: linux-media, Marko Ristola, Manu Abraham, Ben Hutchings,
	Niklas Claesson, Tuxoholic

Mauro Carvalho Chehab <mchehab@redhat.com> writes:

> Em 12-11-2010 12:43, Bjørn Mork escreveu:
>> 
>>   git://git.mork.no/mantis.git for_2.6.38
>
> Didn't work:
>
> git pull git://git.mork.no/mantis.git for_2.6.38
> fatal: Couldn't find remote ref for_2.6.38

Damn, sorry about that.  Was supposed to be 

git://git.mork.no/mantis.git for_v2.6.38


Bjørn

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [GIT PATCHES FOR 2.6.38] mantis for_2.6.38
  2010-11-13 14:45     ` Bjørn Mork
@ 2010-11-17  2:31       ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 22+ messages in thread
From: Mauro Carvalho Chehab @ 2010-11-17  2:31 UTC (permalink / raw)
  To: Bjørn Mork
  Cc: linux-media, Marko Ristola, Manu Abraham, Ben Hutchings,
	Niklas Claesson, Tuxoholic

Hi Bjørn,

Em 13-11-2010 12:45, Bjørn Mork escreveu:
> Mauro Carvalho Chehab <mchehab@redhat.com> writes:
> 
>> Em 12-11-2010 12:43, Bjørn Mork escreveu:
>>>
>>>   git://git.mork.no/mantis.git for_2.6.38
>>
>> Didn't work:
>>
>> git pull git://git.mork.no/mantis.git for_2.6.38
>> fatal: Couldn't find remote ref for_2.6.38
> 
> Damn, sorry about that.  Was supposed to be 
> 
> git://git.mork.no/mantis.git for_v2.6.38

Except when drivers are not maintained anymore (or when the patch is trivial), 
I wait for the driver author(s) to test the patches and ask me to pull (or for them to
reply that a patch is ok with his ack).

Manu sent a pull request with some of the long-standing Mantis patches tested
plus with some improvements for the frontend tuning. I've applied them today.

Please test, and give us a feedback.

Thanks,
Mauro

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [GIT PATCHES FOR 2.6.38] mantis for_2.6.38
  2010-11-13 14:26   ` Mauro Carvalho Chehab
  2010-11-13 14:45     ` Bjørn Mork
@ 2010-11-17 19:43     ` Marko Ristola
  2010-11-17 21:12       ` Mauro Carvalho Chehab
  1 sibling, 1 reply; 22+ messages in thread
From: Marko Ristola @ 2010-11-17 19:43 UTC (permalink / raw)
  To: Mauro Carvalho Chehab
  Cc: Bjørn Mork, linux-media, Manu Abraham, Ben Hutchings,
	Niklas Claesson, Tuxoholic


Hi Bjørn Mork and Mauro Carvalho Chehab.

The following patch was an experiment and wasn't actually meant for inclusion into the Kernel.

>>> Jul,10 2010: Mantis driver patch: use interrupt for I2C traffic instead of busy reg http://patchwork.kernel.org/patch/111245  Marko Ristola <marko.ristola@kolumbus.fi>

Would you please drop it from the Patchwork?
Would you Bjørn remove it from your Git repository?
The patch is ugly and it doesn't work, the broken part is active only with vp2033.

I'm sorry for the hassle.
Cheers, Marko Ristola

13.11.2010 16:26, Mauro Carvalho Chehab wrote:
> Em 12-11-2010 12:43, Bjørn Mork escreveu:
>> Hello, 
>>
>> I've been waiting for this list of patchwork patches to be included for
>> quite a while, and have now taken the liberty to clean them up as
>> necessary and add them to a git tree, based on the current media_tree
>> for_v2.6.38 branch, with exceptions as noted below:
>>
>>> 		== mantis patches - Waiting for Manu Abraham <abraham.manu@gmail.com> == 
>>>
>>> Jul,10 2010: Mantis driver patch: use interrupt for I2C traffic instead of busy reg http://patchwork.kernel.org/patch/111245  Marko Ristola <marko.ristola@kolumbus.fi>

>> The following changes since commit 
>>
>> af9f14f7fc31f0d7b7cdf8f7f7f15a3c3794aea3    [media] IR: add tv power scancode to rc6 mce keymap
>>
>> are available in the git repository at:
>>
>>   git://git.mork.no/mantis.git for_2.6.38
> 

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [GIT PATCHES FOR 2.6.38] mantis for_2.6.38
  2010-11-17 19:43     ` Marko Ristola
@ 2010-11-17 21:12       ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 22+ messages in thread
From: Mauro Carvalho Chehab @ 2010-11-17 21:12 UTC (permalink / raw)
  To: Marko Ristola
  Cc: Bjørn Mork, linux-media, Manu Abraham, Ben Hutchings,
	Niklas Claesson, Tuxoholic

Em 17-11-2010 17:43, Marko Ristola escreveu:
> 
> Hi Bjørn Mork and Mauro Carvalho Chehab.
> 
> The following patch was an experiment and wasn't actually meant for inclusion into the Kernel.
> 
>>>> Jul,10 2010: Mantis driver patch: use interrupt for I2C traffic instead of busy reg http://patchwork.kernel.org/patch/111245  Marko Ristola <marko.ristola@kolumbus.fi>
> 
> Would you please drop it from the Patchwork?

Updated, thanks.

Cheers,
Mauro

^ permalink raw reply	[flat|nested] 22+ messages in thread

end of thread, other threads:[~2010-11-17 21:13 UTC | newest]

Thread overview: 22+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-10-17 21:20 Old patches sent via the Mailing list Mauro Carvalho Chehab
2010-10-17 22:03 ` Guennadi Liakhovetski
2010-10-17 23:08   ` Mauro Carvalho Chehab
2010-10-17 22:27 ` Antti Palosaari
2010-10-17 22:58   ` Mauro Carvalho Chehab
2010-10-17 23:36 ` Andy Walls
2010-10-18  6:15   ` Mauro Carvalho Chehab
2010-10-20  5:19     ` Sven Barth
2010-10-20 12:00       ` Andy Walls
2010-10-20 18:01         ` Sven Barth
2010-10-21 11:44           ` Mauro Carvalho Chehab
2010-10-18  4:27 ` Németh Márton
2010-10-18  5:28   ` Mauro Carvalho Chehab
2010-10-18  6:34 ` Jean-Francois Moine
2010-10-18 12:22   ` Mauro Carvalho Chehab
2010-10-19 18:00 ` Jarod Wilson
2010-11-12 14:43 ` [GIT PATCHES FOR 2.6.38] mantis for_2.6.38 Bjørn Mork
2010-11-13 14:26   ` Mauro Carvalho Chehab
2010-11-13 14:45     ` Bjørn Mork
2010-11-17  2:31       ` Mauro Carvalho Chehab
2010-11-17 19:43     ` Marko Ristola
2010-11-17 21:12       ` Mauro Carvalho Chehab

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox