OSDN Git Service

media: cxd2099: move driver out of staging into dvb-frontends
authorDaniel Scheller <d.scheller@gmx.net>
Thu, 8 Feb 2018 19:53:17 +0000 (14:53 -0500)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 6 Mar 2018 13:50:18 +0000 (08:50 -0500)
According to the TODO file, this driver only landed in staging because of
the way device nodes and data transfers are handled. Besides that this way
(use of secX devices) has become sort of standard to date (ie. VDR
supports this literally since ages via the ddci plugin, TVHeadend received
this functionality lately, and minisatip being currently worked on
regarding this), most importantly this I2C client only driver isn't even
responsible for setting up device nodes, not for handling data
transfer and so on, but only serves as interface for the dvb_ca_en50221
subsystem, just like every other DVB card out in the wild, with hard-wired
or such flexible CA interfaces. And, it would even work with cards having
the cxd2099 controller hard-wired.

Also, this driver received quite some love and even is a proper I2C client
driver by now. So, as this driver acts as a EN50221 frontend device, move
it to dvb-frontends. There is no need to keep it buried in staging.

This commit also updates all affected Kconfig and Makefile's, and adds
MEDIA_AUTOSELECT depends to ddbridge and ngene.

Signed-off-by: Daniel Scheller <d.scheller@gmx.net>
Signed-off-by: Jasmin Jessich <jasmin@anw.at>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
13 files changed:
drivers/media/dvb-frontends/Kconfig
drivers/media/dvb-frontends/Makefile
drivers/media/dvb-frontends/cxd2099.c [moved from drivers/staging/media/cxd2099/cxd2099.c with 100% similarity]
drivers/media/dvb-frontends/cxd2099.h [moved from drivers/staging/media/cxd2099/cxd2099.h with 100% similarity]
drivers/media/pci/ddbridge/Kconfig
drivers/media/pci/ddbridge/Makefile
drivers/media/pci/ngene/Kconfig
drivers/media/pci/ngene/Makefile
drivers/staging/media/Kconfig
drivers/staging/media/Makefile
drivers/staging/media/cxd2099/Kconfig [deleted file]
drivers/staging/media/cxd2099/Makefile [deleted file]
drivers/staging/media/cxd2099/TODO [deleted file]

index d17722e..ca8c7ed 100644 (file)
@@ -904,6 +904,18 @@ config DVB_HELENE
        help
          Say Y when you want to support this frontend.
 
+comment "Common Interface (EN50221) controller drivers"
+       depends on DVB_CORE
+
+config DVB_CXD2099
+       tristate "CXD2099AR Common Interface driver"
+       depends on DVB_CORE && I2C
+       ---help---
+         A driver for the CI controller currently found mostly on
+         Digital Devices DuoFlex CI (single) addon modules.
+
+         Say Y when you want to support these devices.
+
 comment "Tools to develop new frontends"
 
 config DVB_DUMMY_FE
index 4be59fe..abbd76e 100644 (file)
@@ -129,3 +129,4 @@ obj-$(CONFIG_DVB_HORUS3A) += horus3a.o
 obj-$(CONFIG_DVB_ASCOT2E) += ascot2e.o
 obj-$(CONFIG_DVB_HELENE) += helene.o
 obj-$(CONFIG_DVB_ZD1301_DEMOD) += zd1301_demod.o
+obj-$(CONFIG_DVB_CXD2099) += cxd2099.o
index f43d0b8..a422dde 100644 (file)
@@ -13,6 +13,7 @@ config DVB_DDBRIDGE
        select DVB_LNBH25 if MEDIA_SUBDRV_AUTOSELECT
        select MEDIA_TUNER_TDA18212 if MEDIA_SUBDRV_AUTOSELECT
        select DVB_MXL5XX if MEDIA_SUBDRV_AUTOSELECT
+       select DVB_CXD2099 if MEDIA_SUBDRV_AUTOSELECT
        ---help---
          Support for cards with the Digital Devices PCI express bridge:
          - Octopus PCIe Bridge
index f58fdec..745b37d 100644 (file)
@@ -10,6 +10,3 @@ obj-$(CONFIG_DVB_DDBRIDGE) += ddbridge.o
 
 ccflags-y += -Idrivers/media/dvb-frontends/
 ccflags-y += -Idrivers/media/tuners/
-
-# For the staging CI driver cxd2099
-ccflags-y += -Idrivers/staging/media/cxd2099/
index 637d506..390ed75 100644 (file)
@@ -8,6 +8,7 @@ config DVB_NGENE
        select DVB_DRXK if MEDIA_SUBDRV_AUTOSELECT
        select DVB_TDA18271C2DD if MEDIA_SUBDRV_AUTOSELECT
        select MEDIA_TUNER_MT2131 if MEDIA_SUBDRV_AUTOSELECT
+       select DVB_CXD2099 if MEDIA_SUBDRV_AUTOSELECT
        ---help---
          Support for Micronas PCI express cards with nGene bridge.
 
index e4208f5..ec450ad 100644 (file)
@@ -9,6 +9,3 @@ obj-$(CONFIG_DVB_NGENE) += ngene.o
 
 ccflags-y += -Idrivers/media/dvb-frontends/
 ccflags-y += -Idrivers/media/tuners/
-
-# For the staging CI driver cxd2099
-ccflags-y += -Idrivers/staging/media/cxd2099/
index f99287e..4c495a1 100644 (file)
@@ -23,8 +23,6 @@ source "drivers/staging/media/atomisp/Kconfig"
 
 source "drivers/staging/media/bcm2048/Kconfig"
 
-source "drivers/staging/media/cxd2099/Kconfig"
-
 source "drivers/staging/media/davinci_vpfe/Kconfig"
 
 source "drivers/staging/media/imx/Kconfig"
index a98efd5..61a5765 100644 (file)
@@ -1,6 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0
 obj-$(CONFIG_I2C_BCM2048)      += bcm2048/
-obj-$(CONFIG_DVB_CXD2099)      += cxd2099/
 obj-$(CONFIG_VIDEO_IMX_MEDIA)  += imx/
 obj-$(CONFIG_SOC_CAMERA_IMX074)        += imx074/
 obj-$(CONFIG_SOC_CAMERA_MT9T031)       += mt9t031/
diff --git a/drivers/staging/media/cxd2099/Kconfig b/drivers/staging/media/cxd2099/Kconfig
deleted file mode 100644 (file)
index b48aefd..0000000
+++ /dev/null
@@ -1,12 +0,0 @@
-config DVB_CXD2099
-       tristate "CXD2099AR Common Interface driver"
-       depends on DVB_CORE && PCI && I2C
-       ---help---
-         Support for the CI module found on cards based on
-         - Micronas ngene PCIe bridge: cineS2 etc.
-         - Digital Devices PCIe bridge: Octopus series
-
-         For now, data is passed through '/dev/dvb/adapterX/sec0':
-           - Encrypted data must be written to 'sec0'.
-           - Decrypted data can be read from 'sec0'.
-           - Setup the CAM using device 'ca0'.
diff --git a/drivers/staging/media/cxd2099/Makefile b/drivers/staging/media/cxd2099/Makefile
deleted file mode 100644 (file)
index 30432c9..0000000
+++ /dev/null
@@ -1,4 +0,0 @@
-obj-$(CONFIG_DVB_CXD2099) += cxd2099.o
-
-ccflags-y += -Idrivers/media/dvb-frontends/
-ccflags-y += -Idrivers/media/tuners/
diff --git a/drivers/staging/media/cxd2099/TODO b/drivers/staging/media/cxd2099/TODO
deleted file mode 100644 (file)
index 375bb6f..0000000
+++ /dev/null
@@ -1,12 +0,0 @@
-For now, data is passed through '/dev/dvb/adapterX/sec0':
- - Encrypted data must be written to 'sec0'.
- - Decrypted data can be read from 'sec0'.
- - Setup the CAM using device 'ca0'.
-
-But this is wrong. There are some discussions about the proper way for
-doing it, as seen at:
-       http://www.mail-archive.com/linux-media@vger.kernel.org/msg22196.html
-
-While there's no proper fix for it, the driver should be kept in staging.
-
-Patches should be submitted to: linux-media@vger.kernel.org.