Hello Sakari, On 01/09/2016 08:03 PM, Sakari Ailus wrote: > Hi Javier, > > On Thu, Jan 07, 2016 at 03:27:14PM -0300, Javier Martinez Canillas wrote: >> Hello, >> >> When discussing a patch [0] with Laurent Pinchart for another series I >> mentioned to him that most callers of v4l2_of_parse_endpoint() weren't >> checking the return value. This is likely due the function kernel-doc >> stating incorrectly that the return value is always 0 but can return a >> negative error code on failure. >> >> This trivial patch series fixes the function kernel-doc and add proper >> error checking in all the drivers that are currently not doing so. > > After fixing patches 5 and 6, > Done, posted a v2 fixing the issues you pointed out. > Acked-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > Thanks a lot for your feedback and review! Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America -- 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