On Sat, Oct 24, 2020 at 03:58:54PM +0800, Jiaxin Yu wrote: > +/* mtkaif dmic */ > +static const char * const mt8192_adda_off_on_str[] = { > + "Off", "On" > +}; > + > +static const struct soc_enum mt8192_adda_enum[] = { > + SOC_ENUM_SINGLE_EXT(ARRAY_SIZE(mt8192_adda_off_on_str), > + mt8192_adda_off_on_str), > +}; This should be a standard boolean control with a name ending in Switch rather than an enum.