[PATCH RFC 26/46] drivers/base: provide an infrastructure for componentised subsystems

Guennadi Liakhovetski
2014-02-26T21:00:25Z

Hi Russell

(I suspect this my email will be rejected by ALKML too like other my
recent emails, but at least other MLs will pick it up and individual CCs
too, so, if replying, maybe it would be good to keep my entire reply, all
the more that it's going to be very short)

On Thu, 2 Jan 2014, Russell King wrote:

> Subsystems such as ALSA, DRM and others require a single card-level
> device structure to represent a subsystem. However, firmware tends to
> describe the individual devices and the connections between them.
>
> Therefore, we need a way to gather up the individual component devices
> together, and indicate when we have all the component devices.
>
> We do this in DT by providing a "superdevice" node which specifies
> the components, eg:
>
> imx-drm {
> compatible = "fsl,drm";
> crtcs = <&ipu1>;
> connectors = <&hdmi>;
> };

It is a pity linux-media wasn't CC'ed and apparently V4L developers didn't
notice this and other related patches in a "clean up" series, and now this
patch is already in the mainline. But at least I'd like to ask whether the
bindings, defined in
Documentation/devicetree/bindings/media/video-interfaces.txt and
implemented in drivers/media/v4l2-core/v4l2-of.c have been considered for
this job, and - if so - why have they been found unsuitable? Wouldn't it
have been better to use and - if needed - extend them to cover any
deficiencies? Even though the implementation is currently located under
drivers/media/v4l2-code/ it's pretty generic and should be easily
transferable to a more generic location.

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer