Re: [PATCH v8 2/2] memory: mtk-smi: mt8188: Add SMI reset and clamp for MT8188

["Friday Yang (杨阳)" <Friday.Yang@mediatek.com>]

On Thu, 2025-07-24 at 10:55 +0200, Krzysztof Kozlowski wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> On 24/07/2025 10:43, Friday Yang (杨阳) wrote:
> > > 
> > > > ignore this tag. What I intended to explain here was that I
> > > > decided
> > > > not
> > > > to use 'devm_pm_runtime_enable' to replace 'pm_runtime_enable'
> > > > functions. Unfortunately, the fake tag didn't explain this
> > > > clearly
> > > > in
> > > > the changelog, which was my fault. To address this, I updated
> > > > patchset
> > > > v8 to include an explanation.
> > > > 
> > > > In patchset v6, I replaced 'pm_runtime_enable' with
> > > > 'devm_pm_runtime_enable'. However, in patchset v8, I reverted
> > > > this
> > > > change and included the reason for this decision in the
> > > > changelog.
> > > > Apologize for the delay and the trouble again.
> > > 
> > > Nothing above is related to my question about the
> > > fake/invented/questioned tag.
> > > 
> > 
> > I got your point, you refer to the 'reviewed-by' and 'acked-by' tag
> > in
> > this patch.
> > These are the tags from two reviewers.
> > 
> > 
https://lore.kernel.org/lkml/174172361378.44650.15345202042780383326.robh@kernel.org/
> 
> 
> You are really not responding to my initial comments and keep
> dragging
> this discussion in some confused directions. Do we talk here about
> that
> patch? No.
> 

I understand what you're referring to. For the 'dt-binding' patch, I
can keep the tags. However, for the 'smi driver' patch, I should remove
the tags because there was a change after v6.
I will update v9 soon. Thank you!


> Best regards,
> Krzysztof