mbox series

[v2,0/6] Introduce devm_kmemdup_array() helper

Message ID 20241126172240.6044-1-raag.jadav@intel.com (mailing list archive)
Headers show
Series Introduce devm_kmemdup_array() helper | expand

Message

Raag Jadav Nov. 26, 2024, 5:22 p.m. UTC
This series introduces a more robust and cleaner devm_kmemdup_array()
helper and uses it across drivers.

v2: Use size_mul() for multiplication (Dmitry)
    Update commit message (Dmitry)

Raag Jadav (6):
  devres: Introduce devm_kmemdup_array()
  pinctrl: intel: copy communities using devm_kmemdup_array()
  pinctrl: tangier: use devm_kmemdup_array()
  pinctrl: pxa2xx: use devm_kmemdup_array()
  input: sparse-keymap: use devm_kmemdup_array()
  ASoC: Intel: avs: use devm_kmemdup_array()

 drivers/input/sparse-keymap.c           | 3 +--
 drivers/pinctrl/intel/pinctrl-intel.c   | 6 ++----
 drivers/pinctrl/intel/pinctrl-tangier.c | 5 ++---
 drivers/pinctrl/pxa/pinctrl-pxa2xx.c    | 8 ++++----
 include/linux/device.h                  | 5 +++++
 sound/soc/intel/avs/boards/da7219.c     | 3 ++-
 sound/soc/intel/avs/boards/es8336.c     | 3 ++-
 sound/soc/intel/avs/boards/nau8825.c    | 3 ++-
 sound/soc/intel/avs/boards/rt274.c      | 3 ++-
 sound/soc/intel/avs/boards/rt286.c      | 3 ++-
 sound/soc/intel/avs/boards/rt298.c      | 3 ++-
 sound/soc/intel/avs/boards/rt5663.c     | 3 ++-
 sound/soc/intel/avs/boards/rt5682.c     | 2 +-
 13 files changed, 29 insertions(+), 21 deletions(-)

Comments

Andy Shevchenko Nov. 26, 2024, 10:40 p.m. UTC | #1
On Tue, Nov 26, 2024 at 10:52:34PM +0530, Raag Jadav wrote:
> This series introduces a more robust and cleaner devm_kmemdup_array()
> helper and uses it across drivers.


I believe the best is to route this via Intel pin control tree, but we need
an Ack from Greg and Dmitry. Or share your vision on this.

> Raag Jadav (6):
>   devres: Introduce devm_kmemdup_array()
>   pinctrl: intel: copy communities using devm_kmemdup_array()
>   pinctrl: tangier: use devm_kmemdup_array()
>   pinctrl: pxa2xx: use devm_kmemdup_array()
>   input: sparse-keymap: use devm_kmemdup_array()
>   ASoC: Intel: avs: use devm_kmemdup_array()
> 
>  drivers/input/sparse-keymap.c           | 3 +--
>  drivers/pinctrl/intel/pinctrl-intel.c   | 6 ++----
>  drivers/pinctrl/intel/pinctrl-tangier.c | 5 ++---
>  drivers/pinctrl/pxa/pinctrl-pxa2xx.c    | 8 ++++----
>  include/linux/device.h                  | 5 +++++
>  sound/soc/intel/avs/boards/da7219.c     | 3 ++-
>  sound/soc/intel/avs/boards/es8336.c     | 3 ++-
>  sound/soc/intel/avs/boards/nau8825.c    | 3 ++-
>  sound/soc/intel/avs/boards/rt274.c      | 3 ++-
>  sound/soc/intel/avs/boards/rt286.c      | 3 ++-
>  sound/soc/intel/avs/boards/rt298.c      | 3 ++-
>  sound/soc/intel/avs/boards/rt5663.c     | 3 ++-
>  sound/soc/intel/avs/boards/rt5682.c     | 2 +-
>  13 files changed, 29 insertions(+), 21 deletions(-)
Dmitry Torokhov Nov. 27, 2024, 1:37 a.m. UTC | #2
On Wed, Nov 27, 2024 at 12:40:18AM +0200, Andy Shevchenko wrote:
> On Tue, Nov 26, 2024 at 10:52:34PM +0530, Raag Jadav wrote:
> > This series introduces a more robust and cleaner devm_kmemdup_array()
> > helper and uses it across drivers.
> 
> 
> I believe the best is to route this via Intel pin control tree, but we need
> an Ack from Greg and Dmitry. Or share your vision on this.

Sounds good to me.

Thanks.
Linus Walleij Nov. 29, 2024, 10:17 a.m. UTC | #3
On Tue, Nov 26, 2024 at 6:22 PM Raag Jadav <raag.jadav@intel.com> wrote:

> This series introduces a more robust and cleaner devm_kmemdup_array()
> helper and uses it across drivers.

For the series:
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

It seems like Andy will push it to me which is excellent.

Yours,
Linus Walleij
Andy Shevchenko Nov. 29, 2024, 2:20 p.m. UTC | #4
On Fri, Nov 29, 2024 at 11:17:02AM +0100, Linus Walleij wrote:
> On Tue, Nov 26, 2024 at 6:22 PM Raag Jadav <raag.jadav@intel.com> wrote:
> 
> > This series introduces a more robust and cleaner devm_kmemdup_array()
> > helper and uses it across drivers.
> 
> For the series:
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
> 
> It seems like Andy will push it to me which is excellent.

Yep, that's the plan after we get all necessary ACKs.