All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>,
	Mauro Carvalho Chehab <mchehab@infradead.org>
Subject: Re: [PATCHv4 12/25] [media] dvb_ca_en50221: add support for CA node at the media controller
Date: Mon, 16 Feb 2015 08:54:06 -0200	[thread overview]
Message-ID: <20150216085406.4b2264c3@recife.lan> (raw)
In-Reply-To: <54E1B2B2.3010906@xs4all.nl>

Em Mon, 16 Feb 2015 10:04:50 +0100
Hans Verkuil <hverkuil@xs4all.nl> escreveu:

> On 02/13/2015 11:57 PM, Mauro Carvalho Chehab wrote:
> > Make the dvb core CA support aware of the media controller and
> > register the corresponding devices.
> > 
> > Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
> > 
> > diff --git a/drivers/media/dvb-core/dvb_ca_en50221.c b/drivers/media/dvb-core/dvb_ca_en50221.c
> > index 0aac3096728e..2bf28eb97a64 100644
> > --- a/drivers/media/dvb-core/dvb_ca_en50221.c
> > +++ b/drivers/media/dvb-core/dvb_ca_en50221.c
> > @@ -1638,15 +1638,17 @@ static const struct file_operations dvb_ca_fops = {
> >  	.llseek = noop_llseek,
> >  };
> >  
> > -static struct dvb_device dvbdev_ca = {
> > +static const struct dvb_device dvbdev_ca = {
> >  	.priv = NULL,
> >  	.users = 1,
> >  	.readers = 1,
> >  	.writers = 1,
> > +#if defined(CONFIG_MEDIA_CONTROLLER_DVB)
> > +	.name = "ca_en50221",
> 
> I'd use 'dvb-ca-en50221': the dvb prefix makes it clear that this is a 
> dvb core device, and personally I prefer '-' over '_'.

Ok, will do, in order to standardize. Yet, at dvb, the core uses dvb_foo
for the several c files.

Regards,
Mauro

  reply	other threads:[~2015-02-16 10:54 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-13 22:57 [PATCHv4 00/25] dvb core: add basic support for the media controller Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 01/25] [media] ir-hix5hd2: remove writel/readl_relaxed define Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 02/25] [media] media: Fix DVB devnode representation at media controller Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 03/25] [media] Docbook: Fix documentation for media controller devnodes Mauro Carvalho Chehab
     [not found] ` <cover.1423867976.git.mchehab-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
2015-02-13 22:57   ` [PATCHv4 04/25] [media] media: add new types for DVB devnodes Mauro Carvalho Chehab
2015-02-13 22:57     ` Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 05/25] [media] DocBook: Document the DVB API devnodes at the media controller Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 06/25] [media] media: add a subdev type for tuner Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 07/25] [media] DocBook: Add tuner subdev at documentation Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 08/25] [media] dvbdev: add support for media controller Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 09/25] [media] cx231xx: add media controller support Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 10/25] [media] dvb_frontend: add media controller support for DVB frontend Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 11/25] [media] dmxdev: add support for demux/dvr nodes at media controller Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 12/25] [media] dvb_ca_en50221: add support for CA node at the " Mauro Carvalho Chehab
2015-02-16  9:04   ` Hans Verkuil
2015-02-16 10:54     ` Mauro Carvalho Chehab [this message]
2015-02-13 22:57 ` [PATCHv4 13/25] [media] dvb_net: add support for DVB net " Mauro Carvalho Chehab
2015-02-16  9:03   ` Hans Verkuil
2015-02-16 10:53     ` Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 14/25] [media] dvbdev: add pad for the DVB devnodes Mauro Carvalho Chehab
2015-02-13 22:57 ` [PATCHv4 15/25] [media] tuner-core: properly initialize media controller subdev Mauro Carvalho Chehab
2015-02-16  9:10   ` Hans Verkuil
2015-02-16 10:59     ` Mauro Carvalho Chehab
2015-02-16 14:39       ` Devin Heitmueller
2015-02-16 14:46         ` Hans Verkuil
2015-02-16 15:24           ` Devin Heitmueller
2015-02-13 22:57 ` [PATCHv4 16/25] [media] cx25840: fill the media controller entity Mauro Carvalho Chehab
2015-02-16  9:11   ` Hans Verkuil
2015-02-16 11:11     ` Mauro Carvalho Chehab
2015-02-16 11:16       ` Hans Verkuil
2015-02-16 11:42         ` Mauro Carvalho Chehab
2015-02-18 22:48   ` Lad, Prabhakar
2015-02-19 19:50     ` Mauro Carvalho Chehab
2015-02-19 21:50       ` Mauro Carvalho Chehab
2015-02-13 22:58 ` [PATCHv4 17/25] [media] cx231xx: initialize video/vbi pads Mauro Carvalho Chehab
2015-02-13 22:58 ` [PATCHv4 18/25] [media] cx231xx: create media links for analog mode Mauro Carvalho Chehab
2015-02-13 22:58 ` [PATCHv4 19/25] [media] dvbdev: represent frontend with two pads Mauro Carvalho Chehab
2015-02-13 22:58 ` [PATCHv4 20/25] [media] dvbdev: add a function to create DVB media graph Mauro Carvalho Chehab
2015-02-13 22:58 ` [PATCHv4 21/25] [media] cx231xx: create DVB graph Mauro Carvalho Chehab
2015-02-13 22:58 ` [PATCHv4 22/25] [media] dvbdev: enable DVB-specific links Mauro Carvalho Chehab
2015-02-13 22:58 ` [PATCHv4 23/25] [media] dvb-frontend: enable tuner link when the FE thread starts Mauro Carvalho Chehab
2015-02-13 22:58 ` [PATCHv4 24/25] [media] cx231xx: enable tuner->decoder link at videobuf start Mauro Carvalho Chehab
2015-02-16  9:27   ` Hans Verkuil
2015-02-16 11:19     ` Mauro Carvalho Chehab
2015-02-13 22:58 ` [PATCHv4 25/25] [media] dvb_frontend: start media pipeline while thread is running Mauro Carvalho Chehab
2015-02-16 12:52   ` Hans Verkuil
2015-02-14  9:32 ` [PATCHv4 00/25] dvb core: add basic support for the media controller Hans Verkuil
2015-02-14 11:00   ` Mauro Carvalho Chehab
2015-02-14 11:43     ` Hans Verkuil
2015-02-15 10:27       ` Mauro Carvalho Chehab
2015-02-16  9:55         ` Hans Verkuil
2015-02-16 10:50           ` Mauro Carvalho Chehab
2015-02-16 11:08             ` Hans Verkuil
2015-02-16  9:57 ` Hans Verkuil

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=20150216085406.4b2264c3@recife.lan \
    --to=mchehab@osg.samsung.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@infradead.org \
    /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.