Skip to content

arch: sc5xx: skip CDU0 CLKINSEL write on SC846 - #175

Open
swinchen wants to merge 1 commit into
adi-u-boot-2025.10.yfrom
adsp-sc846-no-cgu1-clkinsel
Open

swinchen wants to merge 1 commit into
adi-u-boot-2025.10.yfrom
adsp-sc846-no-cgu1-clkinsel

Conversation

@swinchen

Copy link
Copy Markdown

cdu_init() unconditionally writes CONFIG_CDU0_CGU1_CLKIN to REG_CDU0_CLKINSEL (0x3108F044) to select the CGU1 input clock.

The CLKINSEL register is optional in the CDU block and is not implemented on ADSP-SC846.

Skip the write on SC846 so the SPL does not access an unimplemented register during early clock initialisation.

Fixes: 3285a56 ("arch: sc5xx: Add SC846 DDR init hooks")

@swinchen
swinchen requested review from a team, qasim-ijaz and sipraga October 19, 2026 21:01
@swinchen
swinchen force-pushed the adsp-sc846-no-cgu1-clkinsel branch from a9afa44 to 21389b1 Compare October 19, 2026 21:03
@qasim-ijaz qasim-ijaz added this to ADSP Oct 19, 2026
Comment thread arch/arm/mach-sc5xx/init/clkinit.c Outdated
Comment thread arch/arm/mach-sc5xx/init/clkinit.c Outdated
@sipraga

sipraga commented Oct 20, 2026 •

Copy link
Copy Markdown

Oh, just noticed this from @nunojsa in his SMP PR:

bfe5f08

I think it's in conflict with this change.

cdu_init() unconditionally writes CONFIG_CDU0_CGU1_CLKIN to
REG_CDU0_CLKINSEL (0x3108F044) to select the CGU1 input clock.

The CLKINSEL register is optional in the CDU block and is not
implemented on ADSP-SC846.

Skip the write on SC846 so the SPL does not access an unimplemented
register during early clock initialisation.

Fixes: 3285a56 ("arch: sc5xx: Add SC846 DDR init hooks")
Signed-off-by: Sam Winchenbach <sam.winchenbach@analog.com>
@swinchen
swinchen force-pushed the adsp-sc846-no-cgu1-clkinsel branch from 21389b1 to b108f75 Compare October 20, 2026 14:16
@swinchen
swinchen marked this pull request as ready for review October 20, 2026 14:20
@swinchen swinchen self-assigned this Oct 20, 2026
@swinchen
swinchen requested a review from qasim-ijaz October 20, 2026 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants