Hi Florian,
On Thursday, 23. October 2014 08:42:31 Florian Preinstorfer wrote:
> > you are right, eeprom->rs232_inversion is the same as eeprom->invert.
> > To replacing it is good. However i propose to split up yout patch in two
> > parts:
> > - first replacing rs232_inversio vs. invert.
> > - second introducing the printout function.
> >
> > This will help regression testing.
>
> Hi Uwe.
>
> Good point, thanks for reviewing the patch. Please find two patches
> attached, that split up the rs232_inversion replacement and the printout
> function.
both patches have been applied. I've double checked the removal
of the "RS232_INVERSION" eeprom API definition and it was safe to remove it:
It was never part of an official release and also just added on 2014-09-14.
I guess the only one that needs to change his code is Uwe :D
Cheers,
Thomas
signature.asc
Description: This is a digitally signed message part.
|