Re: [PATCH] [media] mt9v032: Remove duplicate test for I2C_FUNC_SMBUS_WORD_DATA functionality

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Axel,

Thank you for the patch.

On Sunday 10 August 2014 17:41:49 Axel Lin wrote:
> Since commit b42261078a91 ("regmap: i2c: fallback to SMBus if the adapter
> does not support standard I2C"), regmap-i2c will check the
> I2C_FUNC_SMBUS_[BYTE|WORD]_DATA functionality based on the regmap_config
> setting if the adapter does not support standard I2C.
> 
> So remove the I2C_FUNC_SMBUS_WORD_DATA functionality check in the driver
> code.
> 
> Signed-off-by: Axel Lin <axel.lin@ingics.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

and applied to my tree.

> ---
>  drivers/media/i2c/mt9v032.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/drivers/media/i2c/mt9v032.c b/drivers/media/i2c/mt9v032.c
> index d044bce..f9e4bf7 100644
> --- a/drivers/media/i2c/mt9v032.c
> +++ b/drivers/media/i2c/mt9v032.c
> @@ -879,13 +879,6 @@ static int mt9v032_probe(struct i2c_client *client,
>  	unsigned int i;
>  	int ret;
> 
> -	if (!i2c_check_functionality(client->adapter,
> -				     I2C_FUNC_SMBUS_WORD_DATA)) {
> -		dev_warn(&client->adapter->dev,
> -			 "I2C-Adapter doesn't support I2C_FUNC_SMBUS_WORD\n");
> -		return -EIO;
> -	}
> -
>  	mt9v032 = devm_kzalloc(&client->dev, sizeof(*mt9v032), GFP_KERNEL);
>  	if (!mt9v032)
>  		return -ENOMEM;

-- 
Regards,

Laurent Pinchart