Re: [PATCH 4/6] media: i2c: s5c73m3: Convert to new i2c device probe()

[Wolfram Sang]

On Wed, Jul 10, 2019 at 10:51:47PM +0100, Kieran Bingham wrote:
> The I2C core framework provides a simplified probe framework from commit
> b8a1a4cd5a98 ("i2c: Provide a temporary .probe_new() call-back type").
> 
> This driver does not utilise the i2c_device_id table in the probe, so we can
> easily convert it to utilise the simplfied i2c driver registration.
> 
> Signed-off-by: Kieran Bingham 

Reviewed-by: Wolfram Sang 



signature.asc
Description: PGP signature

Re: [PATCH 4/6] media: i2c: s5c73m3: Convert to new i2c device probe()

[Andrzej Hajda]

On 10.07.2019 23:51, Kieran Bingham wrote:
> The I2C core framework provides a simplified probe framework from commit
> b8a1a4cd5a98 ("i2c: Provide a temporary .probe_new() call-back type").
>
> This driver does not utilise the i2c_device_id table in the probe, so we can
> easily convert it to utilise the simplfied i2c driver registration.
>
> Signed-off-by: Kieran Bingham 


Acked-by: Andrzej Hajda 


 --
Regards
Andrzej

[PATCH 4/6] media: i2c: s5c73m3: Convert to new i2c device probe()

[Kieran Bingham]

The I2C core framework provides a simplified probe framework from commit
b8a1a4cd5a98 ("i2c: Provide a temporary .probe_new() call-back type").

This driver does not utilise the i2c_device_id table in the probe, so we can
easily convert it to utilise the simplfied i2c driver registration.

Signed-off-by: Kieran Bingham 
---
 drivers/media/i2c/s5c73m3/s5c73m3-core.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/media/i2c/s5c73m3/s5c73m3-core.c 
b/drivers/media/i2c/s5c73m3/s5c73m3-core.c
index 7633aebd8c06..5b4c4a3547c9 100644
--- a/drivers/media/i2c/s5c73m3/s5c73m3-core.c
+++ b/drivers/media/i2c/s5c73m3/s5c73m3-core.c
@@ -1650,8 +1650,7 @@ static int s5c73m3_get_platform_data(struct s5c73m3 
*state)
return 0;
 }
 
-static int s5c73m3_probe(struct i2c_client *client,
-   const struct i2c_device_id *id)
+static int s5c73m3_probe(struct i2c_client *client)
 {
struct device *dev = >dev;
struct v4l2_subdev *sd;
@@ -1806,7 +1805,7 @@ static struct i2c_driver s5c73m3_i2c_driver = {
.of_match_table = of_match_ptr(s5c73m3_of_match),
.name   = DRIVER_NAME,
},
-   .probe  = s5c73m3_probe,
+   .probe_new  = s5c73m3_probe,
.remove = s5c73m3_remove,
.id_table   = s5c73m3_id,
 };
-- 
2.20.1