Skip to content

[ADI] ARM: dts: adi: Add support for SC598-HTOL board#3239

Open
ozan956 wants to merge 2 commits intoadsp-6.12.0-yfrom
add-htol-support
Open

[ADI] ARM: dts: adi: Add support for SC598-HTOL board#3239
ozan956 wants to merge 2 commits intoadsp-6.12.0-yfrom
add-htol-support

Conversation

@ozan956
Copy link
Copy Markdown

@ozan956 ozan956 commented Apr 8, 2026

Add device tree and defconfig support for the ADI
SC598 HTOL SBC based on the reference board designs.

PR Description

  • Please replace this comment with a summary of your changes, and add any context
    necessary to understand them. List any dependencies required for this change.
  • To check the checkboxes below, insert a 'x' between square brackets (without
    any space), or simply check them after publishing the PR.
  • If you changes include a breaking change, please specify dependent PRs in the
    description and try to push all related PRs simultaneously.

PR Type

  • Bug fix (a change that fixes an issue)
  • New feature (a change that adds new functionality)
  • Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

  • I have conducted a self-review of my own code changes
  • I have compiled my changes, including the documentation
  • I have tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly
  • I have provided links for the relevant upstream lore

@ozan956 ozan956 requested review from a team and pamolloy April 8, 2026 12:05
@ozan956 ozan956 self-assigned this Apr 8, 2026
@CalebEthridgeADI
Copy link
Copy Markdown

Are you planning to add the defconfig for the HTOL board in this PR as well?

@ozan956
Copy link
Copy Markdown
Author

ozan956 commented Apr 8, 2026

Are you planning to add the defconfig for the HTOL board in this PR as well?

I believe there is no need for a separate config. We already have the device tree, and its enough to describe the system. Only possible disadvantage is that some unused drivers may still be compiled, which could slightly increase the size. However, since they are not referenced in the device tree, they will not cause any issues in the end.

@CalebEthridgeADI
Copy link
Copy Markdown

Okay, I would check the ARCH_SC5XX and ARCH_SC59X_64 symbols just to make sure there isn't anything there that we need, but otherwise LGTM.

@pamolloy
Copy link
Copy Markdown
Collaborator

pamolloy commented Apr 8, 2026

Are you planning to add the defconfig for the HTOL board in this PR as well?

I believe there is no need for a separate config. We already have the device tree, and its enough to describe the system. Only possible disadvantage is that some unused drivers may still be compiled, which could slightly increase the size. However, since they are not referenced in the device tree, they will not cause any issues in the end.

👍 one of my many things we will clean-up in the long run is to reduce the number of defconfigs in Linux (and U-Boot) to one for each architecture

@pamolloy pamolloy requested a review from a team April 8, 2026 13:40
@pamolloy pamolloy added this to ADSP Apr 8, 2026
@ozan956
Copy link
Copy Markdown
Author

ozan956 commented Apr 8, 2026

Okay, I would check the ARCH_SC5XX and ARCH_SC59X_64 symbols just to make sure there isn't anything there that we need, but otherwise LGTM.

Thank you! 😄

CONFIG_ARCH_SC59X_64 is already enabled in both ezkit and ezlite defconfigs, which covers all the necessary platform drivers. The architecture selections are essentially the same across these SC598 boards, so HTOL has everything it needs.

Ideally, in the future, we could unify these into a single architecture level defconfig since the core platform requirements are identical across all SC598 variants. The device tree already handles the board specific differences, so having separate defconfigs is somewhat redundant.

#size-cells = <0>;

adin1300: ethernet-phy@0 {
compatible = "ethernet-phy-ieee802.3-c22";
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Very interesting to see how this goes since it is currently misconfigured for at least SC598 (see #2883)

Add device tree support for SC598 HTOL SBC based on the
reference board designs.

Signed-off-by: Ozan Durgut <ozan.durgut@analog.com>
@ozan956 ozan956 force-pushed the add-htol-support branch from 984f676 to 7ac1edd Compare April 9, 2026 08:36
Signed-off-by: Ozan Durgut <ozan.durgut@analog.com>
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