Isn't it like there need to be {} for both "if" and "else" when
there is more than one line in either block?
Regards,
--
Sylwester Nawrocki
> -----Original Message-----
> From: linux-media-owner@xxxxxxxxxxxxxxx [mailto:linux-media-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Karicheri, Muralidharan
> Sent: Wednesday, July 07, 2010 4:15 PM
> To: Laurent Pinchart; linux-media@xxxxxxxxxxxxxxx
> Cc: sakari.ailus@xxxxxxxxxxxxxxxxxxxxxxxxxx
> Subject: RE: [RFC/PATCH 2/6] v4l: subdev: Add device node support
>
>
>
> >v4l2_device *v4l2_dev,
> > if (err && err != -ENOIOCTLCMD) {
> > v4l2_device_unregister_subdev(sd);
> > sd = NULL;
> >+ } else {
> >+ sd->initialized = 1;
> > }
>
> Wouldn't checkpatch.pl script complain about { } on the else part since
> there is only one statement?
> > }
> >
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media"
> in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
