linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 00/10] Don't let i2c adapters declare I2C_CLASS_SPD support if they support I2C_CLASS_HWMON
@ 2023-11-24 10:16 Heiner Kallweit
  2023-11-24 10:16 ` [PATCH v2 03/10] i2c: pasemi: " Heiner Kallweit
  2023-12-16 20:23 ` [PATCH v2 00/10] " Heiner Kallweit
  0 siblings, 2 replies; 5+ messages in thread
From: Heiner Kallweit @ 2023-11-24 10:16 UTC (permalink / raw)
  To: Wolfram Sang, linuxppc-dev
  Cc: linux-media, linux-staging, linux-kernel, Nicholas Piggin,
	greybus-dev, linux-i2c, Heiner Kallweit

After removal of the legacy eeprom driver the only remaining I2C
client device driver supporting I2C_CLASS_SPD is jc42. Because this
driver also supports I2C_CLASS_HWMON, adapters don't have to
declare support for I2C_CLASS_SPD if they support I2C_CLASS_HWMON.
It's one step towards getting rid of I2C_CLASS_SPD mid-term.

Series was created supported by Coccinelle and its splitpatch.

v2:
- fix style issue in patch 4
- add ack in patch 2
- set proper subject prefix for all patches

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>

---

 drivers/i2c/busses/i2c-ali1535.c                  |    2 +-
 drivers/i2c/busses/i2c-ali1563.c                  |    2 +-
 drivers/i2c/busses/i2c-ali15x3.c                  |    2 +-
 drivers/i2c/busses/i2c-amd756.c                   |    2 +-
 drivers/i2c/busses/i2c-amd8111.c                  |    2 +-
 drivers/i2c/busses/i2c-elektor.c                  |    2 +-
 drivers/i2c/busses/i2c-gpio.c                     |    2 +-
 drivers/i2c/busses/i2c-ibm_iic.c                  |    2 +-
 drivers/i2c/busses/i2c-iop3xx.c                   |    2 +-
 drivers/i2c/busses/i2c-isch.c                     |    2 +-
 drivers/i2c/busses/i2c-kempld.c                   |    3 +--
 drivers/i2c/busses/i2c-mlxcpld.c                  |    2 +-
 drivers/i2c/busses/i2c-nforce2.c                  |    2 +-
 drivers/i2c/busses/i2c-pasemi-pci.c               |    2 +-
 drivers/i2c/busses/i2c-piix4.c                    |    2 +-
 drivers/i2c/busses/i2c-scmi.c                     |    2 +-
 drivers/i2c/busses/i2c-sh7760.c                   |    2 +-
 drivers/i2c/busses/i2c-sibyte.c                   |    4 ++--
 drivers/i2c/busses/i2c-sis5595.c                  |    2 +-
 drivers/i2c/busses/i2c-sis630.c                   |    2 +-
 drivers/i2c/busses/i2c-sis96x.c                   |    2 +-
 drivers/i2c/busses/i2c-via.c                      |    2 +-
 drivers/i2c/busses/i2c-viapro.c                   |    2 +-
 drivers/i2c/busses/scx200_acb.c                   |    2 +-
 drivers/i2c/i2c-stub.c                            |    2 +-
 drivers/media/pci/netup_unidvb/netup_unidvb_i2c.c |    2 +-
 drivers/staging/greybus/i2c.c                     |    2 +-
 27 files changed, 28 insertions(+), 29 deletions(-)

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v2 03/10] i2c: pasemi: Don't let i2c adapters declare I2C_CLASS_SPD support if they support I2C_CLASS_HWMON
  2023-11-24 10:16 [PATCH v2 00/10] Don't let i2c adapters declare I2C_CLASS_SPD support if they support I2C_CLASS_HWMON Heiner Kallweit
@ 2023-11-24 10:16 ` Heiner Kallweit
  2023-11-26 14:22   ` Andi Shyti
  2023-12-16 20:23 ` [PATCH v2 00/10] " Heiner Kallweit
  1 sibling, 1 reply; 5+ messages in thread
From: Heiner Kallweit @ 2023-11-24 10:16 UTC (permalink / raw)
  To: Wolfram Sang, Michael Ellerman
  Cc: Andi Shyti, Nicholas Piggin, linux-kernel, linux-i2c,
	linuxppc-dev, Heiner Kallweit

After removal of the legacy eeprom driver the only remaining I2C
client device driver supporting I2C_CLASS_SPD is jc42. Because this
driver also supports I2C_CLASS_HWMON, adapters don't have to
declare support for I2C_CLASS_SPD if they support I2C_CLASS_HWMON.
It's one step towards getting rid of I2C_CLASS_SPD mid-term.

Series was created supported by Coccinelle and its splitpatch.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>

---
 drivers/i2c/busses/i2c-pasemi-pci.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-pasemi-pci.c b/drivers/i2c/busses/i2c-pasemi-pci.c
index cfc89e04e..77f90c743 100644
--- a/drivers/i2c/busses/i2c-pasemi-pci.c
+++ b/drivers/i2c/busses/i2c-pasemi-pci.c
@@ -56,7 +56,7 @@ static int pasemi_smb_pci_probe(struct pci_dev *dev,
 	if (!smbus->ioaddr)
 		return -EBUSY;
 
-	smbus->adapter.class = I2C_CLASS_HWMON | I2C_CLASS_SPD;
+	smbus->adapter.class = I2C_CLASS_HWMON;
 	error = pasemi_i2c_common_probe(smbus);
 	if (error)
 		return error;


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH v2 03/10] i2c: pasemi: Don't let i2c adapters declare I2C_CLASS_SPD support if they support I2C_CLASS_HWMON
  2023-11-24 10:16 ` [PATCH v2 03/10] i2c: pasemi: " Heiner Kallweit
@ 2023-11-26 14:22   ` Andi Shyti
  0 siblings, 0 replies; 5+ messages in thread
From: Andi Shyti @ 2023-11-26 14:22 UTC (permalink / raw)
  To: Heiner Kallweit
  Cc: Nicholas Piggin, linux-kernel, Wolfram Sang, linux-i2c,
	linuxppc-dev

Hi Heiner,

On Fri, Nov 24, 2023 at 11:16:12AM +0100, Heiner Kallweit wrote:
> After removal of the legacy eeprom driver the only remaining I2C
> client device driver supporting I2C_CLASS_SPD is jc42. Because this
> driver also supports I2C_CLASS_HWMON, adapters don't have to
> declare support for I2C_CLASS_SPD if they support I2C_CLASS_HWMON.
> It's one step towards getting rid of I2C_CLASS_SPD mid-term.
> 
> Series was created supported by Coccinelle and its splitpatch.
> 
> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>

Acked-by: Andi Shyti <andi.shyti@kernel.org>

Thanks,
Andi

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2 00/10] Don't let i2c adapters declare I2C_CLASS_SPD support if they support I2C_CLASS_HWMON
  2023-11-24 10:16 [PATCH v2 00/10] Don't let i2c adapters declare I2C_CLASS_SPD support if they support I2C_CLASS_HWMON Heiner Kallweit
  2023-11-24 10:16 ` [PATCH v2 03/10] i2c: pasemi: " Heiner Kallweit
@ 2023-12-16 20:23 ` Heiner Kallweit
  2023-12-19 12:36   ` Wolfram Sang
  1 sibling, 1 reply; 5+ messages in thread
From: Heiner Kallweit @ 2023-12-16 20:23 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-staging, linux-kernel, Nicholas Piggin, greybus-dev,
	linux-i2c, linuxppc-dev, linux-media

On 24.11.2023 11:16, Heiner Kallweit wrote:
> After removal of the legacy eeprom driver the only remaining I2C
> client device driver supporting I2C_CLASS_SPD is jc42. Because this
> driver also supports I2C_CLASS_HWMON, adapters don't have to
> declare support for I2C_CLASS_SPD if they support I2C_CLASS_HWMON.
> It's one step towards getting rid of I2C_CLASS_SPD mid-term.
> 
> Series was created supported by Coccinelle and its splitpatch.
> 
> v2:
> - fix style issue in patch 4
> - add ack in patch 2
> - set proper subject prefix for all patches
> 
> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
> 
> ---
> 
>  drivers/i2c/busses/i2c-ali1535.c                  |    2 +-
>  drivers/i2c/busses/i2c-ali1563.c                  |    2 +-
>  drivers/i2c/busses/i2c-ali15x3.c                  |    2 +-
>  drivers/i2c/busses/i2c-amd756.c                   |    2 +-
>  drivers/i2c/busses/i2c-amd8111.c                  |    2 +-
>  drivers/i2c/busses/i2c-elektor.c                  |    2 +-
>  drivers/i2c/busses/i2c-gpio.c                     |    2 +-
>  drivers/i2c/busses/i2c-ibm_iic.c                  |    2 +-
>  drivers/i2c/busses/i2c-iop3xx.c                   |    2 +-
>  drivers/i2c/busses/i2c-isch.c                     |    2 +-
>  drivers/i2c/busses/i2c-kempld.c                   |    3 +--
>  drivers/i2c/busses/i2c-mlxcpld.c                  |    2 +-
>  drivers/i2c/busses/i2c-nforce2.c                  |    2 +-
>  drivers/i2c/busses/i2c-pasemi-pci.c               |    2 +-
>  drivers/i2c/busses/i2c-piix4.c                    |    2 +-
>  drivers/i2c/busses/i2c-scmi.c                     |    2 +-
>  drivers/i2c/busses/i2c-sh7760.c                   |    2 +-
>  drivers/i2c/busses/i2c-sibyte.c                   |    4 ++--
>  drivers/i2c/busses/i2c-sis5595.c                  |    2 +-
>  drivers/i2c/busses/i2c-sis630.c                   |    2 +-
>  drivers/i2c/busses/i2c-sis96x.c                   |    2 +-
>  drivers/i2c/busses/i2c-via.c                      |    2 +-
>  drivers/i2c/busses/i2c-viapro.c                   |    2 +-
>  drivers/i2c/busses/scx200_acb.c                   |    2 +-
>  drivers/i2c/i2c-stub.c                            |    2 +-
>  drivers/media/pci/netup_unidvb/netup_unidvb_i2c.c |    2 +-
>  drivers/staging/greybus/i2c.c                     |    2 +-
>  27 files changed, 28 insertions(+), 29 deletions(-)

This series and my other series are sitting idle in patchwork
for 3 weeks now. AFAICS they have the needed ack's.
Anything missing before they can be applied?


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2 00/10] Don't let i2c adapters declare I2C_CLASS_SPD support if they support I2C_CLASS_HWMON
  2023-12-16 20:23 ` [PATCH v2 00/10] " Heiner Kallweit
@ 2023-12-19 12:36   ` Wolfram Sang
  0 siblings, 0 replies; 5+ messages in thread
From: Wolfram Sang @ 2023-12-19 12:36 UTC (permalink / raw)
  To: Heiner Kallweit
  Cc: linux-staging, linux-kernel, Nicholas Piggin, greybus-dev,
	linux-i2c, linuxppc-dev, linux-media

[-- Attachment #1: Type: text/plain, Size: 197 bytes --]


> This series and my other series are sitting idle in patchwork
> for 3 weeks now. AFAICS they have the needed ack's.
> Anything missing before they can be applied?

Applied to for-next, thanks!


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-12-19 12:37 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-24 10:16 [PATCH v2 00/10] Don't let i2c adapters declare I2C_CLASS_SPD support if they support I2C_CLASS_HWMON Heiner Kallweit
2023-11-24 10:16 ` [PATCH v2 03/10] i2c: pasemi: " Heiner Kallweit
2023-11-26 14:22   ` Andi Shyti
2023-12-16 20:23 ` [PATCH v2 00/10] " Heiner Kallweit
2023-12-19 12:36   ` Wolfram Sang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).