[072/107] Input: properly assign return value of clamp() macro.

From: Greg KH
Date: Fri Jul 08 2011 - 01:04:28 EST


2.6.39-stable review patch. If anyone has any objections, please let us know.

------------------

From: Hans Petter Selasky <hselasky@xxxxxxx>

commit 8c127f0717b438e6abc3d92d4ae248c4224b9dcb upstream.

[dtor@xxxxxxx: added mousedev changes]
Signed-off-by: Hans Petter Selasky <hselasky@xxxxxxx>
Signed-off-by: Dmitry Torokhov <dtor@xxxxxxx>
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxx>

---
drivers/input/input.c | 2 +-
drivers/input/mousedev.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)

--- a/drivers/input/input.c
+++ b/drivers/input/input.c
@@ -1757,7 +1757,7 @@ static unsigned int input_estimate_event
} else if (test_bit(ABS_MT_TRACKING_ID, dev->absbit)) {
mt_slots = dev->absinfo[ABS_MT_TRACKING_ID].maximum -
dev->absinfo[ABS_MT_TRACKING_ID].minimum + 1,
- clamp(mt_slots, 2, 32);
+ mt_slots = clamp(mt_slots, 2, 32);
} else if (test_bit(ABS_MT_POSITION_X, dev->absbit)) {
mt_slots = 2;
} else {
--- a/drivers/input/mousedev.c
+++ b/drivers/input/mousedev.c
@@ -187,7 +187,7 @@ static void mousedev_abs_event(struct in
if (size == 0)
size = xres ? : 1;

- clamp(value, min, max);
+ value = clamp(value, min, max);

mousedev->packet.x = ((value - min) * xres) / size;
mousedev->packet.abs_event = 1;
@@ -201,7 +201,7 @@ static void mousedev_abs_event(struct in
if (size == 0)
size = yres ? : 1;

- clamp(value, min, max);
+ value = clamp(value, min, max);

mousedev->packet.y = yres - ((value - min) * yres) / size;
mousedev->packet.abs_event = 1;


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/