The variable types are simply larger than they need to be. Shrink to signed and unsigned chars. Signed-off-by: George Spelvin <linux@xxxxxxxxxxx> --- drivers/media/rc/ati_remote.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/rc/ati_remote.c b/drivers/media/rc/ati_remote.c index 3ddd66a23d..b92da56e9a 100644 --- a/drivers/media/rc/ati_remote.c +++ b/drivers/media/rc/ati_remote.c @@ -289,11 +289,11 @@ struct ati_remote { /* Translation table from hardware messages to input events. */ static const struct { - short kind; + unsigned char kind; unsigned char data; - int type; - unsigned int code; - int value; + unsigned char type; + unsigned short code; + signed char value; } ati_remote_tbl[] = { /* Directional control pad axes */ {KIND_ACCEL, 0x70, EV_REL, REL_X, -1}, /* left */ -- 1.9.2 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html