| Message ID | 1462099023-11819-2-git-send-email-yamada.masahiro@socionext.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Sunday 01 May 2016 04:06 PM, Masahiro Yamada wrote: > Add a device managed API for reset_controller_register(). > > This helps in reducing code in .remove callbacks and sometimes > dropping .remove callbacks entirely. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > I liked it. Acked-by: Laxman Dewangan <ldewangan@nvidia.com>
Hi Masahiro, Am Sonntag, den 01.05.2016, 19:36 +0900 schrieb Masahiro Yamada: > Add a device managed API for reset_controller_register(). > > This helps in reducing code in .remove callbacks and sometimes > dropping .remove callbacks entirely. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> Thank you for these patches. Except for the issue with the lpc18xx patch they all look good to me. If you don't mind, I'll drop the lpc18xx patch for now and apply the others. regards Philipp
2016-05-03 19:17 GMT+09:00 Philipp Zabel <p.zabel@pengutronix.de>: > Hi Masahiro, > > Am Sonntag, den 01.05.2016, 19:36 +0900 schrieb Masahiro Yamada: >> Add a device managed API for reset_controller_register(). >> >> This helps in reducing code in .remove callbacks and sometimes >> dropping .remove callbacks entirely. >> >> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > Thank you for these patches. Except for the issue with the lpc18xx patch > they all look good to me. > If you don't mind, I'll drop the lpc18xx patch for now and apply the > others. > It is OK with me, but I do not understand what is the problem. Could you answer my question in the 3/7 thread?
2016-05-03 19:26 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>: > 2016-05-03 19:17 GMT+09:00 Philipp Zabel <p.zabel@pengutronix.de>: >> Hi Masahiro, >> >> Am Sonntag, den 01.05.2016, 19:36 +0900 schrieb Masahiro Yamada: >>> Add a device managed API for reset_controller_register(). >>> >>> This helps in reducing code in .remove callbacks and sometimes >>> dropping .remove callbacks entirely. >>> >>> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> >> >> Thank you for these patches. Except for the issue with the lpc18xx patch >> they all look good to me. >> If you don't mind, I'll drop the lpc18xx patch for now and apply the >> others. >> > > It is OK with me, but I do not understand what is the problem. > > Could you answer my question in the 3/7 thread? Now I am convinced. I leave 3/7 to your decision.
Am Dienstag, den 03.05.2016, 20:41 +0900 schrieb Masahiro Yamada: > 2016-05-03 19:26 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>: > > 2016-05-03 19:17 GMT+09:00 Philipp Zabel <p.zabel@pengutronix.de>: > >> Hi Masahiro, > >> > >> Am Sonntag, den 01.05.2016, 19:36 +0900 schrieb Masahiro Yamada: > >>> Add a device managed API for reset_controller_register(). > >>> > >>> This helps in reducing code in .remove callbacks and sometimes > >>> dropping .remove callbacks entirely. > >>> > >>> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > >> > >> Thank you for these patches. Except for the issue with the lpc18xx patch > >> they all look good to me. > >> If you don't mind, I'll drop the lpc18xx patch for now and apply the > >> others. > >> > > > > It is OK with me, but I do not understand what is the problem. > > > > Could you answer my question in the 3/7 thread? > > Now I am convinced. > > I leave 3/7 to your decision. Ok, I applied all but 3/7 to my reset/next branch. thanks Philipp
diff --git a/Documentation/driver-model/devres.txt b/Documentation/driver-model/devres.txt index 108d455..5270435 100644 --- a/Documentation/driver-model/devres.txt +++ b/Documentation/driver-model/devres.txt @@ -340,6 +340,10 @@ REGULATOR devm_regulator_put() devm_regulator_register() +RESET + devm_reset_control_get() + devm_reset_controller_register() + SLAVE DMA ENGINE devm_acpi_dma_controller_register() diff --git a/drivers/reset/core.c b/drivers/reset/core.c index f15f150..181b05d 100644 --- a/drivers/reset/core.c +++ b/drivers/reset/core.c @@ -82,6 +82,43 @@ void reset_controller_unregister(struct reset_controller_dev *rcdev) } EXPORT_SYMBOL_GPL(reset_controller_unregister); +static void devm_reset_controller_release(struct device *dev, void *res) +{ + reset_controller_unregister(*(struct reset_controller_dev **)res); +} + +/** + * devm_reset_controller_register - resource managed reset_controller_register() + * @dev: device that is registering this reset controller + * @rcdev: a pointer to the initialized reset controller device + * + * Managed reset_controller_register(). For reset controllers registered by + * this function, reset_controller_unregister() is automatically called on + * driver detach. See reset_controller_register() for more information. + */ +int devm_reset_controller_register(struct device *dev, + struct reset_controller_dev *rcdev) +{ + struct reset_controller_dev **rcdevp; + int ret; + + rcdevp = devres_alloc(devm_reset_controller_release, sizeof(*rcdevp), + GFP_KERNEL); + if (!rcdevp) + return -ENOMEM; + + ret = reset_controller_register(rcdev); + if (!ret) { + *rcdevp = rcdev; + devres_add(dev, rcdevp); + } else { + devres_free(rcdevp); + } + + return ret; +} +EXPORT_SYMBOL_GPL(devm_reset_controller_register); + /** * reset_control_reset - reset the controlled device * @rstc: reset controller diff --git a/include/linux/reset-controller.h b/include/linux/reset-controller.h index a3a5bcd..a4eaf1c 100644 --- a/include/linux/reset-controller.h +++ b/include/linux/reset-controller.h @@ -51,4 +51,8 @@ struct reset_controller_dev { int reset_controller_register(struct reset_controller_dev *rcdev); void reset_controller_unregister(struct reset_controller_dev *rcdev); +struct device; +int devm_reset_controller_register(struct device *dev, + struct reset_controller_dev *rcdev); + #endif
Add a device managed API for reset_controller_register(). This helps in reducing code in .remove callbacks and sometimes dropping .remove callbacks entirely. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> --- Documentation/driver-model/devres.txt | 4 ++++ drivers/reset/core.c | 37 +++++++++++++++++++++++++++++++++++ include/linux/reset-controller.h | 4 ++++ 3 files changed, 45 insertions(+)