Make code warnings-free with -pedantic, this should guard against more programming errors in the future

This removes a function that's no longer used, and inline specifiers
from include file (where they're useless anyway). This makes the whole
default project warnings-free with -Wall -pedantic.

Signed-off-by: Paul Fertser <fercerpav@gmail.com>
OK!
On Jul 21, 2014 1:31 AM, "luggi" <notifications@github.com> wrote:

> In src/telemetry_frsky.c:
>
> > @@ -210,19 +210,6 @@ static void sendVoltage(void)
> >      currentCell %= batteryCellCount;
> >  }
> >
> > -/*
> > - * Send voltage with ID_VOLTAGE_AMP
> > - */
> > -static void sendVoltageAmp(void)
> > -{
> > -    uint16_t voltage = (vbat * 110) / 21;
> > -
> > -    sendDataHead(ID_VOLTAGE_AMP_BP);
> > -    serialize16(voltage / 100);
> > -    sendDataHead(ID_VOLTAGE_AMP_AP);
> > -    serialize16(((voltage % 100) + 5) / 10);
> > -}
> > -
>
> From what i can tell, this function didn't send anything useful to the
> frsky rx. The battery voltage is allready transmitted via sendVoltage(). I
> guess it can be removed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/multiwii/baseflight/pull/141/files#r15150482>.
>