From: "John W. Linville" <linville@xxxxxxxxxxxxx> Date: Wed, 10 Feb 2010 13:58:16 -0500 > Please let me know if there are problems! I see one: > diff --git a/drivers/ssb/main.c b/drivers/ssb/main.c > index 5681ebe..03dfd27 100644 > --- a/drivers/ssb/main.c > +++ b/drivers/ssb/main.c > @@ -494,8 +494,7 @@ static int ssb_devices_register(struct ssb_bus *bus) > #endif > break; > case SSB_BUSTYPE_SDIO: > -#ifdef CONFIG_SSB_SDIO > - sdev->irq = bus->host_sdio->dev.irq; > +#ifdef CONFIG_SSB_SDIOHOST > dev->parent = &bus->host_sdio->dev; > #endif > break; The commit message says "fix a typo" but it's also removing the assignment of sdev->irq All the bus types assign the sdev->irq, so it looks like this was removed unintentionally. If it was intentional, it needs to be explained in the commit message or a code comment. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
