On 28.09.2023 19:09, Conor Dooley wrote: > On Wed, Sep 27, 2023 at 03:19:01PM +0200, Konrad Dybcio wrote: >> Raydium RM692E5 is a display driver IC used to drive AMOLED DSI panels. >> Describe it. >> >> Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx> >> --- >> .../bindings/display/panel/raydium,rm692e5.yaml | 73 ++++++++++++++++++++++ >> 1 file changed, 73 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/display/panel/raydium,rm692e5.yaml b/Documentation/devicetree/bindings/display/panel/raydium,rm692e5.yaml >> new file mode 100644 >> index 000000000000..423a85616c1c >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/display/panel/raydium,rm692e5.yaml >> @@ -0,0 +1,73 @@ >> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/display/panel/raydium,rm692e5.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Raydium RM692E5 based DSI display Panels > > The capitalisation here drives me crazy, but I can live with it... Oh that's what I get for copypasting a different yaml as template :D willfix > >> + >> +maintainers: >> + - Konrad Dybcio <konradybcio@xxxxxxxxxx> > > In case it's a typo, you sharing the d between first and surnames in > your email addr? Intentional design decision! > >> + >> +description: | > > This | should not be needed. Ack > > Otherwise, this seems fine to me. > Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> Thanks! Konrad