Skip to content

arch: imx9: allow last CCM root index#19336

Merged
xiaoxiang781216 merged 1 commit into
apache:masterfrom
Old-Ding:codex/imx9-ccm-root-last-index
Jul 6, 2026
Merged

arch: imx9: allow last CCM root index#19336
xiaoxiang781216 merged 1 commit into
apache:masterfrom
Old-Ding:codex/imx9-ccm-root-last-index

Conversation

@Old-Ding

@Old-Ding Old-Ding commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • allow i.MX9 non-SCMI CCM helpers to accept the last root index
  • keep the fix in the register-level CCM boundary checks for arm and arm64

Root cause

CCM_CR_COUNT is defined as 94 in the non-SCMI CCM headers, and CCM_CR_PALCAMESCAN is also 94. The g_ccm_root_mux tables contain 95 entries, so root 94 is a valid table/register index. The root >= CCM_CR_COUNT checks rejected that last valid root before reaching the CCM helpers.

Testing

  • git diff --check HEAD~1 HEAD
  • git show --stat --check --format=fuller HEAD
  • PowerShell consistency check: arm imx93, arm64 imx93, and arm64 imx95 CCM tables all report CCM_CR_COUNT=94, CCM_CR_PALCAMESCAN=94, MuxEntries=95

Not run locally:

  • tools/checkpatch.sh -f arch/arm/src/imx9/imx9_ccm.c arch/arm64/src/imx9/imx9_ccm.c (local bash opens the Windows WSL install prompt)
  • build tests (local make, gcc, arm-none-eabi-gcc, and clang are not installed)

CCM_CR_COUNT is defined as the last CCM root index in the non-SCMI tables, and CCM_CR_PALCAMESCAN also uses that value. Use a strict greater-than check so root 94 reaches the CCM register helpers.

Generated-by: OpenAI Codex
Signed-off-by: Old-Ding <35417409+Old-Ding@users.noreply.github.com>
@Old-Ding Old-Ding requested a review from pussuw as a code owner July 6, 2026 03:18
@github-actions github-actions Bot added Arch: arm Issues related to ARM (32-bit) architecture Arch: arm64 Issues related to ARM64 (64-bit) architecture Size: XS The size of the change in this PR is very small labels Jul 6, 2026
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

@xiaoxiang781216 xiaoxiang781216 merged commit 0841476 into apache:master Jul 6, 2026
53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Arch: arm Issues related to ARM (32-bit) architecture Arch: arm64 Issues related to ARM64 (64-bit) architecture Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants