[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 06/14] i2c:pm_smbus: Add interrupt handling
From: |
minyard |
Subject: |
[Qemu-devel] [PATCH 06/14] i2c:pm_smbus: Add interrupt handling |
Date: |
Thu, 7 Dec 2017 15:46:13 -0600 |
From: Corey Minyard <address@hidden>
Add the necessary code so that interrupts actually work from
the pm_smbus device.
Signed-off-by: Corey Minyard <address@hidden>
Cc: Michael S. Tsirkin <address@hidden>
Cc: Paolo Bonzini <address@hidden>
---
hw/i2c/pm_smbus.c | 14 +++++++++++++-
hw/i2c/smbus_ich9.c | 17 +++++++++++++++++
include/hw/i2c/pm_smbus.h | 2 ++
3 files changed, 32 insertions(+), 1 deletion(-)
diff --git a/hw/i2c/pm_smbus.c b/hw/i2c/pm_smbus.c
index eb2df4d..2fb00d0 100644
--- a/hw/i2c/pm_smbus.c
+++ b/hw/i2c/pm_smbus.c
@@ -205,6 +205,12 @@ error:
return;
}
+static bool
+smb_irq_value(PMSMBus *s)
+{
+ return ((s->smb_stat & ~STS_HOST_BUSY) != 0) && (s->smb_ctl & CTL_INTREN);
+}
+
static void smb_ioport_writeb(void *opaque, hwaddr addr, uint64_t val,
unsigned width)
{
@@ -300,7 +306,9 @@ static void smb_ioport_writeb(void *opaque, hwaddr addr,
uint64_t val,
}
out:
- return;
+ if (s->set_irq) {
+ s->set_irq(s, smb_irq_value(s));
+ }
}
static uint64_t smb_ioport_readb(void *opaque, hwaddr addr, unsigned width)
@@ -352,6 +360,10 @@ static uint64_t smb_ioport_readb(void *opaque, hwaddr
addr, unsigned width)
SMBUS_DPRINTF("SMB readb port=0x%04" HWADDR_PRIx " val=0x%02x\n",
addr, val);
+ if (s->set_irq) {
+ s->set_irq(s, smb_irq_value(s));
+ }
+
return val;
}
diff --git a/hw/i2c/smbus_ich9.c b/hw/i2c/smbus_ich9.c
index 706b9ec..d029816 100644
--- a/hw/i2c/smbus_ich9.c
+++ b/hw/i2c/smbus_ich9.c
@@ -41,6 +41,8 @@
typedef struct ICH9SMBState {
PCIDevice dev;
+ bool irq_enabled;
+
PMSMBus smb;
} ICH9SMBState;
@@ -50,6 +52,7 @@ static const VMStateDescription vmstate_ich9_smbus = {
.minimum_version_id = 1,
.fields = (VMStateField[]) {
VMSTATE_PCI_DEVICE(dev, ICH9SMBState),
+ VMSTATE_BOOL(irq_enabled, ICH9SMBState),
VMSTATE_STRUCT(smb, ICH9SMBState, 1, pmsmb_vmstate, PMSMBus),
VMSTATE_END_OF_LIST()
}
@@ -111,11 +114,25 @@ static void ich9_smb_class_init(ObjectClass *klass, void
*data)
dc->user_creatable = false;
}
+static void ich9_smb_set_irq(PMSMBus *pmsmb, bool enabled)
+{
+ ICH9SMBState *s = pmsmb->opaque;
+
+ if (enabled == s->irq_enabled) {
+ return;
+ }
+
+ s->irq_enabled = enabled;
+ pci_set_irq(&s->dev, enabled);
+}
+
I2CBus *ich9_smb_init(PCIBus *bus, int devfn, uint32_t smb_io_base)
{
PCIDevice *d =
pci_create_simple_multifunction(bus, devfn, true,
TYPE_ICH9_SMB_DEVICE);
ICH9SMBState *s = ICH9_SMB_DEVICE(d);
+ s->smb.set_irq = ich9_smb_set_irq;
+ s->smb.opaque = s;
return s->smb.smbus;
}
diff --git a/include/hw/i2c/pm_smbus.h b/include/hw/i2c/pm_smbus.h
index b1e1970..cfe596f 100644
--- a/include/hw/i2c/pm_smbus.h
+++ b/include/hw/i2c/pm_smbus.h
@@ -23,6 +23,8 @@ typedef struct PMSMBus {
/* Set by the user. */
bool i2c_enable;
+ void (*set_irq)(struct PMSMBus *s, bool enabled);
+ void *opaque;
/* Internally used by pm_smbus. */
--
2.7.4
- [Qemu-devel] [PATCH 00/13] pm_smbus fixes and and IPMI SMBus device, minyard, 2017/12/07
- [Qemu-devel] [PATCH 02/14] i2c:pm_smbus: Fix the semantics of block I2C transfers, minyard, 2017/12/07
- [Qemu-devel] [PATCH 04/14] i2c:pm_smbus: Add block transfer capability, minyard, 2017/12/07
- [Qemu-devel] [PATCH 05/14] i2c:pm_smbus: Fix state transfer, minyard, 2017/12/07
- [Qemu-devel] [PATCH 06/14] i2c:pm_smbus: Add interrupt handling,
minyard <=
- [Qemu-devel] [PATCH 09/14] i2c: Add vmstate handling to the smbus eeprom, minyard, 2017/12/07
- [Qemu-devel] [PATCH 01/14] i2c:pm_smbus: Clean up some style issues, minyard, 2017/12/07
- [Qemu-devel] [PATCH 07/14] i2c:pm_smbus: Add the ability to force block transfer enable, minyard, 2017/12/07
- [Qemu-devel] [PATCH 03/14] i2c:pm_smbus: Make the I2C block read command read-only, minyard, 2017/12/07
- [Qemu-devel] [PATCH 10/14] ipmi: Add an SMBus IPMI interface, minyard, 2017/12/07
- [Qemu-devel] [PATCH 08/14] i2c: Add an SMBus vmstate structure, minyard, 2017/12/07
- [Qemu-devel] [PATCH 12/14] ipmi: Fix SSIF ACPI handling to use the right CRS, minyard, 2017/12/07
- [Qemu-devel] [PATCH 11/14] acpi: Add i2c serial bus CRS handling, minyard, 2017/12/07
- [Qemu-devel] [PATCH 13/14] i386: Rename bools in PCMachineState to end in _enabled, minyard, 2017/12/07
- [Qemu-devel] [PATCH 14/14] pc: Add an SMB0 ACPI device to q35, minyard, 2017/12/07