I was fixing a sparse warning in media_entity_pads_init() and I noticed that 
that
function always returns 0. Any reason why this can't be changed to a void 
function?

That return value is checked a zillion times in the media code. By making it 
void
it should simplify code all over.

See e.g. uvc_mc_init_entity in drivers/media/usb/uvc/uvc_entity.c: that whole
function can become a void function itself.

Regards,

        Hans
--
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

Reply via email to