Subject: Re: [PATCH 1/3] ASoC: Make OpenFirmware helper include file conditional

From: Grant Likely <[email-masked]>
Date: Tue, 22 Jul 2008 17:24:52 -0400

On Tue, Jul 22, 2008 at 12:53:07PM +0100, Mark Brown wrote:
> The OpenFirmware API headers don't build on all platforms so ensure
> that they are not included unless they are being used.
> 
> Signed-off-by: Mark Brown <[email-masked]>

Acked-by: Grant Likely <[email-masked]>

I actually fixed this in my tree, but I forgot to roll it into the patch
before I emailed it.  Thanks.

g.

> ---
>  include/sound/soc-of-simple.h |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/include/sound/soc-of-simple.h b/include/sound/soc-of-simple.h
> index 696fc51..a064e19 100644
> --- a/include/sound/soc-of-simple.h
> +++ b/include/sound/soc-of-simple.h
> @@ -7,6 +7,8 @@
>  #ifndef _INCLUDE_SOC_OF_H_
>  #define _INCLUDE_SOC_OF_H_
>  
> +#if defined(CONFIG_SND_SOC_OF_SIMPLE) || defined(CONFIG_SND_SOC_OF_SIMPLE_MODULE)
> +
>  #include <linux/of.h>
>  #include <sound/soc.h>
>  
> @@ -18,4 +20,6 @@ int of_snd_soc_register_platform(struct snd_soc_platform *platform,
>  				 struct device_node *node,
>  				 struct snd_soc_dai *cpu_dai);
>  
> +#endif
> +
>  #endif /* _INCLUDE_SOC_OF_H_ */
> -- 
> 1.5.6.3
>