Sorry for the delay, this looks fine. Acked-by: Scott Murray <scott@xxxxxxxxxxxx> On Mon, Jan 30, 2012 at 5:00 PM, Danny Kukawka <danny.kukawka@xxxxxxxxx> wrote: > > Fix debug variable from module parameter to be really bool to > fix 'warning: return from incompatible pointer type'. > > Signed-off-by: Danny Kukawka <danny.kukawka@xxxxxxxxx> > --- > drivers/pci/hotplug/cpcihp_generic.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/pci/hotplug/cpcihp_generic.c b/drivers/pci/hotplug/cpcihp_generic.c > index fb3f846..81af764 100644 > --- a/drivers/pci/hotplug/cpcihp_generic.c > +++ b/drivers/pci/hotplug/cpcihp_generic.c > @@ -62,7 +62,7 @@ > #define warn(format, arg...) printk(KERN_WARNING "%s: " format "\n", MY_NAME , ## arg) > > /* local variables */ > -static int debug; > +static bool debug; > static char *bridge; > static u8 bridge_busnr; > static u8 bridge_slot; > -- > 1.7.7.3 > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html