Skip to content

kboot_atc: Add t8122 tunables#555

Merged
svenpeter42 merged 1 commit intoAsahiLinux:mainfrom
noopwafel:t8122-atc-tunables
Apr 16, 2026
Merged

kboot_atc: Add t8122 tunables#555
svenpeter42 merged 1 commit intoAsahiLinux:mainfrom
noopwafel:t8122-atc-tunables

Conversation

@noopwafel
Copy link
Copy Markdown
Contributor

Add support for t8122 USB tunables; a few are commented out since they're not used on my j613 (w/14.8.3) and so I don't know the offsets.

Signed-off-by: Alyssa Milburn <[email protected]>
@noopwafel noopwafel force-pushed the t8122-atc-tunables branch from 2f1c08c to 1403cad Compare March 31, 2026 09:43
Comment thread src/kboot_atc.c
ret = dt_append_atc_tunable(dt, adt_node, fdt_node, &atc_tunables[i]);
if (adt_is_compatible_at(adt, adt_node, "atc-phy,t8122", 0)) {
tunables = &atc_tunables_t8122[0];
tunable_count = sizeof(atc_tunables_t8122) / sizeof(*atc_tunables_t8122);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: this could use ARRAY_SIZE macro but not really important as we don't strictly use it

Comment thread src/kboot_atc.c

for (size_t i = 0; i < sizeof(atc_tunables) / sizeof(*atc_tunables); ++i) {
ret = dt_append_atc_tunable(dt, adt_node, fdt_node, &atc_tunables[i]);
if (adt_is_compatible_at(adt, adt_node, "atc-phy,t8122", 0)) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we want to copy the approach for atc_fuses but I guess we can still refactor once we add support for t603x or t814x. One possible issue is that the DT compaitbles probably won't allow this.

@svenpeter42 svenpeter42 merged commit bcee7f2 into AsahiLinux:main Apr 16, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants