From mboxrd@z Thu Jan 1 00:00:00 1970 From: rklein@nvidia.com (Rhyland Klein) Date: Mon, 28 Nov 2011 11:13:21 -0800 Subject: [PATCH v5] power: bq20z75: devicetree init support In-Reply-To: <20111124184215.GA29135@oksana.dev.rtsoft.ru> References: <1316031547-18281-1-git-send-email-rklein@nvidia.com> <20111124184215.GA29135@oksana.dev.rtsoft.ru> Message-ID: <1322507601.3888.2.camel@rklein-linux2> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, 2011-11-24 at 10:42 -0800, Anton Vorontsov wrote: > Hello Rhyland, > > Got this: > > CC [M] drivers/power/bq20z75.o > drivers/power/bq20z75.c: In function ?bq20z75_of_populate_pdata?: > drivers/power/bq20z75.c:630:21: error: storage size of ?gpio_flags? isn?t known > drivers/power/bq20z75.c:668:2: error: implicit declaration of function ?of_get_named_gpio_flags? [-Werror=implicit-function-declaration] > drivers/power/bq20z75.c:671:19: error: ?OF_GPIO_ACTIVE_LOW? undeclared (first use in this function) > drivers/power/bq20z75.c:671:19: note: each undeclared identifier is reported only once for each function it appears in > drivers/power/bq20z75.c:630:21: warning: unused variable ?gpio_flags? [-Wunused-variable] > drivers/power/bq20z75.c:627:29: warning: unused variable ?dtid? [-Wunused-variable] > > I fixed this by... > > On Wed, Sep 14, 2011 at 01:19:07PM -0700, Rhyland Klein wrote: > > > +#if defined(CONFIG_OF) > > +#include > > Adding linux/of_gpio.h > > > +static const struct of_device_id bq20z75_dt_ids[] = { > > + { .compatible = "ti,bq20z75" }, > > + { } > > +}; > > +MODULE_DEVICE_TABLE(i2c, bq20z75_dt_ids); > > + > > +static struct bq20z75_platform_data *bq20z75_of_populate_pdata( > > + struct i2c_client *client) > > +{ > > + const struct of_device_id *dtid; > > And deleting that line. > > > Applied to battery-2.6.git, but please double check if everything > is OK. > > Thanks, > Yah, sorry about that. Seems things have changed a bit since I originally sent out the patch. I synced battery-2.6 and everything looks great. thanks, rhyland