lkml.org 
[lkml]   [2016]   [Jan]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Subject[PATCH 0/8] [media] Check v4l2_of_parse_endpoint() ret val in all drivers
    Date
    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.

    [0]: https://lkml.org/lkml/2016/1/6/307

    Best regards,
    Javier


    Javier Martinez Canillas (8):
    [media] v4l: of: Correct v4l2_of_parse_endpoint() kernel-doc
    [media] adv7604: Check v4l2_of_parse_endpoint() return value
    [media] s5c73m3: Check v4l2_of_parse_endpoint() return value
    [media] s5k5baf: Check v4l2_of_parse_endpoint() return value
    [media] tvp514x: Check v4l2_of_parse_endpoint() return value
    [media] tvp7002: Check v4l2_of_parse_endpoint() return value
    [media] exynos4-is: Check v4l2_of_parse_endpoint() return value
    [media] omap3isp: Check v4l2_of_parse_endpoint() return value

    drivers/media/i2c/adv7604.c | 7 ++++++-
    drivers/media/i2c/s5c73m3/s5c73m3-core.c | 4 +++-
    drivers/media/i2c/s5k5baf.c | 5 ++++-
    drivers/media/i2c/tvp514x.c | 4 +++-
    drivers/media/i2c/tvp7002.c | 4 +++-
    drivers/media/platform/exynos4-is/media-dev.c | 8 +++++++-
    drivers/media/platform/exynos4-is/mipi-csis.c | 10 +++++++---
    drivers/media/platform/omap3isp/isp.c | 5 ++++-
    drivers/media/v4l2-core/v4l2-of.c | 2 +-
    9 files changed, 38 insertions(+), 11 deletions(-)

    --
    2.4.3

    \
     
     \ /
      Last update: 2016-01-07 19:41    [W:3.192 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site