diff options
author | Kevin Cernekee <cernekee@gmail.com> | 2013-02-22 01:58:20 -0500 |
---|---|---|
committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2013-02-22 03:10:19 -0500 |
commit | d18e53fce2f6e42bfb8ac157547dd3f804385749 (patch) | |
tree | 0a599949b2c90566a7db915eea95f64008821ab7 /drivers/input | |
parent | 81bb5d31fbf3893a8e041c649dea704dd11d5272 (diff) |
Input: ALPS - remove unused argument to alps_enter_command_mode()
Now that alps_identify() explicitly issues an EC report using
alps_rpt_cmd(), we no longer need to look at the magic numbers returned
by alps_enter_command_mode().
Signed-off-by: Kevin Cernekee <cernekee@gmail.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Diffstat (limited to 'drivers/input')
-rw-r--r-- | drivers/input/mouse/alps.c | 18 |
1 files changed, 7 insertions, 11 deletions
diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c index 7b99fc7c9438..9c97531bd2cd 100644 --- a/drivers/input/mouse/alps.c +++ b/drivers/input/mouse/alps.c | |||
@@ -994,8 +994,7 @@ static int alps_rpt_cmd(struct psmouse *psmouse, int init_command, | |||
994 | return 0; | 994 | return 0; |
995 | } | 995 | } |
996 | 996 | ||
997 | static int alps_enter_command_mode(struct psmouse *psmouse, | 997 | static int alps_enter_command_mode(struct psmouse *psmouse) |
998 | unsigned char *resp) | ||
999 | { | 998 | { |
1000 | unsigned char param[4]; | 999 | unsigned char param[4]; |
1001 | 1000 | ||
@@ -1009,9 +1008,6 @@ static int alps_enter_command_mode(struct psmouse *psmouse, | |||
1009 | "unknown response while entering command mode\n"); | 1008 | "unknown response while entering command mode\n"); |
1010 | return -1; | 1009 | return -1; |
1011 | } | 1010 | } |
1012 | |||
1013 | if (resp) | ||
1014 | *resp = param[2]; | ||
1015 | return 0; | 1011 | return 0; |
1016 | } | 1012 | } |
1017 | 1013 | ||
@@ -1176,7 +1172,7 @@ static int alps_passthrough_mode_v3(struct psmouse *psmouse, | |||
1176 | { | 1172 | { |
1177 | int reg_val, ret = -1; | 1173 | int reg_val, ret = -1; |
1178 | 1174 | ||
1179 | if (alps_enter_command_mode(psmouse, NULL)) | 1175 | if (alps_enter_command_mode(psmouse)) |
1180 | return -1; | 1176 | return -1; |
1181 | 1177 | ||
1182 | reg_val = alps_command_mode_read_reg(psmouse, reg_base + 0x0008); | 1178 | reg_val = alps_command_mode_read_reg(psmouse, reg_base + 0x0008); |
@@ -1216,7 +1212,7 @@ static int alps_probe_trackstick_v3(struct psmouse *psmouse, int reg_base) | |||
1216 | { | 1212 | { |
1217 | int ret = -EIO, reg_val; | 1213 | int ret = -EIO, reg_val; |
1218 | 1214 | ||
1219 | if (alps_enter_command_mode(psmouse, NULL)) | 1215 | if (alps_enter_command_mode(psmouse)) |
1220 | goto error; | 1216 | goto error; |
1221 | 1217 | ||
1222 | reg_val = alps_command_mode_read_reg(psmouse, reg_base + 0x08); | 1218 | reg_val = alps_command_mode_read_reg(psmouse, reg_base + 0x08); |
@@ -1279,7 +1275,7 @@ static int alps_setup_trackstick_v3(struct psmouse *psmouse, int reg_base) | |||
1279 | * supported by this driver. If bit 1 isn't set the packet | 1275 | * supported by this driver. If bit 1 isn't set the packet |
1280 | * format is different. | 1276 | * format is different. |
1281 | */ | 1277 | */ |
1282 | if (alps_enter_command_mode(psmouse, NULL) || | 1278 | if (alps_enter_command_mode(psmouse) || |
1283 | alps_command_mode_write_reg(psmouse, | 1279 | alps_command_mode_write_reg(psmouse, |
1284 | reg_base + 0x08, 0x82) || | 1280 | reg_base + 0x08, 0x82) || |
1285 | alps_exit_command_mode(psmouse)) | 1281 | alps_exit_command_mode(psmouse)) |
@@ -1306,7 +1302,7 @@ static int alps_hw_init_v3(struct psmouse *psmouse) | |||
1306 | alps_setup_trackstick_v3(psmouse, ALPS_REG_BASE_PINNACLE) == -EIO) | 1302 | alps_setup_trackstick_v3(psmouse, ALPS_REG_BASE_PINNACLE) == -EIO) |
1307 | goto error; | 1303 | goto error; |
1308 | 1304 | ||
1309 | if (alps_enter_command_mode(psmouse, NULL) || | 1305 | if (alps_enter_command_mode(psmouse) || |
1310 | alps_absolute_mode_v3(psmouse)) { | 1306 | alps_absolute_mode_v3(psmouse)) { |
1311 | psmouse_err(psmouse, "Failed to enter absolute mode\n"); | 1307 | psmouse_err(psmouse, "Failed to enter absolute mode\n"); |
1312 | goto error; | 1308 | goto error; |
@@ -1381,7 +1377,7 @@ static int alps_hw_init_rushmore_v3(struct psmouse *psmouse) | |||
1381 | priv->flags &= ~ALPS_DUALPOINT; | 1377 | priv->flags &= ~ALPS_DUALPOINT; |
1382 | } | 1378 | } |
1383 | 1379 | ||
1384 | if (alps_enter_command_mode(psmouse, NULL) || | 1380 | if (alps_enter_command_mode(psmouse) || |
1385 | alps_command_mode_read_reg(psmouse, 0xc2d9) == -1 || | 1381 | alps_command_mode_read_reg(psmouse, 0xc2d9) == -1 || |
1386 | alps_command_mode_write_reg(psmouse, 0xc2cb, 0x00)) | 1382 | alps_command_mode_write_reg(psmouse, 0xc2cb, 0x00)) |
1387 | goto error; | 1383 | goto error; |
@@ -1431,7 +1427,7 @@ static int alps_hw_init_v4(struct psmouse *psmouse) | |||
1431 | struct ps2dev *ps2dev = &psmouse->ps2dev; | 1427 | struct ps2dev *ps2dev = &psmouse->ps2dev; |
1432 | unsigned char param[4]; | 1428 | unsigned char param[4]; |
1433 | 1429 | ||
1434 | if (alps_enter_command_mode(psmouse, NULL)) | 1430 | if (alps_enter_command_mode(psmouse)) |
1435 | goto error; | 1431 | goto error; |
1436 | 1432 | ||
1437 | if (alps_absolute_mode_v4(psmouse)) { | 1433 | if (alps_absolute_mode_v4(psmouse)) { |