Messages in this thread |  | | | Date | Mon, 5 May 2008 14:15:14 +1000 | | From | Paul Mackerras <> | | Subject | Re: [PATCH] [POWERPC] Fix of_i2c include for module compilation |
Jochen Friedrich writes:
> -#ifdef CONFIG_OF_I2C
> +#if defined(CONFIG_OF_I2C) || defined(CONFIG_OF_I2C_MODULE)
>
> void of_register_i2c_devices(struct i2c_adapter *adap,
> struct device_node *adap_node);
Why do we have that ifdef there at all? There's only that one
external declaration within it, so the #ifdef and #endif could just be
removed. If the ifdef hadn't been there in the first place we
wouldn't have had this problem.
Paul.
|  |