Looks good to me.
Thanks Regards Jee Heng
-----Original Message----- From: Nathan Chancellor natechancellor@gmail.com Sent: Wednesday, June 17, 2020 9:03 AM To: Rojewski, Cezary cezary.rojewski@intel.com; Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com; Liam Girdwood liam.r.girdwood@linux.intel.com; Jie Yang yang.jie@linux.intel.com; Mark Brown broonie@kernel.org Cc: Sia, Jee Heng jee.heng.sia@intel.com; alsa-devel@alsa-project.org; linux-kernel@vger.kernel.org; clang-built-linux@googlegroups.com; Nathan Chancellor natechancellor@gmail.com Subject: [PATCH] ASoC: Intel: KeemBay: Fix header guard
Clang warns:
In file included from sound/soc/intel/keembay/kmb_platform.c:14: sound/soc/intel/keembay/kmb_platform.h:9:9: warning: 'KMB_PLATFORM_H_' is used as a header guard here, followed by #define of a different macro [-Wheader-guard] #ifndef KMB_PLATFORM_H_ ^~~~~~~~~~~~~~~ sound/soc/intel/keembay/kmb_platform.h:10:9: note: 'KMB_PLATFORMP_H_' is defined here; did you mean 'KMB_PLATFORM_H_'? #define KMB_PLATFORMP_H_ ^~~~~~~~~~~~~~~~ KMB_PLATFORM_H_ 1 warning generated.
Fix the typo so that the header guard works as intended.
Fixes: c5477e966728 ("ASoC: Intel: Add KeemBay platform driver") Link: https://github.com/ClangBuiltLinux/linux/issues/1053 Signed-off-by: Nathan Chancellor natechancellor@gmail.com --- sound/soc/intel/keembay/kmb_platform.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/intel/keembay/kmb_platform.h b/sound/soc/intel/keembay/kmb_platform.h index 29600652d8f4..6bf221aa8fff 100644 --- a/sound/soc/intel/keembay/kmb_platform.h +++ b/sound/soc/intel/keembay/kmb_platform.h @@ -7,7 +7,7 @@ */
#ifndef KMB_PLATFORM_H_ -#define KMB_PLATFORMP_H_ +#define KMB_PLATFORM_H_
#include <linux/bits.h> #include <linux/bitfield.h>
base-commit: 27f70ec4fa0e0f419031f1b8d61b1a788244e313 -- 2.27.0