On Mon, Feb 24, 2014 at 11:25:12AM +0100, Gerd Hoffmann wrote: > -static acpi_status vmbus_walk_resources(struct acpi_resource *res, void *irq) > +static acpi_status vmbus_walk_resources(struct acpi_resource *res, void *ctx) > { > + switch (res->type) { > + case ACPI_RESOURCE_TYPE_IRQ: > + irq = res->data.irq.interrupts[0]; > > - if (res->type == ACPI_RESOURCE_TYPE_IRQ) { > - struct acpi_resource_irq *irqp; > - irqp = &res->data.irq; > - > - *((unsigned int *)irq) = irqp->interrupts[0]; Add a /* fall through */ comment here. > + case ACPI_RESOURCE_TYPE_ADDRESS64: > + hyperv_mmio_start = res->data.address64.minimum; > + hyperv_mmio_size = res->data.address64.address_length; > } regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel