From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]:23478 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753293Ab3AFMRV (ORCPT ); Sun, 6 Jan 2013 07:17:21 -0500 Date: Sun, 6 Jan 2013 10:16:42 -0200 From: Mauro Carvalho Chehab To: Mauro Carvalho Chehab Cc: Antti Palosaari , linux-media@vger.kernel.org Subject: Re: [PATCH RFC 01/11] dvb_usb_v2: make remote controller optional Message-ID: <20130106101642.2336a090@redhat.com> In-Reply-To: <20130106101129.552100d3@redhat.com> References: <1355100335-2123-1-git-send-email-crope@iki.fi> <20130106101129.552100d3@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Em Sun, 6 Jan 2013 10:11:29 -0200 Mauro Carvalho Chehab escreveu: > Em Mon, 10 Dec 2012 02:45:25 +0200 > Antti Palosaari escreveu: > > > Make it possible to compile dvb_usb_v2 driver without the remote > > controller (RC-core). > > > > Signed-off-by: Antti Palosaari > > --- > > drivers/media/usb/dvb-usb-v2/Kconfig | 3 ++- > > drivers/media/usb/dvb-usb-v2/dvb_usb.h | 9 +++++++++ > > drivers/media/usb/dvb-usb-v2/dvb_usb_core.c | 12 ++++++++++++ > > 3 files changed, 23 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/media/usb/dvb-usb-v2/Kconfig b/drivers/media/usb/dvb-usb-v2/Kconfig > > index 834bfec..60f4240 100644 > > --- a/drivers/media/usb/dvb-usb-v2/Kconfig > > +++ b/drivers/media/usb/dvb-usb-v2/Kconfig > > @@ -1,6 +1,6 @@ > > config DVB_USB_V2 > > tristate "Support for various USB DVB devices v2" > > - depends on DVB_CORE && USB && I2C && RC_CORE > > + depends on DVB_CORE && USB && I2C > > help > > By enabling this you will be able to choose the various supported > > USB1.1 and USB2.0 DVB devices. > > @@ -113,6 +113,7 @@ config DVB_USB_IT913X > > config DVB_USB_LME2510 > > tristate "LME DM04/QQBOX DVB-S USB2.0 support" > > depends on DVB_USB_V2 > > + depends on RC_CORE > > select DVB_TDA10086 if MEDIA_SUBDRV_AUTOSELECT > > select DVB_TDA826X if MEDIA_SUBDRV_AUTOSELECT > > select DVB_STV0288 if MEDIA_SUBDRV_AUTOSELECT > > diff --git a/drivers/media/usb/dvb-usb-v2/dvb_usb.h b/drivers/media/usb/dvb-usb-v2/dvb_usb.h > > index 059291b..e2678a7 100644 > > --- a/drivers/media/usb/dvb-usb-v2/dvb_usb.h > > +++ b/drivers/media/usb/dvb-usb-v2/dvb_usb.h > > @@ -400,4 +400,13 @@ extern int dvb_usbv2_reset_resume(struct usb_interface *); > > extern int dvb_usbv2_generic_rw(struct dvb_usb_device *, u8 *, u16, u8 *, u16); > > extern int dvb_usbv2_generic_write(struct dvb_usb_device *, u8 *, u16); > > > > +/* stub implementations that will be never called when RC-core is disabled */ > > +#if !defined(CONFIG_RC_CORE) && !defined(CONFIG_RC_CORE_MODULE) > > +#define rc_repeat(args...) > > +#define rc_keydown(args...) > > +#define rc_keydown_notimeout(args...) > > +#define rc_keyup(args...) > > +#define rc_g_keycode_from_table(args...) 0 > > +#endif > > + > > Those stub seem to be miss-placed: they belong to rc-core, and not to dvb-usb-v2. > So, those changes should be, instead, at: include/media/rc-core.h Hmm.. you removed those later. Ok, I'll apply this series, but I'll then add an additional patch at the end putting the above at rc-core.h, as it may help with other drivers. -- Cheers, Mauro