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

2014-02-24 Thread Russell King - ARM Linux
On Tue, Feb 18, 2014 at 12:36:03PM +0100, Philipp Zabel wrote:
 From: Philipp Zabel philipp.za...@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.za...@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.

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.

-- 
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


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

2014-02-24 Thread Philipp Zabel
Am Montag, den 24.02.2014, 15:52 + schrieb Russell King - ARM Linux:
 On Tue, Feb 18, 2014 at 12:36:03PM +0100, Philipp Zabel wrote:
  From: Philipp Zabel philipp.za...@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.za...@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

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel