[0/4] drivers/perf: Constify static struct attribute_group

Message

Rikard Falkeborn Jan. 17, 2021, 9:28 p.m. UTC

Thie series makes a number of static struct attribute_group const. The
only usage of the structs is to put their address in an array of pointers
to const struct * attribute_group. With this series applied, all but two
static struct attribute_group in drivers/perf are const (and the two
remaining are modified at runtime and can't be const).

Patches are independent and split based on output from get_maintainers.pl.
I can of course split differently if that's desired.

Done with the help of coccinelle.

Rikard Falkeborn (4):
  perf: qcom: Constify static struct attribute_group
  perf/imx_ddr: Constify static struct attribute_group
  perf: hisi: Constify static struct attribute_group
  perf: Constify static struct attribute_group

 drivers/perf/arm-cci.c                        |  2 +-
 drivers/perf/arm-cmn.c                        |  2 +-
 drivers/perf/arm_dmc620_pmu.c                 |  4 ++--
 drivers/perf/arm_pmu.c                        |  2 +-
 drivers/perf/arm_smmuv3_pmu.c                 |  8 ++++----
 drivers/perf/arm_spe_pmu.c                    |  6 +++---
 drivers/perf/fsl_imx8_ddr_perf.c              | 10 +++++-----
 drivers/perf/hisilicon/hisi_uncore_ddrc_pmu.c |  2 +-
 drivers/perf/hisilicon/hisi_uncore_hha_pmu.c  |  2 +-
 drivers/perf/hisilicon/hisi_uncore_l3c_pmu.c  |  2 +-
 drivers/perf/qcom_l2_pmu.c                    |  6 +++---
 drivers/perf/qcom_l3_pmu.c                    |  6 +++---
 12 files changed, 26 insertions(+), 26 deletions(-)

Comments

Will Deacon Jan. 20, 2021, 6:13 p.m. UTC | #1

On Sun, 17 Jan 2021 22:28:43 +0100, Rikard Falkeborn wrote:
> Thie series makes a number of static struct attribute_group const. The
> only usage of the structs is to put their address in an array of pointers
> to const struct * attribute_group. With this series applied, all but two
> static struct attribute_group in drivers/perf are const (and the two
> remaining are modified at runtime and can't be const).
> 
> Patches are independent and split based on output from get_maintainers.pl.
> I can of course split differently if that's desired.
> 
> [...]

Applied to will (for-next/perf), thanks!

[1/4] perf: qcom: Constify static struct attribute_group
      https://git.kernel.org/will/c/30b34c4833ea
[2/4] perf/imx_ddr: Constify static struct attribute_group
      https://git.kernel.org/will/c/3cb7d2da183f
[3/4] perf: hisi: Constify static struct attribute_group
      https://git.kernel.org/will/c/c2c4d5c051b2
[4/4] perf: Constify static struct attribute_group
      https://git.kernel.org/will/c/f0c140481d1b

Cheers,

[e-mail] March 1, 2021, 7:59 p.m. UTC | #2

Hello:

This series was applied to qcom/linux.git (refs/heads/for-next):

On Sun, 17 Jan 2021 22:28:43 +0100 you wrote:
> Thie series makes a number of static struct attribute_group const. The
> only usage of the structs is to put their address in an array of pointers
> to const struct * attribute_group. With this series applied, all but two
> static struct attribute_group in drivers/perf are const (and the two
> remaining are modified at runtime and can't be const).
> 
> Patches are independent and split based on output from get_maintainers.pl.
> I can of course split differently if that's desired.
> 
> [...]

Here is the summary with links:
  - [1/4] perf: qcom: Constify static struct attribute_group
    https://git.kernel.org/qcom/c/30b34c4833ea
  - [2/4] perf/imx_ddr: Constify static struct attribute_group
    https://git.kernel.org/qcom/c/3cb7d2da183f
  - [3/4] perf: hisi: Constify static struct attribute_group
    https://git.kernel.org/qcom/c/c2c4d5c051b2
  - [4/4] perf: Constify static struct attribute_group
    https://git.kernel.org/qcom/c/f0c140481d1b

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html