From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13671 invoked by alias); 17 Jan 2011 13:18:06 -0000 Received: (qmail 13659 invoked by uid 22791); 17 Jan 2011 13:18:06 -0000 X-SWARE-Spam-Status: No, hits=-1.8 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 Jan 2011 13:18:01 +0000 Received: from localhost (hagrid.ecoscentric.com [127.0.0.1]) by mail.ecoscentric.com (Postfix) with ESMTP id 522072F78006 for ; Mon, 17 Jan 2011 13:17:59 +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 xwQ6IJ2l6dzA; Mon, 17 Jan 2011 13:17:59 +0000 (GMT) From: bugzilla-daemon@bugs.ecos.sourceware.org To: ecos-bugs@ecos.sourceware.org Subject: [Bug 1001090] Added option to run system timer with internal clock 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: sergei.gavrikov@gmail.com X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Priority: low X-Bugzilla-Assigned-To: unassigned@bugs.ecos.sourceware.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: CC 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 Jan 2011 13:18:00 -0000 Message-Id: <20110117131757.E31A02F7800A@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: 2011/txt/msg00086.txt.bz2 Please do not reply to this email. Use the web interface provided at: http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001090 Sergei Gavrikov changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sergei.gavrikov@gmail.com --- Comment #4 from Sergei Gavrikov 2011-01-17 13:17:56 GMT --- (In reply to comment #3) > (In reply to comment #2) > > CYGSEM_HAL_SYSTICK_CLK_INTERNAL is a CDL option to be defined in the variant > > configuration. Is that ok or shall it be defined in the cortexm CDL? > > I downloaded your parch and produced a CDL bool option into LPC17xx port ( Bug > 1001114 ). Needles to say, works fine. Of course there is (always) a question > of flavor: a bool as is now or data - CDL defines CYGARC_REG_SYSTICK_CLK_SRC as > a choice between Int/Ext (no need for CYGSEM_HAL_SYSTICK_CLK_INTERNAL ). > > Regarding your question: IMO this bit is a part of the Cortex-M architecture > and it would be good if Cortex-M architecture CDL reflects it. IMO, yet another solution would be to move the clock management in the variant HALs and in the architecture HAL to have those famous 'Clock control' macros: #define HAL_CLOCK_INITIALIZE( _period_ ) hal_clock_initialize( _period_ ) #define HAL_CLOCK_RESET( _vec_, _period_ ) hal_clock_reset( _vec_, _period_ ) #define HAL_CLOCK_READ( _pvalue_ ) hal_clock_read( _pvalue_ ) ... It would be nice to know the opinion of Nick Garnett. -- 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.