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

dockerfiles: Add container for DT validation #352

Merged
merged 1 commit into from
May 22, 2020

Conversation

broonie
Copy link
Member

@broonie broonie commented Apr 9, 2020

The kernel has started moving devicetree schemas to a format which
allows for automatic validation. This requires a separate build target
and a toolset (currently not packaged for any distro) so create a new
docker container with the toolset installed that we can use to run these
tests.

Signed-off-by: Mark Brown broonie@kernel.org

The kernel has started moving devicetree schemas to a format which
allows for automatic validation.  This requires a separate build target
and a toolset (currently not packaged for any distro) so create a new
docker container with the toolset installed that we can use to run these
tests.

Signed-off-by: Mark Brown <broonie@kernel.org>
@gctucker gctucker requested a review from a team April 9, 2020 15:32
@gctucker gctucker added this to In Progress in KernelCI project board via automation Apr 9, 2020
@gctucker
Copy link
Contributor

Sounds good, that would be something to add to the "static analysis" category of tests we're starting to add.

We should just need to run a DT validation in this Docker image manually to get this PR merged and then work on how to automate things. It should get pushed to the Docker hub later today.

@gctucker
Copy link
Contributor

The image is there: https://hub.docker.com/r/kernelci/staging-dt-validation

@broonie Can you please try to run dt-validate using this Docker image with an actual kernel source tree (from git or a kernelci tarball) and post a comment with the results?

@broonie
Copy link
Member Author

broonie commented May 22, 2020

Yes, it works for me.

Copy link
Contributor

@gctucker gctucker left a comment

Choose a reason for hiding this comment

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

Thanks.

@gctucker gctucker merged commit 18ed548 into kernelci:master May 22, 2020
KernelCI project board automation moved this from In Progress to Done May 22, 2020
@gctucker gctucker mentioned this pull request Jun 5, 2020
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants