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: PC - larger PCI spaces


>>>>> Gary Thomas writes:

> On Wed, 2002-08-14 at 07:47, Mark Salter wrote:
>> >>>>> Gary Thomas writes:
>> 
>> > Index: hal/i386/pcmb/current/ChangeLog
>> > ===================================================================
>> > RCS file: /misc/cvsfiles/ecos/packages/hal/i386/pcmb/current/ChangeLog,v
>> > retrieving revision 1.15
>> > diff -u -5 -p -r1.15 ChangeLog
>> > --- hal/i386/pcmb/current/ChangeLog	23 May 2002 23:03:14 -0000	1.15
>> > +++ hal/i386/pcmb/current/ChangeLog	14 Aug 2002 12:51:42 -0000
>> > @@ -1,5 +1,11 @@
>> > +2002-08-14  Gary Thomas  <gthomas@ecoscentric.com>
>> > +2002-08-14  Drew Moseley <drew.moseley@intel.com>	
>> > +
>> > +	* cdl/hal_i386_pcmb.cdl: 
>> > +	* include/pcmb_io.h: Increase PCI space to include all possible
>> > +	busses, slots, etc.  Required because defaults are often too small.	
>> 
>> Wow. I wouldn't expect there to be too many systems with >7 PCI bridges
>> in them.
>> 
>> Anyway, I was thinking this might be better handled by cdl in io/pci.
>> Give the user a chance to enter an actual number and avoid the
>> overhead of running config cycles for busses which don't exist.
>> 
>> Just a thought.

> Which is why I left it where it was - local to the aberrant
> system and also why I made it conditional (normally off).
 
My point was that it would be nicer to have an io/pci cdl option setting
the default. With platform cdl or configtool user overriding it. That
way, you only need a cdl change at most for other platforms that may
have a large number of busses.

Then again, I don't feel strongly enough to actually implement that at
the moment. (I should have have thought of this when I added bridge
support way back when ;-)

--Mark


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