[PATCH v4 0/2] Add SMI reset and clamp for MediaTek MT8188 SoC

Friday Yang posted 2 patches 9 months, 4 weeks ago
There is a newer version of this series
.../mediatek,smi-common.yaml                  |   2 +
.../memory-controllers/mediatek,smi-larb.yaml |  20 +++
drivers/memory/mtk-smi.c                      | 141 +++++++++++++++++-
3 files changed, 159 insertions(+), 4 deletions(-)
[PATCH v4 0/2] Add SMI reset and clamp for MediaTek MT8188 SoC
Posted by Friday Yang 9 months, 4 weeks ago
Based on tag: next-20250220, linux-next/master

On the MediaTek MT8188 SoC platform, we encountered power-off failures
and SMI bus hang issues during camera stress tests. The issue arises
because bus glitches are sometimes produced when MTCMOS powers on or
off. While this is fairly normal, the software must handle these
glitches to avoid mistaking them for transaction signals. What's
more, this issue emerged only after the initial upstreaming of SMI
driver.

The software solutions can be summarized as follows:

1. Use CLAMP to disable the SMI sub-common port after turning off the
   LARB CG and before turning off the LARB MTCMOS.
2. Use CLAMP to disable/enable the SMI sub-common port.
3. Implement an AXI reset for SMI LARBs.

Changes v4:
- Use 'devm_reset_control_get_optional_exclusive' instead of
  'devm_reset_control_get'.

v3:
https://lore.kernel.org/lkml/20250121064934.13482-2-
friday.yang@mediatek.com/
https://lore.kernel.org/lkml/20250121064934.13482-3-
friday.yang@mediatek.com/

friday.yang (2):
  dt-bindings: memory: mediatek: Add SMI reset and clamp for MT8188
  memory: mtk-smi: mt8188: Add SMI reset and clamp for MT8188

 .../mediatek,smi-common.yaml                  |   2 +
 .../memory-controllers/mediatek,smi-larb.yaml |  20 +++
 drivers/memory/mtk-smi.c                      | 141 +++++++++++++++++-
 3 files changed, 159 insertions(+), 4 deletions(-)

--
2.46.0
Re: [PATCH v4 0/2] Add SMI reset and clamp for MediaTek MT8188 SoC
Posted by Krzysztof Kozlowski 9 months, 3 weeks ago
On Fri, Feb 21, 2025 at 03:48:30PM +0800, Friday Yang wrote:
> Based on tag: next-20250220, linux-next/master
> 
> On the MediaTek MT8188 SoC platform, we encountered power-off failures
> and SMI bus hang issues during camera stress tests. The issue arises
> because bus glitches are sometimes produced when MTCMOS powers on or
> off. While this is fairly normal, the software must handle these
> glitches to avoid mistaking them for transaction signals. What's
> more, this issue emerged only after the initial upstreaming of SMI
> driver.
> 
> The software solutions can be summarized as follows:
> 
> 1. Use CLAMP to disable the SMI sub-common port after turning off the
>    LARB CG and before turning off the LARB MTCMOS.
> 2. Use CLAMP to disable/enable the SMI sub-common port.
> 3. Implement an AXI reset for SMI LARBs.
> 
> Changes v4:
> - Use 'devm_reset_control_get_optional_exclusive' instead of
>   'devm_reset_control_get'.
> 
> v3:
> https://lore.kernel.org/lkml/20250121064934.13482-2-

That's not a valid link. Fix your email client.

Best regards,
Krzysztof
Re: [PATCH v4 0/2] Add SMI reset and clamp for MediaTek MT8188 SoC
Posted by Friday Yang (杨阳) 9 months, 3 weeks ago
On Sat, 2025-02-22 at 10:45 +0100, Krzysztof Kozlowski wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> On Fri, Feb 21, 2025 at 03:48:30PM +0800, Friday Yang wrote:
> > Based on tag: next-20250220, linux-next/master
> > 
> > On the MediaTek MT8188 SoC platform, we encountered power-off
> > failures
> > and SMI bus hang issues during camera stress tests. The issue
> > arises
> > because bus glitches are sometimes produced when MTCMOS powers on
> > or
> > off. While this is fairly normal, the software must handle these
> > glitches to avoid mistaking them for transaction signals. What's
> > more, this issue emerged only after the initial upstreaming of SMI
> > driver.
> > 
> > The software solutions can be summarized as follows:
> > 
> > 1. Use CLAMP to disable the SMI sub-common port after turning off
> > the
> >    LARB CG and before turning off the LARB MTCMOS.
> > 2. Use CLAMP to disable/enable the SMI sub-common port.
> > 3. Implement an AXI reset for SMI LARBs.
> > 
> > Changes v4:
> > - Use 'devm_reset_control_get_optional_exclusive' instead of
> >   'devm_reset_control_get'.
> > 
> > v3:
> > https://lore.kernel.org/lkml/20250121064934.13482-2-
> 
> That's not a valid link. Fix your email client.
> 

Sorry for the inconvenience. The Web link is quite long, so I have
split it into two lines. Please copy the entire link and paste it into
the browser, it sholuld work. 


> Best regards,
> Krzysztof
>