mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Bjorn Helgaas <bjorn.helgaas@hp.com>
To: Corey Minyard <cminyard@mvista.com>
Cc: Peter Martuccelli <peterm@redhat.com>,
	len.brown@intel.com, akpm@osdl.com, linux-kernel@vger.kernel.org,
	minyard@mvista.com
Subject: Re: [PATCH 2.6.12.4] ACPI oops during ipmi_si driver init
Date: Wed, 17 Aug 2005 15:09:17 -0600	[thread overview]
Message-ID: <200508171509.17627.bjorn.helgaas@hp.com> (raw)
In-Reply-To: <43039E57.2020607@mvista.com>

On Wednesday 17 August 2005 2:30 pm, Corey Minyard wrote:
> Basically, the IPMI system interface needs information from a specific 
> IPMI table to know how to configure itself.  Those tables can reference 
> GPEs, so the driver can use those (though AFAIK it has never been tested).

The information in the SPMI table *should* also be in the ACPI
namespace.  In general, drivers should claim devices based on the
namespace, not based on tables like SPMI.  The tables are mainly
there for the times when you need a device before the ACPI namespace
is available.

drivers/serial/8250_pnp.c is a basic example of claiming PNP
devices.  In particular, see serial_pnp_probe(), which gets
called for every device with a PNP ID found in pnp_dev_table[].

drivers/serial/8250_acpi.c is an example of claiming a device
directly from the ACPI namespace.  It claims everything with
PNP ID "PNP0501".

If you need to handle GPEs, you probably would need the
8250_acpi.c style, since I don't think PNP can deal with
those.  You would use acpi_register_driver(), and pass it
an acpi_driver struct containing '.ids = "IPI0001"'.  The
add() function you supply will get called for every active
IPI0001 device in the namespace.  Use acpi_walk_resources()
on its _CRS to extract the I/O port or MMIO address of the
controller and its interrupt information.  If the _CRS
contains no interrupt information, look for a _GPE method.

      reply	other threads:[~2005-08-17 21:11 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-12 19:44 Peter Martuccelli
2005-08-15 22:13 ` Bjorn Helgaas
2005-08-16 20:50   ` Peter Martuccelli
2005-08-17 20:30     ` Corey Minyard
2005-08-17 21:09       ` Bjorn Helgaas [this message]

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=200508171509.17627.bjorn.helgaas@hp.com \
    --to=bjorn.helgaas@hp.com \
    --cc=akpm@osdl.com \
    --cc=cminyard@mvista.com \
    --cc=len.brown@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=minyard@mvista.com \
    --cc=peterm@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®