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@xxxxxxxxxxxx>
---
.../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@xxxxxxxxxx>
+ - Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
+
+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
