mbox series

[RFC,PATCH-for-9.1,0/4] hw/i2c: Convert to spec v7 (inclusive) terminology

Message ID 20240408213339.83149-1-philmd@linaro.org (mailing list archive)
Headers show
Series hw/i2c: Convert to spec v7 (inclusive) terminology | expand

Message

Philippe Mathieu-Daudé April 8, 2024, 9:33 p.m. UTC
Mechanical (mostly) conversion inspired by Wolfram [*] to
use inclusive terminology, similarly to the other renames
we did 3 years ago, shortly before the I2C spec v7 was
published.

Posted as RFC to get feedback, if no objection I plan to
finish the conversion (SMBus and rest if hw/i2c/).

[*] https://lore.kernel.org/all/20240322132619.6389-1-wsa+renesas@sang-engineering.com/

Philippe Mathieu-Daudé (4):
  hw/i2c: Fix checkpatch block comment warnings
  hw/i2c: Fix checkpatch line over 80 chars warnings
  hw/i2c: Convert to spec v7 terminology (automatically)
  hw/i2c: Convert to spec v7 terminology (manually)

 include/hw/display/i2c-ddc.h     |   2 +-
 include/hw/gpio/pca9552.h        |   2 +-
 include/hw/gpio/pca9554.h        |   2 +-
 include/hw/i2c/aspeed_i2c.h      |   4 +-
 include/hw/i2c/i2c.h             | 123 ++++++-------
 include/hw/i2c/i2c_mux_pca954x.h |   2 +-
 include/hw/i2c/smbus_slave.h     |   4 +-
 include/hw/nvram/eeprom_at24c.h  |   8 +-
 include/hw/sensor/tmp105.h       |   2 +-
 hw/arm/aspeed.c                  | 290 +++++++++++++++++--------------
 hw/arm/bananapi_m2u.c            |   2 +-
 hw/arm/cubieboard.c              |   2 +-
 hw/arm/musicpal.c                |   6 +-
 hw/arm/npcm7xx_boards.c          |  44 ++---
 hw/arm/nseries.c                 |   6 +-
 hw/arm/pxa2xx.c                  |  36 ++--
 hw/arm/realview.c                |   2 +-
 hw/arm/spitz.c                   |  12 +-
 hw/arm/stellaris.c               |   2 +-
 hw/arm/tosa.c                    |  14 +-
 hw/arm/versatilepb.c             |   2 +-
 hw/arm/vexpress.c                |   2 +-
 hw/arm/z2.c                      |  20 +--
 hw/audio/wm8750.c                |  18 +-
 hw/display/ati.c                 |   4 +-
 hw/display/i2c-ddc.c             |  10 +-
 hw/display/sii9022.c             |  16 +-
 hw/display/sm501.c               |   2 +-
 hw/display/ssd0303.c             |  14 +-
 hw/display/xlnx_dp.c             |   2 +-
 hw/gpio/max7310.c                |  14 +-
 hw/gpio/pca9552.c                |  14 +-
 hw/gpio/pca9554.c                |  14 +-
 hw/gpio/pcf8574.c                |  12 +-
 hw/i2c/aspeed_i2c.c              |  16 +-
 hw/i2c/core.c                    |  90 +++++-----
 hw/i2c/i2c_mux_pca954x.c         |   6 +-
 hw/i2c/imx_i2c.c                 |   2 +-
 hw/i2c/smbus_slave.c             |  12 +-
 hw/input/lm832x.c                |  14 +-
 hw/misc/axp2xx.c                 |  14 +-
 hw/misc/i2c-echo.c               |  14 +-
 hw/nvram/eeprom_at24c.c          |  26 +--
 hw/ppc/e500.c                    |   2 +-
 hw/ppc/pnv.c                     |   4 +-
 hw/ppc/sam460ex.c                |   2 +-
 hw/rtc/ds1338.c                  |  14 +-
 hw/rtc/m41t80.c                  |  12 +-
 hw/rtc/twl92230.c                |  16 +-
 hw/sensor/dps310.c               |  14 +-
 hw/sensor/emc141x.c              |  16 +-
 hw/sensor/lsm303dlhc_mag.c       |  16 +-
 hw/sensor/tmp105.c               |  16 +-
 hw/sensor/tmp421.c               |  20 +--
 hw/tpm/tpm_tis_i2c.c             |  12 +-
 55 files changed, 541 insertions(+), 506 deletions(-)

Comments

Corey Minyard May 16, 2024, 1:48 a.m. UTC | #1
On Mon, Apr 08, 2024 at 11:33:34PM +0200, Philippe Mathieu-Daudé wrote:
> Mechanical (mostly) conversion inspired by Wolfram [*] to
> use inclusive terminology, similarly to the other renames
> we did 3 years ago, shortly before the I2C spec v7 was
> published.

Sorry, I've been extremely busy on this.

Since the spec has been redone, I'm good with the renames.

As far as the >80 character lines, I'm fine either way.  I actually like
to keep them in 80 characters, but the Linux kernel has moved away from
that, and if it's easier to read with a longer line, that's probably
better.

-corey

> 
> Posted as RFC to get feedback, if no objection I plan to
> finish the conversion (SMBus and rest if hw/i2c/).
> 
> [*] https://lore.kernel.org/all/20240322132619.6389-1-wsa+renesas@sang-engineering.com/
> 
> Philippe Mathieu-Daudé (4):
>   hw/i2c: Fix checkpatch block comment warnings
>   hw/i2c: Fix checkpatch line over 80 chars warnings
>   hw/i2c: Convert to spec v7 terminology (automatically)
>   hw/i2c: Convert to spec v7 terminology (manually)
> 
>  include/hw/display/i2c-ddc.h     |   2 +-
>  include/hw/gpio/pca9552.h        |   2 +-
>  include/hw/gpio/pca9554.h        |   2 +-
>  include/hw/i2c/aspeed_i2c.h      |   4 +-
>  include/hw/i2c/i2c.h             | 123 ++++++-------
>  include/hw/i2c/i2c_mux_pca954x.h |   2 +-
>  include/hw/i2c/smbus_slave.h     |   4 +-
>  include/hw/nvram/eeprom_at24c.h  |   8 +-
>  include/hw/sensor/tmp105.h       |   2 +-
>  hw/arm/aspeed.c                  | 290 +++++++++++++++++--------------
>  hw/arm/bananapi_m2u.c            |   2 +-
>  hw/arm/cubieboard.c              |   2 +-
>  hw/arm/musicpal.c                |   6 +-
>  hw/arm/npcm7xx_boards.c          |  44 ++---
>  hw/arm/nseries.c                 |   6 +-
>  hw/arm/pxa2xx.c                  |  36 ++--
>  hw/arm/realview.c                |   2 +-
>  hw/arm/spitz.c                   |  12 +-
>  hw/arm/stellaris.c               |   2 +-
>  hw/arm/tosa.c                    |  14 +-
>  hw/arm/versatilepb.c             |   2 +-
>  hw/arm/vexpress.c                |   2 +-
>  hw/arm/z2.c                      |  20 +--
>  hw/audio/wm8750.c                |  18 +-
>  hw/display/ati.c                 |   4 +-
>  hw/display/i2c-ddc.c             |  10 +-
>  hw/display/sii9022.c             |  16 +-
>  hw/display/sm501.c               |   2 +-
>  hw/display/ssd0303.c             |  14 +-
>  hw/display/xlnx_dp.c             |   2 +-
>  hw/gpio/max7310.c                |  14 +-
>  hw/gpio/pca9552.c                |  14 +-
>  hw/gpio/pca9554.c                |  14 +-
>  hw/gpio/pcf8574.c                |  12 +-
>  hw/i2c/aspeed_i2c.c              |  16 +-
>  hw/i2c/core.c                    |  90 +++++-----
>  hw/i2c/i2c_mux_pca954x.c         |   6 +-
>  hw/i2c/imx_i2c.c                 |   2 +-
>  hw/i2c/smbus_slave.c             |  12 +-
>  hw/input/lm832x.c                |  14 +-
>  hw/misc/axp2xx.c                 |  14 +-
>  hw/misc/i2c-echo.c               |  14 +-
>  hw/nvram/eeprom_at24c.c          |  26 +--
>  hw/ppc/e500.c                    |   2 +-
>  hw/ppc/pnv.c                     |   4 +-
>  hw/ppc/sam460ex.c                |   2 +-
>  hw/rtc/ds1338.c                  |  14 +-
>  hw/rtc/m41t80.c                  |  12 +-
>  hw/rtc/twl92230.c                |  16 +-
>  hw/sensor/dps310.c               |  14 +-
>  hw/sensor/emc141x.c              |  16 +-
>  hw/sensor/lsm303dlhc_mag.c       |  16 +-
>  hw/sensor/tmp105.c               |  16 +-
>  hw/sensor/tmp421.c               |  20 +--
>  hw/tpm/tpm_tis_i2c.c             |  12 +-
>  55 files changed, 541 insertions(+), 506 deletions(-)
> 
> -- 
> 2.41.0
> 
>