Hi Sakari,

On Wed, Mar 7, 2018 at 5:13 AM, Sakari Ailus <sakari.ai...@iki.fi> wrote:

>> @@ -2497,16 +2504,22 @@ static int ov5640_probe(struct i2c_client *client,
>>       struct fwnode_handle *endpoint;
>>       struct ov5640_dev *sensor;
>>       int ret;
>> +     struct v4l2_mbus_framefmt *fmt;
>
> This one I'd arrange before ret. The local variable declarations should
> generally look like a Christmas tree but upside down.

It seems Mauro is not happy with reverse Christmas tree ordering:
https://www.mail-archive.com/linux-media@vger.kernel.org/msg127221.html

Reply via email to