On 23.11.2023 21:59, Wolfram Sang wrote: > On Wed, Nov 08, 2023 at 07:42:37AM +0100, Heiner Kallweit wrote: >> This I2C bus is used by the firmware only and it seems I2C_CLASS_SPD >> device auto-detection has never been used. So we can safely remove it. >> That's one further step towards removing I2C_CLASS_SPD completely. >> >> Signed-off-by: Heiner Kallweit <hkallweit1@xxxxxxxxx> > > As I want to apply this cleanup early in the cycle, could we have an ack > for me to modify this subsystem via I2C, please? > Today I received the following, patch has been applied via the linux-media tree already. [git:media_stage/master] media: exynos4-is: fimc-is-i2c: remove I2C_CLASS_SPD support This is an automatic generated email to let you know that the following patch were queued: Subject: media: exynos4-is: fimc-is-i2c: remove I2C_CLASS_SPD support Author: Heiner Kallweit <hkallweit1@xxxxxxxxx> Date: Wed Nov 8 07:42:37 2023 +0100 This I2C bus is used by the firmware only and it seems I2C_CLASS_SPD device auto-detection has never been used. So we can safely remove it. That's one further step towards removing I2C_CLASS_SPD completely. Signed-off-by: Heiner Kallweit <hkallweit1@xxxxxxxxx> Signed-off-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx> Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> >> --- >> drivers/media/platform/samsung/exynos4-is/fimc-is-i2c.c | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/drivers/media/platform/samsung/exynos4-is/fimc-is-i2c.c b/drivers/media/platform/samsung/exynos4-is/fimc-is-i2c.c >> index bef6e9b4a..44363c424 100644 >> --- a/drivers/media/platform/samsung/exynos4-is/fimc-is-i2c.c >> +++ b/drivers/media/platform/samsung/exynos4-is/fimc-is-i2c.c >> @@ -57,7 +57,6 @@ static int fimc_is_i2c_probe(struct platform_device *pdev) >> strscpy(i2c_adap->name, "exynos4x12-isp-i2c", sizeof(i2c_adap->name)); >> i2c_adap->owner = THIS_MODULE; >> i2c_adap->algo = &fimc_is_i2c_algorithm; >> - i2c_adap->class = I2C_CLASS_SPD; >> >> platform_set_drvdata(pdev, isp_i2c); >> pm_runtime_enable(&pdev->dev);
