video: use clamp() macro from kernel.h to replace local defintions

PATCH In review Harvey Harrison 1 participant 1 message Mar 11, 2008 Media (V4L/DVB)

lore

Share & tools

lore

Harvey Harrison

author

2008-03-11 21:13 UTC #0 lore raw

PATCH 2/6 video: use clamp() macro from kernel.h to replace local defintions

Clamps a value to be within a given range with strict typechecking.

Acked-by Mauro Carvalho Chehab <[email protected]>

Signed-off-by Harvey Harrison <[email protected]>

---
 drivers/media/video/bt8xx/bttvp.h    |    2 --
 drivers/media/video/usbvideo/vicam.c |    6 ------
 2 files changed, 0 insertions(+), 8 deletions(-)

drivers/media/video/bt8xx/bttvp.h +0 -2

index 1305d31..b38e3a0 100644
@@ -82,8 +82,6 @@
 /* Limits scaled width, which must be a multiple of 4. */
 #define MAX_HACTIVE (0x3FF & -4)
 
-#define clamp(x, low, high) min (max (low, x), high)
-
 #define BTTV_NORMS    (\
 		V4L2_STD_PAL    | V4L2_STD_PAL_N | \
 		V4L2_STD_PAL_Nc | V4L2_STD_SECAM | \

drivers/media/video/usbvideo/vicam.c +0 -6

index da1ba02..938a60d 100644
@@ -70,12 +70,6 @@
 
 #define VICAM_HEADER_SIZE       64
 
-#define clamp( x, l, h )        max_t( __typeof__( x ),         \
-				       ( l ),                   \
-				       min_t( __typeof__( x ),  \
-					      ( h ),            \
-					      ( x ) ) )
-
 /* Not sure what all the bytes in these char
  * arrays do, but they're necessary to make
  * the camera work.

Signature · 1 line

1.5.4.4.592.g32d4c