From: Daniel Scheller <d.scheller.oss@gmail.com>
To: linux-media@vger.kernel.org, mchehab@kernel.org,
mchehab@s-opensource.com
Cc: rjkm@metzlerbros.de, jasmin@anw.at
Subject: [PATCH 0/9] STV0910/STV6111 drivers, ddbridge CineS2 V7 support
Date: Sat, 24 Jun 2017 18:02:52 +0200 [thread overview]
Message-ID: <20170624160301.17710-1-d.scheller.oss@gmail.com> (raw)
From: Daniel Scheller <d.scheller@gmx.net>
For Linux 4.14.
This series adds drivers for the ST STV0910 DVB-S/S2 demodulator ICs and
the ST STV6111 DVB-S/S2 tuners, and utilises them to enable ddbridge to
support the current line of Digital Devices DVB-S/S2 hardware (e.g. Cine
S2 V7/V7A adapters, DuoFlex S2 V4 addon modules and maybe more, with
similar components).
The two new drivers have been picked up from Digital Devices' vendor-
provided dddvb driver package, as of release 0.9.29. Permission to reuse
(and mainline) them was formally granted by Ralph Metzler
<rjkm@metzlerbros.de>.
Drivers have been cleaned up alot (formatting fixes, dead code removal,
features depending on not-yet-available API changes removed). Checkpatch
complaints left:
WARNING: please write a paragraph that describes the config symbol fully
#39: FILE: drivers/media/dvb-frontends/Kconfig:31:
+config DVB_STV0910
Not sure what checkpatch demands, since a module description exists in
Kconfig... Applies to the stv6111 aswell.
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#64:
new file mode 100644
See below.
WARNING: 'VALIDE' may be misspelled - perhaps 'VALID'?
#3314: FILE: drivers/media/dvb-frontends/stv0910_regs.h:1467:
+#define FSTV0910_P2_NOSRAM_VALIDE 0xf30e0004
Picked from upstream. Since I'm not sure if this really is a mistake
(maybe the hardware vendor really wants to name the reg like this?) so
kept as-is.
Also, all uppercase writing an camel case are still there yet, and if you
don't ultimately insist of having this changed (there are drivers all over
the place which have uppercase vars), I'll prefer to keep it like it is
at the moment to not diverge even more from upstream, that'll ease syncing
later on that way. Else, I can change this ofc.
Patch 2 is a fix for an issue found while preparing this series for
posting, and was in parallel submitted to the vendor's GIT repository.
Patch 8 eventually needs fixup (printk -> dev_*) wrt
https://patchwork.linuxtv.org/patch/42034/
Regarding MAINTAINERS and maintainership: I already offered to volunteer
to regularly check for upstream updates and sync them to mainline, as
outlined at
http://www.mail-archive.com/linux-media@vger.kernel.org/msg114469.html .
If this qualifies for an entry in MAINTAINERS, then that one's missing.
Please advise.
Mauro, I assume you're the one who reviews this (since these are new
drivers). It'd be very great to have this in for 4.14 to tackle the
ddbridge bump for 4.15.
Daniel Scheller (9):
[media] dvb-frontends: add ST STV0910 DVB-S/S2 demodulator frontend
driver
[media] dvb-frontends/stv0910: Fix possible buffer overflow
[media] dvb-frontends/stv0910: add multistream (ISI) and PLS
capabilities
[media] dvb-frontends/stv0910: Fix signal strength reporting
[media] dvb-frontends/stv0910: Add missing set_frontend fe-op
[media] dvb-frontends: add ST STV6111 DVB-S/S2 tuner frontend driver
[media] ddbridge: return stv09xx id in port_has_stv0900_aa()
[media] ddbridge: support for CineS2 V7(A) and DuoFlex S2 V4 hardware
[media] ddbridge: stv0910 single demod mode module option
drivers/media/dvb-frontends/Kconfig | 18 +
drivers/media/dvb-frontends/Makefile | 2 +
drivers/media/dvb-frontends/stv0910.c | 1765 +++++++++++
drivers/media/dvb-frontends/stv0910.h | 32 +
drivers/media/dvb-frontends/stv0910_regs.h | 4759 ++++++++++++++++++++++++++++
drivers/media/dvb-frontends/stv6111.c | 675 ++++
drivers/media/dvb-frontends/stv6111.h | 20 +
drivers/media/pci/ddbridge/Kconfig | 4 +
drivers/media/pci/ddbridge/ddbridge-core.c | 149 +-
drivers/media/pci/ddbridge/ddbridge.h | 2 +
10 files changed, 7418 insertions(+), 8 deletions(-)
create mode 100644 drivers/media/dvb-frontends/stv0910.c
create mode 100644 drivers/media/dvb-frontends/stv0910.h
create mode 100644 drivers/media/dvb-frontends/stv0910_regs.h
create mode 100644 drivers/media/dvb-frontends/stv6111.c
create mode 100644 drivers/media/dvb-frontends/stv6111.h
--
2.13.0
next reply other threads:[~2017-06-24 16:03 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-24 16:02 Daniel Scheller [this message]
2017-06-24 16:02 ` [PATCH 1/9] [media] dvb-frontends: add ST STV0910 DVB-S/S2 demodulator frontend driver Daniel Scheller
2017-06-24 16:02 ` [PATCH 2/9] [media] dvb-frontends/stv0910: Fix possible buffer overflow Daniel Scheller
2017-06-24 16:02 ` [PATCH 3/9] [media] dvb-frontends/stv0910: add multistream (ISI) and PLS capabilities Daniel Scheller
2017-06-24 16:02 ` [PATCH 4/9] [media] dvb-frontends/stv0910: Fix signal strength reporting Daniel Scheller
2017-06-26 8:55 ` Ralph Metzler
2017-06-26 10:00 ` Mauro Carvalho Chehab
2017-06-26 10:14 ` Ralph Metzler
2017-06-26 15:39 ` Daniel Scheller
2017-06-24 16:02 ` [PATCH 5/9] [media] dvb-frontends/stv0910: Add missing set_frontend fe-op Daniel Scheller
2017-06-24 16:02 ` [PATCH 6/9] [media] dvb-frontends: add ST STV6111 DVB-S/S2 tuner frontend driver Daniel Scheller
2017-06-24 16:02 ` [PATCH 7/9] [media] ddbridge: return stv09xx id in port_has_stv0900_aa() Daniel Scheller
2017-06-24 16:03 ` [PATCH 8/9] [media] ddbridge: support for CineS2 V7(A) and DuoFlex S2 V4 hardware Daniel Scheller
2017-06-24 16:03 ` [PATCH 9/9] [media] ddbridge: stv0910 single demod mode module option Daniel Scheller
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=20170624160301.17710-1-d.scheller.oss@gmail.com \
--to=d.scheller.oss@gmail.com \
--cc=jasmin@anw.at \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=mchehab@s-opensource.com \
--cc=rjkm@metzlerbros.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.