From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: linux-media@vger.kernel.org
Subject: Re: What would be a good time to move subdev drivers to a subdev directory?
Date: Sun, 28 Mar 2010 12:38:31 -0300 [thread overview]
Message-ID: <4BAF77F7.3070205@redhat.com> (raw)
In-Reply-To: <201003281224.17678.hverkuil@xs4all.nl>
Hans Verkuil wrote:
> Hi Mauro,
>
> Currently drivers/media/video is a mix of subdev drivers and bridge/platform
> drivers. I think it would be good to create a drivers/media/subdev directory
> where subdev drivers can go.
>
> We discussed in the past whether we should have categories for audio subdevs,
> video subdevs, etc. but I think that will cause problems, especially with
> future multifunction devices.
Due to the discussions we had on the last time, I'm not so sure that such move
would be good: There are some cases where the division of a subdev is more a
matter of a logical organization than a physical device division. for example,
cx231xx is just one chip, but, as it has internally the same functionalities as
a cx2584x, the cx2584x is a subdev used by the driver. There are other similar
examples on other IC's and SoC.
I remember that Oliver argued on that time that the better would be to reduce the
number of subdirs, instead of increasing. On that discussions, I got convinced
that he was right, but maybe we have some new reasons to create a subdev dir.
So, let's get some feedback from developers about this again. Whatever decided,
we should clearly document the used criteria, to avoid having drivers misplaced.
Ah, as we're talking about drivers directory, I'm intending to move the Remote
Controller common code to another place, likely drivers/input/rc or drivers/rc.
The idea is to use this subsystem for pure input devices as well. By keeping it
at drivers/media, it will be missplaced.
> What is your opinion on this, and what would be a good time to start moving
> drivers?
If we're doing this change, I prefer to generate the patch by the end of a
merge window, after merging from everybody else and being sure that trivial patches
also got merged.
Comments?
--
Cheers,
Mauro
next prev parent reply other threads:[~2010-03-28 15:38 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-28 10:24 What would be a good time to move subdev drivers to a subdev directory? Hans Verkuil
2010-03-28 15:38 ` Mauro Carvalho Chehab [this message]
2010-03-28 16:03 ` Hans Verkuil
2010-03-28 23:08 ` Andy Walls
2010-03-29 6:07 ` Hans Verkuil
2010-03-29 7:31 ` Laurent Pinchart
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=4BAF77F7.3070205@redhat.com \
--to=mchehab@redhat.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-media@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox