Message ID | 1414528565-10907-6-git-send-email-pantelis.antoniou@konsulko.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Tue, 28 Oct 2014 22:36:02 +0200 , Pantelis Antoniou <pantelis.antoniou@konsulko.com> wrote: > Dynamically inserting i2c client device nodes requires the use > of a single device registration method. Rework and export it. > > Signed-off-by: Pantelis Antoniou <pantelis.antoniou@konsulko.com> Acked-by: Grant Likely <grant.likely@linaro.org> Wolfram, this patch and the next one only make sense in conjunction with the overlay patches, but they don't necessarily need to be merged via the same tree. I'm happy to take them, but am equally happy to have you take them (assuming I ack the next patch; I haven't looked at it yet). What is your preference? g. > --- > drivers/i2c/i2c-core.c | 99 +++++++++++++++++++++++++++----------------------- > include/linux/i2c.h | 10 +++++ > 2 files changed, 64 insertions(+), 45 deletions(-) > > diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c > index 2f90ac6..e6da9d3 100644 > --- a/drivers/i2c/i2c-core.c > +++ b/drivers/i2c/i2c-core.c > @@ -54,6 +54,7 @@ > #include <linux/acpi.h> > #include <linux/jump_label.h> > #include <asm/uaccess.h> > +#include <linux/err.h> > > #include "i2c-core.h" > > @@ -1370,63 +1371,71 @@ static void i2c_scan_static_board_info(struct i2c_adapter *adapter) > /* OF support code */ > > #if IS_ENABLED(CONFIG_OF) > -static void of_i2c_register_devices(struct i2c_adapter *adap) > +struct i2c_client * > +of_i2c_register_device(struct i2c_adapter *adap, > + struct device_node *node) > { > - void *result; > - struct device_node *node; > + struct i2c_client *result; > + struct i2c_board_info info = {}; > + struct dev_archdata dev_ad = {}; > + const __be32 *addr; > + int len; > > - /* Only register child devices if the adapter has a node pointer set */ > - if (!adap->dev.of_node) > - return; > + dev_dbg(&adap->dev, "of_i2c: register %s\n", node->full_name); > > - dev_dbg(&adap->dev, "of_i2c: walking child nodes\n"); > + if (of_modalias_node(node, info.type, sizeof(info.type)) < 0) { > + dev_err(&adap->dev, "of_i2c: modalias failure on %s\n", > + node->full_name); > + return ERR_PTR(-EINVAL); > + } > > - for_each_available_child_of_node(adap->dev.of_node, node) { > - struct i2c_board_info info = {}; > - struct dev_archdata dev_ad = {}; > - const __be32 *addr; > - int len; > + addr = of_get_property(node, "reg", &len); > + if (!addr || (len < sizeof(int))) { > + dev_err(&adap->dev, "of_i2c: invalid reg on %s\n", > + node->full_name); > + return ERR_PTR(-EINVAL); > + } > > - dev_dbg(&adap->dev, "of_i2c: register %s\n", node->full_name); > + info.addr = be32_to_cpup(addr); > + if (info.addr > (1 << 10) - 1) { > + dev_err(&adap->dev, "of_i2c: invalid addr=%x on %s\n", > + info.addr, node->full_name); > + return ERR_PTR(-EINVAL); > + } > > - if (of_modalias_node(node, info.type, sizeof(info.type)) < 0) { > - dev_err(&adap->dev, "of_i2c: modalias failure on %s\n", > - node->full_name); > - continue; > - } > + info.irq = irq_of_parse_and_map(node, 0); > + info.of_node = of_node_get(node); > + info.archdata = &dev_ad; > > - addr = of_get_property(node, "reg", &len); > - if (!addr || (len < sizeof(int))) { > - dev_err(&adap->dev, "of_i2c: invalid reg on %s\n", > - node->full_name); > - continue; > - } > + if (of_get_property(node, "wakeup-source", NULL)) > + info.flags |= I2C_CLIENT_WAKE; > > - info.addr = be32_to_cpup(addr); > - if (info.addr > (1 << 10) - 1) { > - dev_err(&adap->dev, "of_i2c: invalid addr=%x on %s\n", > - info.addr, node->full_name); > - continue; > - } > + request_module("%s%s", I2C_MODULE_PREFIX, info.type); > > - info.irq = irq_of_parse_and_map(node, 0); > - info.of_node = of_node_get(node); > - info.archdata = &dev_ad; > + result = i2c_new_device(adap, &info); > + if (result == NULL) { > + dev_err(&adap->dev, "of_i2c: Failure registering %s\n", > + node->full_name); > + of_node_put(node); > + irq_dispose_mapping(info.irq); > + return ERR_PTR(-EINVAL); > + } > + return result; > +} > +EXPORT_SYMBOL(of_i2c_register_device); > > - if (of_get_property(node, "wakeup-source", NULL)) > - info.flags |= I2C_CLIENT_WAKE; > +static void of_i2c_register_devices(struct i2c_adapter *adap) > +{ > + struct device_node *node; > > - request_module("%s%s", I2C_MODULE_PREFIX, info.type); > + /* Only register child devices if the adapter has a node pointer set */ > + if (!adap->dev.of_node) > + return; > > - result = i2c_new_device(adap, &info); > - if (result == NULL) { > - dev_err(&adap->dev, "of_i2c: Failure registering %s\n", > - node->full_name); > - of_node_put(node); > - irq_dispose_mapping(info.irq); > - continue; > - } > - } > + dev_dbg(&adap->dev, "of_i2c: walking child nodes\n"); > + > + for_each_available_child_of_node(adap->dev.of_node, node) > + of_i2c_register_device(adap, node); > } > > static int of_dev_node_match(struct device *dev, void *data) > diff --git a/include/linux/i2c.h b/include/linux/i2c.h > index b556e0a..22a8f44 100644 > --- a/include/linux/i2c.h > +++ b/include/linux/i2c.h > @@ -558,6 +558,9 @@ static inline int i2c_adapter_id(struct i2c_adapter *adap) > #endif /* I2C */ > > #if IS_ENABLED(CONFIG_OF) > +struct i2c_client * > +of_i2c_register_device(struct i2c_adapter *adap, struct device_node *node); > + > /* must call put_device() when done with returned i2c_client device */ > extern struct i2c_client *of_find_i2c_device_by_node(struct device_node *node); > > @@ -566,6 +569,13 @@ extern struct i2c_adapter *of_find_i2c_adapter_by_node(struct device_node *node) > > #else > > +static inline struct i2c_client * > +of_i2c_register_device(struct i2c_adapter *adap, > + struct device_node *node) > +{ > + return ERR_PTR(-ENODEV); > +} > + > static inline struct i2c_client *of_find_i2c_device_by_node(struct device_node *node) > { > return NULL; > -- > 1.7.12 > -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Nov 21, 2014 at 01:46:31AM +0000, Grant Likely wrote: > On Tue, 28 Oct 2014 22:36:02 +0200 > , Pantelis Antoniou <pantelis.antoniou@konsulko.com> > wrote: > > Dynamically inserting i2c client device nodes requires the use > > of a single device registration method. Rework and export it. > > > > Signed-off-by: Pantelis Antoniou <pantelis.antoniou@konsulko.com> > > Acked-by: Grant Likely <grant.likely@linaro.org> > > Wolfram, this patch and the next one only make sense in conjunction with the overlay > patches, but they don't necessarily need to be merged via the same tree. > I'm happy to take them, but am equally happy to have you take them > (assuming I ack the next patch; I haven't looked at it yet). What is > your preference? OK, now I had the time to look at it, phew. Is this targetted for 3.19? Then you can take it: Reviewed-by: Wolfram Sang <wsa@the-dreams.de> There is a minor conflict caused by this patch in i2c/for-next which should be easy to solve, just remove the irq handling. 2fd36c552649 2014-10-30 15:59 Laurent Pinchart o i2c: core: Map OF IRQ at probe time If you target for later than 3.19, then we need to discuss again, because there might be other i2c core changes coming in.
diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c index 2f90ac6..e6da9d3 100644 --- a/drivers/i2c/i2c-core.c +++ b/drivers/i2c/i2c-core.c @@ -54,6 +54,7 @@ #include <linux/acpi.h> #include <linux/jump_label.h> #include <asm/uaccess.h> +#include <linux/err.h> #include "i2c-core.h" @@ -1370,63 +1371,71 @@ static void i2c_scan_static_board_info(struct i2c_adapter *adapter) /* OF support code */ #if IS_ENABLED(CONFIG_OF) -static void of_i2c_register_devices(struct i2c_adapter *adap) +struct i2c_client * +of_i2c_register_device(struct i2c_adapter *adap, + struct device_node *node) { - void *result; - struct device_node *node; + struct i2c_client *result; + struct i2c_board_info info = {}; + struct dev_archdata dev_ad = {}; + const __be32 *addr; + int len; - /* Only register child devices if the adapter has a node pointer set */ - if (!adap->dev.of_node) - return; + dev_dbg(&adap->dev, "of_i2c: register %s\n", node->full_name); - dev_dbg(&adap->dev, "of_i2c: walking child nodes\n"); + if (of_modalias_node(node, info.type, sizeof(info.type)) < 0) { + dev_err(&adap->dev, "of_i2c: modalias failure on %s\n", + node->full_name); + return ERR_PTR(-EINVAL); + } - for_each_available_child_of_node(adap->dev.of_node, node) { - struct i2c_board_info info = {}; - struct dev_archdata dev_ad = {}; - const __be32 *addr; - int len; + addr = of_get_property(node, "reg", &len); + if (!addr || (len < sizeof(int))) { + dev_err(&adap->dev, "of_i2c: invalid reg on %s\n", + node->full_name); + return ERR_PTR(-EINVAL); + } - dev_dbg(&adap->dev, "of_i2c: register %s\n", node->full_name); + info.addr = be32_to_cpup(addr); + if (info.addr > (1 << 10) - 1) { + dev_err(&adap->dev, "of_i2c: invalid addr=%x on %s\n", + info.addr, node->full_name); + return ERR_PTR(-EINVAL); + } - if (of_modalias_node(node, info.type, sizeof(info.type)) < 0) { - dev_err(&adap->dev, "of_i2c: modalias failure on %s\n", - node->full_name); - continue; - } + info.irq = irq_of_parse_and_map(node, 0); + info.of_node = of_node_get(node); + info.archdata = &dev_ad; - addr = of_get_property(node, "reg", &len); - if (!addr || (len < sizeof(int))) { - dev_err(&adap->dev, "of_i2c: invalid reg on %s\n", - node->full_name); - continue; - } + if (of_get_property(node, "wakeup-source", NULL)) + info.flags |= I2C_CLIENT_WAKE; - info.addr = be32_to_cpup(addr); - if (info.addr > (1 << 10) - 1) { - dev_err(&adap->dev, "of_i2c: invalid addr=%x on %s\n", - info.addr, node->full_name); - continue; - } + request_module("%s%s", I2C_MODULE_PREFIX, info.type); - info.irq = irq_of_parse_and_map(node, 0); - info.of_node = of_node_get(node); - info.archdata = &dev_ad; + result = i2c_new_device(adap, &info); + if (result == NULL) { + dev_err(&adap->dev, "of_i2c: Failure registering %s\n", + node->full_name); + of_node_put(node); + irq_dispose_mapping(info.irq); + return ERR_PTR(-EINVAL); + } + return result; +} +EXPORT_SYMBOL(of_i2c_register_device); - if (of_get_property(node, "wakeup-source", NULL)) - info.flags |= I2C_CLIENT_WAKE; +static void of_i2c_register_devices(struct i2c_adapter *adap) +{ + struct device_node *node; - request_module("%s%s", I2C_MODULE_PREFIX, info.type); + /* Only register child devices if the adapter has a node pointer set */ + if (!adap->dev.of_node) + return; - result = i2c_new_device(adap, &info); - if (result == NULL) { - dev_err(&adap->dev, "of_i2c: Failure registering %s\n", - node->full_name); - of_node_put(node); - irq_dispose_mapping(info.irq); - continue; - } - } + dev_dbg(&adap->dev, "of_i2c: walking child nodes\n"); + + for_each_available_child_of_node(adap->dev.of_node, node) + of_i2c_register_device(adap, node); } static int of_dev_node_match(struct device *dev, void *data) diff --git a/include/linux/i2c.h b/include/linux/i2c.h index b556e0a..22a8f44 100644 --- a/include/linux/i2c.h +++ b/include/linux/i2c.h @@ -558,6 +558,9 @@ static inline int i2c_adapter_id(struct i2c_adapter *adap) #endif /* I2C */ #if IS_ENABLED(CONFIG_OF) +struct i2c_client * +of_i2c_register_device(struct i2c_adapter *adap, struct device_node *node); + /* must call put_device() when done with returned i2c_client device */ extern struct i2c_client *of_find_i2c_device_by_node(struct device_node *node); @@ -566,6 +569,13 @@ extern struct i2c_adapter *of_find_i2c_adapter_by_node(struct device_node *node) #else +static inline struct i2c_client * +of_i2c_register_device(struct i2c_adapter *adap, + struct device_node *node) +{ + return ERR_PTR(-ENODEV); +} + static inline struct i2c_client *of_find_i2c_device_by_node(struct device_node *node) { return NULL;
Dynamically inserting i2c client device nodes requires the use of a single device registration method. Rework and export it. Signed-off-by: Pantelis Antoniou <pantelis.antoniou@konsulko.com> --- drivers/i2c/i2c-core.c | 99 +++++++++++++++++++++++++++----------------------- include/linux/i2c.h | 10 +++++ 2 files changed, 64 insertions(+), 45 deletions(-)