From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24680 invoked by alias); 25 Sep 2012 20:17:56 -0000 Received: (qmail 24428 invoked by uid 22791); 25 Sep 2012 20:17:53 -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; Tue, 25 Sep 2012 20:17:39 +0000 Received: from localhost (hagrid.ecoscentric.com [127.0.0.1]) by mail.ecoscentric.com (Postfix) with ESMTP id 3BFC02F78006 for ; Tue, 25 Sep 2012 21:17:38 +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 B5W3BFtXoG35; Tue, 25 Sep 2012 21:17:36 +0100 (BST) From: bugzilla-daemon@bugs.ecos.sourceware.org To: ecos-patches@ecos.sourceware.org Subject: [Bug 1001561] Internal flash driver for Freescale TWR-K60N512 board 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: ilijak@siva.com.mk X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Priority: low X-Bugzilla-Assigned-To: ilijak@siva.com.mk X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: Attachment #1875 is obsolete 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: Tue, 25 Sep 2012 20:17:00 -0000 Message-Id: <20120925201736.82D842F78001@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: 2012-09/txt/msg00011.txt.bz2 Please do not reply to this email. Use the web interface provided at: http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001561 Ilija Kocho changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #1875|0 |1 is obsolete| | --- Comment #46 from Ilija Kocho 2012-09-25 21:17:29 BST --- Created an attachment (id=1940) --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1940) Kinetis Flash device driver 120925 (In reply to comment #45) Hi Jifl, Many thanks for your comments. > Hi Ilija, > > After my summer holidays, I've been exceptionally busy for various reasons, > sorry about that. But before you commit here... I hope you had nice holidays. I was busy as well past few weeks and now I'm trying to fight the backlog. > > There's no way to disable the flash driver. Once you add CYGPKG_IO_FLASH this > driver will be present even if you don't want to use it, e.g. to only support > off-chip flash. Not impossible if you want to keep on-chip flash for your > program and off-chip for data (e.g. flash filesystem). Thank you for pointing this out, I wasn't aware of this. Btw, can't application select where will FS reside in runtime? Anyway, I attach a solution (not tested yet). The realisation is somehow different than in STM32 but should be functionally equivalent. Is there some severe reason for tab entry to live with HAL rather than driver? Also, maybe #ifdef could enclose complete driver code. > > I assume you are happy leaving the flash configuration area unprotected for > now? I gave up due to lack of RedBoot user control (that you have pointed out). Also, during practical use I realised that attempt for rewriting is pretty much unlikely, because the critical bytes are within RedBoot area. Btw, maybe we could consider RedBoot protecting itself from overwriting, in general (CDL option). Then, the critical functions: /fis load/, etc. could have -f option to enforce writing in critical regions. > > We could temporarily do something crude and specifically check for the > 0x400-0x40f addresses and have a CDL option (default on) which refuses to > program that area. Someone would have to disable the CDL option to program > them. Not as sophisticated as what I suggested first of all, but perhaps a > safety net? I could look at this, but I realize it would need some printouts..., so I would rather think about general RedBoot solution above. 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.