Re: Old patches sent via the Mailing list

2010-12-10 Thread Ozan Çağlayan
Pazartesi 18 Ekim 2010 günü (saat 00:20:31) Mauro Carvalho Chehab şunları 
yazmıştı:
 Hi,

 Jun,21 2010: Mantis, hopper: use MODULE_DEVICE_TABLE use the macro to make 
modules  http://patchwork.kernel.org/patch/107147  Manu

This was first spotted by me in February 2010 in this list. Then several 
people in this list spotted that the modules were missing MODULE_DEVICE_TABLE. 
Well I thought that it should already be merged during the last 10 months but 
I'm surprised to see that it's not.

Note that without this patch those drivers are not automatically loaded by 
udev so all other efforts of improving the mantis stuff stays insignificant 
for end-users.

Can this be fixed ASAP?

Thanks,


---
Ozan Çağlayan
TUBITAK/UEKAE - Pardus Linux
http://www.pardus.org.tr/eng
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-12-10 Thread Bjørn Mork
Ozan Çağlayan o...@pardus.org.tr writes:
 Pazartesi 18 Ekim 2010 günü (saat 00:20:31) Mauro Carvalho Chehab şunları 
 yazmıştı:
 Hi,

 Jun,21 2010: Mantis, hopper: use MODULE_DEVICE_TABLE use the macro to make 
 modules  http://patchwork.kernel.org/patch/107147  Manu

 This was first spotted by me in February 2010 in this list. Then several 
 people in this list spotted that the modules were missing 
 MODULE_DEVICE_TABLE. 
 Well I thought that it should already be merged during the last 10 months but 
 I'm surprised to see that it's not.

 Note that without this patch those drivers are not automatically loaded by 
 udev so all other efforts of improving the mantis stuff stays insignificant 
 for end-users.

well, the excellent Debian kernel team actually accepted this patch when
adding the backported mantis driver to their 2.6.32 kernel, so at least
Debian end users do get auto loading:

bj...@nemi:~$ modinfo mantis
filename:   
/lib/modules/2.6.32-5-amd64/kernel/drivers/media/dvb/mantis/mantis.ko
license:GPL
author: Manu Abraham
description:MANTIS driver
alias:  pci:v1822d4E35sv1822sd0024bc*sc*i*
alias:  pci:v1822d4E35sv153Bsd1178bc*sc*i*
alias:  pci:v1822d4E35sv1AE4sd0002bc*sc*i*
alias:  pci:v1822d4E35sv1822sd0043bc*sc*i*
alias:  pci:v1822d4E35sv1822sd0008bc*sc*i*
alias:  pci:v1822d4E35sv153Bsd1179bc*sc*i*
alias:  pci:v1822d4E35sv1AE4sd0003bc*sc*i*
alias:  pci:v1822d4E35sv1AE4sd0001bc*sc*i*
alias:  pci:v1822d4E35sv1822sd0031bc*sc*i*
alias:  pci:v1822d4E35sv1822sd0014bc*sc*i*
alias:  pci:v1822d4E35sv1822sd0016bc*sc*i*
depends:
mantis_core,stv0299,i2c-core,stb0899,zl10353,tda10023,tda10021,stb6100,mb86a16,lnbp21,tda665x
vermagic:   2.6.32-5-amd64 SMP mod_unload modversions 
parm:   verbose:verbose startup messages, default is 1 (yes) (int)
bj...@nemi:~$ dpkg -S 
/lib/modules/2.6.32-5-amd64/kernel/drivers/media/dvb/mantis/mantis.ko
linux-image-2.6.32-5-amd64: 
/lib/modules/2.6.32-5-amd64/kernel/drivers/media/dvb/mantis/mantis.ko
bj...@nemi:~$ apt-cache policy linux-image-2.6.32-5-amd64
linux-image-2.6.32-5-amd64:
  Installed: 2.6.32-28
  Candidate: 2.6.32-28
  Version table:
 *** 2.6.32-28 0
990 http://ftp.no.debian.org/debian/ squeeze/main amd64 Packages
600 http://ftp.no.debian.org/debian/ sid/main amd64 Packages
100 /var/lib/dpkg/status


(Note that the extensive depends shows that it's still missing one of
the other critical patches)


 Can this be fixed ASAP?

At least it's now queued for 2.6.38:


bj...@canardo:/usr/local/src/git/linux-2.6$ git shortlog 
origin..media_tree/staging/for_v2.6.38 drivers/media/dvb/mantis/
Ben Hutchings (1):
  [media] Mantis: Rename gpio_set_bits to mantis_gpio_set_bits

Bjørn Mork (1):
  [media] Mantis: use dvb_attach to avoid double dereferencing on module 
removal

David Härdeman (1):
  [media] ir-core: make struct rc_dev the primary interface

Manu Abraham (1):
  [media] Mantis, hopper: use MODULE_DEVICE_TABLE

Marko Ristola (1):
  [media] Mantis: append tasklet maintenance for DVB stream delivery

Mauro Carvalho Chehab (4):
  [media] rc: rename the remaining things to rc_core
  [media] rc: Rename remote controller type to rc_type instead of ir_type
  [media] rc: Name RC keymap tables as rc_map_table
  [media] rc: use rc_map_ prefix for all rc map tables




And I will submit it to stable as soon as it's in Linus' tree, unless
someone beats me to it...



Bjørn

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-12-10 Thread Ozan Çağlayan
Cuma 10 Aralık 2010 günü (saat 14:38:21) Bjørn Mork şunları yazmıştı:

 At least it's now queued for 2.6.38:

Thanks :)

---
Ozan Çağlayan
TUBITAK/UEKAE - Pardus Linux
http://www.pardus.org.tr/eng
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-21 Thread Mauro Carvalho Chehab
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 
 Barthpascaldra...@googlemail.com
 Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400
http://patchwork.kernel.org/patch/94960   Sven 
 Barthpascaldra...@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
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-20 Thread Andy Walls
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 
  Barthpascaldra...@googlemail.com
 Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400  
   http://patchwork.kernel.org/patch/94960   Sven 
  Barthpascaldra...@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



--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-20 Thread Sven Barth

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 Barthpascaldra...@googlemail.com
Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400 
  http://patchwork.kernel.org/patch/94960   Sven 
Barthpascaldra...@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
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-19 Thread Jarod Wilson
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 ja...@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 j...@perches.com

Acked.

-- 
Jarod Wilson
ja...@redhat.com

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-19 Thread Sven Barth

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 Barthpascaldra...@googlemail.com
   Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400  
 http://patchwork.kernel.org/patch/94960   Sven 
Barthpascaldra...@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
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-18 Thread 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 pascaldra...@googlemail.com
   Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400  
  http://patchwork.kernel.org/patch/94960   Sven Barth 
 pascaldra...@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
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-18 Thread Jean-Francois Moine
On Sun, 17 Oct 2010 19:20:31 -0200
Mauro Carvalho Chehab mche...@redhat.com wrote:

   == Gspca patches - Waiting Jean-Francois Moine
 moin...@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 moin...@free.fr

Hi Mauro,

This patch is already applied.

Cheers.

-- 
Ken ar c'hentañ | ** Breizh ha Linux atav! **
Jef |   http://moinejf.free.fr/
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-18 Thread Mauro Carvalho Chehab
Em 18-10-2010 04:34, Jean-Francois Moine escreveu:
 On Sun, 17 Oct 2010 19:20:31 -0200
 Mauro Carvalho Chehab mche...@redhat.com wrote:
 
  == Gspca patches - Waiting Jean-Francois Moine
 moin...@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 moin...@free.fr
 
 Hi Mauro,
 
 This patch is already applied.

Updated, thanks!
 
 Cheers.
 

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Old patches sent via the Mailing list

2010-10-17 Thread Mauro Carvalho Chehab
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 pascaldra...@googlemail.com
  Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400 
  http://patchwork.kernel.org/patch/94960   Sven Barth 
pascaldra...@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.rist...@kolumbus.fi
  Mar, 1 2010: firedtv: add parameter to fake ca_system_ids in CA_INFO  
  http://patchwork.kernel.org/patch/82912   Henrik Kurelid 
hen...@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 
arn...@mind.be
  Jul,27 2010: videobuf_dma_sg: a new implementation for mmap   
  http://patchwork.kernel.org/patch/114520  Figo.zhang figo1...@gmail.com
  Jul,28 2010: [v2] videobuf_dma_sg: a new implementation for mmap  
  http://patchwork.kernel.org/patch/114760  Figo.zhang figo1...@gmail.com
  Jul,30 2010: [v2] Resend:videobuf_dma_sg: a new implementation for mmap   
  http://patchwork.kernel.org/patch/115348  Figo.zhang figo1...@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.m...@gmail.com == 

Apr,15 2010: [5/8] ir-core: convert mantis from ir-functions.c  
http://patchwork.kernel.org/patch/92961   David Härdeman 
da...@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.rist...@kolumbus.fi
Jun,20 2010: [2/2] DVB/V4L: mantis: remove unused files 
http://patchwork.kernel.org/patch/107062  Bjørn Mork bj...@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 bj...@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.m...@gmail.com
Jul, 3 2010: mantis: Rename gpio_set_bits to mantis_gpio_set_bits   
http://patchwork.kernel.org/patch/109972  Ben Hutchings 
b...@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.rist...@kolumbus.fi
Jul, 9 2010: Mantis: append tasklet maintenance for DVB stream delivery 
http://patchwork.kernel.org/patch/111090  Marko Ristola 
marko.rist...@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.rist...@kolumbus.fi
Jul,19 2010: Twinhan DTV Ter-CI (3030 mantis)   
http://patchwork.kernel.org/patch/112708  Niklas Claesson 
nicke.claes...@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.rist...@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 
tuxoho...@hotmail.de

== Soc_camera waiting for Guennadi Liakhovetski 
g.liakhovet...@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   

Re: Old patches sent via the Mailing list

2010-10-17 Thread Guennadi Liakhovetski
Hi Mauro

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

   == Soc_camera waiting for Guennadi Liakhovetski 
 g.liakhovet...@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 
 d...@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 
 bar...@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 
 bar...@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 
 bar...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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/
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-17 Thread Antti Palosaari

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 Palosaaricr...@iki.fi  review ==

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



Mark as dropped.

Antti

--
http://palosaari.fi/
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-17 Thread Mauro Carvalho Chehab
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.liakhovet...@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 
 d...@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 
 bar...@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 
 bar...@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 
 bar...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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.grzesc...@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
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-17 Thread Andy Walls
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 pascaldra...@googlemail.com
   Apr,25 2010: Problem with cx25840 and Terratec Grabster AV400   
 http://patchwork.kernel.org/patch/94960   Sven Barth 
 pascaldra...@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


--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Old patches sent via the Mailing list

2010-10-17 Thread Németh Márton
Mauro Carvalho Chehab wrote:
   == Gspca patches - Waiting Hans de Goede hdego...@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 
 nm...@freemail.hu

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

Márton Németh
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html