mbox

[GIT,PULL,FOR,3.9] em28xx videobuf2 support and v4l2-compliance fixes

Message ID CAGoCfiyPaaE5aAkjQdPGD_e9s3K6L+sv+fwGHxeoY5K1+iBYpQ@mail.gmail.com (mailing list archive)
State Accepted, archived
Headers

Pull-request

git://git.kernellabs.com/dheitmueller/linuxtv.git v39staging

Message

Devin Heitmueller Jan. 4, 2013, 7:41 p.m. UTC 
  Hello Mauro,

Please pull the following series for 3.9, which ports em28xx to VB2 as
well as applying Hans Verkuil's v4l2-compliance fixes for em28xx.

Thanks,

Devin

The following changes since commit 8cd7085ff460ead3aba6174052a408f4ad52ac36:

  [media] get_dvb_firmware: Fix the location of firmware for Terratec
HTC (2013-01-01 11:18:26 -0200)

are available in the git repository at:

  git://git.kernellabs.com/dheitmueller/linuxtv.git v39staging

for you to fetch changes up to 381abfc158c2dad81a558a3d3ff924fc7f80d277:

  em28xx: convert to videobuf2 (2013-01-04 14:16:24 -0500)

----------------------------------------------------------------
Devin Heitmueller (1):
      em28xx: convert to videobuf2

Hans Verkuil (14):
      em28xx: fix querycap.
      em28xx: remove bogus input/audio ioctls for the radio device.
      em28xx: fix VIDIOC_DBG_G_CHIP_IDENT compliance errors.
      em28xx: fix tuner/frequency handling
      v4l2-ctrls: add a notify callback.
      em28xx: convert to the control framework.
      em28xx: convert to v4l2_fh, fix priority handling.
      em28xx: add support for control events.
      em28xx: fill in readbuffers and fix incorrect return code.
      em28xx: fix broken TRY_FMT.
      tvp5150: remove compat control ops.
      em28xx: std fixes: don't implement in webcam mode, and fix std changes.
      em28xx: remove sliced VBI support.
      em28xx: zero vbi_format reserved array and add try_vbi_fmt.

 Documentation/video4linux/v4l2-controls.txt |   22 +-
 drivers/media/i2c/tvp5150.c                 |    7 -
 drivers/media/usb/em28xx/Kconfig            |    3 +-
 drivers/media/usb/em28xx/em28xx-cards.c     |   31 +-
 drivers/media/usb/em28xx/em28xx-dvb.c       |    4 +-
 drivers/media/usb/em28xx/em28xx-vbi.c       |  123 ++-
 drivers/media/usb/em28xx/em28xx-video.c     | 1159 ++++++++-------------------
 drivers/media/usb/em28xx/em28xx.h           |   38 +-
 drivers/media/v4l2-core/v4l2-ctrls.c        |   18 +
 include/media/v4l2-ctrls.h                  |   25 +
 10 files changed, 504 insertions(+), 926 deletions(-)

Comments

Ezequiel Garcia Jan. 4, 2013, 7:59 p.m. UTC | #1 
On Fri, Jan 4, 2013 at 4:41 PM, Devin Heitmueller
<dheitmueller@kernellabs.com> wrote:
[snip]
> are available in the git repository at:
>
>   git://git.kernellabs.com/dheitmueller/linuxtv.git v39staging
>
> for you to fetch changes up to 381abfc158c2dad81a558a3d3ff924fc7f80d277:
>
>   em28xx: convert to videobuf2 (2013-01-04 14:16:24 -0500)
>
> ----------------------------------------------------------------
> Devin Heitmueller (1):
>       em28xx: convert to videobuf2
>

Maybe I'm wrong, but weren't **all** changes supposed to be sent as a PATCH
to the mailing list for community review, before the PULL request was sent?
Devin Heitmueller Jan. 4, 2013, 8:59 p.m. UTC | #2 
On Fri, Jan 4, 2013 at 2:59 PM, Ezequiel Garcia <elezegarcia@gmail.com> wrote:
> Maybe I'm wrong, but weren't **all** changes supposed to be sent as a PATCH
> to the mailing list for community review, before the PULL request was sent?

No, you are correct.  I did a PULL specifically because Mauro asked me
to.  I'll resend the individual patches now to the list.

Devin
Frank Schaefer Jan. 5, 2013, 12:49 p.m. UTC | #3 
First of all:
Hans, Devin thank you for working on this stuff !

Am 04.01.2013 21:59, schrieb Devin Heitmueller:
> On Fri, Jan 4, 2013 at 2:59 PM, Ezequiel Garcia <elezegarcia@gmail.com> wrote:
>> Maybe I'm wrong, but weren't **all** changes supposed to be sent as a PATCH
>> to the mailing list for community review, before the PULL request was sent?
> No, you are correct.  I did a PULL specifically because Mauro asked me
> to. 

Interesting. Can you shed some light on whats going on behind the scenes
? ;)
How does this "request for pull-request" thing work and shouldn't this
happen on the public ML, too ?

> I'll resend the individual patches now to the list.

Do you still want me to review/test it ?
I'm willing to invest some time for it, but as there is already a pull
request, it seems to be ready and I really don't want to waste my time... !?

Regards,
Frank

> Devin
>

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Frank Schaefer Jan. 5, 2013, 12:51 p.m. UTC | #4 
Am 04.01.2013 20:41, schrieb Devin Heitmueller:
> Hello Mauro,
>
> Please pull the following series for 3.9, which ports em28xx to VB2 as
> well as applying Hans Verkuil's v4l2-compliance fixes for em28xx.
>
> Thanks,
>
> Devin
>
> The following changes since commit 8cd7085ff460ead3aba6174052a408f4ad52ac36:
>
>   [media] get_dvb_firmware: Fix the location of firmware for Terratec
> HTC (2013-01-01 11:18:26 -0200)
>
> are available in the git repository at:
>
>   git://git.kernellabs.com/dheitmueller/linuxtv.git v39staging
>
> for you to fetch changes up to 381abfc158c2dad81a558a3d3ff924fc7f80d277:
>
>   em28xx: convert to videobuf2 (2013-01-04 14:16:24 -0500)
>
> ----------------------------------------------------------------
> Devin Heitmueller (1):
>       em28xx: convert to videobuf2
>
> Hans Verkuil (14):
>       em28xx: fix querycap.
>       em28xx: remove bogus input/audio ioctls for the radio device.
>       em28xx: fix VIDIOC_DBG_G_CHIP_IDENT compliance errors.
>       em28xx: fix tuner/frequency handling
>       v4l2-ctrls: add a notify callback.
>       em28xx: convert to the control framework.
>       em28xx: convert to v4l2_fh, fix priority handling.
>       em28xx: add support for control events.
>       em28xx: fill in readbuffers and fix incorrect return code.
>       em28xx: fix broken TRY_FMT.
>       tvp5150: remove compat control ops.
>       em28xx: std fixes: don't implement in webcam mode, and fix std changes.
>       em28xx: remove sliced VBI support.
>       em28xx: zero vbi_format reserved array and add try_vbi_fmt.
>
>  Documentation/video4linux/v4l2-controls.txt |   22 +-
>  drivers/media/i2c/tvp5150.c                 |    7 -
>  drivers/media/usb/em28xx/Kconfig            |    3 +-
>  drivers/media/usb/em28xx/em28xx-cards.c     |   31 +-
>  drivers/media/usb/em28xx/em28xx-dvb.c       |    4 +-
>  drivers/media/usb/em28xx/em28xx-vbi.c       |  123 ++-
>  drivers/media/usb/em28xx/em28xx-video.c     | 1159 ++++++++-------------------
>  drivers/media/usb/em28xx/em28xx.h           |   38 +-
>  drivers/media/v4l2-core/v4l2-ctrls.c        |   18 +
>  include/media/v4l2-ctrls.h                  |   25 +
>  10 files changed, 504 insertions(+), 926 deletions(-)
>

The server is down ?


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