public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
* RE: [ACPI] acpi_irq_to_vector()
@ 2003-06-23  6:02 Yu, Luming
       [not found] ` <3ACA40606221794F80A5670F0AF15F8427237D-4yWAQGcml64gGBtAFL8yw7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: Yu, Luming @ 2003-06-23  6:02 UTC (permalink / raw)
  To: Matthew Wilcox
  Cc: acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
	linux-ia64-WygotPe7DYIabbyE177sbg, Grover, Andrew,
	Mallick, Asit K, Brown, Len

>For PCI devices, we just read the irq out of the struct pci_dev.
>The platform code gets the opportunity to mangle the irq in whatever
>way it chooses before the pci_dev is passed to the driver.  Can we do
>something similar for ACPI instead?

The acpi_irq_to_vector is called in acpi_os_install_interrupt_handler which is in acpi_init that is before pci_init get called.
So I don't think it is reasonable to use pci_dev in acpi_init. 


-- 
"It's not Hollywood.  War is real, war is primarily not about defeat or
victory, it is about death.  I've seen thousands and thousands of dead bodies.
Do you think I want to have an academic debate on this subject?" -- Robert Fisk


-------------------------------------------------------
This SF.NET email is sponsored by: eBay
Great deals on office technology -- on eBay now! Click here:
http://adfarm.mediaplex.com/ad/ck/711-11697-6916-5
_______________________________________________
Acpi-devel mailing list
Acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/acpi-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: RE: [ACPI] acpi_irq_to_vector()
       [not found] ` <3ACA40606221794F80A5670F0AF15F8427237D-4yWAQGcml64gGBtAFL8yw7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
@ 2003-06-23  7:13   ` Grant Grundler
  0 siblings, 0 replies; 3+ messages in thread
From: Grant Grundler @ 2003-06-23  7:13 UTC (permalink / raw)
  To: Yu, Luming
  Cc: Matthew Wilcox, acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
	linux-ia64-WygotPe7DYIabbyE177sbg, Grover, Andrew,
	Mallick, Asit K, Brown, Len

On Mon, Jun 23, 2003 at 02:02:08PM +0800, Yu, Luming wrote:
> So I don't think it is reasonable to use pci_dev in acpi_init. 

That's not what willy was proposing.
I understood he was proposing a similar scheme; not to use pci_dev.

grant

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: RE: [ACPI] acpi_irq_to_vector()
       [not found] ` <3ACA40606221794F80A5670F0AF15F8427237E-4yWAQGcml64gGBtAFL8yw7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
@ 2003-06-23 18:47   ` David Mosberger
  0 siblings, 0 replies; 3+ messages in thread
From: David Mosberger @ 2003-06-23 18:47 UTC (permalink / raw)
  To: Yu, Luming
  Cc: Matthew Wilcox, acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
	linux-ia64-WygotPe7DYIabbyE177sbg, Grover, Andrew,
	Mallick, Asit K, Brown, Len

>>>>> On Mon, 23 Jun 2003 14:15:07 +0800, "Yu, Luming" <luming.yu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> said:

  Linux> To remove the ugly ifdef in the common part of ACPI code, I
  Linux> think the simple way is to define a acpi_irq_to_vector in
  Linux> arch/i386/acpi which only return acpi_fadt.sci_int.  Or
  Linux> someone can invent a method that can return correct value for
  Linux> each platform.

Shouldn't this be called acpi_gsi_to_irq(), though?

	--david

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2003-06-23 18:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-06-23  6:02 [ACPI] acpi_irq_to_vector() Yu, Luming
     [not found] ` <3ACA40606221794F80A5670F0AF15F8427237D-4yWAQGcml64gGBtAFL8yw7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
2003-06-23  7:13   ` Grant Grundler
  -- strict thread matches above, loose matches on Subject: below --
2003-06-23  6:15 Yu, Luming
     [not found] ` <3ACA40606221794F80A5670F0AF15F8427237E-4yWAQGcml64gGBtAFL8yw7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
2003-06-23 18:47   ` David Mosberger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox