linux-kernel - Re: [PATCH v3 4/6] dt-bindings: display: mediatek: disp-tdshp: Add support for MT8196

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <e2511655-1907-42db-bed5-8943a8c835c2@kernel.org>
Date: Tue, 7 Oct 2025 15:21:53 +0900
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Jay Liu (刘博) <Jay.Liu@...iatek.com>
Cc: "linux-mediatek@...ts.infradead.org"
 <linux-mediatek@...ts.infradead.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "chunkuang.hu@...nel.org" <chunkuang.hu@...nel.org>,
 "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
 "tzimmermann@...e.de" <tzimmermann@...e.de>,
 "simona@...ll.ch" <simona@...ll.ch>, "mripard@...nel.org"
 <mripard@...nel.org>, "p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
 "maarten.lankhorst@...ux.intel.com" <maarten.lankhorst@...ux.intel.com>,
 "conor+dt@...nel.org" <conor+dt@...nel.org>,
 "robh@...nel.org" <robh@...nel.org>,
 "dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
 "airlied@...il.com" <airlied@...il.com>,
 "linux-arm-kernel@...ts.infradead.org"
 <linux-arm-kernel@...ts.infradead.org>,
 "matthias.bgg@...il.com" <matthias.bgg@...il.com>,
 "krzk+dt@...nel.org" <krzk+dt@...nel.org>,
 AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
Subject: Re: [PATCH v3 4/6] dt-bindings: display: mediatek: disp-tdshp: Add
 support for MT8196

On 21/09/2025 13:39, Jay Liu (刘博) wrote:
>>> +
>>> +maintainers:
>>> +  - Chun-Kuang Hu <chunkuang.hu@...nel.org>
>>> +  - Philipp Zabel <p.zabel@...gutronix.de>
>>> +
>>> +description: |
>>> +  MediaTek display 2D sharpness processor, namely TDSHP, provides
>>> a
>>> +  operation used to adjust sharpness in display system.
>>> +  TDSHP device node must be siblings to the central MMSYS_CONFIG
>>> node.
>>
>> Heh? Why would this have to be a sibling? This is really odd, because
>> you cannto actually rely on that.
>>
> hi Krzysztof,
> disp-tdshp is one of components of display pipeline. just like OVL,
> color, etc. MMSYS_CONFIG controls the clock of display pipeline,


Read feedback again.

Unfortunately Mediatek produces way too many poor quality patches and
review has so many obstacles like not addressing the comments. Example
above. You need to pay more attention and invest more time, before
pinging or replying to maintainers.

> 
>> You just added unverifiable unusual ABI with no explanation.

Best regards,
Krzysztof