public inbox for ecos-bugs@sourceware.org
help / color / mirror / Atom feed
From: bugzilla-daemon@bugs.ecos.sourceware.org
To: unassigned@bugs.ecos.sourceware.org
Subject: [Bug 1001864] Data not relocated to RAM during ROMINT startup
Date: Tue, 07 Oct 2014 16:54:00 -0000	[thread overview]
Message-ID: <bug-1001864-777-3ZaH0QpQJW@http.bugs.ecos.sourceware.org/> (raw)
In-Reply-To: <bug-1001864-777@http.bugs.ecos.sourceware.org/>

Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001864

--- Comment #6 from Ilija Kocho [Илија Кочо] <ilijak@siva.com.mk> ---
Hi Jifl

Thank you for your consideration.

(In reply to comment #5)
> Hi Ilija,
> 
> The specific proposal in comment #1 to use CYG_HAL_STARTUP_ROM doesn't feel
> quite right to me:
> 
> a) it would probably be better for this property to be accessible via CDL,
> not just source. A CDL interface presumably.
> b) It might make the code appear confusing in the case where you do need to
> distinguish "real" ROM startup from ROMINT/FLASH/whatever.

May be confusing indeed. I developed the presented method in order to avoid
changes at architecture level. Now, provided that it is not a taboo - why not?

then for protection of legacy code we can use something like

#if defined(CYG_HAL_STARTUP_ROM) || defined(CYGINT_HAL_...)

> 
> But this is easily dealt with just by using a different name for the
> abstract startup type. It could be CYGINT_HAL_STARTUP_ROM or
> CYGINT_HAL_ROM_STARTUP, but perhaps it might better express what we're after
> here if we called it CYGINT_HAL_STANDALONE_STARTUP.

/ROM/ does not imply /standalone/. FLASH startup from Bug #1001623 is being
started from RedBoot. And it shall implement both RAM and ROM interfaces.

> But i've no strong
> preferences.

Regarding remaining two options, I have no strong preference either, but
CYGINT_HAL_ROM_STARTUP seems slightly better distinguished from
CYG_HAL_STARTUP_ROM than the other one.

Ilija

-- 
You are receiving this mail because:
You are the assignee for the bug.
>From ecos-bugs-return-10904-listarch-ecos-bugs=sources.redhat.com@sourceware.org Tue Oct 07 16:54:25 2014
Return-Path: <ecos-bugs-return-10904-listarch-ecos-bugs=sources.redhat.com@sourceware.org>
Delivered-To: listarch-ecos-bugs@sources.redhat.com
Received: (qmail 27159 invoked by alias); 7 Oct 2014 16:54:23 -0000
Mailing-List: contact ecos-bugs-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <ecos-bugs.sourceware.org>
List-Subscribe: <mailto:ecos-bugs-subscribe@sourceware.org>
List-Post: <mailto:ecos-bugs@sourceware.org>
List-Help: <mailto:ecos-bugs-help@sourceware.org>, <http://sourceware.org/lists.html#faqs>
Sender: ecos-bugs-owner@sourceware.org
Delivered-To: mailing list ecos-bugs@sourceware.org
Received: (qmail 27133 invoked by uid 89); 7 Oct 2014 16:54:22 -0000
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=1.8 required=5.0 tests=AWL,BAYES_00,BODY_8BITS,GARBLED_BODY autolearn=no version=3.3.2
X-HELO: mail.ecoscentric.com
Received: from albus.ecoscentric.com (HELO mail.ecoscentric.com) (212.13.207.200)
 by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Tue, 07 Oct 2014 16:54:20 +0000
Received: from localhost (localhost [127.0.0.1])
	by mail.ecoscentric.com (Postfix) with ESMTP id 03F1EA88899
	for <ecos-bugs@ecos.sourceware.org>; Tue,  7 Oct 2014 17:54:18 +0100 (BST)
Received: from mail.ecoscentric.com ([127.0.0.1])
	by localhost (albus.ecoscentric.com [127.0.0.1]) (amavisd-new, port 10024)
	with ESMTP id iaQinz48zT_S; Tue,  7 Oct 2014 17:54:17 +0100 (BST)
From: bugzilla-daemon@bugs.ecos.sourceware.org
To: ecos-bugs@ecos.sourceware.org
Subject: [Bug 1001864] Data not relocated to RAM during ROMINT startup
Date: Tue, 07 Oct 2014 16:54:00 -0000
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: normal
X-Bugzilla-Who: ilijak@siva.com.mk
X-Bugzilla-Status: NEW
X-Bugzilla-Priority: normal
X-Bugzilla-Assigned-To: unassigned@bugs.ecos.sourceware.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Changed-Fields:
Message-ID: <bug-1001864-13-6F99LTFh1A@http.bugs.ecos.sourceware.org/>
In-Reply-To: <bug-1001864-13@http.bugs.ecos.sourceware.org/>
References: <bug-1001864-13@http.bugs.ecos.sourceware.org/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://bugs.ecos.sourceware.org/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2014/txt/msg00271.txt.bz2
Content-length: 1698

Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001864

--- Comment #6 from Ilija Kocho [Илија Кочо] <ilijak@siva.com.mk> ---
Hi Jifl

Thank you for your consideration.

(In reply to comment #5)
> Hi Ilija,
> 
> The specific proposal in comment #1 to use CYG_HAL_STARTUP_ROM doesn't feel
> quite right to me:
> 
> a) it would probably be better for this property to be accessible via CDL,
> not just source. A CDL interface presumably.
> b) It might make the code appear confusing in the case where you do need to
> distinguish "real" ROM startup from ROMINT/FLASH/whatever.

May be confusing indeed. I developed the presented method in order to avoid
changes at architecture level. Now, provided that it is not a taboo - why not?

then for protection of legacy code we can use something like

#if defined(CYG_HAL_STARTUP_ROM) || defined(CYGINT_HAL_...)

> 
> But this is easily dealt with just by using a different name for the
> abstract startup type. It could be CYGINT_HAL_STARTUP_ROM or
> CYGINT_HAL_ROM_STARTUP, but perhaps it might better express what we're after
> here if we called it CYGINT_HAL_STANDALONE_STARTUP.

/ROM/ does not imply /standalone/. FLASH startup from Bug #1001623 is being
started from RedBoot. And it shall implement both RAM and ROM interfaces.

> But i've no strong
> preferences.

Regarding remaining two options, I have no strong preference either, but
CYGINT_HAL_ROM_STARTUP seems slightly better distinguished from
CYG_HAL_STARTUP_ROM than the other one.

Ilija

-- 
You are receiving this mail because:
You are on the CC list for the bug.
>From ecos-bugs-return-10905-listarch-ecos-bugs=sources.redhat.com@sourceware.org Tue Oct 07 19:39:03 2014
Return-Path: <ecos-bugs-return-10905-listarch-ecos-bugs=sources.redhat.com@sourceware.org>
Delivered-To: listarch-ecos-bugs@sources.redhat.com
Received: (qmail 25014 invoked by alias); 7 Oct 2014 19:39:02 -0000
Mailing-List: contact ecos-bugs-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <ecos-bugs.sourceware.org>
List-Subscribe: <mailto:ecos-bugs-subscribe@sourceware.org>
List-Post: <mailto:ecos-bugs@sourceware.org>
List-Help: <mailto:ecos-bugs-help@sourceware.org>, <http://sourceware.org/lists.html#faqs>
Sender: ecos-bugs-owner@sourceware.org
Delivered-To: mailing list ecos-bugs@sourceware.org
Received: (qmail 25002 invoked by uid 89); 7 Oct 2014 19:39:02 -0000
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-2.2 required=5.0 tests=AWL,BAYES_00,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2
X-HELO: mail.ecoscentric.com
Received: from albus.ecoscentric.com (HELO mail.ecoscentric.com) (212.13.207.200)
 by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Tue, 07 Oct 2014 19:39:01 +0000
Received: by mail.ecoscentric.com (Postfix, from userid 512)
	id 47EF4A888DC; Tue,  7 Oct 2014 20:38:58 +0100 (BST)
X-Original-To: unassigned@bugs.ecos.sourceware.org
Delivered-To: unassigned@bugs.ecos.sourceware.org
From: bugzilla-daemon@bugs.ecos.sourceware.org
To: unassigned@bugs.ecos.sourceware.org
Subject: [Bug 1001864] Data not relocated to RAM during ROMINT startup
Date: Tue, 07 Oct 2014 19:39:00 -0000
X-Bugzilla-Reason: AssignedTo
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: eCos
X-Bugzilla-Component: HAL
X-Bugzilla-Keywords:
X-Bugzilla-Severity: normal
X-Bugzilla-Who: jifl@ecoscentric.com
X-Bugzilla-Status: NEW
X-Bugzilla-Priority: normal
X-Bugzilla-Assigned-To: unassigned@bugs.ecos.sourceware.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Changed-Fields:
Message-ID: <bug-1001864-777-1LZXgaL7kJ@http.bugs.ecos.sourceware.org/>
In-Reply-To: <bug-1001864-777@http.bugs.ecos.sourceware.org/>
References: <bug-1001864-777@http.bugs.ecos.sourceware.org/>
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/txt/msg00272.txt.bz2
Content-length: 497

Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id\x1001864

--- Comment #7 from Jonathan Larmour <jifl@ecoscentric.com> ---
(In reply to comment #6)
>
> Regarding remaining two options, I have no strong preference either, but
> CYGINT_HAL_ROM_STARTUP seems slightly better distinguished from
> CYG_HAL_STARTUP_ROM than the other one.

Let's go with that then.

Jifl

--
You are receiving this mail because:
You are the assignee for the bug.


  parent reply	other threads:[~2014-10-07 16:54 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-12 20:12 [Bug 1001864] New: " bugzilla-daemon
2013-06-12 22:47 ` [Bug 1001864] " bugzilla-daemon
2014-10-03 15:03 ` bugzilla-daemon
2014-10-04 12:52 ` bugzilla-daemon
2014-10-05 10:20 ` bugzilla-daemon
2014-10-07 13:44 ` bugzilla-daemon
2014-10-07 16:54 ` bugzilla-daemon [this message]
2013-06-12 20:12 [Bug 1001864] New: " bugzilla-daemon
2013-06-12 22:47 ` [Bug 1001864] " bugzilla-daemon
2014-10-03 15:03 ` bugzilla-daemon
2014-10-04 12:52 ` bugzilla-daemon
2014-10-07 13:44 ` bugzilla-daemon
2014-10-07 19:39 ` bugzilla-daemon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-1001864-777-3ZaH0QpQJW@http.bugs.ecos.sourceware.org/ \
    --to=bugzilla-daemon@bugs.ecos.sourceware.org \
    --cc=unassigned@bugs.ecos.sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).