Change the soc from ace30_ptl to ace30

[gbernatxintel]

Renamed soc from ace30_ptl to ace30.
We were previously using the wrong soc name.
The correct name is ace30.

There is only one ptl platform, but there can be several ace30 platforms.

Check the commit on the sof project repo:
Change the soc from ace30_ptl to ace30

Dependencies:
thesofproject/sof#9475

[github-actions]

Hello @gbernatxintel, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

[nordicjm]

these should match SOC_* should be the value of * in the string

[gbernatxintel]

In my opinion it is ok, if you see other platforms the names are entered without soc_* just simply string. For other manufacturers I randomly checked and also they don't all use soc_* names but just the name of the specific chip

[nordicjm]

I mean this should either be default "intel_ace30" if SOC_INTEL_ACE30 or default "ace30" if SOC_ACE30 if not then this will fail at some point in the future when the build system generates these symbols automatically from the soc.yml files

[gbernatxintel]

Oki, I changed to "ace30" if SOC_ACE30

[dcpleung]

Also, could you separate the whitespace changes into another commit? A simple rename without any changes should show up in git as a simple rename, and makes it easier to review the diff.

[gbernatxintel]

Also, could you separate the whitespace changes into another commit? A simple rename without any changes should show up in git as a simple rename, and makes it easier to review the diff.

In fact, it's hard to carve out a separate commit, because the whole commit is a name change from ace30_ptl to ace30. The general problem was that the soc was misspelled. If there is for example another board based on the same soc then there will be confusion with the definitions.
There are no functional changes in this commit

[dcpleung]

Also, could you separate the whitespace changes into another commit? A simple rename without any changes should show up in git as a simple rename, and makes it easier to review the diff.

In fact, it's hard to carve out a separate commit, because the whole commit is a name change from ace30_ptl to ace30. The general problem was that the soc was misspelled. If there is for example another board based on the same soc then there will be confusion with the definitions. There are no functional changes in this commit

When you look at the diff, there are bunch of header files with only whitespace changes and nothing else. There are not ace30_ptl to ace30 changes in there. All I ask is to separate those simple whitespace changes into another commit so that the renaming would be more simple and direct in git log.

[dcpleung]

Also, keep the toolchain name as intel_ace30_ptl as it is related to the Zephyr SDK, and the toolchain there is named with _ptl. Changing this would mean Zephyr SDK can no longer build this platform.

[github-actions]

Hi @gbernatxintel!
Congratulations on getting your very first Zephyr pull request merged 🎉🥳. This is a fantastic achievement, and we're thrilled to have you as part of our community!

To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge.

Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁

[nordicjm]

This has broken CI https://github.com/zephyrproject-rtos/zephyr/actions/runs/11009734491/job/30570116372 please submit a fix

[aescolar]

This has broken CI https://github.com/zephyrproject-rtos/zephyr/actions/runs/11009734491/job/30570116372 please submit a fix

Thanks @nordicjm, revert queued in #78930

[gbernatxintel]

Thank you @nordicjm ,I'm checking what's happening in this CI test