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)) { |
