Re: [PATCH 11/12] [media] tsin: c8sectpfe: Add Kconfig and Makefile for the driver.
Hi Mauro, Thanks for reviewing. Sending my reply again, as it looks like I dropped the CC list on my first reply, and my second reply bounced on the mailing lists :-( On Wed, 22 Jul 2015, Mauro Carvalho Chehab wrote: > Em Wed, 24 Jun 2015 16:11:09 +0100 > Peter Griffin escreveu: > > > This patch adds the Kconfig and Makefile for the c8sectpfe driver > > so it will be built. It also selects additional demodulator and tuners > > which are required by the supported NIM cards. > > > > Signed-off-by: Peter Griffin > > --- > > drivers/media/Kconfig | 1 + > > drivers/media/Makefile| 1 + > > drivers/media/tsin/c8sectpfe/Kconfig | 26 ++ > > drivers/media/tsin/c8sectpfe/Makefile | 11 +++ > > 4 files changed, 39 insertions(+) > > create mode 100644 drivers/media/tsin/c8sectpfe/Kconfig > > create mode 100644 drivers/media/tsin/c8sectpfe/Makefile > > > > diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig > > index 1570992..82bc1dc 100644 > > --- a/drivers/media/Kconfig > > +++ b/drivers/media/Kconfig > > @@ -170,6 +170,7 @@ source "drivers/media/pci/Kconfig" > > source "drivers/media/platform/Kconfig" > > source "drivers/media/mmc/Kconfig" > > source "drivers/media/radio/Kconfig" > > +source "drivers/media/tsin/c8sectpfe/Kconfig" > > > > comment "Supported FireWire (IEEE 1394) Adapters" > > depends on DVB_CORE && FIREWIRE > > diff --git a/drivers/media/Makefile b/drivers/media/Makefile > > index e608bbc..0a567b8 100644 > > --- a/drivers/media/Makefile > > +++ b/drivers/media/Makefile > > @@ -29,5 +29,6 @@ obj-y += rc/ > > # > > > > obj-y += common/ platform/ pci/ usb/ mmc/ firewire/ > > +obj-$(CONFIG_DVB_C8SECTPFE) += tsin/c8sectpfe/ > > Hmm... why are you adding it at a new "tsin" directory? We're putting > those SoC platform drivers under platform/. I didn't realise that. I will move this under there in the V2 patchset then? The rationale behind a new 'tsin' directory was that all the current DVB drivers seemed to be grouped by the underlying bus on which TS data enters the system (e.g. pci / usb). As this didn't fit in with that scheme I created a new tsin directory for SoC's which have dedicated hardware for Transport Stream INput (tsin) into the SoC. regards, Peter. p.s. Mauro - appologies again for spaming you -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 11/12] [media] tsin: c8sectpfe: Add Kconfig and Makefile for the driver.
Em Wed, 24 Jun 2015 16:11:09 +0100 Peter Griffin escreveu: > This patch adds the Kconfig and Makefile for the c8sectpfe driver > so it will be built. It also selects additional demodulator and tuners > which are required by the supported NIM cards. > > Signed-off-by: Peter Griffin > --- > drivers/media/Kconfig | 1 + > drivers/media/Makefile| 1 + > drivers/media/tsin/c8sectpfe/Kconfig | 26 ++ > drivers/media/tsin/c8sectpfe/Makefile | 11 +++ > 4 files changed, 39 insertions(+) > create mode 100644 drivers/media/tsin/c8sectpfe/Kconfig > create mode 100644 drivers/media/tsin/c8sectpfe/Makefile > > diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig > index 1570992..82bc1dc 100644 > --- a/drivers/media/Kconfig > +++ b/drivers/media/Kconfig > @@ -170,6 +170,7 @@ source "drivers/media/pci/Kconfig" > source "drivers/media/platform/Kconfig" > source "drivers/media/mmc/Kconfig" > source "drivers/media/radio/Kconfig" > +source "drivers/media/tsin/c8sectpfe/Kconfig" > > comment "Supported FireWire (IEEE 1394) Adapters" > depends on DVB_CORE && FIREWIRE > diff --git a/drivers/media/Makefile b/drivers/media/Makefile > index e608bbc..0a567b8 100644 > --- a/drivers/media/Makefile > +++ b/drivers/media/Makefile > @@ -29,5 +29,6 @@ obj-y += rc/ > # > > obj-y += common/ platform/ pci/ usb/ mmc/ firewire/ > +obj-$(CONFIG_DVB_C8SECTPFE) += tsin/c8sectpfe/ Hmm... why are you adding it at a new "tsin" directory? We're putting those SoC platform drivers under platform/. > obj-$(CONFIG_VIDEO_DEV) += radio/ > > diff --git a/drivers/media/tsin/c8sectpfe/Kconfig > b/drivers/media/tsin/c8sectpfe/Kconfig > new file mode 100644 > index 000..8d99a87 > --- /dev/null > +++ b/drivers/media/tsin/c8sectpfe/Kconfig > @@ -0,0 +1,26 @@ > +config DVB_C8SECTPFE > + tristate "STMicroelectronics C8SECTPFE DVB support" > + depends on DVB_CORE && I2C && (ARCH_STI || ARCH_MULTIPLATFORM) > + select DVB_LNBP21 if MEDIA_SUBDRV_AUTOSELECT > + select DVB_STV090x if MEDIA_SUBDRV_AUTOSELECT > + select DVB_STB6100 if MEDIA_SUBDRV_AUTOSELECT > + select DVB_STV6110 if MEDIA_SUBDRV_AUTOSELECT > + select DVB_STV0900 if MEDIA_SUBDRV_AUTOSELECT > + select DVB_STV0367 if MEDIA_SUBDRV_AUTOSELECT > + select DVB_PLL if MEDIA_SUBDRV_AUTOSELECT > + select MEDIA_TUNER_TDA18212 if MEDIA_SUBDRV_AUTOSELECT > + > + ---help--- > + This adds support for DVB front-end cards connected > + to TS inputs of STiH407/410 SoC. > + > + The driver currently supports C8SECTPFE's TS input block, > + memdma engine, and HW PID filtering. > + > + Supported DVB front-end cards are: > + - STMicroelectronics DVB-T B2100A (STV0367 + TDA18212) > + - STMicroelectronics DVB-T STV0367 PLL board (STV0367 + DTT7546X) > + - STMicroelectronics DVB-S/S2 STV0903 + STV6110 + LNBP24 board > + > + To compile this driver as a module, choose M here: the > + module will be called c8sectpfe. > diff --git a/drivers/media/tsin/c8sectpfe/Makefile > b/drivers/media/tsin/c8sectpfe/Makefile > new file mode 100644 > index 000..777f06d > --- /dev/null > +++ b/drivers/media/tsin/c8sectpfe/Makefile > @@ -0,0 +1,11 @@ > +c8sectpfe-y += c8sectpfe-core.o c8sectpfe-common.o c8sectpfe-dvb.o > + > +obj-$(CONFIG_DVB_C8SECTPFE) += c8sectpfe.o > + > +ifneq ($(CONFIG_DVB_C8SECTPFE),) > + c8sectpfe-y += c8sectpfe-debugfs.o > +endif > + > +ccflags-y += -Idrivers/media/i2c > +ccflags-y += -Idrivers/media/common > +ccflags-y += -Idrivers/media/dvb-core/ -Idrivers/media/dvb-frontends/ > -Idrivers/media/tuners/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 11/12] [media] tsin: c8sectpfe: Add Kconfig and Makefile for the driver.
Hi Paul, Thanks for reviewing. On Thu, 25 Jun 2015, Paul Bolle wrote: > On Wed, 2015-06-24 at 16:11 +0100, Peter Griffin wrote: > > --- /dev/null > > +++ b/drivers/media/tsin/c8sectpfe/Makefile > > > +c8sectpfe-y += c8sectpfe-core.o c8sectpfe-common.o c8sectpfe-dvb.o > > + > > +obj-$(CONFIG_DVB_C8SECTPFE) += c8sectpfe.o > > + > > +ifneq ($(CONFIG_DVB_C8SECTPFE),) > > + c8sectpfe-y += c8sectpfe-debugfs.o > > +endif > > Isn't the above equivalent to > c8sectpfe-y += c8sectpfe-core.o c8sectpfe-common.o c8sectpfe-dvb.o > c8sectpfe-debugfs.o > > obj-$(CONFIG_DVB_C8SECTPFE) += c8sectpfe.o > > Or am I missing something subtle here? No I think I just messed up. Will fix in v2. I suspect what happened was I was starting to add a CONFIG_DVB_C8SECTPFE_DEBUGFS Kconfig option, and then forgot ;) In v2 I have added a "select DEBUG_FS" to Kconfig for the driver, and put it all on one line. Also at the same time fixing some other Kconfig dependencies I noticed so it now has 'select LIBELF_32' and 'select FW_LOADER'. regards, Peter. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 11/12] [media] tsin: c8sectpfe: Add Kconfig and Makefile for the driver.
On Wed, 2015-06-24 at 16:11 +0100, Peter Griffin wrote: > --- /dev/null > +++ b/drivers/media/tsin/c8sectpfe/Makefile > +c8sectpfe-y += c8sectpfe-core.o c8sectpfe-common.o c8sectpfe-dvb.o > + > +obj-$(CONFIG_DVB_C8SECTPFE) += c8sectpfe.o > + > +ifneq ($(CONFIG_DVB_C8SECTPFE),) > + c8sectpfe-y += c8sectpfe-debugfs.o > +endif Isn't the above equivalent to c8sectpfe-y += c8sectpfe-core.o c8sectpfe-common.o c8sectpfe-dvb.o c8sectpfe-debugfs.o obj-$(CONFIG_DVB_C8SECTPFE) += c8sectpfe.o Or am I missing something subtle here? Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 11/12] [media] tsin: c8sectpfe: Add Kconfig and Makefile for the driver.
This patch adds the Kconfig and Makefile for the c8sectpfe driver so it will be built. It also selects additional demodulator and tuners which are required by the supported NIM cards. Signed-off-by: Peter Griffin --- drivers/media/Kconfig | 1 + drivers/media/Makefile| 1 + drivers/media/tsin/c8sectpfe/Kconfig | 26 ++ drivers/media/tsin/c8sectpfe/Makefile | 11 +++ 4 files changed, 39 insertions(+) create mode 100644 drivers/media/tsin/c8sectpfe/Kconfig create mode 100644 drivers/media/tsin/c8sectpfe/Makefile diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig index 1570992..82bc1dc 100644 --- a/drivers/media/Kconfig +++ b/drivers/media/Kconfig @@ -170,6 +170,7 @@ source "drivers/media/pci/Kconfig" source "drivers/media/platform/Kconfig" source "drivers/media/mmc/Kconfig" source "drivers/media/radio/Kconfig" +source "drivers/media/tsin/c8sectpfe/Kconfig" comment "Supported FireWire (IEEE 1394) Adapters" depends on DVB_CORE && FIREWIRE diff --git a/drivers/media/Makefile b/drivers/media/Makefile index e608bbc..0a567b8 100644 --- a/drivers/media/Makefile +++ b/drivers/media/Makefile @@ -29,5 +29,6 @@ obj-y += rc/ # obj-y += common/ platform/ pci/ usb/ mmc/ firewire/ +obj-$(CONFIG_DVB_C8SECTPFE) += tsin/c8sectpfe/ obj-$(CONFIG_VIDEO_DEV) += radio/ diff --git a/drivers/media/tsin/c8sectpfe/Kconfig b/drivers/media/tsin/c8sectpfe/Kconfig new file mode 100644 index 000..8d99a87 --- /dev/null +++ b/drivers/media/tsin/c8sectpfe/Kconfig @@ -0,0 +1,26 @@ +config DVB_C8SECTPFE + tristate "STMicroelectronics C8SECTPFE DVB support" + depends on DVB_CORE && I2C && (ARCH_STI || ARCH_MULTIPLATFORM) + select DVB_LNBP21 if MEDIA_SUBDRV_AUTOSELECT + select DVB_STV090x if MEDIA_SUBDRV_AUTOSELECT + select DVB_STB6100 if MEDIA_SUBDRV_AUTOSELECT + select DVB_STV6110 if MEDIA_SUBDRV_AUTOSELECT + select DVB_STV0900 if MEDIA_SUBDRV_AUTOSELECT + select DVB_STV0367 if MEDIA_SUBDRV_AUTOSELECT + select DVB_PLL if MEDIA_SUBDRV_AUTOSELECT + select MEDIA_TUNER_TDA18212 if MEDIA_SUBDRV_AUTOSELECT + + ---help--- + This adds support for DVB front-end cards connected + to TS inputs of STiH407/410 SoC. + + The driver currently supports C8SECTPFE's TS input block, + memdma engine, and HW PID filtering. + + Supported DVB front-end cards are: + - STMicroelectronics DVB-T B2100A (STV0367 + TDA18212) + - STMicroelectronics DVB-T STV0367 PLL board (STV0367 + DTT7546X) + - STMicroelectronics DVB-S/S2 STV0903 + STV6110 + LNBP24 board + + To compile this driver as a module, choose M here: the + module will be called c8sectpfe. diff --git a/drivers/media/tsin/c8sectpfe/Makefile b/drivers/media/tsin/c8sectpfe/Makefile new file mode 100644 index 000..777f06d --- /dev/null +++ b/drivers/media/tsin/c8sectpfe/Makefile @@ -0,0 +1,11 @@ +c8sectpfe-y += c8sectpfe-core.o c8sectpfe-common.o c8sectpfe-dvb.o + +obj-$(CONFIG_DVB_C8SECTPFE) += c8sectpfe.o + +ifneq ($(CONFIG_DVB_C8SECTPFE),) + c8sectpfe-y += c8sectpfe-debugfs.o +endif + +ccflags-y += -Idrivers/media/i2c +ccflags-y += -Idrivers/media/common +ccflags-y += -Idrivers/media/dvb-core/ -Idrivers/media/dvb-frontends/ -Idrivers/media/tuners/ -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
