| Message ID | e4395454-c500-4d76-af74-42e29d4b40d8@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | e3234e547a4db0572e271e490d044bdb4cb7233b |
| Headers | show |
| Series | media: exynos4-is: fimc-is-i2c: remove I2C_CLASS_SPD support | expand |
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@gmail.com> 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? > --- > 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);
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@gmail.com> > > 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@gmail.com> 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@gmail.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org> >> --- >> 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);
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);
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@gmail.com> --- drivers/media/platform/samsung/exynos4-is/fimc-is-i2c.c | 1 - 1 file changed, 1 deletion(-)