-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[navstik] Added a new board #744
Conversation
@@ -425,6 +425,7 @@ void RadioControlEventImp(void (*frame_handler)(void)) { | |||
#else | |||
/* if we have one receiver and it has new data */ | |||
if(PrimarySpektrumState.RcAvailable) { | |||
LED_ON(1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
debug led I guess ? should be removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, will remove it.
Fixed the comments and also added udev rules for the IvyGS module. |
@@ -24,6 +24,7 @@ | |||
* Inertial Measurement Unit interface. | |||
*/ | |||
|
|||
#include BOARD_CONFIG |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do you need the board config here ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because of the IMU poweron in defined in the board. That was missing from my previous pull request.
@@ -18,6 +18,13 @@ else ifeq ($(BOARD), lisa_l) | |||
BARO_BOARD_CFLAGS += -DUSE_I2C2 | |||
BARO_BOARD_SRCS += $(SRC_BOARD)/baro_board.c | |||
|
|||
# Navstik baro |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be BARO_BOARD_CFLAGS instead of ap.CFLAGS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, will edit that. Probably because of merging this went wrong ;)
@fvantienen fixed... |
Navstik STM32F4 autopilot.
This has the basic support for all the sensors and outputs most autopilots have. Still having issues with the i2c at boot, but seems to be a problem with STM32F4 autopilots.
https://wiki.paparazziuav.org/wiki/NavStik
http://www.navstik.org