diff mbox series

media: exynos4-is: fimc-is-i2c: remove I2C_CLASS_SPD support

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

Commit Message

Heiner Kallweit Nov. 8, 2023, 6:42 a.m. UTC
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(-)

Comments

Wolfram Sang Nov. 23, 2023, 8:59 p.m. UTC | #1
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);
Heiner Kallweit Nov. 23, 2023, 9:37 p.m. UTC | #2
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 mbox series

Patch

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);