diff options
author | John Garry <john.garry@huawei.com> | 2017-01-06 06:02:57 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-01-19 14:18:02 -0500 |
commit | 93c94ec23faeea8d9a501b6188c701fc282672b1 (patch) | |
tree | 32fd1c68bb0f9de07ed179690f42d937cbefe766 | |
parent | 61a8c3372adbfc64ee9bcae400908656602c3a92 (diff) |
i2c: print correct device invalid address
commit 6f724fb3039522486fce2e32e4c0fbe238a6ab02 upstream.
In of_i2c_register_device(), when the check for
device address validity fails we print the info.addr,
which has not been assigned properly.
Fix this by printing the actual invalid address.
Signed-off-by: John Garry <john.garry@huawei.com>
Reviewed-by: Vladimir Zapolskiy <vz@mleia.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Fixes: b4e2f6ac1281 ("i2c: apply DT flags when probing")
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/i2c/i2c-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c index b432b64e307a..7484aac1e14d 100644 --- a/drivers/i2c/i2c-core.c +++ b/drivers/i2c/i2c-core.c | |||
@@ -1657,7 +1657,7 @@ static struct i2c_client *of_i2c_register_device(struct i2c_adapter *adap, | |||
1657 | 1657 | ||
1658 | if (i2c_check_addr_validity(addr, info.flags)) { | 1658 | if (i2c_check_addr_validity(addr, info.flags)) { |
1659 | dev_err(&adap->dev, "of_i2c: invalid addr=%x on %s\n", | 1659 | dev_err(&adap->dev, "of_i2c: invalid addr=%x on %s\n", |
1660 | info.addr, node->full_name); | 1660 | addr, node->full_name); |
1661 | return ERR_PTR(-EINVAL); | 1661 | return ERR_PTR(-EINVAL); |
1662 | } | 1662 | } |
1663 | 1663 | ||