[RFC PATCH v3 2/9] staging: imx-drm: Add temporary copies of v4l2-of parsing functions
Philipp Zabel
p.zabel at pengutronix.de
Mon Feb 24 12:31:30 EST 2014
Am Montag, den 24.02.2014, 15:52 +0000 schrieb Russell King - ARM Linux:
> On Tue, Feb 18, 2014 at 12:36:03PM +0100, Philipp Zabel wrote:
> > From: Philipp Zabel <philipp.zabel at gmail.com>
> >
> > The existing v4l2-of parser functions for the video interface bindings
> > described in Documentation/device-tree/bindings/media/video-interfaces.txt
> > are useful for DRM drivers, too. They will be moved to drivers/media
> > so they can be used by drm drivers, too. Until then, duplicate the
> > v4l2-of parser functions temporarily.
> >
> > Signed-off-by: Philipp Zabel <philipp.zabel at gmail.com>
>
> Ergh. So, because we can't get agreement on where to put the common
> helpers, we have to put up with adding duplicate helpers into the
> staging directory, inflating not only the kernel source code size
> but also the binary size as well.
This part is only for the convenience of prospective reviewers/testers.
I hope I don't have exhausted your patience at this patch, because the
real changes start at 3/9.
This is one reason why I still didn't drop the RFC. Basically, with so
many moving parts, I didn't feel like telling people to fetch three
different series just to make sense of this one.
> Come on people, get agreement on how to deal with this. Staging may
> be a dumping ground for drivers which aren't ready to be merged as
> proper drivers, but this is no reason to ignore proper process.
>
> Do we, or do we not want to get imx-drm out of drivers/staging? If
> we do, the only way that's going to happen is if we stop throwing in
> this kind of stuff.
regards
Philipp
More information about the linux-arm-kernel
mailing list