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? >> >>>> >>>>> + 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? Best regards, Krzysztof