Skip to content

[ADI] github: pass defconfig list into workflow#76

Open
ozan956 wants to merge 1 commit intoadi-u-boot-2025.10.yfrom
qconfig-check-test
Open

[ADI] github: pass defconfig list into workflow#76
ozan956 wants to merge 1 commit intoadi-u-boot-2025.10.yfrom
qconfig-check-test

Conversation

@ozan956
Copy link
Copy Markdown

@ozan956 ozan956 commented Mar 19, 2026

Pass the defconfigs to the checks workflow so we can validate board specific defconfigs and report any files that needs to be formatted.

Pass the defconfigs to the checks workflow so we can validate
board specific defconfigs and report any files that needs to
be formatted.

Signed-off-by: Ozan Durgut <ozan.durgut@analog.com>
@ozan956
Copy link
Copy Markdown
Author

ozan956 commented Mar 19, 2026

@ozan956 ozan956 requested review from gastmaier and pamolloy March 19, 2026 08:39
@ozan956 ozan956 self-assigned this Mar 19, 2026
@ozan956 ozan956 added this to ADSP Mar 19, 2026
@ozan956 ozan956 marked this pull request as ready for review March 19, 2026 08:39
@pamolloy
Copy link
Copy Markdown
Collaborator

@woodrowbarlow you might want to check your O-RAN branch as well

Copy link
Copy Markdown
Collaborator

@pamolloy pamolloy left a comment

Choose a reason for hiding this comment

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

Looks good to me, but let's wait for @gastmaier to take a look next week

Copy link
Copy Markdown
Collaborator

@pamolloy pamolloy left a comment

Choose a reason for hiding this comment

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

See https://github.com/analogdevicesinc/u-boot/actions/runs/23286008647

We should look into why "all checks have passed" even though our workflows didn't run.

@ozan956
Copy link
Copy Markdown
Author

ozan956 commented Mar 19, 2026

See https://github.com/analogdevicesinc/u-boot/actions/runs/23286008647

We should look into why "all checks have passed" even though our workflows didn't run.

It fails because current worflow file in ci repo does not call defconfigs as input I believe.

See the lines I have added in my another PR (which I am currently replacing the implementation with qconfig.py but the lines I am referring is generic):
https://github.com/analogdevicesinc/u-boot/pull/70/changes#diff-3ea54af4839eb75404d71b28252bead7e7ec8f676b1f815e1cde02629a75c165R13

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