juancarlostinoco / ardupilot-mega

Automatically exported from code.google.com/p/ardupilot-mega
0 stars 0 forks source link

RCX_MIN values for aux channels never get stored or overwritten with 0 in ArduPlane rev cb25c7e7c4b3 #718

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. erase EEPROM via CLI and restart
2. perform radio setup
3. restart and use CLI setup show to see that all these are 0 

What is the expected output? What do you see instead?

RCX_MIN values should be stored correctly obviously.

What version of the product are you using? On what operating system?

Revision cb25c7e7c4b3

Please provide any additional information below.

See discussion here: 
http://diydrones.com/forum/topics/upcoming-features-for-arduplane-2-60

One potential problem could be the AP_NESTEDGROUPINFO inside RC_Channel_aux 
which could overwrite the first parameter in the higher group RC_Channel, which 
happens to be the MIN value. This is a wild guess based on seeing so many 
changes in the AP_Param EEPROM parameter structure, relative to 2.50.

Original issue reported on code.google.com by andre.sc...@gmail.com on 21 Aug 2012 at 8:13

GoogleCodeExporter commented 8 years ago
Hi Andre,
I'm not able to reproduce the problem. I followed the steps you gave, and ended 
up with the right value (1061) for RC1_MIN. Note that I used the CLI radio 
setup method (did you use the CLI for radio setup, or Mission Planner?).
I tested with an APM2, with git revision 
cb25c7e7c4b3385a6977f0343308dbf619759692
Does it happen every time for you?
Cheers, Tridge

Original comment by tridg...@gmail.com on 21 Aug 2012 at 11:31

GoogleCodeExporter commented 8 years ago
Hi Tridge,
thanks a lot for the follow up. Yes, unfortunately this occurs every time. 
Should have mentioned that I'm running an APM1 and had to do some changes to 
cut down the size. Here is my git diff output with respect to the revision you 
mention:

http://pastebin.com/M4RFgSH8

I can't see how any of these would affect the issue but of course I don't know 
the code that well.

Thanks a lot for your help! Andre

Original comment by andre.sc...@gmail.com on 22 Aug 2012 at 10:54

GoogleCodeExporter commented 8 years ago
Should mention that I hard-coded the CLI here to simplify the testing..

Original comment by andre.sc...@gmail.com on 22 Aug 2012 at 10:55

GoogleCodeExporter commented 8 years ago
Seems that I finally found a workaround. Basically I removed the AP_NESTEDGROUP 
parameter macro and copypasted instead all the other channel specific parameter 
macros from RC_CHANNEL to RC_CHANNEL_AUX (requires making some fields protected 
that were previously private). Then min values get stored correctly for the aux 
channels.

Since nobody else is reporting problems I wonder whether this is specific to 
APM1 1280. But it would be strange..

Original comment by andre.sc...@gmail.com on 15 Sep 2012 at 2:17