[RFC PATCH v3 2/9] staging: imx-drm: Add temporary copies of v4l2-of parsing functions (original) (raw)

Philipp Zabel p.zabel at pengutronix.de
Mon Feb 24 09:31:30 PST 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 dri-devel mailing list