diff mbox series

staging: olpc_dcon: Remove I2C_CLASS_DDC support

Message ID da3070d9-e016-4167-843f-a08d5b2dc1fe@gmail.com
State New
Headers show
Series staging: olpc_dcon: Remove I2C_CLASS_DDC support | expand

Commit Message

Heiner Kallweit Oct. 26, 2023, 8:49 p.m. UTC
olpc_dcon is the only remaining i2c client device driver declaring
I2C_CLASS_DDC support after the legacy eeprom driver has been removed.

olpc_dcon is only used on olpc devices, connected to an i2c adapter
driven by scx200_acb. This adapter driver declares support for
I2C_CLASS_HWMON and I2C_CLASS_SPD. Therefore we can safely drop
I2C_CLASS_DDC support in olpc_dcon.

That's the last step before I2C_CLASS_DDC can be removed in general.

This patch is solely based on documentation, and I don't have an olpc
device for testing. Therefore some testing would be appreciated before
patch is applied.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
---
 drivers/staging/olpc_dcon/olpc_dcon.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Wolfram Sang Oct. 28, 2023, 12:30 p.m. UTC | #1
On Thu, Oct 26, 2023 at 10:49:21PM +0200, Heiner Kallweit wrote:
> olpc_dcon is the only remaining i2c client device driver declaring
> I2C_CLASS_DDC support after the legacy eeprom driver has been removed.
> 
> olpc_dcon is only used on olpc devices, connected to an i2c adapter
> driven by scx200_acb. This adapter driver declares support for
> I2C_CLASS_HWMON and I2C_CLASS_SPD. Therefore we can safely drop
> I2C_CLASS_DDC support in olpc_dcon.
> 
> That's the last step before I2C_CLASS_DDC can be removed in general.
> 
> This patch is solely based on documentation, and I don't have an olpc
> device for testing. Therefore some testing would be appreciated before
> patch is applied.

Yes, testing would be good, for sure. Still, I think we can apply it
even without. Because we are not dealing with HW features here, but only
with the Linux internal handling of i2c adapter classes. So:

Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

> 
> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
> ---
>  drivers/staging/olpc_dcon/olpc_dcon.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/olpc_dcon/olpc_dcon.c b/drivers/staging/olpc_dcon/olpc_dcon.c
> index d5271eac1..08ec3aae9 100644
> --- a/drivers/staging/olpc_dcon/olpc_dcon.c
> +++ b/drivers/staging/olpc_dcon/olpc_dcon.c
> @@ -777,7 +777,7 @@ static struct i2c_driver dcon_driver = {
>  		.name	= "olpc_dcon",
>  		.pm = &dcon_pm_ops,
>  	},
> -	.class = I2C_CLASS_DDC | I2C_CLASS_HWMON,
> +	.class = I2C_CLASS_HWMON,
>  	.id_table = dcon_idtable,
>  	.probe = dcon_probe,
>  	.remove = dcon_remove,
> -- 
> 2.42.0
>
diff mbox series

Patch

diff --git a/drivers/staging/olpc_dcon/olpc_dcon.c b/drivers/staging/olpc_dcon/olpc_dcon.c
index d5271eac1..08ec3aae9 100644
--- a/drivers/staging/olpc_dcon/olpc_dcon.c
+++ b/drivers/staging/olpc_dcon/olpc_dcon.c
@@ -777,7 +777,7 @@  static struct i2c_driver dcon_driver = {
 		.name	= "olpc_dcon",
 		.pm = &dcon_pm_ops,
 	},
-	.class = I2C_CLASS_DDC | I2C_CLASS_HWMON,
+	.class = I2C_CLASS_HWMON,
 	.id_table = dcon_idtable,
 	.probe = dcon_probe,
 	.remove = dcon_remove,