From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1534 invoked by alias); 3 Feb 2014 19:33:30 -0000 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 Received: (qmail 1512 invoked by uid 89); 3 Feb 2014 19:33:29 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW autolearn=ham version=3.3.2 X-HELO: mail.ecoscentric.com Received: from hagrid.ecoscentric.com (HELO mail.ecoscentric.com) (212.13.207.197) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Mon, 03 Feb 2014 19:33:28 +0000 Received: from localhost (hagrid.ecoscentric.com [127.0.0.1]) by mail.ecoscentric.com (Postfix) with ESMTP id AEE254680001 for ; Mon, 3 Feb 2014 19:33:24 +0000 (GMT) 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 YiMxFr34GgZZ; Mon, 3 Feb 2014 19:33:19 +0000 (GMT) From: bugzilla-daemon@bugs.ecos.sourceware.org To: ecos-patches@ecos.sourceware.org Subject: [Bug 1001933] New HAL for the M4 core of Freescale Vybrid targets Date: Mon, 03 Feb 2014 19:33:00 -0000 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: Stefan.Singer@freescale.com X-Bugzilla-Status: ASSIGNED X-Bugzilla-Priority: low X-Bugzilla-Assigned-To: ilijak@siva.com.mk X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: http://bugs.ecos.sourceware.org/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-SW-Source: 2014-02/txt/msg00000.txt.bz2 Please do not reply to this email, use the link below. http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001933 --- Comment #6 from Stefan Singer --- Hi Ilija, sorry for the slow response, but some urgent things came in the middle. Let me try to address some of the points: drivers: We only verified certain elements, e.g. ENET works. We will do some more checking. "There are a lot of "reg*.h" files in HAL include directory. Devices in eCos are decoupled from HAL in order to enable usage of device drivers with different architectures. I know that this philosophy is not perfectly implemented, but I aim to do best for new packages. It is especially benefit for Freescale's chips that re-use peripherals on different architectures." This is exactly the reason for us having those files. For example we have designed an Audio Framework, that works on MPC5xxx (Big Endian Power Architecture) and Vybrid (Little Endian ARM CortexM), because all of those peripheral definitions are taken from the HAL, so it e.g. includes the "sai_reg.h" file for the definition of the SAI (Serial Audio Interface) Peripheral, which are endianess swapped files between those two architectures. Isn't that exactly what you want to achieve ? How else would I achieve that ? "I'm glad to see SGML docs, but they seem incomplete and their compilation raises errors." Quite frankly I do not even know how to correctly view SGML. I just opened your Kinetis files with a text editor and tried to get something semi correct. "I have noticed CDL for Compiler selection. It is unnecessary because user can specify compiler prefix and flags in "Global Build Options". Please remove it." Actually we driving a lot of things from that selection, e.g. whether we can move some critical code into the TCM (Tightly coupled Memory). This requires, that the Compiler can generate some trampolin code for that purpose, which the standard eCOS Compiler will not, but the Codesourcery will. That checkbox will avoid a lot of user erros and all members of our team really like that feature. Stefan -- You are receiving this mail because: You are on the CC list for the bug.