staging: media: omap24xx: Use min_t instead of min
authorAybuke Ozdemir <aybuke.147@gmail.com>
Mon, 20 Oct 2014 22:39:34 +0000 (01:39 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 23 Oct 2014 06:10:49 +0000 (14:10 +0800)
This patch focuses on fixing the following warning generated by
checkpatch.pl for the file tcm825x.c:

WARNÄ°NG: min() should probably be min_t(u32, tgt_xclk, TCM825X_XCLK_MAX)

The changes were applied using the following coccinelle rule:
@@
expression e1, e2;
typedef u32;
@@

-tgt_xclk = min(e1, (u32)e2);
+tgt_xclk = min_t(u32, e1, e2);

Signed-off-by: Aybuke Ozdemir <aybuke.147@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/media/omap24xx/tcm825x.c

index 9d9ecf1..6a1f479 100644 (file)
@@ -751,7 +751,7 @@ static int ioctl_g_ifparm(struct v4l2_int_device *s, struct v4l2_ifparm *p)
                (2457 * tgt_fps) / MAX_FPS;
        tgt_xclk *= 10000;
 
-       tgt_xclk = min(tgt_xclk, (u32)TCM825X_XCLK_MAX);
+       tgt_xclk = min_t(u32, tgt_xclk, TCM825X_XCLK_MAX);
        tgt_xclk = max(tgt_xclk, (u32)TCM825X_XCLK_MIN);
 
        p->u.bt656.clock_curr = tgt_xclk;