[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CACRpkdanNPaBOAo80wSvNU=ynkS5AVc6gV02jTfq-AePye+fGg@mail.gmail.com>
Date: Mon, 26 Jul 2021 09:14:57 +0200
From: Linus Walleij <linus.walleij@...aro.org>
To: Iskren Chernev <iskren.chernev@...il.com>
Cc: Rob Herring <robh+dt@...nel.org>, Sam Ravnborg <sam@...nborg.org>,
Thierry Reding <thierry.reding@...il.com>,
Daniel Vetter <daniel@...ll.ch>,
David Airlie <airlied@...ux.ie>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@...r.kernel.org>,
"open list:DRM PANEL DRIVERS" <dri-devel@...ts.freedesktop.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
phone-devel@...r.kernel.org,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
<devicetree@...r.kernel.org>, Hans de Goede <hdegoede@...hat.com>, Andy
Shevchenko <andy.shevchenko@...il.com>,"
<~postmarketos/upstreaming@...ts.sr.ht>
Subject: Re: [PATCH v3 1/2] dt-bindings: panel: Add Samsung S6E3FA2 panel
Hi Iskren,
thanks for your patch!
On Mon, Feb 1, 2021 at 5:56 PM Iskren Chernev <iskren.chernev@...il.com> wrote:
> +required:
> + - compatible
> + - reset-gpios
> + - iovdd-supply
> + - vddr-supply
> + - port
I do not think port should be required because the DSI
framework allows panels to be put as children right inside
the DSI controller node without any port linking the two.
Other than that it looks good!
Yours,
Linus Walleij
Powered by blists - more mailing lists