Re: [Xen-merge] [PATCH] merge i386's agp.h

[Christian Limpach <Christian.Limpach@xxxxxxxxxxxx>]

On Tue, Dec 20, 2005 at 03:15:00PM +0100, Jan Beulich wrote:
> Subject says it all.

How about having a mach-{default,xen}/mach_agp.h file included at the top
of agp.h with:

#define phys_to_gart(x) phys_to_machine(x)
#define gart_to_phys(x) machine_to_phys(x)
#define HAVE_MACH_PHYS_GART_MACROS

and then in asm-i386/agp.h:
/* Convert a physical address to an address suitable for the GART. */
#ifndef HAVE_MACH_PHYS_GART_MACROS
#define phys_to_gart(x) (x)
#define gart_to_phys(x) (x)
#endif

Same for the {alloc,free}_gatt_pages macros.

That would be my preference since it preserves the default definitions
in the regular file moving the overrides into machine specific files.

    christian


_______________________________________________
Xen-merge mailing list
Xen-merge@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-merge