public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Rob Herring <robherring2@gmail.com>
Cc: Philipp Zabel <p.zabel@pengutronix.de>,
	Mauro Carvalho Chehab <m.chehab@samsung.com>,
	Grant Likely <grant.likely@linaro.org>,
	Rob Herring <robh+dt@kernel.org>,
	Sylwester Nawrocki <s.nawrocki@samsung.com>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Philipp Zabel <philipp.zabel@gmail.com>
Subject: Re: [RFC PATCH] [media]: of: move graph helpers from drivers/media/v4l2-core to drivers/of
Date: Tue, 11 Feb 2014 14:52:48 +0000	[thread overview]
Message-ID: <20140211145248.GI26684@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <CAL_Jsq+U9zU1i+STLHMBjY5BeEP6djYnJVE5X1ix-D2q_zWztQ@mail.gmail.com>

On Tue, Feb 11, 2014 at 07:56:33AM -0600, Rob Herring wrote:
> On Tue, Feb 11, 2014 at 5:45 AM, Philipp Zabel <p.zabel@pengutronix.de> wrote:
> > This allows to reuse the same parser code from outside the V4L2 framework,
> > most importantly from display drivers. There have been patches that duplicate
> > the code (and I am going to send one of my own), such as
> > http://lists.freedesktop.org/archives/dri-devel/2013-August/043308.html
> > and others that parse the same binding in a different way:
> > https://www.mail-archive.com/linux-omap@vger.kernel.org/msg100761.html
> >
> > I think that all common video interface parsing helpers should be moved to a
> > single place, outside of the specific subsystems, so that it can be reused
> > by all drivers.
> 
> Perhaps that should be done rather than moving to drivers/of now and
> then again to somewhere else.

Do you have a better suggestion where it should move to?

drivers/gpu/drm - no, because v4l2 wants to use it
drivers/media/video - no, because DRM drivers want to use it
drivers/video - no, because v4l2 and drm drivers want to use it

Maybe drivers/of-graph/ ?  Or maybe it's just as good a place to move it
into drivers/of ?

-- 
FTTC broadband for 0.8mile line: 5.8Mbps down 500kbps up.  Estimation
in database were 13.1 to 19Mbit for a good line, about 7.5+ for a bad.
Estimate before purchase was "up to 13.2Mbit".

  reply	other threads:[~2014-02-11 14:53 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-11 11:45 [RFC PATCH] [media]: of: move graph helpers from drivers/media/v4l2-core to drivers/of Philipp Zabel
     [not found] ` < CAL_Jsq+U9zU1i+STLHMBjY5BeEP6djYnJVE5X1ix-D2q_zWztQ@mail.gmail.com>
     [not found]   ` < 20140217181451.7EB7FC4044D@trevor.secretlab.ca>
     [not found] ` < 4339286.FzhQ2m6hoA@avalon>
     [not found] ` < 20140310145815.17595C405FA@trevor.secretlab.ca>
     [not found] ` < 20140320153804.35d5b835@samsung.com>
2014-02-11 13:56 ` Rob Herring
2014-02-11 14:52   ` Russell King - ARM Linux [this message]
2014-02-11 15:23     ` Laurent Pinchart
2014-02-11 16:36       ` Philipp Zabel
2014-02-11 17:22         ` Sylwester Nawrocki
2014-02-11 17:41           ` Mauro Carvalho Chehab
2014-02-11 21:00             ` Guennadi Liakhovetski
2014-02-11 21:46           ` Philipp Zabel
2014-02-11 17:24         ` Laurent Pinchart
2014-02-11 16:30     ` Rob Herring
2014-02-11 15:27   ` Philipp Zabel
2014-02-17 18:14   ` Grant Likely
2014-02-18  7:06     ` Sascha Hauer
2014-02-18 16:26       ` Grant Likely
     [not found]         ` < 1393263389.3091.82.camel@pizza.hi.pengutronix.de>
     [not found]           ` <20140226110114. CF2C7C40A89@trevor.secretlab.ca>
2014-02-24 17:36         ` Philipp Zabel
2014-02-26 11:01           ` Grant Likely
2014-02-26 14:48             ` Philipp Zabel
2014-02-27  8:36               ` Tomi Valkeinen
2014-03-07 17:06                 ` Grant Likely
2014-03-08 10:57                   ` Tomi Valkeinen
2014-03-07 17:05               ` Grant Likely
2014-03-08 10:33                 ` Tomi Valkeinen
2014-03-08 11:41                   ` Grant Likely
2014-03-08 23:25                     ` Sylwester Nawrocki
2014-03-10 10:18                     ` Tomi Valkeinen
2014-03-10 13:52                       ` Laurent Pinchart
2014-03-10 14:10                         ` Tomi Valkeinen
2014-03-10 14:58                         ` Grant Likely
2014-03-10 15:15                           ` Laurent Pinchart
2014-03-10 15:40                             ` Philipp Zabel
2014-03-11 11:43                               ` Laurent Pinchart
2014-03-11 12:59                                 ` Tomi Valkeinen
     [not found]                                   ` <1883687.VdfitvQEN3@samsung.com>
     [not found]                                     ` <avalon@samsung.com>
2014-03-11 13:16                                   ` Laurent Pinchart
2014-03-11 13:27                                     ` Tomi Valkeinen
2014-03-11 13:44                                       ` Philipp Zabel
2014-03-20 17:23                                     ` Grant Likely
2014-03-21 10:44                                       ` Andrzej Hajda
2014-03-21 11:47                                         ` Grant Likely
2014-03-21 12:16                                           ` Tomi Valkeinen
2014-03-21 14:33                                             ` Grant Likely
2014-03-20 17:09                                 ` Grant Likely
2014-03-11 13:04                             ` Andrzej Hajda
2014-03-20 17:33                               ` Grant Likely
2014-03-20 17:01                             ` Grant Likely
2014-03-21 12:10                               ` Tomi Valkeinen
2014-03-11 15:07                           ` Philipp Zabel
2014-03-11 15:21                             ` Laurent Pinchart
2014-03-11 18:52                               ` Philipp Zabel
2014-03-20 17:53                               ` Grant Likely
2014-03-20 17:36                             ` Grant Likely
2014-03-12 10:25                         ` Russell King - ARM Linux
2014-03-12 10:47                           ` Tomi Valkeinen
2014-03-12 11:21                             ` Laurent Pinchart
2014-03-20 17:54                           ` Grant Likely
2014-03-20 18:16                             ` Laurent Pinchart
2014-03-20 18:18                               ` Russell King - ARM Linux
2014-03-20 18:50                                 ` Laurent Pinchart
2014-03-20 19:27                                 ` Laurent Pinchart
2014-03-20 18:38                             ` Mauro Carvalho Chehab
2014-03-20 18:43                               ` Russell King - ARM Linux
2014-03-20 18:49                                 ` Laurent Pinchart
2014-03-21  8:20                                   ` Tomi Valkeinen
2014-03-20 18:48                               ` Grant Likely
2014-03-20 18:52                                 ` Laurent Pinchart
2014-03-20 23:12                                   ` Grant Likely
2014-03-20 23:26                                     ` Laurent Pinchart
2014-03-21  8:15                                       ` Grant Likely
2014-03-21 12:44                                         ` Laurent Pinchart
2014-03-21 13:26                                           ` Grant Likely
2014-03-10 14:41                       ` Grant Likely
2014-03-08 11:54                 ` Russell King - ARM Linux
2014-02-18 10:43     ` Sylwester Nawrocki
2014-02-18 13:41     ` Philipp Zabel

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=20140211145248.GI26684@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=devicetree@vger.kernel.org \
    --cc=grant.likely@linaro.org \
    --cc=kyungmin.park@samsung.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=m.chehab@samsung.com \
    --cc=p.zabel@pengutronix.de \
    --cc=philipp.zabel@gmail.com \
    --cc=robh+dt@kernel.org \
    --cc=robherring2@gmail.com \
    --cc=s.nawrocki@samsung.com \
    --cc=tomi.valkeinen@ti.com \
    /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