Skip to content

Conversation

@scottwcpg
Copy link
Contributor

We are replacing the multiple GPIO drivers for Microchip MEC family with a single driver. The hardware is nearly identical in across MEC15xx and MEC172x/4x/5x SoCs.

@scottwcpg
Copy link
Contributor Author

Why is compliance check failing related to python script import ordering?

@scottwcpg
Copy link
Contributor Author

Running check_compliance.py locally produces:
Running LicenseAndCopyrightCheck tests in /home/c21969/devel/zephyrproject/zephyr ...
WARNING : Could not resolve SPDX License Identifier of /home/c21969/devel/zephyrproject/zephyr/LICENSES/README.md, resolving to README. Make sure the license is in the license list found at https://spdx.org/licenses/ or that it starts with 'LicenseRef-', and that it has a file extension.

and
Running Pylint tests in /home/c21969/devel/zephyrproject/zephyr ...
WARNING : Skipped DevicetreeLinting
1 check(s) failed
ERROR : Test Ruff failed:
Python lint error (I001) see https://docs.astral.sh/ruff/rules/unsorted-imports:Import block is un-sorted or un-formatted
File:/home/c21969/devel/zephyrproject/zephyr/scripts/west_commands/tests/west_build/test_dir_fmt.py
Line:5
Column:1
EndLine:12
EndColumn:24

Do these tests require a specific Python version?
I have python 3.13.9 as a virtual env.

@scottwcpg scottwcpg force-pushed the mec_common_gpio_driver branch from 3db36e2 to 7cfefa3 Compare December 1, 2025 15:25
@scottwcpg
Copy link
Contributor Author

Had to install and configure a clean zephyr development environment using Python 3.10 not 3.11.
Rebase PR with upstream.
Check compliance script error due to ruff python package is gone. License compliance warning in zephyr/LICENSES.README.md is still present.

@scottwcpg
Copy link
Contributor Author

@nashif @fabiobaltieri Does anyone know why check_compliance.py is failing?

@scottwcpg scottwcpg force-pushed the mec_common_gpio_driver branch from 7cfefa3 to c63c3bb Compare December 1, 2025 15:59
@scottwcpg
Copy link
Contributor Author

CI is running DeviceTree linter but local invocation of check_compliance.py is skipping the DeviceTree linter. Adding logging does not show why. Node.js npx and npm versions 10.9.3 are installed.

@scottwcpg
Copy link
Contributor Author

Zephyr environment missing JS dts-linter module. Running npx command to link a single file triggered npx to request download of [email protected]. Local check_compliance.py now runs DeviceTree linter.

Microchip MEC parts have a similar GPIO peripheral block. We
create a unified driver for all parts. NOTE: MEC GPIO interrupt
detection sets active status when changed from interrupt detect
disabled to any enabled mode. Driver ISR and interrupt configuration
implementation includes work-arounds for this issue.

Signed-off-by: Scott Worley <[email protected]>
We added Microchip MEC15xx, MEC172x, MEC174x, and MEC175x evalutaion
boards to the GPIO driver gpio_basic_api, gpio_api_1pin, and
gpio_get_direction tests.

Signed-off-by: Scott Worley <[email protected]>
@scottwcpg scottwcpg force-pushed the mec_common_gpio_driver branch from c63c3bb to 050cb7a Compare December 1, 2025 21:16
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 1, 2025

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.

2 participants