Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Major bug fixing and improvements to upcoming Zephyr v4.1 #291

Merged
merged 20 commits into from
Feb 6, 2025
Merged

Major bug fixing and improvements to upcoming Zephyr v4.1 #291

merged 20 commits into from
Feb 6, 2025

Conversation

rexut
Copy link
Member

@rexut rexut commented Feb 5, 2025

  • Remove our own Maxim DS3231 RTC and temperature sensor driver.
  • Support new revision C of NXP MIMXRT1060-EVK.
  • Use new unified DTS property names.
  • Remove xtools toolchain variant references.
  • Update all test output messages.
  • Porting to new doxyrunner.

rexut added 4 commits February 6, 2025 00:22
Since Zephyr upstream PR 83492 it is no longer possible to install
additional timeouts. Therefore, the second attempt fails in the test
cases if the watchdog has already been activated during booting.

Signed-off-by: Stephan Linz <[email protected]>
Take over new rules and enhancements comming with upcoming Zephyr v4.1.

Signed-off-by: Stephan Linz <[email protected]>
Some platforms already have the sysbuild flag set and as long as our
tests are not ready or prepared for the new sysbuild feature, these
platforms need to be explicitly filtered out by our quarantine file.

Unfortunately, there is no way to filter out these platforms with any
of the Twister filter options in the testcase.yaml files.

Signed-off-by: Stephan Linz <[email protected]>
This commit removes all references to the `xtools` toolchain variant
in the board YAML files.

Note that the `xtools` toolchain variant has been deprecated since
Zephyr v3.3.0 and now removed.

The removal process was automated using the following command line:

  git grep -l xtools -- boards/*.{yml,yaml} | \
  xargs -n 1 -P $(nproc) \
  yq -i 'del(.toolchain[] | select(. == "xtools"))'

Signed-off-by: Stephan Linz <[email protected]>
Copy link

github-actions bot commented Feb 5, 2025

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

rexut added 15 commits February 6, 2025 07:00
With Zephyr upstream v4.1 the brand new revision C can be used.
With the new board support files the hyperflash variant was changed
to be valid for board revision A to C, but visible for Twister only
in board revision C. Thus the revision tag have to be removed.

Signed-off-by: Stephan Linz <[email protected]>
With Zephyr upstream v4.1 property names in bindings will be unified,
using hyphens (-) instead of underscores (_) as separators.

Signed-off-by: Stephan Linz <[email protected]>
With Zephyr upstream v4.1 the system wide provided macro
IS_BIT_SET() in util_macro.h should be used.

Signed-off-by: Stephan Linz <[email protected]>
This reverts commit f5566fa.
This reverts commit d13069f.
This reverts commit 41725b3.

Signed-off-by: Stephan Linz <[email protected]>
This reverts commit 1302ffe.
This reverts commit 87b26dc.

Signed-off-by: Stephan Linz <[email protected]>
This reverts commit 83406e4.

Signed-off-by: Stephan Linz <[email protected]>
With Zephyr upstream v4.1 the new MFD driver for the Maxim DS3231
with full RTC (get, set and alarm) and digital temperature sensing
can be used.

Signed-off-by: Stephan Linz <[email protected]>
With Zephyr upstream v4.1 the new MFD driver for the Maxim DS3231
with full RTC (get, set and alarm) and digital temperature sensing
can be used.

Signed-off-by: Stephan Linz <[email protected]>
With Zephyr upstream v4.1 the new MFD driver for the Maxim DS3231
with full RTC (get, set and alarm) and digital temperature sensing
can be used.

Signed-off-by: Stephan Linz <[email protected]>
With Zephyr upstream v4.1 the new MFD driver for the Maxim DS3231
with full RTC (get, set and alarm) and digital temperature sensing
can be used.

Signed-off-by: Stephan Linz <[email protected]>
The 'doxyrunner' extension now supports multiple Doxygen projects.

Signed-off-by: Stephan Linz <[email protected]>
The 'doxyrunner' extension now supports multiple Doxygen projects.

Signed-off-by: Stephan Linz <[email protected]>
Update copyright notice to reflect current year, 2025.

Signed-off-by: Stephan Linz <[email protected]>
Update all test command lines and output messages for upcoming v4.1.0.

Signed-off-by: Stephan Linz <[email protected]>
Clarified information about the unofficial SoC implementation
within our downstream project.

Signed-off-by: Stephan Linz <[email protected]>
- Remove our own Maxim DS3231 RTC and temperature sensor driver.
- Support new revision C of NXP MIMXRT1060-EVK.
- Use new unified DTS property names.
- Remove xtools toolchain variant references.
- Update all test output messages.
- Porting to new doxyrunner.

Signed-off-by: Stephan Linz <[email protected]>
@rexut
Copy link
Member Author

rexut commented Feb 6, 2025

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@rexut rexut added this to the v4.1.0 milestone Feb 6, 2025
@rexut rexut linked an issue Feb 6, 2025 that may be closed by this pull request
@rexut rexut removed a link to an issue Feb 6, 2025
@rexut rexut self-assigned this Feb 6, 2025
@rexut rexut linked an issue Feb 6, 2025 that may be closed by this pull request
@rexut rexut removed a link to an issue Feb 6, 2025
@rexut rexut linked an issue Feb 6, 2025 that may be closed by this pull request
@rexut rexut requested a review from tobiaskaestner February 6, 2025 07:37
Copy link
Contributor

@tobiaskaestner tobiaskaestner left a comment

Choose a reason for hiding this comment

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

Lgtm

@rexut rexut merged commit 2058125 into main Feb 6, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bring back QA workflows to green
2 participants