| Message ID | 20250219092040.11227-8-jay.liu@mediatek.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | porting pq compnent for MT8196 | expand |
On 19/02/2025 10:20, Jay Liu wrote: > Add a compatible string for MediaTek MT8196 SoC No, this is just bogus commit msg. You did not try enough, just pasted same useless and incorrect message to every patch. > > Signed-off-by: Jay Liu <jay.liu@mediatek.com> > --- > .../display/mediatek/mediatek,tdshp.yaml | 51 +++++++++++++++++++ > 1 file changed, 51 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml > > diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml > new file mode 100644 > index 000000000000..5ed7bdd53d0e > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml Filename matching exactly compatible. > @@ -0,0 +1,51 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/display/mediatek/mediatek,tdshp.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: MediaTek display clarity correction > + > +maintainers: > + - Chun-Kuang Hu <chunkuang.hu@kernel.org> > + - Philipp Zabel <p.zabel@pengutronix.de> > + > +description: | Do not need '|' unless you need to preserve formatting. > + MediaTek display clarity correction, namely TDSHP, provides a > + operation used to adjust sharpness in display system. > + TDSHP device node must be siblings to the central MMSYS_CONFIG node. > + For a description of the MMSYS_CONFIG binding, see > + Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml > + for details. Missing blank line. Do not introduce own style. > +properties: > + compatible: > + oneOf: Drop, unless this is already going to grow? > + - enum: > + - mediatek,mt8196-disp-tdshp > + > + reg: > + maxItems: 1 > + > + clocks: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + - clocks > + > +additionalProperties: false > + > +examples: > + - | > + Drop Best regards, Krzysztof
Il 19/02/25 10:25, Krzysztof Kozlowski ha scritto: > On 19/02/2025 10:20, Jay Liu wrote: >> Add a compatible string for MediaTek MT8196 SoC > > No, this is just bogus commit msg. > > You did not try enough, just pasted same useless and incorrect message > to every patch. > >> >> Signed-off-by: Jay Liu <jay.liu@mediatek.com> >> --- >> .../display/mediatek/mediatek,tdshp.yaml | 51 +++++++++++++++++++ >> 1 file changed, 51 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml >> >> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml >> new file mode 100644 >> index 000000000000..5ed7bdd53d0e >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml > > Filename matching exactly compatible. > >> @@ -0,0 +1,51 @@ >> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/display/mediatek/mediatek,tdshp.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: MediaTek display clarity correction Adding more comments to Krzysztof's review.... What does "TDSHP" stand for? "Display Clarity Correction" rolls up as "DCC" which is not "TDSHP". Please clarify the title by unrolling "TDSHP" title: MediaTek T.. Display... S... H... P >> + >> +maintainers: >> + - Chun-Kuang Hu <chunkuang.hu@kernel.org> >> + - Philipp Zabel <p.zabel@pengutronix.de> >> + >> +description: | > > Do not need '|' unless you need to preserve formatting. > >> + MediaTek display clarity correction, namely TDSHP, provides a Again, TDSHP does not stand for "display clarity correction" - that's what it is for, and it is ok to say what it is for, but say what TDSHP stands for. MediaTek T... Display Sharpness? (TDSHP) provides means to adjust the image sharpness displayed on a physical screen, therefore this IP is meant to perform display clarity correction. ...rest of the blurb, etc. >> + operation used to adjust sharpness in display system. >> + TDSHP device node must be siblings to the central MMSYS_CONFIG node. >> + For a description of the MMSYS_CONFIG binding, see >> + Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml >> + for details. > > Missing blank line. Do not introduce own style. > >> +properties: >> + compatible: >> + oneOf: > > Drop, unless this is already going to grow? > > krzk: oh, it is, guaranteed!! but ... not exactly right now (not very soon), so dropping the oneOf is a sane recommendation, I agree. Cheers, Angelo >> + - enum: >> + - mediatek,mt8196-disp-tdshp >> + >> + reg: >> + maxItems: 1 >> + >> + clocks: >> + maxItems: 1 >> + >> +required: >> + - compatible >> + - reg >> + - clocks >> + >> +additionalProperties: false >> + >> +examples: >> + - | >> + > > Drop > > > Best regards, > Krzysztof
diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml new file mode 100644 index 000000000000..5ed7bdd53d0e --- /dev/null +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml @@ -0,0 +1,51 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/mediatek/mediatek,tdshp.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: MediaTek display clarity correction + +maintainers: + - Chun-Kuang Hu <chunkuang.hu@kernel.org> + - Philipp Zabel <p.zabel@pengutronix.de> + +description: | + MediaTek display clarity correction, namely TDSHP, provides a + operation used to adjust sharpness in display system. + TDSHP device node must be siblings to the central MMSYS_CONFIG node. + For a description of the MMSYS_CONFIG binding, see + Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml + for details. +properties: + compatible: + oneOf: + - enum: + - mediatek,mt8196-disp-tdshp + + reg: + maxItems: 1 + + clocks: + maxItems: 1 + +required: + - compatible + - reg + - clocks + +additionalProperties: false + +examples: + - | + + soc { + #address-cells = <2>; + #size-cells = <2>; + + tdshp@321e0000 { + compatible = "mediatek,mt8196-disp-tdshp"; + reg = <0 0x321e0000 0 0x1000>; + clocks = <&dispsys_config_clk 107>; + }; + };
Add a compatible string for MediaTek MT8196 SoC Signed-off-by: Jay Liu <jay.liu@mediatek.com> --- .../display/mediatek/mediatek,tdshp.yaml | 51 +++++++++++++++++++ 1 file changed, 51 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/mediatek/mediatek,tdshp.yaml