# Re: \[PATCH\] \[POWERPC\] Fix of\_i2c include for module compilation

- From: Paul Mackerras \<hidden\>
- Date: 2008-05-05 04:24:53 
- List: linuxppc-dev
- Page: https://ratatoskr.run/linuxppc-dev/2008/05/1095708
- Thread: https://ratatoskr.run/lkml/2008/05/1095142/t.md

```text
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.
```