Hi,> err = devm_add_action(sfp->dev, sfp_cleanup, sfp);> - if (err < 0)> + if (err < 0) {> + sfp_cleanup(sfp);> return err;> + }Better use devm_add_action_or_reset(), no?-michael