From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15561 invoked by alias); 18 Jun 2012 21:16:46 -0000 Received: (qmail 15549 invoked by uid 22791); 18 Jun 2012 21:16:46 -0000 X-SWARE-Spam-Status: No, hits=-2.8 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED 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, 18 Jun 2012 21:16:32 +0000 Received: from localhost (hagrid.ecoscentric.com [127.0.0.1]) by mail.ecoscentric.com (Postfix) with ESMTP id 316072F78008 for ; Mon, 18 Jun 2012 22:16:31 +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 rXEKQSKat06p; Mon, 18 Jun 2012 22:16:25 +0100 (BST) From: bugzilla-daemon@bugs.ecos.sourceware.org To: ecos-bugs@ecos.sourceware.org Subject: [Bug 1001606] Enable the cache on Kinetis in RAM startup mode X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: eCos X-Bugzilla-Component: HAL X-Bugzilla-Keywords: X-Bugzilla-Severity: enhancement X-Bugzilla-Who: ilijak@siva.com.mk X-Bugzilla-Status: NEEDINFO X-Bugzilla-Priority: low X-Bugzilla-Assigned-To: jifl@ecoscentric.com 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, 18 Jun 2012 21:16:00 -0000 Message-Id: <20120618211625.BBED02F78005@mail.ecoscentric.com> Mailing-List: contact ecos-bugs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: ecos-bugs-owner@sourceware.org X-SW-Source: 2012/txt/msg01039.txt.bz2 Please do not reply to this email. Use the web interface provided at: http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001606 --- Comment #12 from Ilija Kocho 2012-06-18 22:16:14 BST --- (In reply to comment #11) > Hi Ilija, I am extracting the discussion on FPU as topic in its own right, that might get it's final shape in context of Cortex-M FPU support. The caches, on the other hand have some options that need more detailed consideration. > > > - It appears that CYGHWR_HAL_CORTEXM_KINETIS_FPU is effectively redundant > > > (since it's keyed off CYGHWR_HAL_CORTEXM_FPU), but I suspect this has now > > > been dealt with in your recent patches. > > > > It's actually Kinetis' specific. FPU is optional on Kinetis and it is > > reflected in part names. If you toggle CYGHWR_HAL_CORTEXM_KINETIS_FPU in > > configtool you'll notice changes in the part's name ('F' <-> 'D'). > > Optionally this option could be a data CDL with legal values "D" and "F" > > (more consistent with other part name segments) but bool seems more natural > > to me. Ref: The /part name building/ is described in SGML doc. > > Sure, but where CYGHWR_HAL_CORTEXM_KINETIS currently has: > calculated { "MK" . CYGHWR_HAL_CORTEXM_KINETIS_SUBFAM . > (CYGHWR_HAL_CORTEXM_KINETIS_FPU ? "F" : "D") . > (CYGHWR_HAL_CORTEXM_KINETIS_FLEXNVM ? "X" : "N") . > CYGHWR_HAL_CORTEXM_KINETIS_FLASH_NAME } > you could just change that to: > calculated { "MK" . CYGHWR_HAL_CORTEXM_KINETIS_SUBFAM . > (CYGHWR_HAL_CORTEXM_FPU ? "F" : "D") . > (CYGHWR_HAL_CORTEXM_KINETIS_FLEXNVM ? "X" : "N") . > CYGHWR_HAL_CORTEXM_KINETIS_FLASH_NAME } > > and preserve that behaviour? It seems better than have two options which > effectively do exactly the same thing, and have to have the same setting as > each other. > I want to preserve also integral functionality of the part builder CYGHWR_HAL_CORTEXM_KINETIS, including FPU option. CYGHWR_HAL_CORTEXM_KINETIS_FPU selects FPU as Kinetis option and CYGHWR_HAL_CORTEXM_FPU enables/disables FPU function so they aren't identical. But it's true that they look alike so below is CYGHWR_HAL_CORTEXM_KINETIS_FPU reshaped to resemble the other entries of the part builder (CYGHWR_HAL_CORTEXM_KINETIS_FLEXNVM can be reshaped likewise). cdl_option CYGHWR_HAL_CORTEXM_KINETIS_FPU { display "Floating Point Unit part name option" flavor data no_define legal_values { "D" "F" } default_value { "D" } requires { CYGHWR_HAL_CORTEXM_FPU implies CYGHWR_HAL_CORTEXM_KINETIS_FPU == "F" } description "Select/indicate whether the part has Floating Point Unit. \"F\" - stands for parts with FPU, while \"D\" for ones without." } This form will distinguish CYGHWR_HAL_CORTEXM_KINETIS_FPU and CYGHWR_HAL_CORTEXM_FPU in configtool and slightly simplify CYGHWR_HAL_CORTEXM_KINETIS. Ilija -- 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.