From: Sakari Ailus Date: Wed, 30 Sep 2020 14:34:43 +0000 (+0200) Subject: media: v4l2-fwnode: Say it's fine to use v4l2_fwnode_endpoint_parse X-Git-Tag: accepted/tizen/unified/20230118.172025~8355^2~326 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3be682fac66ca5829bb18c623c15147628a2d6f8;p=platform%2Fkernel%2Flinux-rpi.git media: v4l2-fwnode: Say it's fine to use v4l2_fwnode_endpoint_parse Earlier it was expected that there would be more variable size endpoint properties and that most if not all drivers would need them. For that reason it was expected also that v4l2_fwnode_endpoint_parse would no longer be needed. What actually happened that not all drivers require "link-frequencies", the only variable size media endpoint property without a small upper limit. Therefore drivers that do not need that information are fine using v4l2_fwnode_endpoint_parse. So don't tell drivers to use v4l2_fwnode_endpoint_alloc_parse in all cases. Signed-off-by: Sakari Ailus Reviewed-by: Niklas Söderlund Reviewed-by: Laurent Pinchart Signed-off-by: Mauro Carvalho Chehab --- diff --git a/include/media/v4l2-fwnode.h b/include/media/v4l2-fwnode.h index 20b30d7..be30e06 100644 --- a/include/media/v4l2-fwnode.h +++ b/include/media/v4l2-fwnode.h @@ -245,9 +245,9 @@ struct v4l2_fwnode_connector { * * The function does not change the V4L2 fwnode endpoint state if it fails. * - * NOTE: This function does not parse properties the size of which is variable - * without a low fixed limit. Please use v4l2_fwnode_endpoint_alloc_parse() in - * new drivers instead. + * NOTE: This function does not parse "link-frequencies" property as its size is + * not known in advance. Please use v4l2_fwnode_endpoint_alloc_parse() if you + * need properties of variable size. * * Return: %0 on success or a negative error code on failure: * %-ENOMEM on memory allocation failure