* Re: [PATCH v2] media: staging/intel-ipu3: Reformat pointer initialization
[not found] ` <20200323151710.GA22110@deeUbuntu>
@ 2020-03-25 20:48 ` DEEPAK VARMA
0 siblings, 0 replies; only message in thread
From: DEEPAK VARMA @ 2020-03-25 20:48 UTC (permalink / raw)
To: Andy Shevchenko
Cc: outreachy-kernel, Greg Kroah-Hartman, Daniel Baluta,
kieran.bingham, Sakari Ailus, Mauro Carvalho Chehab, linux-media
Adding Linux-media list as advised by Sakari Ailus.
On Mon, Mar 23, 2020 at 08:47:12PM +0530, DEEPAK VARMA wrote:
> On Mon, Mar 23, 2020 at 01:35:58AM +0200, Andy Shevchenko wrote:
> > On Sun, Mar 22, 2020 at 8:54 PM Deepak R Varma <mh12gx2825@gmail.com> wrote:
> > >
> > > Reformat pointer initialization to make it more readable as per the
> > > coding standards. Problem detected by checkpatch.
> >
> > > - struct imgu_v4l2_subdev *imgu_sd =
> > > - container_of(ctrl->handler, struct imgu_v4l2_subdev, ctrl_handler);
> > > + struct imgu_v4l2_subdev *imgu_sd = container_of(ctrl->handler,
> > > + struct imgu_v4l2_subdev,
> > > + ctrl_handler);
> >
> > I'm sorry I don't see benefit of this change, actually an opposite.
>
> Hello Andy,
> Thank you for your comment. To me, it helps quickly and clearly spot
> *imgu_sd. Also the initialization looks more structured. But that's just
> me.
>
> I will wait to see if anyone else wants to share their thoughts, and
> then circle back to you on next steps. Is that fine?
>
Hello Andy and all,
I have not seen any additional comments on this thread. Let me know if
you approve the proposed change or want me to leave it as is.
Thank you,
Deepak.
> Deepak.
> >
> > --
> > With Best Regards,
> > Andy Shevchenko
^ permalink raw reply [flat|nested] only message in thread