This is the mail archive of the ecos-patches@sources.redhat.com 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: i386: allow platforms to override HAL_MEM_REAL_REGION_TOP


On Thu, 2005-04-21 at 20:30 +0200, Andrew Lunn wrote:
> On Thu, Apr 21, 2005 at 03:47:12PM +0100, David Vrabel wrote:
> > Hi,
> > 
> > This patch allows i386 platforms to provide their own
> > HAL_MEM_REAL_REGION_TOP. Handy for i386 platforms that don't have a BIOS.
> 
> Er, maybe im missing something here....  Why cannot a platform provide
> its own implementation of hal_i386_mem_real_region_top()?

It's been a while since we did this. I believe it is because the pcmb
package already provides it and our platforms are based on pcmb in the
same way that the pc platform is. Some of them differ in this detail
since they don't have a BIOS. This patch allows the platform to override
the pcmb provided default.

Perhaps the correct solution is to move hal_i386_mem_real_region_top
from pcmb to the pc package? Or a weak symbol in pcmb (not too sure how
those work though).

Ian.

-- 
Ian Campbell, Senior Design Engineer
                                        Web: http://www.arcom.com
Arcom, Clifton Road,                    Direct: +44 (0)1223 403 465
Cambridge CB1 7EA, United Kingdom       Phone:  +44 (0)1223 411 200


_____________________________________________________________________
The message in this transmission is sent in confidence for the attention of the addressee only and should not be disclosed to any other party. Unauthorised recipients are requested to preserve this confidentiality. Please advise the sender if the addressee is not resident at the receiving end.  Email to and from Arcom is automatically monitored for operational and lawful business reasons.

This message has been virus scanned by MessageLabs.


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