Re: HEADS UP: riastradh-drm2 branch merged

[David Laight <david%l8s.co.uk@localhost>]

On Thu, Mar 20, 2014 at 06:27:55PM -0400, David H. Gutteridge wrote:
> On Tue, 18 Mar 2014 at 19:17:01, Taylor R Campbell wrote:
> From looking at the code, it's clear you're already aware of the
> issue, given your "XXX" notation.
> 
> error = bus_space_alloc(bst, start, 0xffffffffffffffffULL /* XXX */,
>           size, align, 0, 0, &resource->start, &resource->r_bsh);
> 
> I don't know if it's the preferred NetBSD way to handle this, but I'd
> be inclined to add a macro that defines that literal value differently
> depending on whether it's an amd64/i386 PAE build or a plain i386
> build.

        ((uint64_t)0xfffffffful << 32 | 0xffffffff)
works for both.

        David

-- 
David Laight: david%l8s.co.uk@localhost