diff options
author | Corey Minyard <cminyard@mvista.com> | 2015-04-22 14:25:40 -0400 |
---|---|---|
committer | Corey Minyard <cminyard@mvista.com> | 2015-05-05 20:33:48 -0400 |
commit | a182a4b2b3e85a559ea2cd3545f4311db41325f2 (patch) | |
tree | 29b6c15c5593636938eb2eb31b964b8516fbf463 /drivers/char/ipmi | |
parent | 15c5725e6b86cb8dfc4ca655a22005cc678a6f6f (diff) |
ipmi: Report an error if ACPI _IFT doesn't exist
When probing an ACPI table, report a specific error, instead of just
returning an error, if _IFT doesn't exist.
Signed-off-by: Corey Minyard <cminyard@mvista.com>
Diffstat (limited to 'drivers/char/ipmi')
-rw-r--r-- | drivers/char/ipmi/ipmi_si_intf.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c index 461274168d0f..b5a1b450471f 100644 --- a/drivers/char/ipmi/ipmi_si_intf.c +++ b/drivers/char/ipmi/ipmi_si_intf.c | |||
@@ -2262,8 +2262,10 @@ static int ipmi_pnp_probe(struct pnp_dev *dev, | |||
2262 | 2262 | ||
2263 | /* _IFT tells us the interface type: KCS, BT, etc */ | 2263 | /* _IFT tells us the interface type: KCS, BT, etc */ |
2264 | status = acpi_evaluate_integer(handle, "_IFT", NULL, &tmp); | 2264 | status = acpi_evaluate_integer(handle, "_IFT", NULL, &tmp); |
2265 | if (ACPI_FAILURE(status)) | 2265 | if (ACPI_FAILURE(status)) { |
2266 | dev_err(&dev->dev, "Could not find ACPI IPMI interface type\n"); | ||
2266 | goto err_free; | 2267 | goto err_free; |
2268 | } | ||
2267 | 2269 | ||
2268 | switch (tmp) { | 2270 | switch (tmp) { |
2269 | case 1: | 2271 | case 1: |