On Mon, Feb 13, 2023 at 12:23:04PM +0100, Krzysztof Kozlowski wrote: > On 13/02/2023 12:20, Jianhua Lu wrote: > > On Mon, Feb 13, 2023 at 12:08:27PM +0100, Krzysztof Kozlowski wrote: > >> On 13/02/2023 12:04, Jianhua Lu wrote: > >>> On Mon, Feb 13, 2023 at 11:45:23AM +0100, Krzysztof Kozlowski wrote: > >>>> On 10/02/2023 17:19, Jianhua Lu wrote: > >>>>> Novatek NT36523 is a display driver IC that can drive DSI panel. It > >>>>> is also present in the csot and boe video mode panels. It is found > >>>>> in Xiaomi Mi Pad 5 series. > >>>>> > >>>>> Signed-off-by: Jianhua Lu <lujianhua000@xxxxxxxxx> > >>>>> --- > >>>>> .../display/panel/novatek,nt36523.yaml | 101 ++++++++++++++++++ > >>>>> 1 file changed, 101 insertions(+) > >>>>> create mode 100644 Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml > >>>>> > >>>>> diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml > >>>>> new file mode 100644 > >>>>> index 000000000000..a7098d0ce7e1 > >>>>> --- /dev/null > >>>>> +++ b/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml > >>>>> @@ -0,0 +1,101 @@ > >>>>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > >>>>> +%YAML 1.2 > >>>>> +--- > >>>>> +$id: http://devicetree.org/schemas/display/panel/novatek,nt36523.yaml# > >>>>> +$schema: http://devicetree.org/meta-schemas/core.yaml# > >>>>> + > >>>>> +title: Novatek NT36523 based DSI display Panels > >>>>> + > >>>>> +maintainers: > >>>>> + - Jianhua Lu <lujianhua000@xxxxxxxxx> > >>>>> + > >>>>> +description: | > >>>>> + The nt36523 IC from Novatek is a generic DSI Panel IC used to drive dsi > >>>>> + panels. > >>>>> + Right now, only support cost and boe LCD display panels with the > >>>> > >>>> boe? both? > >>> Boe Technology Group Co., Ltd > >> > >> Then what is "cost"? If both are names, then they start with capital > >> letters or how the company officially spells itself. > > both are company name, I will search offical name for it. > > "cost" is company name? Then why do you call binding with different one? "cost" is a typo error, real name is "csot". > > >> > >>>> > >>>>> + resolution of 1600x2560. It is a video mode DSI panel. > >>>> > >>>> The binding or hardware supports only 1600x2560? The how it can be > >>>> "right now"? It's defined, isn't it? > >>> Yes > >> > >> What yes? How it can be "right now"? It implies it will change, so how > >> hardware can change? > > sorry, my wrong. This binding only support 1600x2560. > > How binding can support only 1600x2560? How does it exactly work for > binding? the driver of this binding support 1600x2560 and panels are 1600x2560. > > > Best regards, > Krzysztof >
