Hello, No need for redundant argument check for module_put().
Signed-off-by: Mariusz Kozlowski <[EMAIL PROTECTED]> drivers/media/video/cpia.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff -upr linux-2.6.20-rc2-mm1-a/drivers/media/video/cpia.c linux-2.6.20-rc2-mm1-b/drivers/media/video/cpia.c --- linux-2.6.20-rc2-mm1-a/drivers/media/video/cpia.c 2006-12-24 05:00:32.000000000 +0100 +++ linux-2.6.20-rc2-mm1-b/drivers/media/video/cpia.c 2007-01-02 02:28:01.000000000 +0100 @@ -3153,8 +3153,7 @@ static int reset_camera(struct cam_data static void put_cam(struct cpia_camera_ops* ops) { - if (ops->owner) - module_put(ops->owner); + module_put(ops->owner); } /* ------------------------- V4L interface --------------------- */ -- Regards, Mariusz Kozlowski - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/