On Thu, Oct 16, 2008 at 09:12:56PM +0400, Anton Vorontsov wrote:
> For I2C devices we just setting the the node pointer in the archdata.
> This is needed so that the other code would know device's node.
>
> Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]>
Looks okay to me.
Acked-by: Grant Likely <[EMAIL PROTECTED]>
> ---
> drivers/of/of_i2c.c | 11 +++++++++++
> 1 files changed, 11 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/of/of_i2c.c b/drivers/of/of_i2c.c
> index 6a98dc8..f9e18ed 100644
> --- a/drivers/of/of_i2c.c
> +++ b/drivers/of/of_i2c.c
> @@ -24,6 +24,7 @@ void of_register_i2c_devices(struct i2c_adapter *adap,
>
> for_each_child_of_node(adap_node, node) {
> struct i2c_board_info info = {};
> + struct dev_archdata dev_ad = {};
> const u32 *addr;
> int len;
>
> @@ -41,6 +42,9 @@ void of_register_i2c_devices(struct i2c_adapter *adap,
>
> info.addr = *addr;
>
> + dev_archdata_set_node(&dev_ad, node);
> + info.archdata = &dev_ad;
> +
> request_module(info.type);
>
> result = i2c_new_device(adap, &info);
> @@ -51,6 +55,13 @@ void of_register_i2c_devices(struct i2c_adapter *adap,
> irq_dispose_mapping(info.irq);
> continue;
> }
> +
> + /*
> + * Get the node to not lose the dev_archdata->of_node.
> + * Currently there is no way to put it back, as well as no
> + * of_unregister_i2c_devices() call.
> + */
> + of_node_get(node);
> }
> }
> EXPORT_SYMBOL(of_register_i2c_devices);
> --
> 1.5.6.3
>
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
