Discussion:
[Paparazzi-devel] Baro via I2C on Aspirin 2.1
Helge Walle
2018-01-14 18:48:42 UTC
Permalink
Hi,

I have a Funjet with Lisa 2.0 and Aspirin 2.1. The baro on the Aspirin
board is connected to Lisa via I2C (Alternative configuration).

In the airframe file, under target ap, I have:
<configure name="LISA_M_BARO" value="BARO_MS5611_I2C"/>

With v5.10 I get baro data ok, but after upgrading to v5.12 baro data is
not present. When building the airframe in v5.12, i get these lines:

...
In file included from boards/baro_board_ms5611_i2c.c:31:0:
./boards/bebop/baro_board.h:38:13: warning: redundant redeclaration of
'baro_event' [-Wredundant-decls]
extern void baro_event(void);
^
...

So I looked in the source code, and in
sw/airborne/boards/baro_board_ms5611_i2c.c I found this line in v5.12:
#include "boards/bebop/baro_board.h
This line is not present in v5.10.

So after commenting out this line I got baro data again, and the warning
during build was not there anymore.

Is the new line in v5.12 a permanent change?

Thanks,
Helge.
Michal Podhradsky
2018-01-17 00:58:19 UTC
Permalink
Hi Helge,
this might be related to the compiler change (the new arm-none-eabi gcc)
that has more strict checks. I ll try to have a closer look at this issue
in next few days.
Regards
Michal
Post by Helge Walle
Hi,
I have a Funjet with Lisa 2.0 and Aspirin 2.1. The baro on the Aspirin
board is connected to Lisa via I2C (Alternative configuration).
<configure name="LISA_M_BARO" value="BARO_MS5611_I2C"/>
With v5.10 I get baro data ok, but after upgrading to v5.12 baro data is
...
./boards/bebop/baro_board.h:38:13: warning: redundant redeclaration of
'baro_event' [-Wredundant-decls]
extern void baro_event(void);
^
...
So I looked in the source code, and in sw/airborne/boards/baro_board_ms5611_i2c.c
I found this line in v5.12: #include "boards/bebop/baro_board.h
This line is not present in v5.10.
So after commenting out this line I got baro data again, and the warning
during build was not there anymore.
Is the new line in v5.12 a permanent change?
Thanks,
Helge.
_______________________________________________
Paparazzi-devel mailing list
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
Loading...