Re: [Paparazzi-devel] mag_micromag_fw module build errors

[Felix Ruess]

Hi Stephen,

seems DOWNLINK_DEVICE is not defined when trying to send a message in
these modules (probably include order issue). I just pushed a fix for
this.

You also do NOT need to define USE_MAG_MICROMAG_FW, that does not
exist anywhere.
But you should define MICROMAG_DRDY_VIC_SLOT to an appropriate
otherwise free slot. (You had it misspelled as
MAG_MICROMAG_FW_DRDY_VIC_SLOT).

And you probably need to define SPI_MASTER.

There is some work going on to clean up the SPI defines a bit and make
them more clear.
Martin?

Cheers, Felix

On Fri, Dec 10, 2010 at 10:25 PM, Stephen L Hulme <address@hidden> wrote:
> Hi Felix
>
> Thank you for that. I added spi firmware subsystem "spi"
>
> Now I get the following in ap build mode.
>
> CC /home/stephen/paparazzi/var/FJSR/ap/modules/sensors/mag_micromag_fw.o
> modules/sensors/mag_micromag_fw.c: In function `micromag_event':
> modules/sensors/mag_micromag_fw.c:38: warning: implicit declaration of
> function `DOWNLINK_DEVICECheckFreeSpace'
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICECheckFreeSpace'
> modules/sensors/mag_micromag_fw.c:38: warning: implicit declaration of
> function `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: redundant redeclaration
> of 'DOWNLINK_DEVICETransmit'
> modules/sensors/mag_micromag_fw.c:38: warning: previous implicit
> declaration of 'DOWNLINK_DEVICETransmit' was here
> modules/sensors/mag_micromag_fw.c:38: warning: nested extern declaration
> of `DOWNLINK
> Regards
>
> Stephen
>
> On Fri, 10 Dec 2010 20:17:41 +0100 Felix Reuss wrote:
>
> Hi Stephen,
>
> I think you are missing the
> <subsystem name="spi"/>
>
> Cheers, Felix
>
> On Fri, Dec 10, 2010 at 6:10 AM, Stephen L Hulme <address@hidden> wrote:
>> Hi Martin (I Guess)
>>
>> I am having trouble building the mag_micromag_fw module in ap mode.
>>
>> I get the following message in the paparazzi centre
>>
>> CC
>>
>> /home/stephen/paparazzi/var/FJSR/ap/arch/lpc21/modules/sensors/mag_micromag_fw_hw.o
>> arch/lpc21/modules/sensors/mag_micromag_fw_hw.c: In function `SSP_ISR':
>> arch/lpc21/modules/sensors/mag_micromag_fw_hw.c:22: warning: implicit
>> declaration of function `SpiClearRti'
>> arch/lpc21/modules/sensors/mag_micromag_fw_hw.c:22: warning: nested
>> extern declaration of `SpiClearRti'
>> arch/lpc21/modules/sensors/mag_micromag_fw_hw.c:22: warning: implicit
>> declaration of function `SpiDisableRti'
>> arch/lpc21/modules/sensors/mag_micromag_fw_hw.c:22: warning: nested
>> extern declaration of `SpiDisableRti'
>> arch/lpc21/modules/sensors/mag_micromag_fw_hw.c:22: warning: implicit
>> declaration of function `SpiDisable'
>> arch/lpc21/modules/sensors/mag_micromag_fw_hw.c:22: warning: nested
>> extern declaration of `SpiDisable'
>> arch/lpc21/modules/sensors/mag_micromag_fw_hw.
>>
>> The relevant section of my airframe.xml file is:
>>
>> airframe name="Funjet 3sw Tiny 2.11">
>>
>> Â <firmware name="fixedwing">
>> ÂÂÂ <target name="ap"ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ board="tiny_2.11">
>> ÂÂÂÂÂ <define name="AGR_CLIMB"/>
>> ÂÂÂÂÂ <define name="LOITER_TRIM"/>
>> ÂÂÂÂÂ <define name="ALT_KALMAN"/>
>> ÂÂÂÂÂ <define name="WIND_INFO"/>
>> ÂÂÂÂÂ <define name="WIND_INFO_RET"/>
>> ÂÂÂÂÂ <define name="STRONG_WIND"/>
>> ÂÂÂÂÂ <define name="USE_I2C0"/>
>> ÂÂÂÂÂ <define name="USE_MAG_MICROMAG_FW"/>
>> ÂÂÂÂÂ <define name="MAG_MICROMAG_FW_DRDY_VIC_SLOT" value="9"/>
>> ÂÂÂÂÂ <define name="SSP_VIC_SLOT" value="11"/>
>> ÂÂÂÂÂ <define name="SENSOR_SYNC_SEND"/>
>> ÂÂÂ </target>
>>
>> ÂÂÂ <target name="sim"ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ board="pc">
>> ÂÂÂÂÂ <define name="AGR_CLIMB"/>
>> ÂÂÂÂÂ <define name="LOITER_TRIM"/>
>> ÂÂÂÂÂ <define name="ALT_KALMAN"/>
>> ÂÂÂÂÂ <define name="WIND_INFO"/>
>> ÂÂÂÂÂ <define name="WIND_INFO_RET"/>
>> ÂÂÂ </target>
>>
>> ÂÂÂÂÂÂ <!-- RC -->
>> ÂÂÂ <subsystem name="radio_control"ÂÂÂ type="ppm"/>
>> ÂÂÂ <!-- Communication -->
>> ÂÂÂ <subsystem name="telemetry"ÂÂÂÂÂÂÂ type="transparent">
>> ÂÂÂÂÂ <param name="MODEM_BAUD"ÂÂÂÂÂÂÂÂ value="B9600"/>
>> ÂÂÂ </subsystem>
>> ÂÂÂ <!-- Actuators are automatically chosen according to board-->
>> ÂÂÂ <subsystem name="control"/>
>> ÂÂÂ <!-- Sensors -->
>> ÂÂÂ <subsystem name="attitude"ÂÂÂÂÂÂÂÂ type="infrared"/>
>> ÂÂÂ <subsystem name="gps"ÂÂÂÂÂÂÂÂÂÂÂÂÂ type="ublox_lea4p">
>> ÂÂÂÂÂ <param name="GPS_BAUD"ÂÂÂÂÂÂÂÂÂÂ value="B38400"/>
>> ÂÂÂ </subsystem>
>> ÂÂÂ <!-- Nav -->
>> ÂÂÂ <subsystem name="navigation"/>
>> ÂÂÂ <!-- Interfaces -->
>> ÂÂÂ <subsystem name="i2c"/>
>> Â </firmware>
>>
>> Â <firmware name="setup">
>> ÂÂÂ <target name="tunnel"ÂÂÂÂÂÂÂÂÂÂÂÂÂ board="tiny_2.11"/>
>> ÂÂÂ <target name="usb_tunnel_0"ÂÂÂÂÂÂÂ board="tiny_2.11"/>
>> ÂÂÂ <target name="usb_tunnel_1"ÂÂÂÂÂÂÂ board="tiny_2.11"/>
>> ÂÂÂ <target name="setup_actuators"ÂÂÂÂ board="tiny_2.11"/>
>> Â </firmware>
>>
>> Â <!-- modules -->
>> Â <modules>
>> ÂÂÂ <load name="mag_micromag_fw.xml"/>
>> ÂÂÂ <load name="baro_scp_i2c.xml"/>
>> ÂÂÂ <load name="digital_camo.xml"/>
>> Â </modules>
>>
>>
>> Â<section name="MAG_MICROMAG">
>>
>> ÂÂÂ <define name="MM_SS_PIN" value="20"/>
>> ÂÂÂ <define name="MM_SS_IODIR" value="IO0DIR"/>
>> ÂÂÂ <define name="MM_SS_IOSET" value="IO0SET"/>
>> ÂÂÂ <define name="MM_SS_IOCLR" value="IO0CLR"/>
>>
>> ÂÂÂ <define name="MM_RESET_PIN" value="29"/>
>> ÂÂÂ <define name="MM_RESET_IODIR" value="IO0DIR"/>
>> ÂÂÂ <define name="MM_RESET_IOSET" value="IO0SET"/>
>> ÂÂÂ <define name="MM_RESET_IOCLR" value="IO0CLR"/>
>>
>> ÂÂÂ <define name="MM_DRDY_PINSEL" value="PINSEL1"/>
>> ÂÂÂ <define name="MM_DRDY_PINSEL_BIT" value="0"/>
>> ÂÂÂ <define name="MM_DRDY_PINSEL_VAL" value="1"/>
>> ÂÂÂ <define name="MM_DRDY_EINT" value="0"/>
>> ÂÂÂ <define name="MM_DRDY_VIC_IT" value="VIC_EINT0"/>
>> Â </section>
>>
>>
>> Any help appreciated.
>>
>> Regards
>>
>> Stephen
>>
>>
>>
>>
>>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
> _______________________________________________
> Paparazzi-devel mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/paparazzi-devel
>
>