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 > _______________________________________________ i2c mailing list i2c@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/i2c