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

[TOPIC-GPIO] boards: particle_*: update for new GPIO API #21249

Merged
merged 1 commit into from
Dec 11, 2019

Conversation

pabigot
Copy link
Collaborator

@pabigot pabigot commented Dec 9, 2019

Provide correct active level and pull on all signals. Use
init-to-active when configuring antenna switch.

Signed-off-by: Peter Bigot peter.bigot@nordicsemi.no

Provide correct active level and pull on all signals.  Use
init-to-active when configuring antenna switch.

Signed-off-by: Peter Bigot <peter.bigot@nordicsemi.no>
@zephyrbot
Copy link
Collaborator

All checks passed.

checkpatch (informational only, not a failure)

-:174: WARNING:LONG_LINE: line over 80 characters
#174: FILE: boards/arm/particle_boron/board.h:18:
+#define ANT_UFLn_GPIO_NAME	DT_INST_0_SKYWORKS_SKY13351_VCTL1_GPIOS_CONTROLLER

- total: 0 errors, 1 warnings, 303 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

Your patch has style problems, please review.

NOTE: Ignored message types: AVOID_EXTERNS BRACES CONFIG_EXPERIMENTAL CONST_STRUCT DATE_TIME FILE_PATH_CHANGES MINMAX NETWORKING_BLOCK_COMMENT_STYLE PRINTK_WITHOUT_KERN_LEVEL SPLIT_STRING VOLATILE

NOTE: If any of the errors are false positives, please report
      them to the maintainers.

Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.

@galak galak merged commit dcc84fc into zephyrproject-rtos:topic-gpio Dec 11, 2019
@pabigot pabigot deleted the gpio/particle branch December 19, 2019 14:30
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.

4 participants