mbox series

[v1,0/4] clk: Add kstrdup_and_replace() helper and use it

Message ID 20230628153211.52988-1-andriy.shevchenko@linux.intel.com
Headers show
Series clk: Add kstrdup_and_replace() helper and use it | expand

Message

Andy Shevchenko June 28, 2023, 3:32 p.m. UTC
There are a few existing users and more might come which would like
to have the kstrdup_and_replace() functionality.

Provide this new API and reuse it in a few users.

Since most of that is under CCF, perhaps it makes sense to route it
via that tree.

Andy Shevchenko (4):
  lib/string_helpers: Add kstrdup_and_replace() helper
  driver core: Replace kstrdup() + strreplace() with
    kstrdup_and_replace()
  clk: tegra: Replace kstrdup() + strreplace() with
    kstrdup_and_replace()
  clk: ti: Replace kstrdup() + strreplace() with kstrdup_and_replace()

 drivers/base/core.c            |  5 ++---
 drivers/clk/tegra/clk.c        |  6 ++----
 drivers/clk/ti/clk.c           |  4 ++--
 drivers/clk/ti/clkctrl.c       |  9 +++++----
 include/linux/string_helpers.h |  2 ++
 lib/string_helpers.c           | 15 +++++++++++++++
 6 files changed, 28 insertions(+), 13 deletions(-)

Comments

Andy Shevchenko July 20, 2023, 8:32 a.m. UTC | #1
On Wed, Jul 19, 2023 at 04:47:58PM -0700, Stephen Boyd wrote:
> Quoting Andy Shevchenko (2023-06-28 08:32:07)
> > There are a few existing users and more might come which would like
> > to have the kstrdup_and_replace() functionality.
> > 
> > Provide this new API and reuse it in a few users.
> > 
> > Since most of that is under CCF, perhaps it makes sense to route it
> > via that tree.
> 
> I'm happy to pick it up if you can get an ack from GregKH on the driver
> core bits.

It's also an option to skip that patch.
Greg, can you Ack the patch 2 in this series?