Re: [RESEND PATCH 01/25] media: i2c: imx214: simplify getting state container

Sakari Ailus
2020-10-30T09:17:48Z

Hi Krzysztof,

On Thu, Oct 29, 2020 at 05:42:15PM +0100, Krzysztof Kozlowski wrote:
> The pointer to 'struct v4l2_subdev' is stored in drvdata via
> v4l2_i2c_subdev_init() so there is no point of a dance like:
>
> struct i2c_client *client = to_i2c_client(struct device *dev)
> struct v4l2_subdev *sd = i2c_get_clientdata(client);
>
> This allows to remove local variable 'client' and few pointer
> dereferences. White at it, use 'dev' directly instead of 'imx214->dev'.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

I've applied the original set to my tree here:

<URL:https://git.linuxtv.org/sailus/media_tree.git/log/>

The status of the patches is generally indicated in Patchwork:

<URL:https://patchwork.linuxtv.org/project/linux-media/list/>

"Under review" effectively, but perhaps confusingly, is also used to tell
that the patch is in someone else's tree and on its way to Mauro's. IOW, it
is not necessary to resend them.

--
Regards,

Sakari Ailus