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

boards: shields: add spi support to mikroe weather click shield #82165

Merged

Conversation

iandmorris
Copy link
Contributor

Added support for the SPI interface which is availble on the shield if the jumper configurations are changed.

@zephyrbot zephyrbot added the area: Shields Shields (add-on boards) label Nov 27, 2024
.. note::

By default the Weather Click is configured to use the I2C interface. In
order to use the SPI interface the jumper settings must be changned. See
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
order to use the SPI interface the jumper settings must be changned. See
order to use the SPI interface the jumper settings must be changed. See

Comment on lines 4 to 5
config SHIELD_MIKROE_WEATHER_CLICK
def_bool $(shields_list_contains,mikroe_weather_click)
Copy link
Collaborator

Choose a reason for hiding this comment

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

You don't want to let people build for this "no-op" shield

Suggested change
config SHIELD_MIKROE_WEATHER_CLICK
def_bool $(shields_list_contains,mikroe_weather_click)

Copy link
Collaborator

Choose a reason for hiding this comment

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

drop file

@iandmorris iandmorris force-pushed the mikroe_weather_click_spi branch from 38b77fb to a6b4aa4 Compare December 10, 2024 21:59
@iandmorris iandmorris requested a review from kartben December 10, 2024 22:46
Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

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

Sorry, just spotted this when searching for mikroe_weather_click across the tree to make sure there weren't stale references to the old shield: Line 44 of index.rst needs updating.
In fact, this made me realize that you also want to indicate in the migration guide that the shield was renamed (unless you maybe keep the original name as the "default" i2c and call the new spi variant/revision ..._spi?

Added support for the SPI interface which is availble on the shield
if the jumper configurations are changed.

Signed-off-by: Ian Morris <[email protected]>
@iandmorris iandmorris force-pushed the mikroe_weather_click_spi branch from a6b4aa4 to 5fbdca5 Compare December 23, 2024 12:16
@zephyrbot zephyrbot added the Release Notes To be mentioned in the release notes label Dec 23, 2024
@kartben kartben merged commit 12fada0 into zephyrproject-rtos:main Dec 23, 2024
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Shields Shields (add-on boards) Release Notes To be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants