linux-kernel - Re: [PATCH v11 1/2] dt-bindings: memory: mediatek: Add SMI reset and clamp for MT8188

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <1c84112b5a7fa48d8422f531e660aa1b49ec6487.camel@mediatek.com>
Date: Fri, 31 Oct 2025 06:10:18 +0000
From: Friday Yang (杨阳) <Friday.Yang@...iatek.com>
To: "robh@...nel.org" <robh@...nel.org>, "matthias.bgg@...il.com"
	<matthias.bgg@...il.com>, Yong Wu (吴勇)
	<Yong.Wu@...iatek.com>, "p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
	"conor+dt@...nel.org" <conor+dt@...nel.org>, "krzk@...nel.org"
	<krzk@...nel.org>, AngeloGioacchino Del Regno
	<angelogioacchino.delregno@...labora.com>
CC: "linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "linux-mediatek@...ts.infradead.org"
	<linux-mediatek@...ts.infradead.org>, "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>, Project_Global_Chrome_Upstream_Group
	<Project_Global_Chrome_Upstream_Group@...iatek.com>
Subject: Re: [PATCH v11 1/2] dt-bindings: memory: mediatek: Add SMI reset and
 clamp for MT8188

On Sat, 2025-10-18 at 18:42 +0200, Krzysztof Kozlowski wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> On 17/09/2025 14:07, Friday Yang wrote:
> > Add 'resets' and 'reset-names' properties for SMI LARBs to support
> > SMI reset operations.
> 
> Not informative...
> 
> > diff --git a/Documentation/devicetree/bindings/memory-
> > controllers/mediatek,smi-larb.yaml
> > b/Documentation/devicetree/bindings/memory-
> > controllers/mediatek,smi-larb.yaml
> > index 2e7fac4b5094..fc5feb2eac1f 100644
> > --- a/Documentation/devicetree/bindings/memory-
> > controllers/mediatek,smi-larb.yaml
> > +++ b/Documentation/devicetree/bindings/memory-
> > controllers/mediatek,smi-larb.yaml
> > @@ -70,6 +70,12 @@ properties:
> >      description: the hardware id of this larb. It's only required
> > when this
> >        hardware id is not consecutive from its M4U point of view.
> > 
> > +  resets:
> > +    maxItems: 1
> > +
> > +  reset-names:
> > +    const: larb
> 
> Is the reset valid for all existing devices as well? Commit msg does
> not
> explain that... it is pretty useless - you say what you did. We see
> that
> from the diff. Explain something not obvious.
> 

Thanks, I could add more descriptions here, like below:

On the MediaTek MT8188 SoC, bus glitches may occur during MTCMOS
on/off transitions. To prevent these glitches from causing errors,
SMI requires clamp and reset operations. This issue specifically
affects the image, camera, and IPE subsystems. This change adds the
'resets' and 'reset-names' properties to the SMI LARBs within
these subsystems to support the necessary reset operations.
...

Can I still remain the 'Reviewed-by' and 'Acked-by' tags if I only
change the commit mesg in v12? I will not change the bindings.



> 
> 
> Best regards,
> Krzysztof