Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 9 Jan 2001 14:39:28 -0800 (PST)
From:      Matthew Jacob <mjacob@feral.com>
To:        Mike Smith <msmith@FreeBSD.org>
Cc:        Matt Jacob <mjacob@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/alpha/include bus.h 
Message-ID:  <Pine.LNX.4.21.0101091439110.28132-100000@zeppo.feral.com>
In-Reply-To: <200101092237.f09MbvJ01112@mass.osd.bsdi.com>

next in thread | previous in thread | raw e-mail | index | archive | help


> > mjacob      2001/01/09 10:17:49 PST
> > 
> >   Modified files:
> >     sys/alpha/include    bus.h 
> >   Log:
> >   The alpha architecture is 64 bits, so bus_addr_t && bus_size_t should really
> >   be 64 bits wide. The largest known current actual physical implementation
> >   is 40 bits, so BUS_SPACE_MAXADDR should reflect this. It also seems to
> >   me that BUS_SPACE_UNRESTRICTED should b ~0UL, not ~0.
> 
> Where a constant is of a known type, one should avoid "UL" in favour of 
> the real type, eg. (~(bus_addr_t)0).  There's too much "long" in the bus 
> code already. 8(
> 
> All IMO.

Sure- it's kinda all the same to me. Knock yerself out!



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.LNX.4.21.0101091439110.28132-100000>