On Tue, 2016-11-08 at 21:50 -0800, Kirtika Ruchandani wrote: > shift_param is defined and set in iwl_pcie_load_cpu_sections > but not used. Fix this to avoid -Wunused-but-set-variable > warning. > The code using it turned into dead code with commit dcab8ecd5617 > which added a separate function iwl_pcie_load_given_ucode_8000 > (then 8000b) for IWL_DEVICE_FAMILY_8000. Commit 76f8c0e17edc > removed the dead code but left shift_param as is. > > iwlwifi/pcie/trans.c: In function ‘iwl_pcie_load_cpu_sections’: > iwlwifi/pcie/trans.c:871:6: warning: variable ‘shift_param’ set but not used > [-Wunused-but-set-variable] > > Fixes: dcab8ecd5617 (iwlwifi: mvm: support ucode load for family_8000 B0 only) > Fixes: 76f8c0e17edc (iwlwifi: pcie: remove dead code) > > Signed-off-by: Kirtika Ruchandani <kirt...@google.com> > Cc: Sara Sharon <sara.sha...@intel.com> > Cc: Luca Coelho <luciano.coe...@intel.com> > Cc: Eran Harary <eran.har...@intel.com>
I'll remove Harary's Cc here too, since this email is not valid anymore and I don't know his new one (and he probably doesn't care anymore anyway). -- Luca.
