> +out_pdev: > + platform_device_put(pdev); > +out_gpio: > + gpio_remove(&iocled->chip); I just noticed that the prototype of gpio_remove has __must_check I don't think there is anything to check here; since the thing is not fully initialized here, it is unlikely to be busy. Should there be (void) in front of it? julia