Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not doing the MTD test on FMUv3 in rc.apm #1945

Closed
gmorph opened this issue Mar 5, 2015 · 1 comment
Closed

Not doing the MTD test on FMUv3 in rc.apm #1945

gmorph opened this issue Mar 5, 2015 · 1 comment
Assignees

Comments

@gmorph
Copy link
Contributor

gmorph commented Mar 5, 2015

The if test is specifically checking for FMU v2 - this needs to be fixed.

@OXINARF
Copy link
Member

OXINARF commented Jun 1, 2017

I think this wasn't already the case when this was opened (last version before this was opened doesn't have any board check:

if mtd start /fs/mtd
then
echo "started mtd driver OK"
else
echo "failed to start mtd driver"
echo "failed to start mtd driver" >> $logfile
sh /etc/init.d/rc.error
fi
if mtd readtest /fs/mtd
then
echo "mtd readtest OK"
else
echo "failed to read mtd"
echo "failed to read mtd" >> $logfile
sh /etc/init.d/rc.error
fi
), but it certainly doesn't happen today - this check has been moved to code instead of init file: https://github.com/ArduPilot/ardupilot/blob/master/libraries/AP_HAL_PX4/Storage.cpp#L200-L209.

@OXINARF OXINARF closed this as completed Jun 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants