Skip to content

Conversation

gastmaier
Copy link
Contributor

PR Description

Checkout include/dt-bindings files that don't exist on upstream next master,
with the premise they are non-upstreamable files (e.g., ad*.h)., changes behavior for #2891 .

Only run the second build stage on tracked branches.
The second build stage generate artifacts intended to be deployed on
hardware and include all drivers, hence the longer build times.

On concat to DTS list from DTSI, a space was missing, fixes #2888

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 tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly (if there is the case)

Copy link
Collaborator

@nunojsa nunojsa left a comment

Choose a reason for hiding this comment

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

Just one remark. Git subject fix: ci: ... does not align with style used in the kernel. I now it's ci stuff but we should try to keep git log consistent. Other than that, looks good

On concat to DTS list from DTSI, a space was missing.
Fixes: cd81c99 ("ci: Collect dts when a dtsi is edited")

Signed-off-by: Jorge Marques <[email protected]>
Only run the second build stage on tracked branches.
The second build stage generate artifacts intended to be deployed on
hardware and include all drivers, hence the longer build times.

Signed-off-by: Jorge Marques <[email protected]>
Checkout include/dt-bindings files that don't exist on upstream next master,
with the premise they are non-upstreamable files (e.g., ad*.h).

Signed-off-by: Jorge Marques <[email protected]>
@gastmaier gastmaier changed the title ci: inlclude target branch include/dt-bindings, fix, and coditional ci: include target branch include/dt-bindings, fix, and coditional Aug 18, 2025
@gastmaier gastmaier merged commit 8eae91a into main Aug 18, 2025
2 of 11 checks passed
@gastmaier gastmaier deleted the ci/update branch August 18, 2025 16:26
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.

2 participants