Skip to contentSkip to conversation

Status of the patches under review at LMML (60 patches)

DISCUSSION NAK Mauro Carvalho Chehab 12 participants 19 messages Jul 6, 2010 – Jul 10, 2010 Media (V4L/DVB)

lore
Share & tools
lore

Mauro Carvalho Chehab

author
#0 lore raw

This is the summary of the patches that are currently under review at Linux Media Mailing List <[email protected]>.
Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email).

P.S.: This email is c/c to the developers where some action is expected.
If you were copied, please review the patches, acking/nacking or submitting an update.

== New patches - waiting for some days for more review ==

Jul, 1 2010: v4l2-dev: fix memory leak                                              http://patchwork.kernel.org/patch/109191
Jul, 5 2010: soc-camera: module_put() fix                                           http://patchwork.kernel.org/patch/110202
Jul, 4 2010: IR/mceusb: unify and simplify different gen device init                http://patchwork.kernel.org/patch/110078
Jul, 6 2010: [1/2] Added Technisat Skystar USB HD CI                                http://patchwork.kernel.org/patch/110395
Jul, 6 2010: [2/2] Retrieve firmware for az6027                                     http://patchwork.kernel.org/patch/110394
		== Need more tests/acks from DVB users ==

Jun,27 2010: Avoid unnecessary data copying inside dvb_dmx_swfilter_204() function http://patchwork.kernel.org/patch/108274

		== Better to wait for videobuf changes ==

Mar,17 2010: [2/2] V4L/DVB: buf-dma-sg.c: support non-pageable user-allocated memor http://patchwork.kernel.org/patch/97263
May,26 2010: [1/2] media: Add timberdale video-in driver http://patchwork.kernel.org/patch/102414

		== Waiting for a new version from Jonathan Corbet <[email protected]> ==

Jun, 7 2010: Add the viafb video capture driver http://patchwork.kernel.org/patch/104840

		== Depends on input get_bigkeycode stuff that got postponed ==

May,20 2010: input: fix error at the default input_get_keycode call http://patchwork.kernel.org/patch/101122

		== Waiting for Pawel Osciak <[email protected]> comments - are they ready for review/merge ? ==

Jun,16 2010: [1/7] ARM: Samsung: Add FIMC driver register definition and platform h http://patchwork.kernel.org/patch/106457
Jun,16 2010: [2/7] ARM: Samsung: Add platform definitions for local FIMC/FIMD fifo http://patchwork.kernel.org/patch/106459
Jun,16 2010: [3/7] s3c-fb: Add v4l2 subdevice to support framebuffer local fifo inp http://patchwork.kernel.org/patch/106445
Jun,16 2010: [4/7] v4l: Add Samsung FIMC (video postprocessor) driver http://patchwork.kernel.org/patch/106448
Jun,16 2010: [5/7] ARM: S5PV210: Add fifo link definitions for fimc and framebuffer http://patchwork.kernel.org/patch/106447

Jun,16 2010: [6/7] ARM: S5PV210: enable FIMC on Aquila                              http://patchwork.kernel.org/patch/106449
Jun,16 2010: [7/7] ARM: S5PC100: enable FIMC on SMDKC100                            http://patchwork.kernel.org/patch/106454
Jun,28 2010: v4l: mem2mem_testdev: fix g_fmt NULL pointer dereference               http://patchwork.kernel.org/patch/108321
		== Soc_camera waiting for Guennadi Liakhovetski <[email protected]> review == 
May,11 2010: [v3] soc_camera_platform: Add necessary v4l2_subdev_video_ops method   http://patchwork.kernel.org/patch/98660
Jun,21 2010: mx2_camera: Add soc_camera support for i.MX25/i.MX27                   http://patchwork.kernel.org/patch/107126
Jun,21 2010: mx27: add support for the CSI device                                   http://patchwork.kernel.org/patch/107124
Jun,21 2010: mx25: add support for the CSI device                                   http://patchwork.kernel.org/patch/107125
Jul, 4 2010: mx2_camera: Add soc_camera support for i.MX25/i.MX27                   http://patchwork.kernel.org/patch/110090
		== mantis patches - waiting for Manu Abraham <[email protected]> ==

Apr,15 2010: [5/8] ir-core: convert mantis from ir-functions.c http://patchwork.kernel.org/patch/92961
Jun,11 2010: stb0899: Removed an extra byte sent at init on DiSEqC bus http://patchwork.kernel.org/patch/105621
Jun,20 2010: Mantis DMA transfer cleanup, fixes data corruption and a race, improve http://patchwork.kernel.org/patch/107036
Jun,20 2010: Mantis: append tasklet maintenance for DVB stream delivery http://patchwork.kernel.org/patch/107055
Jun,20 2010: [2/2] DVB/V4L: mantis: remove unused files http://patchwork.kernel.org/patch/107062
Jun,20 2010: mantis: use dvb_attach to avoid double dereferencing on module removal http://patchwork.kernel.org/patch/107063
Jun,21 2010: Mantis, hopper: use MODULE_DEVICE_TABLE use the macro to make modules http://patchwork.kernel.org/patch/107147
Jul, 3 2010: mantis: Rename gpio_set_bits to mantis_gpio_set_bits http://patchwork.kernel.org/patch/109972

		== Waiting for my fixes on Docbook ==

Feb,25 2010: DocBook/Makefile: Make it less verbose http://patchwork.kernel.org/patch/82076
Feb,25 2010: DocBook: Add rules to auto-generate some media docbooks http://patchwork.kernel.org/patch/82075
Feb,25 2010: DocBook/v4l/pixfmt.xml: Add missing formats for gspca cpia1 and sn9c20 http://patchwork.kernel.org/patch/82074
Feb,25 2010: v4l: document new Bayer and monochrome pixel formats http://patchwork.kernel.org/patch/82073

		== Waiting for Antti Palosaari <[email protected]> review ==

Mar,21 2010: af9015 : more robust eeprom parsing http://patchwork.kernel.org/patch/87243
May,20 2010: New NXP tda18218 tuner http://patchwork.kernel.org/patch/101170

		== Waiting for Tobias Lorenz <[email protected]> review ==

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

		== Waiting for its addition at linux-firmware and driver test by David Wong <[email protected]>  ==

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

		== Andy Walls <[email protected]> and Aleksandr Piskunov <[email protected]> are discussing around the solution ==

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

		== Waiting for Andy Walls <[email protected]> ==

Apr,10 2010: cx18: "missing audio" for analog recordings http://patchwork.kernel.org/patch/91879

		== Patches waiting for Patrick Boettcher <[email protected]> review ==

Jan,15 2010: remove obsolete conditionalizing on DVB_DIBCOM_DEBUG http://patchwork.kernel.org/patch/73147
May,25 2010: V4L/DVB: Adding support to the Geniatech/MyGica SBTVD Stick S870 http://patchwork.kernel.org/patch/102314

		== Gspca patches - Waiting Hans de Goede <[email protected]> submission/review ==

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

		== Gspca patches - Waiting Jean-Francois Moine <[email protected]> submission/review ==

Feb,24 2010: gspca pac7302: add USB PID range based on heuristics http://patchwork.kernel.org/patch/81612

		== Waiting for Steven Toth <[email protected]> review == 
Feb, 6 2010: cx23885: Enable Message Signaled Interrupts(MSI).                      http://patchwork.kernel.org/patch/77492
May, 5 2010: tda10048: fix the uncomplete function tda10048_read_ber                http://patchwork.kernel.org/patch/97058
May, 6 2010: tda10048: fix bitmask for the transmission mode                        http://patchwork.kernel.org/patch/97340
May, 6 2010: tda10048: clear the uncorrected packet registers when saturated        http://patchwork.kernel.org/patch/97341
May, 6 2010: dvb_frontend: fix typos in comments and one function                   http://patchwork.kernel.org/patch/97343
		== Waiting for Henrik Kurelid <[email protected]> comments about an userspace patch for MythTV ==

Mar, 1 2010: firedtv: add parameter to fake ca_system_ids in CA_INFO http://patchwork.kernel.org/patch/82912

		== Waiting for Mike Isely <[email protected]> review ==

Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400 http://patchwork.kernel.org/patch/94960

		== Waiting for Igor M. Liplianin <[email protected]> comments/review ==

Mar,10 2010: DM1105: could not attach frontend 195d:1105 http://patchwork.kernel.org/patch/84549

		== Need some fixes from Uwe Kleine-Koenig <[email protected]> ==

Mar,16 2010: mx1-camera: compile fix http://patchwork.kernel.org/patch/86106

		== Waiting for Signed-off-by from Richard Zidlicky <[email protected]> ==

Jun,15 2010: support for Hauppauge WinTV MiniStic IR remote http://patchwork.kernel.org/patch/106247

		== Waiting for Eduardo Valentin <[email protected]> review ==

Jun,13 2010: [1/2] V4L/DVB: radio-si4713: Release i2c adapter in driver cleanup pat http://patchwork.kernel.org/patch/105846
Jun,13 2010: [2/2] V4L/DVB: radio-si4713: Add regulator framework support http://patchwork.kernel.org/patch/105847

		== Waiting for Jarod Wilson <[email protected]> review/ack ==

Jun,20 2010: drivers/media/IR/imon.c: Use pr_err instead of err http://patchwork.kernel.org/patch/107025

		== Waiting for Laurent Pinchart <[email protected]> review/ack/nack ==

Jun,23 2010: [2.6.33.4] V4L/uvcvideo: Add support for Suyin Corp. Lenovo Webcam http://patchwork.kernel.org/patch/107570

		== Patch(broken) - waiting for Manu Abraham <[email protected]> submission ==

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

Cheers,
Mauro

---

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

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

NAK
#1 lore raw

Hi Mauro,

Quoted 8 lines · On Tuesday 06 July 2010 15:06:39 Mauro Carvalho Chehab wrote:

This is the summary of the patches that are currently under review at Linux Media Mailing List <[email protected]>.
Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email).

P.S.: This email is c/c to the developers where some action is expected.
If you were copied, please review the patches, acking/nacking or submitting an update.

[snip]

Quoted 2 lines

Jun,23 2010: [2.6.33.4] V4L/uvcvideo: Add support for Suyin Corp. Lenovo
Webcam http://patchwork.kernel.org/patch/107570

NAK, the patch is not needed.

Signature · 3 lines
Regards,

Laurent Pinchart
#3 lore raw
In reply to Mauro Carvalho Chehab
Quoted 7 lines

               == Waiting for Steven Toth <[email protected]> review ==

Feb, 6 2010: cx23885: Enable Message Signaled Interrupts(MSI).                      http://patchwork.kernel.org/patch/77492
May, 5 2010: tda10048: fix the uncomplete function tda10048_read_ber                http://patchwork.kernel.org/patch/97058
May, 6 2010: tda10048: fix bitmask for the transmission mode                        http://patchwork.kernel.org/patch/97340
May, 6 2010: tda10048: clear the uncorrected packet registers when saturated        http://patchwork.kernel.org/patch/97341
May, 6 2010: dvb_frontend: fix typos in comments and one function                   http://patchwork.kernel.org/patch/97343

Mauro,

I'm fine with all of these.

Signed-off-by Steven Toth <[email protected]>
Signature · 2 lines
Steven Toth - Kernel Labs
http://www.kernellabs.com
author
#2 lore raw
In reply to Laurent Pinchart

Em 06-07-2010 10:13, Laurent Pinchart escreveu:

Quoted 12 lines

Hi Mauro,

Quoted 8 lines · On Tuesday 06 July 2010 15:06:39 Mauro Carvalho Chehab wrote:

This is the summary of the patches that are currently under review at Linux Media Mailing List <[email protected]>.
Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email).

P.S.: This email is c/c to the developers where some action is expected.
If you were copied, please review the patches, acking/nacking or submitting an update.

Quoted 7 lines

[snip]

Quoted 2 lines

Jun,23 2010: [2.6.33.4] V4L/uvcvideo: Add support for Suyin Corp. Lenovo
Webcam http://patchwork.kernel.org/patch/107570

NAK, the patch is not needed.

updated, thanks!

Cheers,
Mauro

#5 lore raw
In reply to Mauro Carvalho Chehab

Hello,

Quoted 7 lines
		== Soc_camera waiting for Guennadi Liakhovetski <[email protected]> review == 
May,11 2010: [v3] soc_camera_platform: Add necessary v4l2_subdev_video_ops method   http://patchwork.kernel.org/patch/98660
Jun,21 2010: mx2_camera: Add soc_camera support for i.MX25/i.MX27                   http://patchwork.kernel.org/patch/107126
Jun,21 2010: mx27: add support for the CSI device                                   http://patchwork.kernel.org/patch/107124
Jun,21 2010: mx25: add support for the CSI device                                   http://patchwork.kernel.org/patch/107125
Jul, 4 2010: mx2_camera: Add soc_camera support for i.MX25/i.MX27                   http://patchwork.kernel.org/patch/110090

The 2nd patch in the list above is obsoleted by the last one. The last one can have my Ack.

Best regards
Uwe

Signature · 2 lines
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
#6 lore raw
In reply to Mauro Carvalho Chehab
Quoted 9 lines · On Tue, 06 Jul 2010 10:06:39 -0300 Mauro Carvalho Chehab wrote:

This is the summary of the patches that are currently under review at Linux Media Mailing List <[email protected]>.
Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email).

P.S.: This email is c/c to the developers where some action is expected.
If you were copied, please review the patches, acking/nacking or submitting an update.

Quoted 7 lines

== New patches - waiting for some days for more review ==

Jul, 1 2010: v4l2-dev: fix memory leak                                              http://patchwork.kernel.org/patch/109191
Jul, 5 2010: soc-camera: module_put() fix                                           http://patchwork.kernel.org/patch/110202
Jul, 4 2010: IR/mceusb: unify and simplify different gen device init                http://patchwork.kernel.org/patch/110078
Jul, 6 2010: [1/2] Added Technisat Skystar USB HD CI                                http://patchwork.kernel.org/patch/110395
Jul, 6 2010: [2/2] Retrieve firmware for az6027                                     http://patchwork.kernel.org/patch/110394

Hi Mauro,
Is it possible to add Author (or From:) to each of these?

thanks,
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

author
#7 lore raw
In reply to Randy Dunlap

Em 06-07-2010 12:21, Randy Dunlap escreveu:

Quoted 20 lines · On Tue, 06 Jul 2010 10:06:39 -0300 Mauro Carvalho Chehab wrote:
Quoted 10 lines

This is the summary of the patches that are currently under review at Linux Media Mailing List <[email protected]>.
Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email).

P.S.: This email is c/c to the developers where some action is expected.
If you were copied, please review the patches, acking/nacking or submitting an update.

Quoted 7 lines

== New patches - waiting for some days for more review ==

Jul, 1 2010: v4l2-dev: fix memory leak                                              http://patchwork.kernel.org/patch/109191
Jul, 5 2010: soc-camera: module_put() fix                                           http://patchwork.kernel.org/patch/110202
Jul, 4 2010: IR/mceusb: unify and simplify different gen device init                http://patchwork.kernel.org/patch/110078
Jul, 6 2010: [1/2] Added Technisat Skystar USB HD CI                                http://patchwork.kernel.org/patch/110395
Jul, 6 2010: [2/2] Retrieve firmware for az6027                                     http://patchwork.kernel.org/patch/110394
Quoted 2 lines

Hi Mauro,
Is it possible to add Author (or From:) to each of these?

It is easier to just modify the script that generates the email ;)

Ok, that's the updated list of patches (52 patches). I've updated the stuff at patchwork to reflect the recent comments I received so far.

Also, one of the new patches got merged.

Cheers,
Mauro.

---

		== New patches == 
Jul, 1 2010: v4l2-dev: fix memory leak                                              http://patchwork.kernel.org/patch/109191  Anatolij Gustschin <[email protected]>
Jul, 5 2010: soc-camera: module_put() fix                                           http://patchwork.kernel.org/patch/110202  Magnus Damm <[email protected]>
Jul, 6 2010: [2/2] Retrieve firmware for az6027                                     http://patchwork.kernel.org/patch/110394  Renzo Dani <[email protected]>
Jul, 6 2010: [1/2] Added Technisat Skystar USB HD CI                                http://patchwork.kernel.org/patch/110395  Renzo Dani <[email protected]>
		== Need more tests/acks from DVB users ==

Jun,27 2010: Avoid unnecessary data copying inside dvb_dmx_swfilter_204() function http://patchwork.kernel.org/patch/108274 Marko Ristola <[email protected]>

		== Better to wait for videobuf changes ==

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 <[email protected]>
May,26 2010: [1/2] media: Add timberdale video-in driver http://patchwork.kernel.org/patch/102414 Richard Röjfors <[email protected]>

		== Waiting for a new version from Jonathan Corbet <[email protected]> ==

Jun, 7 2010: Add the viafb video capture driver http://patchwork.kernel.org/patch/104840 Jonathan Corbet <[email protected]>

		== Depends on input get_bigkeycode stuff that got postponed ==

May,20 2010: input: fix error at the default input_get_keycode call http://patchwork.kernel.org/patch/101122 Mauro Carvalho Chehab <[email protected]>

		== Waiting for Pawel Osciak <[email protected]> comments == 
Jun,16 2010: [1/7] ARM: Samsung: Add FIMC driver register definition and platform h http://patchwork.kernel.org/patch/106457  Sylwester Nawrocki <[email protected]>
Jun,16 2010: [2/7] ARM: Samsung: Add platform definitions for local FIMC/FIMD fifo  http://patchwork.kernel.org/patch/106459  Marek Szyprowski <[email protected]>
Jun,16 2010: [3/7] s3c-fb: Add v4l2 subdevice to support framebuffer local fifo inp http://patchwork.kernel.org/patch/106445  Sylwester Nawrocki <[email protected]>
Jun,16 2010: [4/7] v4l: Add Samsung FIMC (video postprocessor) driver               http://patchwork.kernel.org/patch/106448  Sylwester Nawrocki <[email protected]>
Jun,16 2010: [5/7] ARM: S5PV210: Add fifo link definitions for fimc and framebuffer http://patchwork.kernel.org/patch/106447  Marek Szyprowski <[email protected]>
Jun,16 2010: [6/7] ARM: S5PV210: enable FIMC on Aquila                              http://patchwork.kernel.org/patch/106449  Sylwester Nawrocki <[email protected]>
Jun,16 2010: [7/7] ARM: S5PC100: enable FIMC on SMDKC100                            http://patchwork.kernel.org/patch/106454  Sylwester Nawrocki <[email protected]>
Jun,28 2010: v4l: mem2mem_testdev: fix g_fmt NULL pointer dereference               http://patchwork.kernel.org/patch/108321  Pawel Osciak <[email protected]>
		== Soc_camera waiting for Guennadi Liakhovetski <[email protected]> review ==

May,11 2010: [v3] soc_camera_platform: Add necessary v4l2_subdev_video_ops method http://patchwork.kernel.org/patch/98660 Kuninori Morimoto <[email protected]>

		== lmml_107126_patchv4_1_3_mx2_camera_add_soc_camera_support_for_i_mx25_i_mx27.patch == 
Jun,21 2010: mx27: add support for the CSI device                                   http://patchwork.kernel.org/patch/107124  Baruch Siach <[email protected]>
Jun,21 2010: mx25: add support for the CSI device                                   http://patchwork.kernel.org/patch/107125  Baruch Siach <[email protected]>
Jul, 4 2010: mx2_camera: Add soc_camera support for i.MX25/i.MX27                   http://patchwork.kernel.org/patch/110090  Baruch Siach <[email protected]>
		== mantis patches - waiting for Manu Abraham <[email protected]> == 
Apr,15 2010: [5/8] ir-core: convert mantis from ir-functions.c                      http://patchwork.kernel.org/patch/92961   David Härdeman <[email protected]>
Jun,11 2010: stb0899: Removed an extra byte sent at init on DiSEqC bus              http://patchwork.kernel.org/patch/105621  Florent AUDEBERT <[email protected]>
Jun,20 2010: Mantis DMA transfer cleanup, fixes data corruption and a race, improve http://patchwork.kernel.org/patch/107036  Marko Ristola <[email protected]>
Jun,20 2010: Mantis: append tasklet maintenance for DVB stream delivery             http://patchwork.kernel.org/patch/107055  Marko Ristola <[email protected]>
Jun,20 2010: [2/2] DVB/V4L: mantis: remove unused files                             http://patchwork.kernel.org/patch/107062  Bjørn Mork <[email protected]>
Jun,20 2010: mantis: use dvb_attach to avoid double dereferencing on module removal http://patchwork.kernel.org/patch/107063  Bjørn Mork <[email protected]>
Jun,21 2010: Mantis, hopper: use MODULE_DEVICE_TABLE use the macro to make modules  http://patchwork.kernel.org/patch/107147  Manu Abraham <[email protected]>
Jul, 3 2010: mantis: Rename gpio_set_bits to mantis_gpio_set_bits                   http://patchwork.kernel.org/patch/109972  Ben Hutchings <[email protected]>
		== Waiting for my fixes on Docbook == 
Feb,25 2010: DocBook/Makefile: Make it less verbose                                 http://patchwork.kernel.org/patch/82076   Mauro Carvalho Chehab <[email protected]>
Feb,25 2010: DocBook: Add rules to auto-generate some media docbooks                http://patchwork.kernel.org/patch/82075   Mauro Carvalho Chehab <[email protected]>
Feb,25 2010: DocBook/v4l/pixfmt.xml: Add missing formats for gspca cpia1 and sn9c20 http://patchwork.kernel.org/patch/82074   Mauro Carvalho Chehab <[email protected]>
Feb,25 2010: v4l: document new Bayer and monochrome pixel formats                   http://patchwork.kernel.org/patch/82073   Mauro Carvalho Chehab <[email protected]>
		== Waiting for Antti Palosaari <[email protected]> review == 
Mar,21 2010: af9015 : more robust eeprom parsing                                    http://patchwork.kernel.org/patch/87243   matthieu castet <[email protected]>
May,20 2010: New NXP tda18218 tuner                                                 http://patchwork.kernel.org/patch/101170  Nikola Pajkovsky <[email protected]>
		== Waiting for Tobias Lorenz <[email protected]> review ==

Feb, 3 2010: radio-si470x-common: -EINVAL overwritten in si470x_vidioc_s_tuner() http://patchwork.kernel.org/patch/76786 Roel Kluin <[email protected]>

		== Waiting for its addition at linux-firmware and driver test by David Wong <[email protected]>  ==

Nov, 1 2009: lgs8gxx: remove firmware for lgs8g75 http://patchwork.kernel.org/patch/56822 Ben Hutchings <[email protected]>

		== Andy Walls <[email protected]> and Aleksandr Piskunov <[email protected]> are discussing around the solution ==

Oct,11 2009: AVerTV MCE 116 Plus radio http://patchwork.kernel.org/patch/52981 Aleksandr V. Piskunov <[email protected]>

		== Waiting for Andy Walls <[email protected]> ==

Apr,10 2010: cx18: "missing audio" for analog recordings http://patchwork.kernel.org/patch/91879 Mark Lord <[email protected]>

		== Patches waiting for Patrick Boettcher <[email protected]> review ==

Jan,15 2010: remove obsolete conditionalizing on DVB_DIBCOM_DEBUG http://patchwork.kernel.org/patch/73147 Christoph Egger <[email protected]>
May,25 2010: Adding support to the Geniatech/MyGica SBTVD Stick S870 remote control http://patchwork.kernel.org/patch/102314 Hernán Ordiales <[email protected]>

		== Gspca patches - Waiting Hans de Goede <[email protected]> submission/review ==

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

		== Waiting for Henrik Kurelid <[email protected]> comments about an userspace patch for MythTV ==

Mar, 1 2010: firedtv: add parameter to fake ca_system_ids in CA_INFO http://patchwork.kernel.org/patch/82912 Henrik Kurelid <[email protected]>

		== Waiting for Mike Isely <[email protected]> review ==

Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400 http://patchwork.kernel.org/patch/94960 Sven Barth <[email protected]>

		== Waiting for Igor M. Liplianin <[email protected]> comments/review ==

Mar,10 2010: DM1105: could not attach frontend 195d:1105 http://patchwork.kernel.org/patch/84549 Hendrik Skarpeid <[email protected]>

		== Need some fixes from Uwe Kleine-Koenig <[email protected]> ==

Mar,16 2010: mx1-camera: compile fix http://patchwork.kernel.org/patch/86106 Uwe Kleine-König <[email protected]>

		== Waiting for Signed-off-by from Richard Zidlicky <[email protected]> ==

Jun,15 2010: support for Hauppauge WinTV MiniStic IR remote http://patchwork.kernel.org/patch/106247 Richard Zidlicky <[email protected]>

		== Waiting for Eduardo Valentin <[email protected]> review ==

Jun,13 2010: [1/2] V4L/DVB: radio-si4713: Release i2c adapter in driver cleanup pat http://patchwork.kernel.org/patch/105846 Jarkko Nikula <[email protected]>
Jun,13 2010: [2/2] V4L/DVB: radio-si4713: Add regulator framework support http://patchwork.kernel.org/patch/105847 Jarkko Nikula <[email protected]>

		== Waiting for Jarod Wilson <[email protected]> review/ack ==

Jun,20 2010: drivers/media/IR/imon.c: Use pr_err instead of err http://patchwork.kernel.org/patch/107025 Joe Perches <[email protected]>

		== Patch(broken) - waiting for Manu Abraham <[email protected]> submission ==

Feb,16 2010: Twinhan 1027 + IR Port support http://patchwork.kernel.org/patch/79753 Sergey Ivanov <[email protected]>

		== Patches under review ==

Nov,12 2009: Locking in Siano driver (untested) http://patchwork.kernel.org/patch/59590 Tim Borgeaud <[email protected]>

#8 lore raw
Quoted 21 lines · Mauro Carvalho Chehab

Em 06-07-2010 12:21, Randy Dunlap escreveu:

Quoted 20 lines · On Tue, 06 Jul 2010 10:06:39 -0300 Mauro Carvalho Chehab wrote:
Quoted 17 lines

This is the summary of the patches that are currently under review at Linux Media Mailing List <[email protected]>.
Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email).

P.S.: This email is c/c to the developers where some action is expected.
If you were copied, please review the patches, acking/nacking or submitting an update.

== New patches - waiting for some days for more review ==

Jul, 1 2010: v4l2-dev: fix memory leak                                              http://patchwork.kernel.org/patch/109191
Jul, 5 2010: soc-camera: module_put() fix                                           http://patchwork.kernel.org/patch/110202
Jul, 4 2010: IR/mceusb: unify and simplify different gen device init                http://patchwork.kernel.org/patch/110078
Jul, 6 2010: [1/2] Added Technisat Skystar USB HD CI                                http://patchwork.kernel.org/patch/110395
Jul, 6 2010: [2/2] Retrieve firmware for az6027                                     http://patchwork.kernel.org/patch/110394
Quoted 4 lines
Quoted 2 lines

Hi Mauro,
Is it possible to add Author (or From:) to each of these?

It is easier to just modify the script that generates the email ;)

Yes, thanks for doing that.

Quoted 7 lines

Ok, that's the updated list of patches (52 patches). I've updated the stuff at patchwork to reflect the recent comments I received so far.

Also, one of the new patches got merged.

Cheers,
Mauro.

Signature · 2 lines
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
author
#4 lore raw
In reply to Steven Toth

Em 06-07-2010 10:53, Steven Toth escreveu:

Quoted 8 lines
Quoted 7 lines
               == Waiting for Steven Toth <[email protected]> review ==
Feb, 6 2010: cx23885: Enable Message Signaled Interrupts(MSI).                      http://patchwork.kernel.org/patch/77492
May, 5 2010: tda10048: fix the uncomplete function tda10048_read_ber                http://patchwork.kernel.org/patch/97058
May, 6 2010: tda10048: fix bitmask for the transmission mode                        http://patchwork.kernel.org/patch/97340
May, 6 2010: tda10048: clear the uncorrected packet registers when saturated        http://patchwork.kernel.org/patch/97341
May, 6 2010: dvb_frontend: fix typos in comments and one function                   http://patchwork.kernel.org/patch/97343
Quoted 6 lines

Mauro,

I'm fine with all of these.

Signed-off-by Steven Toth <[email protected]>

Added, thanks!

Cheers,
Mauro

NAK
#10 lore raw
In reply to Mauro Carvalho Chehab
Quoted 5 lines · Mauro Carvalho Chehab

This is the summary of the patches that are currently under review at Linux Media Mailing List<[email protected]>.
Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email).

Quoted 8 lines

P.S.: This email is c/c to the developers where some action is expected.
If you were copied, please review the patches, acking/nacking or submitting an update.

== Waiting for Antti Palosaari<[email protected]> review ==

Mar,21 2010: af9015 : more robust eeprom parsing http://patchwork.kernel.org/patch/87243

NACK, partly. I think it is rather useless.

Quoted 1 line

May,20 2010: New NXP tda18218 tuner http://patchwork.kernel.org/patch/101170

AF9015/AF9013: ACK, partly from the AF9015/AF9013 side. It is safe to merge, it will not break any currently supported device. But I am not sure if all settings are correct since I don't have suitable device (AF9015+TDA18218) to figure out configuration and test.

TDA18218: I don't know. I have reviewed it, feedback can be found from the patchwork. I don't resist to merge, but also I don't want to take any responsibility since I don't have this device.

regards
Antti

Signature · 1 line
http://palosaari.fi/
#11 lore raw

I know that a couple of users(including) tested the patch to be working. Many users(from forum) is still looking for tda18218 support or have been using a very old hg changeset using another patch also written by Nikola.

I have been using that old change set for many months and its working well for me.

Quoted 41 lines · Antti Palosaari
Quoted 11 lines · Mauro Carvalho Chehab

This is the summary of the patches that are currently under review at Linux Media Mailing List<[email protected]>.
Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email).

P.S.: This email is c/c to the developers where some action is expected. If you were copied, please review the patches, acking/nacking or submitting an update.

Quoted 4 lines

               == Waiting for Antti Palosaari<[email protected]>  review ==

Mar,21 2010: af9015 : more robust eeprom parsing
         http://patchwork.kernel.org/patch/87243

NACK, partly. I think it is rather useless.

Quoted 2 lines

May,20 2010: New NXP tda18218 tuner
          http://patchwork.kernel.org/patch/101170

AF9015/AF9013: ACK, partly from the AF9015/AF9013 side. It is safe to merge, it will not break any currently supported device. But I am not sure if all settings are correct since I don't have suitable device (AF9015+TDA18218) to figure out configuration and test.

TDA18218: I don't know. I have reviewed it, feedback can be found from the patchwork. I don't resist to merge, but also I don't want to take any responsibility since I don't have this device.

regards
Antti
--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

#12 lore raw
In reply to Mauro Carvalho Chehab

Hi!

Am 06.07.2010 15:06, schrieb Mauro Carvalho Chehab:

Quoted 3 lines
		== Waiting for Mike Isely<[email protected]>  review ==

Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400

               http://patchwork.kernel.org/patch/94960

Is Mike really the maintainer of the cx25840 module and not only of the pvrusb2 one? If he's not the maintainer you should contact the real one, cause I don't think that Mike can help much regarding patches for the cx25840 in that case.

Also I might need to adjust the patch cause of the recent changes that happened there the last few months. (I don't know when I'll find time for this...)

Regards,
Sven

#13 lore raw
Quoted 13 lines · Sven Barth

Hi!

Am 06.07.2010 15:06, schrieb Mauro Carvalho Chehab:

Quoted 3 lines
		== Waiting for Mike Isely<[email protected]>  review ==

Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400

http://patchwork.kernel.org/patch/94960

Is Mike really the maintainer of the cx25840 module and not only of the pvrusb2 one? If he's not the maintainer you should contact the real one, cause I don't think that Mike can help much regarding patches for the cx25840 in that case.

Quoted 6 lines

Also I might need to adjust the patch cause of the recent changes that happened there the last few months. (I don't know when I'll find time for this...)

Regards,
Sven

No I am definitely not the maintainer of that module, and my knowledge of its inner workings (though improved a lot lately) is still not very good.

All I can do here is verify that it doesn't break the pvrusb2 driver (which is what I was planning on doing).

-Mike

Signature · 3 lines
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
Acked-by
#14 lore raw
In reply to Mauro Carvalho Chehab
Quoted 9 lines · Mauro Carvalho Chehab

This is the summary of the patches that are currently under review at Linux Media Mailing List <[email protected]>.
Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email).

P.S.: This email is c/c to the developers where some action is expected.
If you were copied, please review the patches, acking/nacking or submitting an update.

[...]

Quoted 4 lines
		== Waiting for Mike Isely <[email protected]> review ==

Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400 http://patchwork.kernel.org/patch/94960

These are cx25840 patches and I'm not the maintainer of that module. I can't really speak to the correctness of the changes. Best I can do is to try the patch with a few pvrusb2-driven devices here that use the cx25840 module. I've done that now (HVR-1950 and PVR-USB2 model 24012) and everything continues to work fine. Note, this part of the patch:

int hw_fix = state->pvr150_workaround;
-
- if (std == V4L2_STD_NTSC_M_JP) {
+ if (std == V4L2_STD_NTSC_M_JP) {
/* Japan uses EIAJ audio standard */ cx25840_write(client, 0x808, hw_fix ? 0x2f : 0xf7);
} else if (std == V4L2_STD_NTSC_M_KR) {

is a whitespace-only change which introduces a bogus tab and messes up the indentation of that opening if-statement. It should probably be removed from the patch. Other than that, you have my ack:

Acked-by Mike Isely <[email protected]>

-Mike

Signature · 3 lines
Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
#15 lore raw
In reply to Mike Isely

Hi!

Quoted 5 lines · Mike Isely

These are cx25840 patches and I'm not the maintainer of that module. I can't really speak to the correctness of the changes. Best I can do is to try the patch with a few pvrusb2-driven devices here that use the cx25840 module. I've done that now (HVR-1950 and PVR-USB2 model 24012) and everything continues to work fine.

I also retested the patch (with the recent v4l changes) and my device continues to work as expected (using your current snapshot from July, Mike :) ).

Quoted 10 lines

Note, this part of the patch:

  		int hw_fix = state->pvr150_workaround;
(quoted diff) +1 -2
-
-		if (std == V4L2_STD_NTSC_M_JP) {
+			if (std == V4L2_STD_NTSC_M_JP) {
  			/* Japan uses EIAJ audio standard */
  			cx25840_write(client, 0x808, hw_fix ? 0x2f : 0xf7);
  		} else if (std == V4L2_STD_NTSC_M_KR) {
Quoted 3 lines

is a whitespace-only change which introduces a bogus tab and messes up the indentation of that opening if-statement. It should probably be removed from the patch.

I wonder how that came in there... my excuses for this (and also the removed new line some lines below that).

Quoted 7 lines

Other than that, you have my ack:

Acked-by Mike Isely<[email protected]>

-Mike

Hmm... I've read a bit in the wiki about submitting patches and read that one should sign-off his/her patches... as I didn't do that back then (as I thought that patch would be open for discussion ^^ - note to self: add RFC next time), should I resend the patch with a comment and the sign-off (and excluding the indentation mistake) or should I just send a sign-off in reference to this patch? Or something else?

Regards,
Sven

#18 lore raw
In reply to Mauro Carvalho Chehab
Quoted 10 lines · Mauro Carvalho Chehab

This is the summary of the patches that are currently under review at Linux Media Mailing List <[email protected]>.
Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email).

P.S.: This email is c/c to the developers where some action is expected.
If you were copied, please review the patches, acking/nacking or submitting an update.

Quoted 6 lines

== Andy Walls <[email protected]> and Aleksandr Piskunov <[email protected]> are discussing around the solution ==

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

		== Waiting for Andy Walls <[email protected]> ==

At the end of the thread both Aleksandr and I concluded that adding 50 ms or more to each frequency change was not a good thing to do. Please mark this patach not to be merged. There is not alternative solution at the moment.

Quoted 2 lines

Apr,10 2010: cx18: "missing audio" for analog recordings
http://patchwork.kernel.org/patch/91879

The patch at patchwork is obsolete. Please mark it not to be merged.

The audio problem still exists and I was working a solution with Mark Lord testing, but I haven't done any work on it for a few months. The current fix, for NTSC with BTSC audio, is at least the first two of the three patches here:

	http://linuxtv.org/hg/~awalls/cx18-audio2/

Mark reported the 3rd patch broke things.

Anyone with access to a bona-fide BTSC stereo broadcast signal (which a STB will not produce) feel free to test.

On a general note for all CX2584x and CX23418 audio standard auto-detection problems: without the problem analog signals and detailed specifications on the Merlin audio decoder core, I am not in a good position to fix these audio standard detection problems with these devices.

Regards,
Andy

author
#16 lore raw
In reply to Sven Barth

Em 09-07-2010 16:39, Sven Barth escreveu:

Quoted 29 lines

Hi!

Quoted 5 lines · Mike Isely

These are cx25840 patches and I'm not the maintainer of that module. I can't really speak to the correctness of the changes. Best I can do is to try the patch with a few pvrusb2-driven devices here that use the cx25840 module. I've done that now (HVR-1950 and PVR-USB2 model 24012) and everything continues to work fine.

I also retested the patch (with the recent v4l changes) and my device continues to work as expected (using your current snapshot from July, Mike :) ).

Quoted 10 lines

Note, this part of the patch:

          int hw_fix = state->pvr150_workaround;
(quoted diff) +1 -2
-
-        if (std == V4L2_STD_NTSC_M_JP) {
+            if (std == V4L2_STD_NTSC_M_JP) {
              /* Japan uses EIAJ audio standard */
              cx25840_write(client, 0x808, hw_fix ? 0x2f : 0xf7);
          } else if (std == V4L2_STD_NTSC_M_KR) {
Quoted 3 lines

is a whitespace-only change which introduces a bogus tab and messes up the indentation of that opening if-statement. It should probably be removed from the patch.

I wonder how that came in there... my excuses for this (and also the removed new line some lines below that).

Quoted 2 lines

Other than that, you have my ack:

Quoted 7 lines
Quoted 5 lines
Acked-by Mike Isely<[email protected]>

-Mike

Hmm... I've read a bit in the wiki about submitting patches and read that one should sign-off his/her patches... as I didn't do that back then (as I thought that patch would be open for discussion ^^ - note to self: add RFC next time), should I resend the patch with a comment and the sign-off (and excluding the indentation mistake) or should I just send a sign-off in reference to this patch? Or something else?

Just replying with your Signed-off-by is enough, but, as there's that small fix, the better is if you could send a new email, adding Mike's ack.

Quoted 7 lines

Regards,
Sven

Signature · 3 lines
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html