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