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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxx>
Cc: Liam Girdwood <liam.r.girdwood@xxxxxxxxxxxxxxx>
Cc: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>
Cc: Takashi Iwai <tiwai@xxxxxxx>
Cc: Mark Brown <broonie@xxxxxxxxxx>
---
  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.
   *


_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux