Skip to contentSkip to conversation

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

DISCUSSION Prabhakar Lad 8 participants 10 messages Mar 25, 2013 – Apr 2, 2013 1 message not in the archive Media (V4L/DVB)

lore
Share & tools
lore

Prabhakar Lad

author
#1 lore raw
In reply to a message not in the archive

Hi Mauro,

Quoted 12 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.

It took me a lot of time to handle patches this time. The good news is that there's just one patch without an owner.

[Snip]

Quoted 5 lines
                == Prabhakar Lad <[email protected]> ==

Oct,24 2012: [media] vpif_display: fix return value check in vpif_reqbufs() http://patchwork.linuxtv.org/patch/15167 Wei Yongjun <[email protected]>

This patch can be marked as 'Superseded'.

Regards,
--Prabhakar

#2 lore raw
In reply to a message not in the archive

Hello,

Quoted 12 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.

It took me a lot of time to handle patches this time. The good news is that there's just one patch without an owner.

...

Quoted 3 lines
		== Marek Szyprowski <[email protected]> ==

Nov,12 2012: [media] videobuf2-core: print current state of buffer in vb2_buffer_do http://patchwork.linuxtv.org/patch/15420 Tushar Behera <[email protected]>

This one is ok. I acked it by gmane interface as I don't have original mail, I have no

Quoted 1 line

Mar, 5 2013: [media] dma-mapping: enable no mmu support in dma_common_mmap http://patchwork.linuxtv.org/patch/17112 Scott Jiang <[email protected]>

IMHO this one is RFC. I would like to get some more explanation why it is needed and how it will be used.

Best regards

Signature · 2 lines
Marek Szyprowski
Samsung Poland R&D Center
#3 lore raw
In reply to a message not in the archive

Hi Mauro,

Quoted 8 lines · On Sunday 24 March 2013 15:11:11 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 5 lines
		== Laurent Pinchart <[email protected]> ==

Dec,12 2012: [v2] ad5820: Voice coil motor controller driver
http://patchwork.linuxtv.org/patch/15881
Florian Neuhaus <[email protected]>

Thanks for the reminder, I'll review that one.

Quoted 6 lines

Jan, 4 2013: omap3isp: Add support for interlaced input data
http://patchwork.linuxtv.org/patch/16133
William Swanson <[email protected]>
Jan,14 2013: omap3isp: Add support for interlaced input data
http://patchwork.linuxtv.org/patch/16255
William Swanson <[email protected]>

I've recently received hardware to test interlaced video with the OMAP3 ISP, I'll work on this.

Signature · 3 lines
Regards,

Laurent Pinchart
#4 lore raw
In reply to a message not in the archive
Quoted 15 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.

It took me a lot of time to handle patches this time. The good news is that there's just one patch without an owner.

This patch is there on the pile for a long time. I was actually expecting that either Jarod or David could review it, as it can affect existing NEC devices:

[...]

Quoted 3 lines
		== Sylwester Nawrocki <[email protected]>  ==

Jan, 6 2013: s5p-tv: mixer: fix handling of VIDIOC_S_FMT

http://patchwork.linuxtv.org/patch/16143 Tomasz Stanislawski<[email protected]>

From my side I can't see any applications this patch could cause issues to. And it actually breaks some applications to not have this patch applied. Hence we keep it applied internally. I will likely not be able to address the issue at V4L2 API that some applications rely on drivers returning -EINVAL on an attempt to set unsupported format. I don't have an idea what to do with that at the moment :) I would say it is safe to apply this patch. But if you have concerns please ignore it. Or let's leave it in current state.

Regards,
Sylwester

#6 lore raw
In reply to a message not in the archive

Hi Mauro

Quoted 5 lines · Mauro Carvalho Chehab
		== Guennadi Liakhovetski <[email protected]> ==

I suspect that some of the above may be obsoleted:

Nov,13 2012: sh_vou: Move from videobuf to videobuf2 http://patchwork.linuxtv.org/patch/15433 Laurent Pinchart <[email protected]>

dropped for now

Quoted 1 line

Nov,16 2012: [05/14,media] atmel-isi: Update error check for unsigned variables http://patchwork.linuxtv.org/patch/15475 Tushar Behera <[email protected]>

I was unsure about the above, and then it rolled under the carpet somehow... I think, I'll just apply it.

Quoted 1 line

Jan, 3 2013: [1/3] sh_vou: Don't modify const variable in sh_vou_s_crop() http://patchwork.linuxtv.org/patch/16095 Laurent Pinchart <[email protected]>

Superseded by "[media] media: sh_vou: fix const cropping related warnings"

Quoted 1 line

Mar,18 2013: [v2] soc_camera: Add RGB666 & RGB888 formats http://patchwork.linuxtv.org/patch/17407 [email protected]

An updated version is under review

Quoted 1 line

Mar,14 2013: [v2,1/8] drivers: media: use module_platform_driver_probe() http://patchwork.linuxtv.org/patch/17378 Fabio Porcedda <[email protected]>

scheduled for the second soc-camera 3.10 batch

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

#9 lore raw
In reply to a message not in the archive
Quoted 6 lines · Mauro Carvalho Chehab
                == Michael Krufky <[email protected]> ==

Michael promised to review and apply them on his tree, sending me a pull request, but this never happened. Mike, did you see any issues on those patches?

Apologies, the year 2013 has had me busier than ever. I didn't get to my patch queue as early as I had expected. I'll get these reviewed, tested & pushed to a repository and send you a merge request ASAP.

I'll try very hard to get this done in the coming week.

Best regards,

Mike

#7 lore raw
In reply to Guennadi Liakhovetski

Hi Guennadi,

Quoted 11 lines · On Friday 29 March 2013 16:13:41 Guennadi Liakhovetski wrote:

Hi Mauro

Quoted 7 lines · Mauro Carvalho Chehab
		== Guennadi Liakhovetski <[email protected]> ==

I suspect that some of the above may be obsoleted:

Nov,13 2012: sh_vou: Move from videobuf to videobuf2
http://patchwork.linuxtv.org/patch/15433 Laurent Pinchart
<[email protected]>

dropped for now

As Renesas doesn't seem to be interested in maintaining the sh-vou driver, should it be marked as 'Odd Fixes' in the MAINTAINERS file ?

Signature · 3 lines
Regards,

Laurent Pinchart
#10 lore raw
In reply to a message not in the archive
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]>.

Manu,

Yet another patch adding IR support for mantis. It seems that this is is a long waited feature, as from time to time, people send patches fixing it. Please test and apply it, or otherwise fix and send us a patch properly implementing support for it.

Feb, 7 2013: [media] mantis: add remote control support http://patchwork.linuxtv.org/patch/16732 Jan Klötzke <[email protected]>

I will look into it after these two weeks. Just tied up these two weeks.

Regards,
Manu

#8 lore raw
In reply to Laurent Pinchart
Quoted 10 lines · Laurent Pinchart

Hi Guennadi,

Quoted 7 lines · On Friday 29 March 2013 16:13:41 Guennadi Liakhovetski wrote:

Hi Mauro

Quoted 4 lines · Mauro Carvalho Chehab
		== Guennadi Liakhovetski <[email protected]> ==

I suspect that some of the above may be obsoleted:

Quoted 7 lines
Quoted 4 lines
Quoted 3 lines

Nov,13 2012: sh_vou: Move from videobuf to videobuf2
http://patchwork.linuxtv.org/patch/15433 Laurent Pinchart
<[email protected]>

dropped for now

As Renesas doesn't seem to be interested in maintaining the sh-vou driver, should it be marked as 'Odd Fixes' in the MAINTAINERS file ?

I wouldn't nak such a patch ;-)

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