On Thu, 2010-03-04 at 11:44 +0800, Myron Stowe wrote: > Convert PNP patch (git 9e368fa011d4e0aa050db348d69514900520e40b) to > maintain a pointer to a PNP device, 'pnp_dev', instead of the ACPI > device, 'acpi_dev', that is currently being tracked with PNP based > IPMI device discovery. Hi, Myron Now the IPMI interface defined in ACPI namespace is detected by using PNP device driver. But it seems that this still belongs to ACPI detection mechanism. At the same time IPMI device defined in ACPI namespace is also used to enable the communication between ACPI and IPMI device. In such case it will be better to know whether the IPMI interface is discovered by using ACPI detection mechanism , which can be realized by comparing the info->dev with acpi_dev->dev. Can we still use the acpi_dev->dev to track the IPMI device discovery? thanks. Yakui > > Signed-off-by: Myron Stowe <myron.stowe@xxxxxx> > --- > > drivers/char/ipmi/ipmi_si_intf.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c > index 806ae83..37c6912 100644 > --- a/drivers/char/ipmi/ipmi_si_intf.c > +++ b/drivers/char/ipmi/ipmi_si_intf.c > @@ -1934,7 +1934,7 @@ static int __devinit ipmi_pnp_probe(struct pnp_dev *dev, > info->irq_setup = std_irq_setup; > } > > - info->dev = &acpi_dev->dev; > + info->dev = &dev->dev; > pnp_set_drvdata(dev, info); > > return try_smi_init(info); > > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html