On Tue, Jan 2, 2018 at 3:44 AM, Srinivas Kandagatla <srinivas.kandagatla at linaro.org> wrote: > Thanks for the patch, > > On 01/01/18 23:22, Andrey Smirnov wrote: >> >> Introduce devm_nvmem_register()/devm_nvmem_unregister() to make >> .remove() unnecessary in trivial drivers. >> >> Cc: Srinivas Kandagatla <srinivas.kandagatla at linaro.org> >> Cc: Heiko Stuebner <heiko at sntech.de> >> Cc: Masahiro Yamada <yamada.masahiro at socionext.com> >> Cc: Carlo Caione <carlo at caione.org> >> Cc: Kevin Hilman <khilman at baylibre.com> >> Cc: Matthias Brugger <matthias.bgg at gmail.com> >> Cc: cphealy at gmail.com >> Cc: linux-kernel at vger.kernel.org >> Cc: linux-mediatek at lists.infradead.org >> Cc: linux-rockchip at lists.infradead.org >> Cc: linux-amlogic at lists.infradead.org >> Cc: linux-arm-kernel at lists.infradead.org >> Signed-off-by: Andrey Smirnov <andrew.smirnov at gmail.com> >> --- >> drivers/nvmem/core.c | 41 >> +++++++++++++++++++++++++++++++++++++++++ >> include/linux/nvmem-provider.h | 17 +++++++++++++++++ >> 2 files changed, 58 insertions(+) >> > Patch looks good, kernel doc for these exported symbols are missing, you > should add them in next version! > Good point! Will do in v3. Thanks, Andrey Smirnov