From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32527 invoked by alias); 17 Oct 2011 23:28:46 -0000 Received: (qmail 32509 invoked by uid 22791); 17 Oct 2011 23:28:45 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from hagrid.ecoscentric.com (HELO mail.ecoscentric.com) (212.13.207.197) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 17 Oct 2011 23:28:30 +0000 Received: from localhost (hagrid.ecoscentric.com [127.0.0.1]) by mail.ecoscentric.com (Postfix) with ESMTP id CAC0D2F78014 for ; Tue, 18 Oct 2011 00:28:28 +0100 (BST) Received: from mail.ecoscentric.com ([127.0.0.1]) by localhost (hagrid.ecoscentric.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 0PbT+20NMA2L; Tue, 18 Oct 2011 00:28:27 +0100 (BST) From: bugzilla-daemon@bugs.ecos.sourceware.org To: ecos-patches@ecos.sourceware.org Subject: [Bug 1001187] New port - HAL: Freescale Kinetis variant, TWR-K60N512, TWR-K40X256 plf; Devs: Freescale UART and ENET (Ethernet) X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: eCos X-Bugzilla-Component: Patches and contributions X-Bugzilla-Keywords: X-Bugzilla-Severity: enhancement X-Bugzilla-Who: jifl@ecoscentric.com X-Bugzilla-Status: NEEDINFO X-Bugzilla-Priority: low X-Bugzilla-Assigned-To: unassigned@bugs.ecos.sourceware.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: In-Reply-To: References: X-Bugzilla-URL: http://bugs.ecos.sourceware.org/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Date: Mon, 17 Oct 2011 23:28:00 -0000 Message-Id: <20111017232827.621982F78004@mail.ecoscentric.com> Mailing-List: contact ecos-patches-help@ecos.sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: ecos-patches-owner@ecos.sourceware.org X-SW-Source: 2011-10/txt/msg00039.txt.bz2 Please do not reply to this email. Use the web interface provided at: http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001187 --- Comment #24 from Jonathan Larmour 2011-10-18 00:28:22 BST --- (In reply to comment #22) > (In reply to comment #20) > > As per direct mail, here's a few comments, although I haven't done what I'd > > call a full review: > > > > > > - I'll just mention that if extra TTYs are justified, then you should just > > modify CYGPKG_IO_SERIAL. > > > This has some history. The idea was a provision for adding (as much as needed) > additional tty/termios drivers into target without the need for changing IO. > Luckily after a long session with Sergei the patch was surprisingly short. > Please see: http://ecos.sourceware.org/ml/ecos-devel/2011-03/msg00023.html > Also: Bug #1001180 Personally I would take a pragmatic approach and just increase the number of devices to 6. However using a 'for' loop in the CDL, would be a good rationalisation. > > - The CYGPKG_HAL_CORTEXM_KINETIS .cdl file is quite big - perhaps the > > components related to clocking could be split off into a separate file, and > > included. > > > Yeah. This clock is more complex than some 8-bit micro-controllers. Good idea. > This brings me an idea of extracting clocking code from kinetis_misc.c? > > kinetis_clocking.cdl > kinetis_clocking.c > > Any comment? That would probably be good too. It makes it more maintainable to have all the clock things in one place, and for files not to get too large. Thanks for all the other responses. Jifl -- Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.