Commit 0607512d authored Jun 27, 2017 by Arnd Bergmann Committed by Will Deacon Jun 29, 2017

ras: mark stub functions as 'inline'



With CONFIG_RAS disabled, we get two harmless warnings about
unused functions:

include/linux/ras.h:37:13: error: 'log_arm_hw_error' defined but not used [-Werror=unused-function]
 static void log_arm_hw_error(struct cper_sec_proc_arm *err) { return; }
include/linux/ras.h:33:13: error: 'log_non_standard_event' defined but not used [-Werror=unused-function]
 static void log_non_standard_event(const guid_t *sec_type,

Clearly these are meant to be 'inline', like the other stubs
in the same header.

Fixes: 297b64c7 ("ras: acpi / apei: generate trace event for unrecognized CPER section")
Fixes: e9279e83 ("trace, ras: add ARM processor error trace event")
Acked-by: Borislav Petkov <bp@suse.de>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Will Deacon <will.deacon@arm.com>

parent 3edb1dd1

include/linux/ras.h

+7 −5

Original line number	Diff line number	Diff line
		@@ -30,11 +30,13 @@ void log_non_standard_event(const guid_t *sec_type,
		const u8 sev, const u8 *err, const u32 len);
		void log_arm_hw_error(struct cper_sec_proc_arm *err);
		#else
		static void log_non_standard_event(const guid_t *sec_type,
		static inline void
		log_non_standard_event(const guid_t *sec_type,
		const guid_t fru_id, const char fru_text,
		const u8 sev, const u8 *err,
		const u32 len) { return; }
		static void log_arm_hw_error(struct cper_sec_proc_arm *err) { return; }
		const u8 sev, const u8 *err, const u32 len)
		{ return; }
		static inline void
		log_arm_hw_error(struct cper_sec_proc_arm *err) { return; }
		#endif

		#endif /* __RAS_H__ */

Admin message