From: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
To: Linux Media Mailing List <linux-media@vger.kernel.org>
Cc: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>,
Mauro Carvalho Chehab <mchehab@infradead.org>,
Kees Cook <keescook@chromium.org>,
Devin Heitmueller <dheitmueller@kernellabs.com>,
Philippe Ombredanne <pombredanne@nexb.com>,
Pravin Shedge <pravin.shedge4linux@gmail.com>,
Brian Warner <brian.warner@samsung.com>,
Bhumika Goyal <bhumirks@gmail.com>,
Kate Stewart <kstewart@linuxfoundation.org>,
Max Kellermann <max.kellermann@gmail.com>,
Shuah Khan <shuah@kernel.org>,
Michael Krufky <mkrufky@linuxtv.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Antti Palosaari <crope@iki.fi>,
Nasser Afshin <afshin.nasser@gmail.com>,
Marco Felsch <m.felsch@pengutronix.de>,
Javier Martinez Canillas <javierm@redhat.com>,
Hans Verkuil <hans.verkuil@cisco.com>,
Philipp Zabel <p.zabel@pengutronix.de>,
"Gustavo A. R. Silva" <gustavo@embeddedor.com>,
Thomas Gleixner <tglx@linutronix.de>,
Hans Verkuil <hverkuil@xs4all.nl>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Subject: [PATCH 00/13] Better handle pads for tuning/decoder part of the devices
Date: Wed, 1 Aug 2018 12:55:02 -0300 [thread overview]
Message-ID: <cover.1533138685.git.mchehab+samsung@kernel.org> (raw)
At PC consumer devices, it is very common that the bridge same driver
to be attached to different types of tuners and demods. We need a way
for the Kernel to properly identify what kind of signal is provided by each
PAD, in order to properly setup the pipelines.
The previous approach were to hardcode a fixed number of PADs for all
elements of the same type. This is not good, as different devices may
actually have a different number of pads.
It was acceptable in the past, as there were a promisse of adding "soon"
a properties API that would allow to identify the type for each PADs, but
this was never merged (or even a patchset got submitted).
So, replace this approach by another one: add a "taint" mark to pads that
contain different types of signals.
I tried to minimize the number of signals, in order to make it simpler to
convert from the past way.
For now, it is tested only with a simple grabber device. I intend to do
more tests before merging it, but it would be interesting to have this
merged for Kernel 4.19, as we'll now be exposing the pad index via
the MC API version 2.
Mauro Carvalho Chehab (13):
media: v4l2: remove VBI output pad
media: v4l2: taint pads with the signal types for consumer devices
v4l2-mc: switch it to use the new approach to setup pipelines
media: dvb: use signals to discover pads
media: au0828: use signals instead of hardcoding a pad number
media: au8522: declare its own pads
media: msp3400: declare its own pads
media: saa7115: declare its own pads
media: tvp5150: declare its own pads
media: si2157: declare its own pads
media: saa7134: declare its own pads
media: mxl111sf: declare its own pads
media: v4l2-mc: get rid of global pad indexes
drivers/media/dvb-core/dvbdev.c | 19 +++--
drivers/media/dvb-frontends/au8522_decoder.c | 10 ++-
drivers/media/dvb-frontends/au8522_priv.h | 9 ++-
drivers/media/i2c/msp3400-driver.c | 6 +-
drivers/media/i2c/msp3400-driver.h | 8 +-
drivers/media/i2c/saa7115.c | 18 +++--
drivers/media/i2c/tvp5150.c | 21 ++++--
drivers/media/media-entity.c | 26 +++++++
drivers/media/pci/saa7134/saa7134-core.c | 9 ++-
drivers/media/pci/saa7134/saa7134.h | 8 +-
drivers/media/tuners/si2157.c | 11 ++-
drivers/media/tuners/si2157_priv.h | 9 ++-
drivers/media/usb/au0828/au0828-core.c | 12 +--
drivers/media/usb/dvb-usb-v2/mxl111sf.c | 8 +-
drivers/media/usb/dvb-usb-v2/mxl111sf.h | 8 +-
drivers/media/v4l2-core/tuner-core.c | 18 +++++
drivers/media/v4l2-core/v4l2-mc.c | 73 +++++++++++++-----
include/media/media-entity.h | 54 ++++++++++++++
include/media/v4l2-mc.h | 78 --------------------
19 files changed, 266 insertions(+), 139 deletions(-)
--
2.17.1
next reply other threads:[~2018-08-01 17:41 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-01 15:55 Mauro Carvalho Chehab [this message]
2018-08-01 15:55 ` [PATCH 01/13] media: v4l2: remove VBI output pad Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 02/13] media: v4l2: taint pads with the signal types for consumer devices Mauro Carvalho Chehab
2018-09-26 14:09 ` Laurent Pinchart
2018-09-27 10:01 ` Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 03/13] v4l2-mc: switch it to use the new approach to setup pipelines Mauro Carvalho Chehab
2018-09-26 14:44 ` Laurent Pinchart
2018-09-27 10:40 ` Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 04/13] media: dvb: use signals to discover pads Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 05/13] media: au0828: use signals instead of hardcoding a pad number Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 06/13] media: au8522: declare its own pads Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 07/13] media: msp3400: " Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 08/13] media: saa7115: " Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 09/13] media: tvp5150: " Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 10/13] media: si2157: " Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 11/13] media: saa7134: " Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 12/13] media: mxl111sf: " Mauro Carvalho Chehab
2018-08-01 15:55 ` [PATCH 13/13] media: v4l2-mc: get rid of global pad indexes Mauro Carvalho Chehab
2018-08-02 9:08 ` Hans Verkuil
2018-08-02 9:30 ` Mauro Carvalho Chehab
2018-08-02 9:12 ` [PATCH 00/13] Better handle pads for tuning/decoder part of the devices Hans Verkuil
2018-08-02 9:39 ` Mauro Carvalho Chehab
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1533138685.git.mchehab+samsung@kernel.org \
--to=mchehab+samsung@kernel.org \
--cc=afshin.nasser@gmail.com \
--cc=bhumirks@gmail.com \
--cc=brian.warner@samsung.com \
--cc=crope@iki.fi \
--cc=dheitmueller@kernellabs.com \
--cc=gregkh@linuxfoundation.org \
--cc=gustavo@embeddedor.com \
--cc=hans.verkuil@cisco.com \
--cc=hverkuil@xs4all.nl \
--cc=javierm@redhat.com \
--cc=keescook@chromium.org \
--cc=kstewart@linuxfoundation.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=m.felsch@pengutronix.de \
--cc=max.kellermann@gmail.com \
--cc=mchehab@infradead.org \
--cc=mkrufky@linuxtv.org \
--cc=p.zabel@pengutronix.de \
--cc=pombredanne@nexb.com \
--cc=pravin.shedge4linux@gmail.com \
--cc=sakari.ailus@linux.intel.com \
--cc=shuah@kernel.org \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox