linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL FOR v3.18] Add vivid test driver, remove old vivi test driver
@ 2014-08-29  6:30 Hans Verkuil
  2014-09-02 19:26 ` Mauro Carvalho Chehab
  0 siblings, 1 reply; 2+ messages in thread
From: Hans Verkuil @ 2014-08-29  6:30 UTC (permalink / raw)
  To: Linux Media Mailing List

Hi Mauro,

This adds the new vivid driver as a replacement for the old vivi.

This pull request is identical to the v2 patch series posted earlier:

http://comments.gmane.org/gmane.linux.drivers.video-input-infrastructure/81354

except for the final patch that removes the vivi driver which is new to this
pull request.

One question: the vivid driver (like the vivi driver) is not build by default.
Should that be changed? In my opinion this driver should be enabled by distros,
so I am in favor of changing Kconfig. Let me know if you agree and I'll make a
follow up patch or you can change this yourself.

Regards,

	Hans

The following changes since commit b250392f7b5062cf026b1423e27265e278fd6b30:

  [media] media: ttpci: fix av7110 build to be compatible with CONFIG_INPUT_EVDEV (2014-08-21 15:25:38 -0500)

are available in the git repository at:

  git://linuxtv.org/hverkuil/media_tree.git vivid2

for you to fetch changes up to d5f410f54e87ba420de839dec4e806707cc2aff2:

  vivi: remove driver, it's replaced by vivid. (2014-08-25 13:49:53 +0200)

----------------------------------------------------------------
Hans Verkuil (13):
      vb2: fix multiplanar read() with non-zero data_offset
      vivid.txt: add documentation for the vivid driver
      vivid: add core driver code
      vivid: add the control handling code
      vivid: add the video capture and output parts
      vivid: add VBI capture and output code
      vivid: add the kthread code that controls the video rate
      vivid: add a simple framebuffer device for overlay testing
      vivid: add the Test Pattern Generator
      vivid: add support for radio receivers and transmitters
      vivid: add support for software defined radio
      vivid: enable the vivid driver
      vivi: remove driver, it's replaced by vivid.

 Documentation/video4linux/vivid.txt               | 1109 +++++++++++++++++++++++++++++++++++++++++++++++
 drivers/media/platform/Kconfig                    |   15 +-
 drivers/media/platform/Makefile                   |    2 +-
 drivers/media/platform/vivi.c                     | 1542 -----------------------------------------------------------------
 drivers/media/platform/vivid/Kconfig              |   19 +
 drivers/media/platform/vivid/Makefile             |    6 +
 drivers/media/platform/vivid/vivid-core.c         | 1390 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/media/platform/vivid/vivid-core.h         |  520 ++++++++++++++++++++++
 drivers/media/platform/vivid/vivid-ctrls.c        | 1502 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/media/platform/vivid/vivid-ctrls.h        |   34 ++
 drivers/media/platform/vivid/vivid-kthread-cap.c  |  885 +++++++++++++++++++++++++++++++++++++
 drivers/media/platform/vivid/vivid-kthread-cap.h  |   26 ++
 drivers/media/platform/vivid/vivid-kthread-out.c  |  304 +++++++++++++
 drivers/media/platform/vivid/vivid-kthread-out.h  |   26 ++
 drivers/media/platform/vivid/vivid-osd.c          |  400 +++++++++++++++++
 drivers/media/platform/vivid/vivid-osd.h          |   27 ++
 drivers/media/platform/vivid/vivid-radio-common.c |  189 ++++++++
 drivers/media/platform/vivid/vivid-radio-common.h |   40 ++
 drivers/media/platform/vivid/vivid-radio-rx.c     |  287 ++++++++++++
 drivers/media/platform/vivid/vivid-radio-rx.h     |   31 ++
 drivers/media/platform/vivid/vivid-radio-tx.c     |  141 ++++++
 drivers/media/platform/vivid/vivid-radio-tx.h     |   29 ++
 drivers/media/platform/vivid/vivid-rds-gen.c      |  165 +++++++
 drivers/media/platform/vivid/vivid-rds-gen.h      |   53 +++
 drivers/media/platform/vivid/vivid-sdr-cap.c      |  499 +++++++++++++++++++++
 drivers/media/platform/vivid/vivid-sdr-cap.h      |   34 ++
 drivers/media/platform/vivid/vivid-tpg-colors.c   |  310 +++++++++++++
 drivers/media/platform/vivid/vivid-tpg-colors.h   |   64 +++
 drivers/media/platform/vivid/vivid-tpg.c          | 1439 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/media/platform/vivid/vivid-tpg.h          |  438 +++++++++++++++++++
 drivers/media/platform/vivid/vivid-vbi-cap.c      |  356 +++++++++++++++
 drivers/media/platform/vivid/vivid-vbi-cap.h      |   40 ++
 drivers/media/platform/vivid/vivid-vbi-gen.c      |  248 +++++++++++
 drivers/media/platform/vivid/vivid-vbi-gen.h      |   33 ++
 drivers/media/platform/vivid/vivid-vbi-out.c      |  247 +++++++++++
 drivers/media/platform/vivid/vivid-vbi-out.h      |   34 ++
 drivers/media/platform/vivid/vivid-vid-cap.c      | 1729 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/media/platform/vivid/vivid-vid-cap.h      |   71 +++
 drivers/media/platform/vivid/vivid-vid-common.c   |  571 ++++++++++++++++++++++++
 drivers/media/platform/vivid/vivid-vid-common.h   |   61 +++
 drivers/media/platform/vivid/vivid-vid-out.c      | 1205 +++++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/media/platform/vivid/vivid-vid-out.h      |   57 +++
 drivers/media/v4l2-core/videobuf2-core.c          |    6 +
 43 files changed, 14628 insertions(+), 1556 deletions(-)
 create mode 100644 Documentation/video4linux/vivid.txt
 delete mode 100644 drivers/media/platform/vivi.c
 create mode 100644 drivers/media/platform/vivid/Kconfig
 create mode 100644 drivers/media/platform/vivid/Makefile
 create mode 100644 drivers/media/platform/vivid/vivid-core.c
 create mode 100644 drivers/media/platform/vivid/vivid-core.h
 create mode 100644 drivers/media/platform/vivid/vivid-ctrls.c
 create mode 100644 drivers/media/platform/vivid/vivid-ctrls.h
 create mode 100644 drivers/media/platform/vivid/vivid-kthread-cap.c
 create mode 100644 drivers/media/platform/vivid/vivid-kthread-cap.h
 create mode 100644 drivers/media/platform/vivid/vivid-kthread-out.c
 create mode 100644 drivers/media/platform/vivid/vivid-kthread-out.h
 create mode 100644 drivers/media/platform/vivid/vivid-osd.c
 create mode 100644 drivers/media/platform/vivid/vivid-osd.h
 create mode 100644 drivers/media/platform/vivid/vivid-radio-common.c
 create mode 100644 drivers/media/platform/vivid/vivid-radio-common.h
 create mode 100644 drivers/media/platform/vivid/vivid-radio-rx.c
 create mode 100644 drivers/media/platform/vivid/vivid-radio-rx.h
 create mode 100644 drivers/media/platform/vivid/vivid-radio-tx.c
 create mode 100644 drivers/media/platform/vivid/vivid-radio-tx.h
 create mode 100644 drivers/media/platform/vivid/vivid-rds-gen.c
 create mode 100644 drivers/media/platform/vivid/vivid-rds-gen.h
 create mode 100644 drivers/media/platform/vivid/vivid-sdr-cap.c
 create mode 100644 drivers/media/platform/vivid/vivid-sdr-cap.h
 create mode 100644 drivers/media/platform/vivid/vivid-tpg-colors.c
 create mode 100644 drivers/media/platform/vivid/vivid-tpg-colors.h
 create mode 100644 drivers/media/platform/vivid/vivid-tpg.c
 create mode 100644 drivers/media/platform/vivid/vivid-tpg.h
 create mode 100644 drivers/media/platform/vivid/vivid-vbi-cap.c
 create mode 100644 drivers/media/platform/vivid/vivid-vbi-cap.h
 create mode 100644 drivers/media/platform/vivid/vivid-vbi-gen.c
 create mode 100644 drivers/media/platform/vivid/vivid-vbi-gen.h
 create mode 100644 drivers/media/platform/vivid/vivid-vbi-out.c
 create mode 100644 drivers/media/platform/vivid/vivid-vbi-out.h
 create mode 100644 drivers/media/platform/vivid/vivid-vid-cap.c
 create mode 100644 drivers/media/platform/vivid/vivid-vid-cap.h
 create mode 100644 drivers/media/platform/vivid/vivid-vid-common.c
 create mode 100644 drivers/media/platform/vivid/vivid-vid-common.h
 create mode 100644 drivers/media/platform/vivid/vivid-vid-out.c
 create mode 100644 drivers/media/platform/vivid/vivid-vid-out.h

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [GIT PULL FOR v3.18] Add vivid test driver, remove old vivi test driver
  2014-08-29  6:30 [GIT PULL FOR v3.18] Add vivid test driver, remove old vivi test driver Hans Verkuil
@ 2014-09-02 19:26 ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 2+ messages in thread
From: Mauro Carvalho Chehab @ 2014-09-02 19:26 UTC (permalink / raw)
  To: Hans Verkuil; +Cc: Linux Media Mailing List

Em Fri, 29 Aug 2014 08:30:16 +0200
Hans Verkuil <hverkuil@xs4all.nl> escreveu:

> Hi Mauro,
> 
> This adds the new vivid driver as a replacement for the old vivi.
> 
> This pull request is identical to the v2 patch series posted earlier:
> 
> http://comments.gmane.org/gmane.linux.drivers.video-input-infrastructure/81354
> 
> except for the final patch that removes the vivi driver which is new to this
> pull request.
> 
> One question: the vivid driver (like the vivi driver) is not build by default.

We should never build a driver by default. What makes sense is to enable
dependent drivers by default, not main ones.

> Should that be changed? In my opinion this driver should be enabled by distros,
> so I am in favor of changing Kconfig. Let me know if you agree and I'll make a
> follow up patch or you can change this yourself.

If you want the distros to enable it, you should talk to the distro
maintainers. I think that, on most, the best way of doing that is to
open bugzillas.

IMHO, it doesn't make sense for distros to enable it at their
mainstream Kernel, but it does make sense for them to enable on
their debug kernels (on distros that provide both, like Fedora).
Just my 2 cents.

But, again, it is up to the distro Kernel maintainer to decide.

Regards,
Mauro
> 
> Regards,
> 
> 	Hans
> 
> The following changes since commit b250392f7b5062cf026b1423e27265e278fd6b30:
> 
>   [media] media: ttpci: fix av7110 build to be compatible with CONFIG_INPUT_EVDEV (2014-08-21 15:25:38 -0500)
> 
> are available in the git repository at:
> 
>   git://linuxtv.org/hverkuil/media_tree.git vivid2
> 
> for you to fetch changes up to d5f410f54e87ba420de839dec4e806707cc2aff2:
> 
>   vivi: remove driver, it's replaced by vivid. (2014-08-25 13:49:53 +0200)
> 
> ----------------------------------------------------------------
> Hans Verkuil (13):
>       vb2: fix multiplanar read() with non-zero data_offset
>       vivid.txt: add documentation for the vivid driver
>       vivid: add core driver code
>       vivid: add the control handling code
>       vivid: add the video capture and output parts
>       vivid: add VBI capture and output code
>       vivid: add the kthread code that controls the video rate
>       vivid: add a simple framebuffer device for overlay testing
>       vivid: add the Test Pattern Generator
>       vivid: add support for radio receivers and transmitters
>       vivid: add support for software defined radio
>       vivid: enable the vivid driver
>       vivi: remove driver, it's replaced by vivid.
> 
>  Documentation/video4linux/vivid.txt               | 1109 +++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/media/platform/Kconfig                    |   15 +-
>  drivers/media/platform/Makefile                   |    2 +-
>  drivers/media/platform/vivi.c                     | 1542 -----------------------------------------------------------------
>  drivers/media/platform/vivid/Kconfig              |   19 +
>  drivers/media/platform/vivid/Makefile             |    6 +
>  drivers/media/platform/vivid/vivid-core.c         | 1390 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/media/platform/vivid/vivid-core.h         |  520 ++++++++++++++++++++++
>  drivers/media/platform/vivid/vivid-ctrls.c        | 1502 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/media/platform/vivid/vivid-ctrls.h        |   34 ++
>  drivers/media/platform/vivid/vivid-kthread-cap.c  |  885 +++++++++++++++++++++++++++++++++++++
>  drivers/media/platform/vivid/vivid-kthread-cap.h  |   26 ++
>  drivers/media/platform/vivid/vivid-kthread-out.c  |  304 +++++++++++++
>  drivers/media/platform/vivid/vivid-kthread-out.h  |   26 ++
>  drivers/media/platform/vivid/vivid-osd.c          |  400 +++++++++++++++++
>  drivers/media/platform/vivid/vivid-osd.h          |   27 ++
>  drivers/media/platform/vivid/vivid-radio-common.c |  189 ++++++++
>  drivers/media/platform/vivid/vivid-radio-common.h |   40 ++
>  drivers/media/platform/vivid/vivid-radio-rx.c     |  287 ++++++++++++
>  drivers/media/platform/vivid/vivid-radio-rx.h     |   31 ++
>  drivers/media/platform/vivid/vivid-radio-tx.c     |  141 ++++++
>  drivers/media/platform/vivid/vivid-radio-tx.h     |   29 ++
>  drivers/media/platform/vivid/vivid-rds-gen.c      |  165 +++++++
>  drivers/media/platform/vivid/vivid-rds-gen.h      |   53 +++
>  drivers/media/platform/vivid/vivid-sdr-cap.c      |  499 +++++++++++++++++++++
>  drivers/media/platform/vivid/vivid-sdr-cap.h      |   34 ++
>  drivers/media/platform/vivid/vivid-tpg-colors.c   |  310 +++++++++++++
>  drivers/media/platform/vivid/vivid-tpg-colors.h   |   64 +++
>  drivers/media/platform/vivid/vivid-tpg.c          | 1439 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/media/platform/vivid/vivid-tpg.h          |  438 +++++++++++++++++++
>  drivers/media/platform/vivid/vivid-vbi-cap.c      |  356 +++++++++++++++
>  drivers/media/platform/vivid/vivid-vbi-cap.h      |   40 ++
>  drivers/media/platform/vivid/vivid-vbi-gen.c      |  248 +++++++++++
>  drivers/media/platform/vivid/vivid-vbi-gen.h      |   33 ++
>  drivers/media/platform/vivid/vivid-vbi-out.c      |  247 +++++++++++
>  drivers/media/platform/vivid/vivid-vbi-out.h      |   34 ++
>  drivers/media/platform/vivid/vivid-vid-cap.c      | 1729 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/media/platform/vivid/vivid-vid-cap.h      |   71 +++
>  drivers/media/platform/vivid/vivid-vid-common.c   |  571 ++++++++++++++++++++++++
>  drivers/media/platform/vivid/vivid-vid-common.h   |   61 +++
>  drivers/media/platform/vivid/vivid-vid-out.c      | 1205 +++++++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/media/platform/vivid/vivid-vid-out.h      |   57 +++
>  drivers/media/v4l2-core/videobuf2-core.c          |    6 +
>  43 files changed, 14628 insertions(+), 1556 deletions(-)
>  create mode 100644 Documentation/video4linux/vivid.txt
>  delete mode 100644 drivers/media/platform/vivi.c
>  create mode 100644 drivers/media/platform/vivid/Kconfig
>  create mode 100644 drivers/media/platform/vivid/Makefile
>  create mode 100644 drivers/media/platform/vivid/vivid-core.c
>  create mode 100644 drivers/media/platform/vivid/vivid-core.h
>  create mode 100644 drivers/media/platform/vivid/vivid-ctrls.c
>  create mode 100644 drivers/media/platform/vivid/vivid-ctrls.h
>  create mode 100644 drivers/media/platform/vivid/vivid-kthread-cap.c
>  create mode 100644 drivers/media/platform/vivid/vivid-kthread-cap.h
>  create mode 100644 drivers/media/platform/vivid/vivid-kthread-out.c
>  create mode 100644 drivers/media/platform/vivid/vivid-kthread-out.h
>  create mode 100644 drivers/media/platform/vivid/vivid-osd.c
>  create mode 100644 drivers/media/platform/vivid/vivid-osd.h
>  create mode 100644 drivers/media/platform/vivid/vivid-radio-common.c
>  create mode 100644 drivers/media/platform/vivid/vivid-radio-common.h
>  create mode 100644 drivers/media/platform/vivid/vivid-radio-rx.c
>  create mode 100644 drivers/media/platform/vivid/vivid-radio-rx.h
>  create mode 100644 drivers/media/platform/vivid/vivid-radio-tx.c
>  create mode 100644 drivers/media/platform/vivid/vivid-radio-tx.h
>  create mode 100644 drivers/media/platform/vivid/vivid-rds-gen.c
>  create mode 100644 drivers/media/platform/vivid/vivid-rds-gen.h
>  create mode 100644 drivers/media/platform/vivid/vivid-sdr-cap.c
>  create mode 100644 drivers/media/platform/vivid/vivid-sdr-cap.h
>  create mode 100644 drivers/media/platform/vivid/vivid-tpg-colors.c
>  create mode 100644 drivers/media/platform/vivid/vivid-tpg-colors.h
>  create mode 100644 drivers/media/platform/vivid/vivid-tpg.c
>  create mode 100644 drivers/media/platform/vivid/vivid-tpg.h
>  create mode 100644 drivers/media/platform/vivid/vivid-vbi-cap.c
>  create mode 100644 drivers/media/platform/vivid/vivid-vbi-cap.h
>  create mode 100644 drivers/media/platform/vivid/vivid-vbi-gen.c
>  create mode 100644 drivers/media/platform/vivid/vivid-vbi-gen.h
>  create mode 100644 drivers/media/platform/vivid/vivid-vbi-out.c
>  create mode 100644 drivers/media/platform/vivid/vivid-vbi-out.h
>  create mode 100644 drivers/media/platform/vivid/vivid-vid-cap.c
>  create mode 100644 drivers/media/platform/vivid/vivid-vid-cap.h
>  create mode 100644 drivers/media/platform/vivid/vivid-vid-common.c
>  create mode 100644 drivers/media/platform/vivid/vivid-vid-common.h
>  create mode 100644 drivers/media/platform/vivid/vivid-vid-out.c
>  create mode 100644 drivers/media/platform/vivid/vivid-vid-out.h
> --
> 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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-09-02 19:26 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-08-29  6:30 [GIT PULL FOR v3.18] Add vivid test driver, remove old vivi test driver Hans Verkuil
2014-09-02 19:26 ` Mauro Carvalho Chehab

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).