This is the mail archive of the ecos-devel@sourceware.org mailing list for the eCos project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: hal_platform_setup.h - Approach to a new platform port

From: Ilija Koco <ilijak at siva dot com dot mk>
To: ecos-devel at ecos dot sourceware dot org
Date: Wed, 07 Jun 2006 14:23:02 +0200
Subject: Re: hal_platform_setup.h - Approach to a new platform port
References: <44852E58.4080200@siva.com.mk> <20060606120227.GF15066@lunn.ch> <44857CB6.1030502@siva.com.mk> <20060606161000.GN15066@lunn.ch>

It seems to me there isn't much consistency in naming since
hal_platform_setup.h is assembler too. With respect to this I would stay
with hal_variant_setup.h

Ilija

Andrew Lunn wrote:

It wont be bad, but I derived from hal_platform_setup.h that has "platform" instead of "plf" (as usual). Also these files contain assembler rater than C source so maybe it's better if they are distinguished by their names as they are now.
If you want to indicate it contain assembly language it probably
should be .inc not .h. eg see ./powerpc/mpc5xx/current/include/variant.inc
Andrew

References:
- hal_platform_setup.h - Approach to a new platform port
  - From: Ilija Koco
- Re: hal_platform_setup.h - Approach to a new platform port
  - From: Andrew Lunn
- Re: hal_platform_setup.h - Approach to a new platform port
  - From: Ilija Koco
- Re: hal_platform_setup.h - Approach to a new platform port
  - From: Andrew Lunn

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]