Skip to content

Commit

Permalink
ipmi: Pick up slave address from SMBIOS on an ACPI device
Browse files Browse the repository at this point in the history
When added by ACPI, the information does not contain the slave address
of the BMC.  However, that information is available from SMBIOS.  So
if we add a device that doesn't have a slave address, look at the other
devices that are duplicate interfaces and see if they have a slave
address.

Signed-off-by: Corey Minyard <cminyard@mvista.com>
  • Loading branch information
Corey Minyard committed Nov 25, 2016
1 parent bb2a08c commit 9467171
Show file tree
Hide file tree
Showing 2 changed files with 40 additions and 3 deletions.
10 changes: 9 additions & 1 deletion drivers/char/ipmi/ipmi_si_intf.c
Original file line number Diff line number Diff line change
Expand Up @@ -3448,8 +3448,16 @@ static int is_new_interface(struct smi_info *info)
list_for_each_entry(e, &smi_infos, link) {
if (e->io.addr_type != info->io.addr_type)
continue;
if (e->io.addr_data == info->io.addr_data)
if (e->io.addr_data == info->io.addr_data) {
/*
* This is a cheap hack, ACPI doesn't have a defined
* slave address but SMBIOS does. Pick it up from
* any source that has it available.
*/
if (info->slave_addr && !e->slave_addr)
e->slave_addr = info->slave_addr;
return 0;
}
}

return 1;
Expand Down
33 changes: 31 additions & 2 deletions drivers/char/ipmi/ipmi_ssif.c
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,6 @@ enum ssif_stat_indexes {
};

struct ssif_addr_info {
unsigned short addr;
struct i2c_board_info binfo;
char *adapter_name;
int debug;
Expand Down Expand Up @@ -1401,6 +1400,34 @@ static bool check_acpi(struct ssif_info *ssif_info, struct device *dev)
return false;
}

static int find_slave_address(struct i2c_client *client, int slave_addr)
{
struct ssif_addr_info *info;

if (slave_addr)
return slave_addr;

/*
* Came in without a slave address, search around to see if
* the other sources have a slave address. This lets us pick
* up an SMBIOS slave address when using ACPI.
*/
list_for_each_entry(info, &ssif_infos, link) {
if (info->binfo.addr != client->addr)
continue;
if (info->adapter_name && client->adapter->name &&
strcmp_nospace(info->adapter_name,
client->adapter->name))
continue;
if (info->slave_addr) {
slave_addr = info->slave_addr;
break;
}
}

return slave_addr;
}

/*
* Global enables we care about.
*/
Expand Down Expand Up @@ -1443,6 +1470,8 @@ static int ssif_probe(struct i2c_client *client, const struct i2c_device_id *id)
}
}

slave_addr = find_slave_address(client, slave_addr);

pr_info(PFX "Trying %s-specified SSIF interface at i2c address 0x%x, adapter %s, slave address 0x%x\n",
ipmi_addr_src_to_str(ssif_info->addr_source),
client->addr, client->adapter->name, slave_addr);
Expand Down Expand Up @@ -1931,7 +1960,7 @@ static int decode_dmi(const struct dmi_device *dmi_dev)
slave_addr = data[6];
}

return new_ssif_client(myaddr, NULL, 0, 0, SI_SMBIOS);
return new_ssif_client(myaddr, NULL, 0, slave_addr, SI_SMBIOS);
}

static void dmi_iterator(void)
Expand Down

0 comments on commit 9467171

Please sign in to comment.