All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jesse Barnes <jesse.barnes@intel.com>
To: Bjorn Helgaas <bjorn.helgaas@hp.com>
Cc: eiichiro.oiwa.nm@hitachi.com, akpm@osdl.org, tony.luck@intel.com,
	greg@kroah.com, linux-ia64@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for embedded VGA
Date: Tue, 26 Sep 2006 15:48:53 +0000	[thread overview]
Message-ID: <200609260848.53556.jesse.barnes@intel.com> (raw)
In-Reply-To: <200609260909.12876.bjorn.helgaas@hp.com>

On Tuesday, September 26, 2006 8:09 am, Bjorn Helgaas wrote:
> On Tuesday 26 September 2006 01:42, eiichiro.oiwa.nm@hitachi.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.
>
> What other architectures will need this?  There's nothing ia64-specific
> in the patch below.  Can it be put somewhere more generic?

It could go into drivers/pci, but setting the flag implies that the ROM is 
at 0xc0000, so it does have some arch dependencies (though at least x86, 
x86_64 and ia64 have machines that do this).

Jesse

WARNING: multiple messages have this Message-ID (diff)
From: Jesse Barnes <jesse.barnes@intel.com>
To: Bjorn Helgaas <bjorn.helgaas@hp.com>
Cc: eiichiro.oiwa.nm@hitachi.com, akpm@osdl.org, tony.luck@intel.com,
	greg@kroah.com, linux-ia64@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for embedded VGA
Date: Tue, 26 Sep 2006 08:48:53 -0700	[thread overview]
Message-ID: <200609260848.53556.jesse.barnes@intel.com> (raw)
In-Reply-To: <200609260909.12876.bjorn.helgaas@hp.com>

On Tuesday, September 26, 2006 8:09 am, Bjorn Helgaas wrote:
> On Tuesday 26 September 2006 01:42, eiichiro.oiwa.nm@hitachi.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.
>
> What other architectures will need this?  There's nothing ia64-specific
> in the patch below.  Can it be put somewhere more generic?

It could go into drivers/pci, but setting the flag implies that the ROM is 
at 0xc0000, so it does have some arch dependencies (though at least x86, 
x86_64 and ia64 have machines that do this).

Jesse

  reply	other threads:[~2006-09-26 15:48 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200609260605.k8Q65dAc005670@fire-2.osdl.org>
2006-09-26  7:35 ` [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for eiichiro.oiwa.nm
2006-09-26  7:35   ` [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for embedded VGA eiichiro.oiwa.nm
2006-09-26  7:42   ` [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for eiichiro.oiwa.nm
2006-09-26  7:42     ` [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for embedded VGA eiichiro.oiwa.nm
2006-09-26 15:09     ` Bjorn Helgaas
2006-09-26 15:09       ` Bjorn Helgaas
2006-09-26 15:48       ` Jesse Barnes [this message]
2006-09-26 15:48         ` Jesse Barnes
2006-09-26 15:50     ` Jesse Barnes
2006-09-26 15:50       ` Jesse Barnes
2006-09-27  4:17       ` Re[2]: [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for eiichiro.oiwa.nm
2006-09-27  4:17         ` Re[2]: [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for embedded VG eiichiro.oiwa.nm
2006-09-27 11:23   ` [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for eiichiro.oiwa.nm
2006-09-27 11:23     ` [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for embedded VGA eiichiro.oiwa.nm
2006-09-27 11:48     ` [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for Alan Cox
2006-09-27 12:12       ` [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel for embedded VGA Alan Cox
2006-09-28  5:03       ` Re[2]: [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel eiichiro.oiwa.nm
2006-09-28  5:03         ` Re[2]: [PATCH 2.6.18] IA64: Add pci_fixup_video into IA64 kernel forembedded VGA eiichiro.oiwa.nm

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200609260848.53556.jesse.barnes@intel.com \
    --to=jesse.barnes@intel.com \
    --cc=akpm@osdl.org \
    --cc=bjorn.helgaas@hp.com \
    --cc=eiichiro.oiwa.nm@hitachi.com \
    --cc=greg@kroah.com \
    --cc=linux-ia64@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tony.luck@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.