On Thu, Dec 08, 2016 at 03:30:01PM +0530, Vinod Koul wrote: > On Wed, Dec 07, 2016 at 10:01:42PM -0800, Kirtika Ruchandani wrote: > > Commit 4cd9899f0d16 introduced struct skl_module_fmt* out_fmt without > > using it. Compiling with W=1 gives the below warning, fix it. > > > > sound/soc/intel/skylake/skl-topology.c: In function ‘skl_tplg_update_buffer_size’: > > sound/soc/intel/skylake/skl-topology.c:301:34: warning: variable ‘out_fmt’ set but not used [-Wunused-but-set-variable] > > > > This is a harmless warning and is only being fixed to reduce the noise > > with W=1 in the kernel. > > Heya, > > Can you please tell me which tree this warning was found on? At least this > doesn't exist upstream. The out_fmt is very well used. Sorry I messed up, got confused mcfg->out_fmt variable :( This looks fine: Acked-by: Vinod Koul <vinod.koul@xxxxxxxxx> -- ~Vinod _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
