On 2/9/23 08:32, Mark Brown wrote:
On Thu, Feb 09, 2023 at 08:17:23PM +0800, Zhang Yiqun wrote:
This 2nd variables are all set as true in treewide. So I think it can be removed for easy understanding.
Acked-by: Mark Brown broonie@kernel.org
Reviewed-by: Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com
Btw there's another type of always-true parameter, we have a full_reset parameter that's not used.
sound/soc/intel/skylake$ git grep skl_init_chip skl.c:static int skl_init_chip(struct hdac_bus *bus, bool full_reset) skl.c: skl_init_chip(bus, true); skl.c: skl_init_chip(bus, true); skl.c: err = skl_init_chip(bus, true);
sound/soc/intel/avs$ git grep init_chip core.c:static bool avs_hdac_bus_init_chip(struct hdac_bus *bus, bool full_reset) core.c: ret = snd_hdac_bus_init_chip(bus, full_reset); core.c: avs_hdac_bus_init_chip(bus, true); core.c: avs_hdac_bus_init_chip(bus, true); core.c: avs_hdac_bus_init_chip(bus, true);