Skip to content

Commit 1e5058e

Browse files
committed
ipmi: Remove the device id from ipmi_register_smi()
It's no longer used, dynamic device id handling is in place now. Signed-off-by: Corey Minyard <[email protected]>
1 parent aa9c9ab commit 1e5058e

File tree

5 files changed

+1
-25
lines changed

5 files changed

+1
-25
lines changed

drivers/char/ipmi/ipmi_msghandler.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3112,7 +3112,6 @@ EXPORT_SYMBOL(ipmi_poll_interface);
31123112

31133113
int ipmi_register_smi(const struct ipmi_smi_handlers *handlers,
31143114
void *send_info,
3115-
struct ipmi_device_id *device_id,
31163115
struct device *si_dev,
31173116
unsigned char slave_addr)
31183117
{

drivers/char/ipmi/ipmi_powernv.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323

2424
struct ipmi_smi_powernv {
2525
u64 interface_id;
26-
struct ipmi_device_id ipmi_id;
2726
ipmi_smi_t intf;
2827
unsigned int irq;
2928

@@ -266,8 +265,7 @@ static int ipmi_powernv_probe(struct platform_device *pdev)
266265
}
267266

268267
/* todo: query actual ipmi_device_id */
269-
rc = ipmi_register_smi(&ipmi_powernv_smi_handlers, ipmi,
270-
&ipmi->ipmi_id, dev, 0);
268+
rc = ipmi_register_smi(&ipmi_powernv_smi_handlers, ipmi, dev, 0);
271269
if (rc) {
272270
dev_warn(dev, "IPMI SMI registration failed (%d)\n", rc);
273271
goto err_free_msg;

drivers/char/ipmi/ipmi_si_intf.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3631,7 +3631,6 @@ static int try_smi_init(struct smi_info *new_smi)
36313631

36323632
rv = ipmi_register_smi(&handlers,
36333633
new_smi,
3634-
&new_smi->device_id,
36353634
new_smi->dev,
36363635
new_smi->slave_addr);
36373636
if (rv) {

drivers/char/ipmi/ipmi_ssif.c

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -267,9 +267,6 @@ struct ssif_info {
267267
unsigned char *i2c_data;
268268
unsigned int i2c_size;
269269

270-
/* From the device id response. */
271-
struct ipmi_device_id device_id;
272-
273270
struct timer_list retry_timer;
274271
int retries_left;
275272

@@ -1481,21 +1478,6 @@ static int ssif_probe(struct i2c_client *client, const struct i2c_device_id *id)
14811478
ipmi_addr_src_to_str(ssif_info->addr_source),
14821479
client->addr, client->adapter->name, slave_addr);
14831480

1484-
/*
1485-
* Do a Get Device ID command, since it comes back with some
1486-
* useful info.
1487-
*/
1488-
msg[0] = IPMI_NETFN_APP_REQUEST << 2;
1489-
msg[1] = IPMI_GET_DEVICE_ID_CMD;
1490-
rv = do_cmd(client, 2, msg, &len, resp);
1491-
if (rv)
1492-
goto out;
1493-
1494-
rv = ipmi_demangle_device_id(resp[0] >> 2, resp[1],
1495-
resp + 2, len - 2, &ssif_info->device_id);
1496-
if (rv)
1497-
goto out;
1498-
14991481
ssif_info->client = client;
15001482
i2c_set_clientdata(client, ssif_info);
15011483

@@ -1685,7 +1667,6 @@ static int ssif_probe(struct i2c_client *client, const struct i2c_device_id *id)
16851667

16861668
rv = ipmi_register_smi(&ssif_info->handlers,
16871669
ssif_info,
1688-
&ssif_info->device_id,
16891670
&ssif_info->client->dev,
16901671
slave_addr);
16911672
if (rv) {

include/linux/ipmi_smi.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,6 @@ static inline int ipmi_demangle_device_id(uint8_t netfn, uint8_t cmd,
214214
call. */
215215
int ipmi_register_smi(const struct ipmi_smi_handlers *handlers,
216216
void *send_info,
217-
struct ipmi_device_id *device_id,
218217
struct device *dev,
219218
unsigned char slave_addr);
220219

0 commit comments

Comments
 (0)