Re: [ECOS] ecos + lwip

[Sergei Gavrikov <sergei.gavrikov@gmail.com>]

On Thu, Aug 27, 2009 at 04:10:52PM +0200, Simon Kallweit wrote:
> Mandeep Sandhu wrote:
>> On Thu, Aug 27, 2009 at 7:21 PM, Mandeep
>> Sandhu<mandeepsandhu.chd@gmail.com> wrote:
>>>> Well there was actually a mistake. I did compile sio.c even when serial
>>>> support is not needed (no SLIP or PPP). The warnings in socket.h should not
>>>> matter, but I'll look into them.
>>>>
>>>>> $ find packages/ -name serialio.h
>>>>> packages/io/serial/v3_0/include/serialio.h
>>>> Headers in the eCos repository will not be found when compiling, only the
>>>> headers which are copied over to your build tree are recognized.
>>>>
>>>> Well I fixed the bug with sio.c
>>>>
>>>> Here is the fixed release:
>>>> http://download.westlicht.ch/lwip-20090827-1.tar.gz
>>
>> This one compiled all the way through (leaving a few "warnings")!!
>
> Good news! I'll look into these warnings in socket.c when I get some time.

Simon, all warning will go away if you enter CDL for LWIP_ERR_T, look,
please, at lwip/err.h. By default that is signed char (?!), so its
capacity is quite overflowed. I just tried an ugly addition in err.h

#define LWIP_ERR_T unsigned int

and that fixed all gcc complains in socket.c. I checked it, i.e. with that tweak
all package is compiled silently (arm-elf-gcc 4.3.2), just alone warning:

sequential.c:113: warning: ‘set_ip_addr’ defined but not used

Sergei

>
> Simon

-- 
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss