mbox series

[v2,00/24] mfd: Constify read-only regmap structs

Message ID 20240704-mfd-const-regmap_config-v2-0-0c8785b1331d@gmail.com
Headers show
Series mfd: Constify read-only regmap structs | expand

Message

Javier Carrasco July 4, 2024, 5:23 p.m. UTC
This series adds the const modifier to the remaining regmap_bus,
regmap_config, regmap_irq and regmap_irq_chip structs within mfd
that are effectively used as const (i.e., only read after their
declaration), but kept as writtable data.

Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>

Changes in v2:
- Cover regmap_bus, regmap_irq and regmap_irq_chip too.
- Link to v1: https://lore.kernel.org/r/20240703-mfd-const-regmap_config-v1-0-aa6cd00a03bd@gmail.com

---
Javier Carrasco (24):
      mfd: da9062-core: Constify read-only regmap structs
      mfd: fsl-imx25-tsadc: Constify struct regmap_config
      mfd: hi655x-pmic: Constify struct regmap_config
      mfd: wcd934x: Constify struct regmap_config
      mfd: tps6105x: Constify struct regmap_config
      mfd: rohm-bd9576: Constify read-only regmap structs
      mfd: intel-m10-bmc: Constify struct regmap_config
      mfd: 88pm80x: Constify read-only regmap structs
      mfd: bd9571mwv: Constify struct regmap_irq_chip
      mfd: intel_soc_pmic_bxtwc: Constify struct regmap_irq_chip
      mfd: retu: Constify read-only regmap structs
      mfd: rk8xx-core: Constify struct regmap_irq_chip
      mfd: rohm-bd71828: Constify read-only regmap structs
      mfd: rohm-bd718x7: Constify struct regmap_irq_chip
      mfd: tps65086: Constify struct regmap_irq_chip
      mfd: tps65090: Constify struct regmap_irq_chip
      mfd: tps65218: Constify struct regmap_irq_chip
      mfd: tps65219: Constify read-only regmap structs
      mfd: tps65910: Constify struct regmap_irq_chip
      mfd: tps65912: Constify struct regmap_irq_chip
      mfd: twl6040: Constify struct regmap_irq_chip
      mfd: gateworks-gsc: Constify struct regmap_bus
      mfd: mc13xxx-spi: Constify struct regmap_bus
      mfd: sprd-sc27xx-spi: Constify struct regmap_bus

 drivers/mfd/88pm800.c              |  2 +-
 drivers/mfd/88pm805.c              |  4 ++--
 drivers/mfd/bd9571mwv.c            |  4 ++--
 drivers/mfd/da9062-core.c          | 12 ++++++------
 drivers/mfd/fsl-imx25-tsadc.c      |  2 +-
 drivers/mfd/gateworks-gsc.c        |  2 +-
 drivers/mfd/hi655x-pmic.c          |  2 +-
 drivers/mfd/intel-m10-bmc-pmci.c   |  2 +-
 drivers/mfd/intel-m10-bmc-spi.c    |  2 +-
 drivers/mfd/intel_soc_pmic_bxtwc.c | 14 +++++++-------
 drivers/mfd/mc13xxx-spi.c          |  2 +-
 drivers/mfd/retu-mfd.c             | 12 ++++++------
 drivers/mfd/rk8xx-core.c           |  6 +++---
 drivers/mfd/rohm-bd71828.c         |  8 ++++----
 drivers/mfd/rohm-bd718x7.c         |  2 +-
 drivers/mfd/rohm-bd9576.c          |  6 +++---
 drivers/mfd/sprd-sc27xx-spi.c      |  2 +-
 drivers/mfd/tps6105x.c             |  2 +-
 drivers/mfd/tps65086.c             |  2 +-
 drivers/mfd/tps65090.c             |  2 +-
 drivers/mfd/tps65218.c             |  2 +-
 drivers/mfd/tps65219.c             |  4 ++--
 drivers/mfd/tps65910.c             |  6 +++---
 drivers/mfd/tps65912-core.c        |  2 +-
 drivers/mfd/twl6040.c              |  2 +-
 drivers/mfd/wcd934x.c              |  2 +-
 include/linux/mfd/88pm80x.h        |  2 +-
 27 files changed, 55 insertions(+), 55 deletions(-)
---
base-commit: 0b58e108042b0ed28a71cd7edf5175999955b233
change-id: 20240703-mfd-const-regmap_config-96da5e18a85f

Best regards,

Comments

Matti Vaittinen July 5, 2024, 5:47 a.m. UTC | #1
On 7/4/24 20:23, Javier Carrasco wrote:
> The regmap_irq and regmap_irq_chip structs are not modified and can be
> declared as const to move their data to a read-only section.
> 
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>

Reviewed-by: Matti Vaittinen <mazziesaccount@gmail.com>
Matti Vaittinen July 5, 2024, 5:50 a.m. UTC | #2
On 7/4/24 20:23, Javier Carrasco wrote:
> `bd9571mwv_irq_chip` and `bd9574mwf_irq_chip` are not modified and can
> be declared as const to move their data to a read-only section.
> 
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>

Reviewed-by: Matti Vaittinen <mazziesaccount@gmail.com>
Lee Jones July 25, 2024, 7:38 a.m. UTC | #3
On Thu, 04 Jul 2024 19:23:10 +0200, Javier Carrasco wrote:
> This series adds the const modifier to the remaining regmap_bus,
> regmap_config, regmap_irq and regmap_irq_chip structs within mfd
> that are effectively used as const (i.e., only read after their
> declaration), but kept as writtable data.
> 
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
> 
> [...]

Applied, thanks!

[01/24] mfd: da9062-core: Constify read-only regmap structs
        commit: 7115800617cd41295bb617e865997f227acd480b
[02/24] mfd: fsl-imx25-tsadc: Constify struct regmap_config
        commit: d932e461919cfa45185a0a22fb1a8427154b62cb
[03/24] mfd: hi655x-pmic: Constify struct regmap_config
        commit: d63abd1ced3613df64ed3e1d2e9f96895082f4bc
[04/24] mfd: wcd934x: Constify struct regmap_config
        commit: 3a782d2bf136c965f906f46d6cd35d4a0dabb5c7
[05/24] mfd: tps6105x: Constify struct regmap_config
        commit: 717df26d77b2ccd8d4f0caf3c2eb618dde2ebc81
[06/24] mfd: rohm-bd9576: Constify read-only regmap structs
        commit: 8235b063a2f8dcf2134fe6473da9cf35992ac305
[07/24] mfd: intel-m10-bmc: Constify struct regmap_config
        commit: 00bec2661357619d6eab7ea1cf5c6e8f5c1e1126
[08/24] mfd: 88pm80x: Constify read-only regmap structs
        commit: d038949e314c44db404ca49a4e997541c820e375
[09/24] mfd: bd9571mwv: Constify struct regmap_irq_chip
        commit: cb3394781af9e0047eb8e8c011cff0ad896d01cb
[10/24] mfd: intel_soc_pmic_bxtwc: Constify struct regmap_irq_chip
        commit: b830d915328f3077b3348a755e4178ddde588ce8
[11/24] mfd: retu: Constify read-only regmap structs
        commit: 9b0addc34a642670e73c551fe3625b137ddbae1c
[12/24] mfd: rk8xx-core: Constify struct regmap_irq_chip
        commit: 5ca2a7c1b45ba098e6795a3c93381192f8e430bb
[13/24] mfd: rohm-bd71828: Constify read-only regmap structs
        commit: 2d008372467f94872a47fbc51da4bee5a1fadbd7
[14/24] mfd: rohm-bd718x7: Constify struct regmap_irq_chip
        commit: 0bc081166936b66a600b0d898515032fd8cc0399
[15/24] mfd: tps65086: Constify struct regmap_irq_chip
        commit: 013e84b4d3f99c6e193123a4cf5c0c522361eb71
[16/24] mfd: tps65090: Constify struct regmap_irq_chip
        commit: 75a23bda7f86ebd6142dce395da6a1e30ecd1c58
[17/24] mfd: tps65218: Constify struct regmap_irq_chip
        commit: 9ae187b9b12674742c44a3ce0bb2249f3896b41e
[18/24] mfd: tps65219: Constify read-only regmap structs
        commit: 40ec8b0e239907768d9e3ab07184fde43ffaff9b
[19/24] mfd: tps65910: Constify struct regmap_irq_chip
        commit: 4a8ff7bf4d296e41bb334261be609c8c8bfbda49
[20/24] mfd: tps65912: Constify struct regmap_irq_chip
        commit: 9e94b883c645f7d0579d39b4b72490664bc743f0
[21/24] mfd: twl6040: Constify struct regmap_irq_chip
        commit: 430a8ffb83bf078ad7c63ba4c9eb0c3273302ef7
[22/24] mfd: gateworks-gsc: Constify struct regmap_bus
        commit: 07d43b15688c17a2dc93c9b1ac415c85e81d6bcc
[23/24] mfd: mc13xxx-spi: Constify struct regmap_bus
        commit: e23fd4be80dd6b9b6f55c1968a2b89725e9755b7
[24/24] mfd: sprd-sc27xx-spi: Constify struct regmap_bus
        commit: 8b5a164a9f8396aab242b14d00ad58b8ed7ef7d8

--
Lee Jones [李琼斯]