[alsa-devel] [PATCH] ASoC: SOF: uapi headers - add missing include for stdint.h

Pierre-Louis Bossart pierre-louis.bossart at linux.intel.com
Tue May 28 23:45:46 CEST 2019


Hi Jaroslav,

On 5/28/19 3:02 PM, Jaroslav Kysela wrote:
> The modified header files depend on types defined in <stdint.h>.

Humm, not an objection but more a question: may I ask in which cases the 
compilation would fail or what types were problematic? I see stdint.h 
being used by only 3 files in include/uapi so not sure if we missed 
something?
Also we removed some of those files recently since they were not used by 
the kernel, we only have abi.h, fw.h, header.h and tokens.h in this 
directory, so that patch would not apply against Mark's tree.

> 
> Signed-off-by: Jaroslav Kysela <perex at perex.cz>
> Cc: Liam Girdwood <liam.r.girdwood at linux.intel.com>
> Cc: Pierre-Louis Bossart <pierre-louis.bossart at linux.intel.com>
> Cc: Takashi Iwai <tiwai at suse.de>
> Cc: Mark Brown <broonie at kernel.org>
> ---
>   include/uapi/sound/sof/eq.h       | 4 ++++
>   include/uapi/sound/sof/fw.h       | 4 ++++
>   include/uapi/sound/sof/header.h   | 4 ++++
>   include/uapi/sound/sof/manifest.h | 4 ++++
>   include/uapi/sound/sof/trace.h    | 4 ++++
>   5 files changed, 20 insertions(+)
> 
> diff --git a/include/uapi/sound/sof/eq.h b/include/uapi/sound/sof/eq.h
> index 666c2b6a3229..106d56e357e0 100644
> --- a/include/uapi/sound/sof/eq.h
> +++ b/include/uapi/sound/sof/eq.h
> @@ -9,6 +9,10 @@
>   #ifndef __INCLUDE_UAPI_SOUND_SOF_USER_EQ_H__
>   #define __INCLUDE_UAPI_SOUND_SOF_USER_EQ_H__
>   
> +#ifndef __KERNEL__
> +#include <stdint.h>
> +#endif
> +
>   /* FIR EQ type */
>   
>   #define SOF_EQ_FIR_IDX_SWITCH	0
> diff --git a/include/uapi/sound/sof/fw.h b/include/uapi/sound/sof/fw.h
> index 1afca973eb09..4f2de19a3b86 100644
> --- a/include/uapi/sound/sof/fw.h
> +++ b/include/uapi/sound/sof/fw.h
> @@ -13,6 +13,10 @@
>   #ifndef __INCLUDE_UAPI_SOF_FW_H__
>   #define __INCLUDE_UAPI_SOF_FW_H__
>   
> +#ifndef __KERNEL__
> +#include <stdint.h>
> +#endif
> +
>   #define SND_SOF_FW_SIG_SIZE	4
>   #define SND_SOF_FW_ABI		1
>   #define SND_SOF_FW_SIG		"Reef"
> diff --git a/include/uapi/sound/sof/header.h b/include/uapi/sound/sof/header.h
> index 7868990b0d6f..6a62ae8f0eb9 100644
> --- a/include/uapi/sound/sof/header.h
> +++ b/include/uapi/sound/sof/header.h
> @@ -9,6 +9,10 @@
>   #ifndef __INCLUDE_UAPI_SOUND_SOF_USER_HEADER_H__
>   #define __INCLUDE_UAPI_SOUND_SOF_USER_HEADER_H__
>   
> +#ifndef __KERNEL__
> +#include <stdint.h>
> +#endif
> +
>   /*
>    * Header for all non IPC ABI data.
>    *
> diff --git a/include/uapi/sound/sof/manifest.h b/include/uapi/sound/sof/manifest.h
> index 2009ee30fad0..d57aa2bc4764 100644
> --- a/include/uapi/sound/sof/manifest.h
> +++ b/include/uapi/sound/sof/manifest.h
> @@ -9,6 +9,10 @@
>   #ifndef __INCLUDE_UAPI_SOUND_SOF_USER_MANIFEST_H__
>   #define __INCLUDE_UAPI_SOUND_SOF_USER_MANIFEST_H__
>   
> +#ifndef __KERNEL__
> +#include <stdint.h>
> +#endif
> +
>   /* start offset for base FW module */
>   #define SOF_MAN_ELF_TEXT_OFFSET		0x2000
>   
> diff --git a/include/uapi/sound/sof/trace.h b/include/uapi/sound/sof/trace.h
> index ffa7288a0f16..1652bc08d576 100644
> --- a/include/uapi/sound/sof/trace.h
> +++ b/include/uapi/sound/sof/trace.h
> @@ -9,6 +9,10 @@
>   #ifndef __INCLUDE_UAPI_SOUND_SOF_USER_TRACE_H__
>   #define __INCLUDE_UAPI_SOUND_SOF_USER_TRACE_H__
>   
> +#ifndef __KERNEL__
> +#include <stdint.h>
> +#endif
> +
>   /*
>    * Host system time.
>    *
> 



More information about the Alsa-devel mailing list