From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24347 invoked by alias); 22 Jan 2010 18:51:52 -0000 Received: (qmail 24299 invoked by uid 22791); 22 Jan 2010 18:51:51 -0000 X-SWARE-Spam-Status: No, hits=-2.3 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from mtaout01-winn.ispmail.ntl.com (HELO mtaout01-winn.ispmail.ntl.com) (81.103.221.47) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 22 Jan 2010 18:51:45 +0000 Received: from aamtaout02-winn.ispmail.ntl.com ([81.103.221.35]) by mtaout01-winn.ispmail.ntl.com (InterMail vM.7.08.04.00 201-2186-134-20080326) with ESMTP id <20100122185142.QCRG4204.mtaout01-winn.ispmail.ntl.com@aamtaout02-winn.ispmail.ntl.com>; Fri, 22 Jan 2010 18:51:42 +0000 Received: from cog.dallaway.org.uk ([213.106.93.52]) by aamtaout02-winn.ispmail.ntl.com (InterMail vG.2.02.00.01 201-2161-120-102-20060912) with ESMTP id <20100122185142.KVUL21638.aamtaout02-winn.ispmail.ntl.com@cog.dallaway.org.uk>; Fri, 22 Jan 2010 18:51:42 +0000 Received: from cog.dallaway.org.uk (cog.dallaway.org.uk [127.0.0.1]) by cog.dallaway.org.uk (8.13.8/8.13.8) with ESMTP id o0MIpd06031100; Fri, 22 Jan 2010 18:51:39 GMT Message-ID: <4B59F3BB.3000800@dallaway.org.uk> Date: Fri, 22 Jan 2010 18:51:00 -0000 From: John Dallaway User-Agent: Thunderbird 2.0.0.22 (X11/20090625) MIME-Version: 1.0 To: Simon Kallweit CC: eCos developers Subject: Re: lwip 1.3.2 port References: <73f9997f0907192203h77494e62u58b44ccfc7a738f4@mail.gmail.com> <4A65B8AE.8090204@martinlaabs.de> <73f9997f0907212153i743cfc7as86d2a1205c5dc7ab@mail.gmail.com> <73f9997f0907220421y145a62c3lb2728a132474c900@mail.gmail.com> <4A66F7E0.4050703@intefo.ch> <73f9997f1001220342h7bc1c79cl7180a184bee867f3@mail.gmail.com> <4B599FDA.7020808@intefo.ch> <4B59A8C1.5040903@dallaway.org.uk> <4B59B98F.5010106@intefo.ch> In-Reply-To: <4B59B98F.5010106@intefo.ch> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Mailing-List: contact ecos-devel-help@ecos.sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: ecos-devel-owner@ecos.sourceware.org X-SW-Source: 2010-01/txt/msg00004.txt.bz2 Hi Simon Simon Kallweit wrote: > Ok, I merged the 1.3.2 stable code and did a few quick tests (the > changes are not huge). The tarball is at > http://download.westlicht.ch/lwip-20100122.tar.gz Some initial comments based mainly on diffs against the upstream lwIP 1.3.2 sources and the eCos lwIP 1.1.1 port: a) On the whole, the upstream sources have very little modification. That's good news for future updates. Is it strictly necessary to move the include/ipv4/ headers into include/ as part of the eCos port? This seems like unnecessary effort and will also make it more difficult to support IPv6 in the future. b) Closure of the extern "C" block seems to be missing in network.h. c) There are a lot of small changes under src/netif/ppp/ including function renaming. I understand that you have your own PPP requirements to consider but I think we should stick closer to the master sources for the CVS check-in. Unless your changes have already been accepted upstream? I hope to look at the CDL and run up some tests over the weekend. John Dallaway eCos maintainer