On Mon, 2016-07-11 at 11:18 -0400, Prarit Bhargava wrote: > Didn't get any feedback or review comments on this patch. Resending > ... > > P.
Sorry, this got flooded down my inbox. > ---8<--- > > The iwlwifi driver implements a thermal zone and hwmon device, but > returns -EIO on temperature reads if the firmware isn't loaded. This > results in the error > > iwlwifi-virtual-0 > Adapter: Virtual device > ERROR: Can't get value of subfeature temp1_input: I/O error > temp1: N/A > > being output when using sensors from the lm-sensors package. Since > the temperature cannot be read unless the ucode is loaded there is no > reason to add the interface only to have it return an error 100% of > the time. > > This patch moves the firmware check to > iwl_mvm_thermal_zone_register() and > stops the thermal zone from being created if the ucode hasn't been > loaded. > > Signed-off-by: Prarit Bhargava <[email protected]> > Cc: Johannes Berg <[email protected]> > Cc: Emmanuel Grumbach <[email protected]> > Cc: Luca Coelho <[email protected]> > Cc: Intel Linux Wireless <[email protected]> > Cc: Kalle Valo <[email protected]> > Cc: Chaya Rachel Ivgi <[email protected]> > Cc: Sara Sharon <[email protected]> > Cc: [email protected] > Cc: [email protected] > --- I have now sent it for review on our internal tree. -- Luca.
