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-ow...@vger.kernel.org [mailto:linux-media- > ow...@vger.kernel.org] On Behalf Of Karicheri, Muralidharan > Sent: Wednesday, July 07, 2010 4:15 PM > To: Laurent Pinchart; linux-media@vger.kernel.org > Cc: sakari.ai...@maxwell.research.nokia.com > 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 majord...@vger.kernel.org > 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
