Commit 04ee53a5 authored by Mark Brown's avatar Mark Brown Committed by Will Deacon

arm64/sve: Fix warnings when SVE is disabled

In configurations where SVE is disabled we define but never reference the
functions for retrieving the default vector length, causing warnings. Fix
this by move the ifdef up, marking get_default_vl() inline since it is
referenced from code guarded by an IS_ENABLED() check, and do the same for
the other accessors for consistency.
Reported-by: default avatarCatalin Marinas <catalin.marinas@arm.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20211022141635.2360415-3-broonie@kernel.orgSigned-off-by: default avatarWill Deacon <will@kernel.org>
parent 49ed9204
...@@ -150,24 +150,24 @@ struct vl_config { ...@@ -150,24 +150,24 @@ struct vl_config {
static struct vl_config vl_config[ARM64_VEC_MAX]; static struct vl_config vl_config[ARM64_VEC_MAX];
static int get_default_vl(enum vec_type type) static inline int get_default_vl(enum vec_type type)
{ {
return READ_ONCE(vl_config[type].__default_vl); return READ_ONCE(vl_config[type].__default_vl);
} }
static int get_sve_default_vl(void) #ifdef CONFIG_ARM64_SVE
static inline int get_sve_default_vl(void)
{ {
return get_default_vl(ARM64_VEC_SVE); return get_default_vl(ARM64_VEC_SVE);
} }
#ifdef CONFIG_ARM64_SVE static inline void set_default_vl(enum vec_type type, int val)
static void set_default_vl(enum vec_type type, int val)
{ {
WRITE_ONCE(vl_config[type].__default_vl, val); WRITE_ONCE(vl_config[type].__default_vl, val);
} }
static void set_sve_default_vl(int val) static inline void set_sve_default_vl(int val)
{ {
set_default_vl(ARM64_VEC_SVE, val); set_default_vl(ARM64_VEC_SVE, val);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment