From: Corey Minyard <cminy...@mvista.com>

It's just noise, really, lots of systems don't have it.

Reported-by: Kamlakant Patel <kamlaka...@marvell.com>
Signed-off-by: Corey Minyard <cminy...@mvista.com>
---
 drivers/char/ipmi/ipmi_si_platform.c | 6 ++----
 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);
-- 
2.17.1



_______________________________________________
Openipmi-developer mailing list
Openipmi-developer@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openipmi-developer

Reply via email to