Baro via I2C on Aspirin 2.1

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Baro via I2C on Aspirin 2.1

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).

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.



_______________________________________________
Paparazzi-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
Reply | Threaded
Open this post in threaded view
|

Re: Baro via I2C on Aspirin 2.1

Michal Podhradsky
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

On Sun, Jan 14, 2018 at 10:48 AM, Helge Walle <[hidden email]> wrote:
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.



_______________________________________________
Paparazzi-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel



_______________________________________________
Paparazzi-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel