On Wed, May 11, 2016 at 02:46:05PM -0500, miny...@acm.org wrote: > From: Corey Minyard <cminy...@mvista.com> > > Signed-off-by: Corey Minyard <cminy...@mvista.com> > --- > hw/acpi/ipmi.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/hw/acpi/ipmi.c b/hw/acpi/ipmi.c > index 731f4ad..c187fdd 100644 > --- a/hw/acpi/ipmi.c > +++ b/hw/acpi/ipmi.c > @@ -49,7 +49,9 @@ static Aml *aml_ipmi_crs(IPMIFwInfo *info) > regspacing, info->register_length)); > break; > case IPMI_MEMSPACE_SMBUS: > - aml_append(crs, aml_return(aml_int(info->base_address))); > + aml_append(crs, aml_i2c_serial_bus_device(0, 100000, > + info->base_address, > + info->acpi_parent));
Isn't this fairly new? If so using these opcodes is likely to break some older guests. Maybe they already don't work, but I'd like to see some explanation about that, and what was tested. > break; > default: > abort(); > -- > 2.7.4
