[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <XNM1$9$0$4$$3$3$7$A$9002683U4519fb3c@hitachi.com>
Date: Wed, 27 Sep 2006 13:17:10 +0900
From: <eiichiro.oiwa.nm@...achi.com>
To: "Jesse Barnes" <jesse.barnes@...el.com>
Cc: <akpm@...l.org>, <tony.luck@...el.com>, <greg@...ah.com>,
<linux-ia64@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re[2]: [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for
embedded VG
>On Tuesday, September 26, 2006 12:42 am, eiichiro.oiwa.nm@...achi.com
>wrote:
>> To be compatible with Xorg's handling of PCI, we need pci_fixup_video on
>> IA64 platform like x86 platform. There are also machines, which have VGA
>> embedded into main board, among IA64 platform. Embedded VGA generally
>> don't have PCI ROM, and there are VGA ROM image in System BIOS.
>> Therefore, these machines need pci_fixup_video for the sysfs rom.
>> pci_fixup_video already exists in x86 Linux kernel. However since this
>> function doesn't exist in IA64 kernel, we could not run X server on IA64
>> box has embedded-VGA.
>>
>> I tested pci_fixup_video on IA64 box has embedded-VGA. I confirmed we
>> can read VGA BIOS from the sysfs rom regardless of embedded-VGA.
>
>Looks good, Eiichiro, thanks for posting this.
>
>> +#include <linux/delay.h>
>> +#include <linux/dmi.h>
>> +#include <linux/pci.h>
>> +#include <linux/init.h>
>
>For this version, I don't think you need delay.h or dmi.h. And like Bjorn
>mentioned, this could probably be turned into generic code in drivers/pci
>so we don't have too much duplication with x86 (and like I mentioned,
>x86_64 could probably use this too).
>
>Jesse
>
"PCI-to-PCI Bridge Architecture Specification" describes how to support VGA.
And pci_fixup_video suits this specification because this function checks the
Bridge Control register. I also think pci_fixup_video should be turned into
generic quirks.c in drivers/pci.
Ok, I will modify code and test.
Thanks,
Eiichiro
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists