i2c: icy: Constify the software node
Complete software node can now be supplied to the device with struct i2c_board_info. Signed-off-by: Heikki Krogerus <heikki.krogerus@linux.intel.com> Reviewed-by: Max Staudt <max@enpas.org> Signed-off-by: Wolfram Sang <wsa@kernel.org>
This commit is contained in:
parent
239798f5fb
commit
dd7a37102b
1 changed files with 9 additions and 23 deletions
|
@ -54,7 +54,6 @@ struct icy_i2c {
|
||||||
|
|
||||||
void __iomem *reg_s0;
|
void __iomem *reg_s0;
|
||||||
void __iomem *reg_s1;
|
void __iomem *reg_s1;
|
||||||
struct fwnode_handle *ltc2990_fwnode;
|
|
||||||
struct i2c_client *ltc2990_client;
|
struct i2c_client *ltc2990_client;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
@ -115,6 +114,10 @@ static const struct property_entry icy_ltc2990_props[] = {
|
||||||
{ }
|
{ }
|
||||||
};
|
};
|
||||||
|
|
||||||
|
static const struct software_node icy_ltc2990_node = {
|
||||||
|
.properties = icy_ltc2990_props,
|
||||||
|
};
|
||||||
|
|
||||||
static int icy_probe(struct zorro_dev *z,
|
static int icy_probe(struct zorro_dev *z,
|
||||||
const struct zorro_device_id *ent)
|
const struct zorro_device_id *ent)
|
||||||
{
|
{
|
||||||
|
@ -123,6 +126,7 @@ static int icy_probe(struct zorro_dev *z,
|
||||||
struct fwnode_handle *new_fwnode;
|
struct fwnode_handle *new_fwnode;
|
||||||
struct i2c_board_info ltc2990_info = {
|
struct i2c_board_info ltc2990_info = {
|
||||||
.type = "ltc2990",
|
.type = "ltc2990",
|
||||||
|
.swnode = &icy_ltc2990_node,
|
||||||
};
|
};
|
||||||
|
|
||||||
i2c = devm_kzalloc(&z->dev, sizeof(*i2c), GFP_KERNEL);
|
i2c = devm_kzalloc(&z->dev, sizeof(*i2c), GFP_KERNEL);
|
||||||
|
@ -174,26 +178,10 @@ static int icy_probe(struct zorro_dev *z,
|
||||||
*
|
*
|
||||||
* See property_entry above for in1, in2, temp3.
|
* See property_entry above for in1, in2, temp3.
|
||||||
*/
|
*/
|
||||||
new_fwnode = fwnode_create_software_node(icy_ltc2990_props, NULL);
|
i2c->ltc2990_client = i2c_new_scanned_device(&i2c->adapter,
|
||||||
if (IS_ERR(new_fwnode)) {
|
<c2990_info,
|
||||||
dev_info(&z->dev, "Failed to create fwnode for LTC2990, error: %ld\n",
|
icy_ltc2990_addresses,
|
||||||
PTR_ERR(new_fwnode));
|
NULL);
|
||||||
} else {
|
|
||||||
/*
|
|
||||||
* Store the fwnode so we can destroy it on .remove().
|
|
||||||
* Only store it on success, as fwnode_remove_software_node()
|
|
||||||
* is NULL safe, but not PTR_ERR safe.
|
|
||||||
*/
|
|
||||||
i2c->ltc2990_fwnode = new_fwnode;
|
|
||||||
ltc2990_info.fwnode = new_fwnode;
|
|
||||||
|
|
||||||
i2c->ltc2990_client =
|
|
||||||
i2c_new_scanned_device(&i2c->adapter,
|
|
||||||
<c2990_info,
|
|
||||||
icy_ltc2990_addresses,
|
|
||||||
NULL);
|
|
||||||
}
|
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -202,8 +190,6 @@ static void icy_remove(struct zorro_dev *z)
|
||||||
struct icy_i2c *i2c = dev_get_drvdata(&z->dev);
|
struct icy_i2c *i2c = dev_get_drvdata(&z->dev);
|
||||||
|
|
||||||
i2c_unregister_device(i2c->ltc2990_client);
|
i2c_unregister_device(i2c->ltc2990_client);
|
||||||
fwnode_remove_software_node(i2c->ltc2990_fwnode);
|
|
||||||
|
|
||||||
i2c_del_adapter(&i2c->adapter);
|
i2c_del_adapter(&i2c->adapter);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Reference in a new issue