On Thu, Oct 08, 2009 at 04:04:32PM +0200, Wolfram Sang wrote:
> As Anton introduced archdata support, I wondered if this is a suitable way to
> handle the platform_data/devicetree_property-dualism (at least for some
> drivers).
Yes, we handle OF in a similar way for mmc_spi driver. Though,
[...]
> --- a/drivers/misc/eeprom/at24.c
> +++ b/drivers/misc/eeprom/at24.c
> @@ -22,6 +22,9 @@
[...]
> +#ifdef CONFIG_OF_I2C
> +#include <linux/of.h>
> +#endif
[..]
> +#ifdef CONFIG_OF_I2C
> +static void at24_get_ofdata(struct i2c_client *client, struct
> at24_platform_data *chip)
> +{
> + const u32 *val;
> + struct device_node *node = dev_archdata_get_node(&client->dev.archdata);
> +
> + if (node) {
> + if (of_get_property(node, "read-only", NULL))
> + chip->flags |= AT24_FLAG_READONLY;
> + val = of_get_property(node, "pagesize", NULL);
> + if (val)
> + chip->page_size = *val;
> + }
> +}
> +#else
> +static void at24_get_ofdata(struct i2c_client *client, struct
> at24_platform_data *chip)
> +{ }
> +#endif
#ifdefs are ugly in .c files. I'd suggest to move the OF code
into a separate file. As an example, see
commit 9c43df57910bbba540a6cb5c9132302a9ea5f41a
Author: Anton Vorontsov <[email protected]>
Date: Tue Dec 30 18:15:28 2008 +0300
mmc_spi: Add support for OpenFirmware bindings
Thanks,
--
Anton Vorontsov
email: [email protected]
irc://irc.freenode.net/bd2
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
