* 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