Re: [Paparazzi-devel] Spektrum Receiver & patching main_ap.c/autopilot.h

[Felix Ruess]

Hi Gareth,

I removed the unused autopilot_process_radio_control

You should use fbw_state->channels[RADIO_MODE] instead of radio_control.values in the autopilot on fixedwing.

This is because we still support the old classix which has two processors:

* fbw (fly-by-wire that handles manual mode and actuators)

* ap (autopilot that handles the autonomous stuff).

The communication between the two "processes" is handled by inter_mcu and fbw_state is the data structure shared by both.

Cheers, Felix

On Wed, Jun 13, 2012 at 9:10 PM, Gareth Roberts <address@hidden> wrote:

Hi Tilman,

I think it's right but make sure you test it (a lot!) on the ground.

It wasn't right :)

Just double checked and the autopilot_process_radio_control, despite being defined, isn't used anywhere in the codebase.
I've altered main_ap.c to use the function instead of the #define ternary, but I'm not sure where to get the rc values.
I've pushed the change to the branch from the previous email.

Is the radio_control struct from radio_control.h a valid global to use? It's compiling but I'm not sure if it's right.

If someone who actually knows what they are doing could look at the patch, that would be great.
Tilman, I can check it properly tomorrow if you want to wait, or you can try and see if it works...

Cheers,
Gareth

_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel