diff for duplicates of <1468253261.25088.126.camel@intel.com> diff --git a/a/1.txt b/N1/1.txt index 9a5d227..899cd31 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,26 +1,46 @@ -T24gTW9uLCAyMDE2LTA3LTExIGF0IDExOjE4IC0wNDAwLCBQcmFyaXQgQmhhcmdhdmEgd3JvdGU6 -DQo+IERpZG4ndCBnZXQgYW55IGZlZWRiYWNrIG9yIHJldmlldyBjb21tZW50cyBvbiB0aGlzIHBh -dGNoLsKgwqBSZXNlbmRpbmcNCj4gLi4uDQo+IA0KPiBQLg0KDQpTb3JyeSwgdGhpcyBnb3QgZmxv -b2RlZCBkb3duIG15IGluYm94Lg0KDQoNCj4gLS0tODwtLS0NCj4gDQo+IFRoZSBpd2x3aWZpIGRy -aXZlciBpbXBsZW1lbnRzIGEgdGhlcm1hbCB6b25lIGFuZCBod21vbiBkZXZpY2UsIGJ1dA0KPiBy -ZXR1cm5zIC1FSU8gb24gdGVtcGVyYXR1cmUgcmVhZHMgaWYgdGhlIGZpcm13YXJlIGlzbid0IGxv -YWRlZC7CoMKgVGhpcw0KPiByZXN1bHRzIGluIHRoZSBlcnJvcg0KPiANCj4gaXdsd2lmaS12aXJ0 -dWFsLTANCj4gQWRhcHRlcjogVmlydHVhbCBkZXZpY2UNCj4gRVJST1I6IENhbid0IGdldCB2YWx1 -ZSBvZiBzdWJmZWF0dXJlIHRlbXAxX2lucHV0OiBJL08gZXJyb3INCj4gdGVtcDE6wqDCoMKgwqDC -oMKgwqDCoMKgwqDCoMKgTi9BDQo+IA0KPiBiZWluZyBvdXRwdXQgd2hlbiB1c2luZyBzZW5zb3Jz -IGZyb20gdGhlIGxtLXNlbnNvcnMgcGFja2FnZS7CoMKgU2luY2UNCj4gdGhlIHRlbXBlcmF0dXJl -IGNhbm5vdCBiZSByZWFkIHVubGVzcyB0aGUgdWNvZGUgaXMgbG9hZGVkIHRoZXJlIGlzIG5vDQo+ -IHJlYXNvbiB0byBhZGQgdGhlIGludGVyZmFjZSBvbmx5IHRvIGhhdmUgaXQgcmV0dXJuIGFuIGVy -cm9yIDEwMCUgb2YNCj4gdGhlIHRpbWUuDQo+IA0KPiBUaGlzIHBhdGNoIG1vdmVzIHRoZSBmaXJt -d2FyZSBjaGVjayB0bw0KPiBpd2xfbXZtX3RoZXJtYWxfem9uZV9yZWdpc3RlcigpIGFuZA0KPiBz -dG9wcyB0aGUgdGhlcm1hbCB6b25lIGZyb20gYmVpbmcgY3JlYXRlZCBpZiB0aGUgdWNvZGUgaGFz -bid0IGJlZW4NCj4gbG9hZGVkLg0KPiANCj4gU2lnbmVkLW9mZi1ieTogUHJhcml0IEJoYXJnYXZh -IDxwcmFyaXRAcmVkaGF0LmNvbT4NCj4gQ2M6IEpvaGFubmVzIEJlcmcgPGpvaGFubmVzLmJlcmdA -aW50ZWwuY29tPg0KPiBDYzogRW1tYW51ZWwgR3J1bWJhY2ggPGVtbWFudWVsLmdydW1iYWNoQGlu -dGVsLmNvbT4NCj4gQ2M6IEx1Y2EgQ29lbGhvIDxsdWNpYW5vLmNvZWxob0BpbnRlbC5jb20+DQo+ -IENjOiBJbnRlbCBMaW51eCBXaXJlbGVzcyA8bGludXh3aWZpQGludGVsLmNvbT4NCj4gQ2M6IEth -bGxlIFZhbG8gPGt2YWxvQGNvZGVhdXJvcmEub3JnPg0KPiBDYzogQ2hheWEgUmFjaGVsIEl2Z2kg -PGNoYXlhLnJhY2hlbC5pdmdpQGludGVsLmNvbT4NCj4gQ2M6IFNhcmEgU2hhcm9uIDxzYXJhLnNo -YXJvbkBpbnRlbC5jb20+DQo+IENjOiBsaW51eC13aXJlbGVzc0B2Z2VyLmtlcm5lbC5vcmcNCj4g -Q2M6IG5ldGRldkB2Z2VyLmtlcm5lbC5vcmcNCj4gLS0tDQoNCkkgaGF2ZSBub3cgc2VudCBpdCBm -b3IgcmV2aWV3IG9uIG91ciBpbnRlcm5hbCB0cmVlLg0KDQotLQ0KTHVjYS4= +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 <prarit@redhat.com> +> Cc: Johannes Berg <johannes.berg@intel.com> +> Cc: Emmanuel Grumbach <emmanuel.grumbach@intel.com> +> Cc: Luca Coelho <luciano.coelho@intel.com> +> Cc: Intel Linux Wireless <linuxwifi@intel.com> +> Cc: Kalle Valo <kvalo@codeaurora.org> +> Cc: Chaya Rachel Ivgi <chaya.rachel.ivgi@intel.com> +> Cc: Sara Sharon <sara.sharon@intel.com> +> Cc: linux-wireless@vger.kernel.org +> Cc: netdev@vger.kernel.org +> --- + +I have now sent it for review on our internal tree. + +-- +Luca. diff --git a/a/content_digest b/N1/content_digest index 45a07e2..63dc0ab 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -18,31 +18,51 @@ " Emmanuel <emmanuel.grumbach@intel.com>\0" "\00:1\0" "b\0" - "T24gTW9uLCAyMDE2LTA3LTExIGF0IDExOjE4IC0wNDAwLCBQcmFyaXQgQmhhcmdhdmEgd3JvdGU6\n" - "DQo+IERpZG4ndCBnZXQgYW55IGZlZWRiYWNrIG9yIHJldmlldyBjb21tZW50cyBvbiB0aGlzIHBh\n" - "dGNoLsKgwqBSZXNlbmRpbmcNCj4gLi4uDQo+IA0KPiBQLg0KDQpTb3JyeSwgdGhpcyBnb3QgZmxv\n" - "b2RlZCBkb3duIG15IGluYm94Lg0KDQoNCj4gLS0tODwtLS0NCj4gDQo+IFRoZSBpd2x3aWZpIGRy\n" - "aXZlciBpbXBsZW1lbnRzIGEgdGhlcm1hbCB6b25lIGFuZCBod21vbiBkZXZpY2UsIGJ1dA0KPiBy\n" - "ZXR1cm5zIC1FSU8gb24gdGVtcGVyYXR1cmUgcmVhZHMgaWYgdGhlIGZpcm13YXJlIGlzbid0IGxv\n" - "YWRlZC7CoMKgVGhpcw0KPiByZXN1bHRzIGluIHRoZSBlcnJvcg0KPiANCj4gaXdsd2lmaS12aXJ0\n" - "dWFsLTANCj4gQWRhcHRlcjogVmlydHVhbCBkZXZpY2UNCj4gRVJST1I6IENhbid0IGdldCB2YWx1\n" - "ZSBvZiBzdWJmZWF0dXJlIHRlbXAxX2lucHV0OiBJL08gZXJyb3INCj4gdGVtcDE6wqDCoMKgwqDC\n" - "oMKgwqDCoMKgwqDCoMKgTi9BDQo+IA0KPiBiZWluZyBvdXRwdXQgd2hlbiB1c2luZyBzZW5zb3Jz\n" - "IGZyb20gdGhlIGxtLXNlbnNvcnMgcGFja2FnZS7CoMKgU2luY2UNCj4gdGhlIHRlbXBlcmF0dXJl\n" - "IGNhbm5vdCBiZSByZWFkIHVubGVzcyB0aGUgdWNvZGUgaXMgbG9hZGVkIHRoZXJlIGlzIG5vDQo+\n" - "IHJlYXNvbiB0byBhZGQgdGhlIGludGVyZmFjZSBvbmx5IHRvIGhhdmUgaXQgcmV0dXJuIGFuIGVy\n" - "cm9yIDEwMCUgb2YNCj4gdGhlIHRpbWUuDQo+IA0KPiBUaGlzIHBhdGNoIG1vdmVzIHRoZSBmaXJt\n" - "d2FyZSBjaGVjayB0bw0KPiBpd2xfbXZtX3RoZXJtYWxfem9uZV9yZWdpc3RlcigpIGFuZA0KPiBz\n" - "dG9wcyB0aGUgdGhlcm1hbCB6b25lIGZyb20gYmVpbmcgY3JlYXRlZCBpZiB0aGUgdWNvZGUgaGFz\n" - "bid0IGJlZW4NCj4gbG9hZGVkLg0KPiANCj4gU2lnbmVkLW9mZi1ieTogUHJhcml0IEJoYXJnYXZh\n" - "IDxwcmFyaXRAcmVkaGF0LmNvbT4NCj4gQ2M6IEpvaGFubmVzIEJlcmcgPGpvaGFubmVzLmJlcmdA\n" - "aW50ZWwuY29tPg0KPiBDYzogRW1tYW51ZWwgR3J1bWJhY2ggPGVtbWFudWVsLmdydW1iYWNoQGlu\n" - "dGVsLmNvbT4NCj4gQ2M6IEx1Y2EgQ29lbGhvIDxsdWNpYW5vLmNvZWxob0BpbnRlbC5jb20+DQo+\n" - "IENjOiBJbnRlbCBMaW51eCBXaXJlbGVzcyA8bGludXh3aWZpQGludGVsLmNvbT4NCj4gQ2M6IEth\n" - "bGxlIFZhbG8gPGt2YWxvQGNvZGVhdXJvcmEub3JnPg0KPiBDYzogQ2hheWEgUmFjaGVsIEl2Z2kg\n" - "PGNoYXlhLnJhY2hlbC5pdmdpQGludGVsLmNvbT4NCj4gQ2M6IFNhcmEgU2hhcm9uIDxzYXJhLnNo\n" - "YXJvbkBpbnRlbC5jb20+DQo+IENjOiBsaW51eC13aXJlbGVzc0B2Z2VyLmtlcm5lbC5vcmcNCj4g\n" - "Q2M6IG5ldGRldkB2Z2VyLmtlcm5lbC5vcmcNCj4gLS0tDQoNCkkgaGF2ZSBub3cgc2VudCBpdCBm\n" - b3IgcmV2aWV3IG9uIG91ciBpbnRlcm5hbCB0cmVlLg0KDQotLQ0KTHVjYS4= + "On Mon, 2016-07-11 at 11:18 -0400, Prarit Bhargava wrote:\n" + "> Didn't get any feedback or review comments on this patch.\302\240\302\240Resending\n" + "> ...\n" + "> \n" + "> P.\n" + "\n" + "Sorry, this got flooded down my inbox.\n" + "\n" + "\n" + "> ---8<---\n" + "> \n" + "> The iwlwifi driver implements a thermal zone and hwmon device, but\n" + "> returns -EIO on temperature reads if the firmware isn't loaded.\302\240\302\240This\n" + "> results in the error\n" + "> \n" + "> iwlwifi-virtual-0\n" + "> Adapter: Virtual device\n" + "> ERROR: Can't get value of subfeature temp1_input: I/O error\n" + "> temp1:\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240N/A\n" + "> \n" + "> being output when using sensors from the lm-sensors package.\302\240\302\240Since\n" + "> the temperature cannot be read unless the ucode is loaded there is no\n" + "> reason to add the interface only to have it return an error 100% of\n" + "> the time.\n" + "> \n" + "> This patch moves the firmware check to\n" + "> iwl_mvm_thermal_zone_register() and\n" + "> stops the thermal zone from being created if the ucode hasn't been\n" + "> loaded.\n" + "> \n" + "> Signed-off-by: Prarit Bhargava <prarit@redhat.com>\n" + "> Cc: Johannes Berg <johannes.berg@intel.com>\n" + "> Cc: Emmanuel Grumbach <emmanuel.grumbach@intel.com>\n" + "> Cc: Luca Coelho <luciano.coelho@intel.com>\n" + "> Cc: Intel Linux Wireless <linuxwifi@intel.com>\n" + "> Cc: Kalle Valo <kvalo@codeaurora.org>\n" + "> Cc: Chaya Rachel Ivgi <chaya.rachel.ivgi@intel.com>\n" + "> Cc: Sara Sharon <sara.sharon@intel.com>\n" + "> Cc: linux-wireless@vger.kernel.org\n" + "> Cc: netdev@vger.kernel.org\n" + "> ---\n" + "\n" + "I have now sent it for review on our internal tree.\n" + "\n" + "--\n" + Luca. -3933d8f7e756c5adcad4be919bd3b7ef3076d3e534ca6981d0a53b650b003bb5 +e2105097ea2b77ff61841f5866a08eb2149b53972e6efcd727f1cfd4b6cf4a7d
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.