Re: [PATCH v2 3/4] dt-bindings: display: add Amlogic Meson DRM Bindings

From: Laurent Pinchart
Date: Wed Nov 30 2016 - 10:58:52 EST


Hi Neil,

On Wednesday 30 Nov 2016 16:43:44 Neil Armstrong wrote:
> Signed-off-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx>
> ---
> .../bindings/display/meson/meson-drm.txt | 101 ++++++++++++++++++

I forgot to mention that the file should not be named meson-drm.txt as DRM is
a Linux-specific concept. You can name it meson.txt, but a better option would
be amlogic,meson.txt. By the way does it really need a subdirectory ?

> 1 file changed, 101 insertions(+)
> create mode 100644
> Documentation/devicetree/bindings/display/meson/meson-drm.txt

--
Regards,

Laurent Pinchart