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@xxxxxxxxxx> Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx> 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);
