On Wed, May 11, 2016 at 02:46:06PM -0500, miny...@acm.org wrote: > From: Corey Minyard <cminy...@mvista.com> > > Signed-off-by: Corey Minyard <cminy...@mvista.com> > --- > hw/ipmi/smbus_ipmi.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/hw/ipmi/smbus_ipmi.c b/hw/ipmi/smbus_ipmi.c > index 4e7203b..3a34aaf 100644 > --- a/hw/ipmi/smbus_ipmi.c > +++ b/hw/ipmi/smbus_ipmi.c > @@ -167,6 +167,7 @@ static void smbus_ipmi_realize(DeviceState *dev, Error > **errp) > sid->fwinfo.base_address = sid->parent.i2c.address; > sid->fwinfo.memspace = IPMI_MEMSPACE_SMBUS; > sid->fwinfo.register_spacing = 1; > + sid->fwinfo.acpi_parent = "\\_SB.PCI0.SMB0";
I don't think it's a good idea to spread things like PCI0 outside acpi-build.c. Why do you want to pass in the path at all? > ipmi_add_fwinfo(&sid->fwinfo, errp); > } > > -- > 2.7.4