Re: [PATCH 2.6.12 1/1] pcdp: add PCDP pci interface support

From: Bjorn Helgaas <bjorn.helgaas_at_hp.com>
Date: 2005-04-26 01:26:06
On Saturday 23 April 2005 12:00 pm, Mark Maule wrote:
> Resend with changes per Bjorn Helgaas comments.  Changes from original:
> 
> + Change globals to vga_console_iobase/vga_console_membase.
> + Address style-related comments.

This looks good to me, except that I think we should remove the
EFI_PCDP references below.  The idea of supporting multiple VGA
devices is completely orthogonal to the PCDP.

> +#ifdef CONFIG_EFI_PCDP
> +unsigned long vga_console_iobase;
> +unsigned long vga_console_membase;
> +#endif

> - * videoram directly without any black magic.
> + * videoram directly without any black magic.  If we're in a PCDP
> + * environment, add in the vga mmio offset supplied by the table.
>   */
>  
> +#if defined(CONFIG_EFI_PCDP)
> +extern unsigned long vga_console_iobase;
> +extern unsigned long vga_console_membase;
> +
> +#define VGA_MAP_MEM(x)	((unsigned long) ioremap(vga_console_membase + (x), 0))
> +#else
>  #define VGA_MAP_MEM(x)	((unsigned long) ioremap((x), 0))
> +#endif
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Mon Apr 25 11:26:32 2005

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:37 EST