From mboxrd@z Thu Jan 1 00:00:00 1970 From: narmstrong@baylibre.com (Neil Armstrong) Date: Wed, 30 Nov 2016 17:22:02 +0100 Subject: [PATCH v2 3/4] dt-bindings: display: add Amlogic Meson DRM Bindings In-Reply-To: References: <1480520625-13269-1-git-send-email-narmstrong@baylibre.com> <1480520625-13269-4-git-send-email-narmstrong@baylibre.com> <2734274.BAOZxRirYz@avalon> <2e1b16c6-eec5-8cf3-5795-d6de02f36570@baylibre.com> Message-ID: <6d0450dd-eb7c-a422-ec97-376c1f5fb6c6@baylibre.com> To: linus-amlogic@lists.infradead.org List-Id: linus-amlogic.lists.infradead.org On 11/30/2016 05:21 PM, Kevin Hilman wrote: > Neil Armstrong writes: > >> Hi Laurent, >> On 11/30/2016 04:58 PM, Laurent Pinchart wrote: >>> Hi Neil, >>> >>> On Wednesday 30 Nov 2016 16:43:44 Neil Armstrong wrote: >>>> Signed-off-by: Neil Armstrong >>>> --- >>>> .../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 ? >> >> I took example of the sun4i layout the naming, and no it does not need a subdirector.. >> >> I will move it to amlogic,meson.txt, seems far better. >> > > To me, amlogic,meson is redundant. Probably should be amlogic,vpu.txt? > > Kevin > Yes, seems more coherent. Thanks, Neil