Messages in this thread | | | Date | Fri, 09 Mar 2012 16:00:25 +0800 | From | "Ying-Chun Liu (PaulLiu)" <> | Subject | Re: [PATCH v10] Regulator: Add Anatop regulator driver |
| |
(2012年03月08日 22:18), Jean-Christophe PLAGNIOL-VILLARD wrote: >> +static int __devexit anatop_regulator_remove(struct platform_device *pdev) >> +{ >> + struct regulator_dev *rdev = platform_get_drvdata(pdev); >> + struct anatop_regulator *sreg = rdev_get_drvdata(rdev); >> + kfree(sreg->name); >> + regulator_unregister(rdev); > return from the unregister >> + return 0; >> +} > Best Regards, > J.
Hi Jean-Christophe,
I've modify the patch based on your review. However, the last one cannot be made because regulator_unregister is void return.
Yours Sincerely, Paul -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |