Re: [RFR 2/2] drm/panel: Add simple panel support

Stephen Warren
2013-10-20T22:07:36Z

On 10/17/2013 12:07 PM, Laurent Pinchart wrote:
...
>> As I said, anything that really needs a CDF binding to work
>> likely isn't "simple" anymore, therefore a separate driver can
>> easily be justified.
>
> The system as a whole would be more complex, but the panel could be
> the same. We can't have two drivers for the same piece of hardware
> in the DT world, as there will be a single compatible string and no
> way to choose between the drivers (unlike the board code world that
> could set device names to "foo- encoder-v4l2" or "foo-encoder-drm"
> and live happily with that ever after).

That's not true. We can certainly define two different compatible
values for a piece of HW if we have to. We can easily control whether
they are handled by the same or different drivers in the OS.

Now, we should try to avoid this, because then that means that the
original binding wasn't fully describing the HW. However, at least in
the case of these simple LCD panels, it's hard to see that there is
anything more than what's already in Thierry's binding. Remember, the
binding is a description of the HW, not any Linux-internal details of
how e.g. a CDF or DRM subsystem is supposed to use the HW; that had
better be embodied into the driver or subsystem code, which aren't
ABIs, and hence can be easily modified/enhanced.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html