i2c: of: rename variable to meet expectations
'result' is mostly used in the kernel as int for functions returning errno on failure. Here it is a pointer to the client struct, so let's call it this way (as the parent function does, too). Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
This commit is contained in:
parent
e6db2d3278
commit
c49b0e0775
|
@ -25,7 +25,7 @@
|
||||||
static struct i2c_client *of_i2c_register_device(struct i2c_adapter *adap,
|
static struct i2c_client *of_i2c_register_device(struct i2c_adapter *adap,
|
||||||
struct device_node *node)
|
struct device_node *node)
|
||||||
{
|
{
|
||||||
struct i2c_client *result;
|
struct i2c_client *client;
|
||||||
struct i2c_board_info info = {};
|
struct i2c_board_info info = {};
|
||||||
struct dev_archdata dev_ad = {};
|
struct dev_archdata dev_ad = {};
|
||||||
const __be32 *addr_be;
|
const __be32 *addr_be;
|
||||||
|
@ -73,13 +73,13 @@ static struct i2c_client *of_i2c_register_device(struct i2c_adapter *adap,
|
||||||
if (of_get_property(node, "wakeup-source", NULL))
|
if (of_get_property(node, "wakeup-source", NULL))
|
||||||
info.flags |= I2C_CLIENT_WAKE;
|
info.flags |= I2C_CLIENT_WAKE;
|
||||||
|
|
||||||
result = i2c_new_device(adap, &info);
|
client = i2c_new_device(adap, &info);
|
||||||
if (result == NULL) {
|
if (!client) {
|
||||||
dev_err(&adap->dev, "of_i2c: Failure registering %pOF\n", node);
|
dev_err(&adap->dev, "of_i2c: Failure registering %pOF\n", node);
|
||||||
of_node_put(node);
|
of_node_put(node);
|
||||||
return ERR_PTR(-EINVAL);
|
return ERR_PTR(-EINVAL);
|
||||||
}
|
}
|
||||||
return result;
|
return client;
|
||||||
}
|
}
|
||||||
|
|
||||||
void of_i2c_register_devices(struct i2c_adapter *adap)
|
void of_i2c_register_devices(struct i2c_adapter *adap)
|
||||||
|
|
Loading…
Reference in New Issue