diff options
| author | Corey Minyard <cminyard@mvista.com> | 2019-04-24 12:21:13 -0500 |
|---|---|---|
| committer | Corey Minyard <cminyard@mvista.com> | 2019-04-24 12:29:24 -0500 |
| commit | ed6c3a6d8996659e3bbf4214ba26b5e5a7440b26 (patch) | |
| tree | 653eb1b4d5e8ad7a63050e42fb184b3b24b3212f /drivers/char | |
| parent | 44f56a39addecd6e93b8aa6fa51a6255259edbbb (diff) | |
| download | linux-ed6c3a6d8996659e3bbf4214ba26b5e5a7440b26.tar.bz2 | |
ipmi: Remove warning if no slave address is present
It's just noise, really, lots of systems don't have it.
Reported-by: Kamlakant Patel <kamlakantp@marvell.com>
Signed-off-by: Corey Minyard <cminyard@mvista.com>
Diffstat (limited to 'drivers/char')
| -rw-r--r-- | drivers/char/ipmi/ipmi_si_platform.c | 6 | ||||
| -rw-r--r-- | drivers/char/ipmi/ipmi_ssif.c | 2 |
2 files changed, 3 insertions, 5 deletions
diff --git a/drivers/char/ipmi/ipmi_si_platform.c b/drivers/char/ipmi/ipmi_si_platform.c index 54c7ded2a1ff..f2a91c4d8cab 100644 --- a/drivers/char/ipmi/ipmi_si_platform.c +++ b/drivers/char/ipmi/ipmi_si_platform.c @@ -188,12 +188,10 @@ static int platform_ipmi_probe(struct platform_device *pdev) return -EINVAL; rv = device_property_read_u8(&pdev->dev, "slave-addr", &slave_addr); - if (rv) { - dev_warn(&pdev->dev, "device has no slave-addr property\n"); + if (rv) io.slave_addr = 0x20; - } else { + else io.slave_addr = slave_addr; - } io.irq = platform_get_irq(pdev, 0); if (io.irq > 0) diff --git a/drivers/char/ipmi/ipmi_ssif.c b/drivers/char/ipmi/ipmi_ssif.c index be329f791c95..cf8156d6bc07 100644 --- a/drivers/char/ipmi/ipmi_ssif.c +++ b/drivers/char/ipmi/ipmi_ssif.c @@ -1995,7 +1995,7 @@ static int dmi_ipmi_probe(struct platform_device *pdev) rv = device_property_read_u8(&pdev->dev, "slave-addr", &slave_addr); if (rv) - dev_warn(&pdev->dev, "device has no slave-addr property"); + slave_addr = 0x20; return new_ssif_client(i2c_addr, NULL, 0, slave_addr, SI_SMBIOS, &pdev->dev); |