diff for duplicates of <5516B705.1010800@roeck-us.net> diff --git a/a/1.txt b/N1/1.txt index b488de0..316686b 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,43 +1,66 @@ -T24gMDMvMjgvMjAxNSAwNTo1NCBBTSwgU3RldmVuIEhvbmV5bWFuIHdyb3RlOgo+IE9uIDI4IE1h -cmNoIDIwMTUgYXQgMTE6MDQsIFBhdWwgQm9sbGUgPHBlYm9sbGVAdGlzY2FsaS5ubD4gd3JvdGU6 -Cj4+IE9uIFNhdCwgMjAxNS0wMy0yOCBhdCAxMToyNCArMDEwMCwgUGFsaSBSb2jDoXIgd3JvdGU6 -Cj4+PiAtLS0gYS9kcml2ZXJzL2h3bW9uL0tjb25maWcKPj4+ICsrKyBiL2RyaXZlcnMvaHdtb24v -S2NvbmZpZwo+Pj4gQEAgLTE3MDMsNiArMTcwMywxNyBAQCBjb25maWcgU0VOU09SU19VTFRSQTQ1 -Cj4+PiAgICAgICAgICBUaGlzIGRyaXZlciBwcm92aWRlcyBzdXBwb3J0IGZvciB0aGUgVWx0cmE0 -NSB3b3Jrc3RhdGlvbiBlbnZpcm9ubWVudGFsCj4+PiAgICAgICAgICBzZW5zb3JzLgo+Pj4KPj4+ -ICtjb25maWcgU0VOU09SU19ERUxMX1NNTQo+Pj4gKyAgICAgdHJpc3RhdGUgIkRlbGwgbGFwdG9w -IFNNTSBCSU9TIGh3bW9uIGRyaXZlciIKPj4+ICsgICAgIGRlcGVuZHMgT04gWDg2Cj4+Cj4+IEhv -dyBkaWQgdGhpcyBwYXN0IHlvdXIgdGVzdGluZz8KPj4KPj4+ICsgICAgIC0tLWhlbHAtLS0KPj4+ -ICsgICAgICAgVGhpcyBod21vbiBkcml2ZXIgYWRkcyBzdXBwb3J0IGZvciByZXBvcnRpbmcgdGVt -cGVyYXR1cmUgb2YgZGlmZmVyZW50Cj4+PiArICAgICAgIHNlbnNvcnMgYW5kIGNvbnRyb2xzIHRo -ZSBmYW5zIG9uIERlbGwgbGFwdG9wcyB2aWEgU3lzdGVtIE1hbmFnZW1lbnQKPj4+ICsgICAgICAg -TW9kZSBwcm92aWRlZCBieSBEZWxsIEJJT1MuCj4+PiArCj4+PiArICAgICAgIFdoZW4gb3B0aW9u -IEk4SyBpcyBhbHNvIGVuYWJsZWQgdGhpcyBkcml2ZXIgcHJvdmlkZXMgbGVnYWN5IC9wcm9jL2k4 -awo+Pj4gKyAgICAgICB1c2Vyc3BhY2UgaW50ZXJmYWNlIGZvciBpOGt1dGlscyBwYWNrYWdlLgo+ -Pj4gKwo+Cj4gSXQncyB3b3JraW5nIE9LIGZvciBtZSAoYWZ0ZXIgZml4aW5nIHRoZSBtaXN0YWtl -IHBvaW50ZWQgb3V0IGFib3ZlKS4KPgo+IFtMYXRpdHVkZSBlNjU0MF0KPiAtLS0tLQo+IGNvcmV0 -ZW1wLWlzYS0wMDAwCj4gQWRhcHRlcjogSVNBIGFkYXB0ZXIKPiBQaHlzaWNhbCBpZCAwOiAgKzUz -LjDCsEMgIChoaWdoID0gKzg0LjDCsEMsIGNyaXQgPSArMTAwLjDCsEMpCj4gQ29yZSAwOiAgICAg -ICAgICs1MS4wwrBDICAoaGlnaCA9ICs4NC4wwrBDLCBjcml0ID0gKzEwMC4wwrBDKQo+IENvcmUg -MTogICAgICAgICArNTMuMMKwQyAgKGhpZ2ggPSArODQuMMKwQywgY3JpdCA9ICsxMDAuMMKwQykK -Pgo+IGk4ay12aXJ0dWFsLTAKPiBBZGFwdGVyOiBWaXJ0dWFsIGRldmljZQo+IFByb2Nlc3NvciBG -YW46IDMxNzEgUlBNCj4gQ1BVOiAgICAgICAgICAgICs1Mi4wwrBDCj4gQW1iaWVudDogICAgICAg -ICs0Ni4wwrBDCj4gU09ESU1NOiAgICAgICAgICs0MC4wwrBDCj4gLS0tLS0KPgo+IEFtYmllbnQg -YW5kIFNPRElNTSBsb29rIHN3YXBwZWQgLSBidXQgaXQncyBwcm9iYWJseSBqdXN0IGFub3RoZXIg -RGVsbAo+IGVycm9yLiBJIHJlYWxseSBuZWVkIHRvIGxvb2sgYXQgZ2V0dGluZyBjb3JlYm9vdCBv -biB0aGlzIHRoaW5nLgo+Cj4gT24gMjggTWFyY2ggMjAxNSBhdCAxMDoyNCwgUGFsaSBSb2jDoXIg -PHBhbGkucm9oYXJAZ21haWwuY29tPiB3cm90ZToKPiAuLi4KPj4gICBjb25maWcgSThLCj4+IC0g -ICAgICAgdHJpc3RhdGUgIkRlbGwgbGFwdG9wIHN1cHBvcnQiCj4+IC0gICAgICAgc2VsZWN0IEhX -TU9OCj4+ICsgICAgICAgYm9vbCAiRGVsbCBpOGsgbGVnYWN5IGxhcHRvcCBzdXBwb3J0Igo+PiAr -ICAgICAgIHNlbGVjdCBTRU5TT1JTX0RFTExfU01NCj4gLi4uCj4+ICtjb25maWcgU0VOU09SU19E -RUxMX1NNTQo+PiArICAgICAgIHRyaXN0YXRlICJEZWxsIGxhcHRvcCBTTU0gQklPUyBod21vbiBk -cml2ZXIiCj4KPiBUaGUgb25seSBjaGFuZ2UgSSdkIHN1Z2dlc3QgaXMgbm90IHRvIGNoYW5nZSB0 -aGUgdHJpc3RhdGUgdG8gYm9vbCBmb3IKPiBJOEsuIEZvcmNpbmcgdGhhdCB0byBib29sIG1lYW5z -IHRoYXQgU0VOU09SU19ERUxMX1NNTSBjYW4ndCBiZQo+IGNvbXBpbGVkIGFzIGEgbW9kdWxlIGlm -IHRoZSB1c2VyIHdpc2hlcyB0byBrZWVwIGk4ayBoYW5naW5nIGFyb3VuZAo+ICJqdXN0IGluIGNh -c2UiLiBJdCdsbCBwcm9iYWJseSBhbm5veSBkaXN0cm8ga2VybmVsIHBhY2thZ2VycyB0b28uCj4K -CkFncmVlZC4gSXQgYWxzbyBzdGlsbCBuZWVkcyB0byBzZWxlY3QgSFdNT04gKFNFTlNPUlNfREVM -TF9TTU0gaXMKb25seSB2aXNpYmxlIGlmIEhXTU9OIGlzIHNlbGVjdGVkKS4KCkd1ZW50ZXIKCgpf -X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsbS1zZW5zb3Jz -IG1haWxpbmcgbGlzdApsbS1zZW5zb3JzQGxtLXNlbnNvcnMub3JnCmh0dHA6Ly9saXN0cy5sbS1z -ZW5zb3JzLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xtLXNlbnNvcnM +On 03/28/2015 05:54 AM, Steven Honeyman wrote: +> On 28 March 2015 at 11:04, Paul Bolle <pebolle@tiscali.nl> wrote: +>> On Sat, 2015-03-28 at 11:24 +0100, Pali Rohár wrote: +>>> --- a/drivers/hwmon/Kconfig +>>> +++ b/drivers/hwmon/Kconfig +>>> @@ -1703,6 +1703,17 @@ config SENSORS_ULTRA45 +>>> This driver provides support for the Ultra45 workstation environmental +>>> sensors. +>>> +>>> +config SENSORS_DELL_SMM +>>> + tristate "Dell laptop SMM BIOS hwmon driver" +>>> + depends ON X86 +>> +>> How did this past your testing? +>> +>>> + ---help--- +>>> + This hwmon driver adds support for reporting temperature of different +>>> + sensors and controls the fans on Dell laptops via System Management +>>> + Mode provided by Dell BIOS. +>>> + +>>> + When option I8K is also enabled this driver provides legacy /proc/i8k +>>> + userspace interface for i8kutils package. +>>> + +> +> It's working OK for me (after fixing the mistake pointed out above). +> +> [Latitude e6540] +> ----- +> coretemp-isa-0000 +> Adapter: ISA adapter +> Physical id 0: +53.0°C (high = +84.0°C, crit = +100.0°C) +> Core 0: +51.0°C (high = +84.0°C, crit = +100.0°C) +> Core 1: +53.0°C (high = +84.0°C, crit = +100.0°C) +> +> i8k-virtual-0 +> Adapter: Virtual device +> Processor Fan: 3171 RPM +> CPU: +52.0°C +> Ambient: +46.0°C +> SODIMM: +40.0°C +> ----- +> +> Ambient and SODIMM look swapped - but it's probably just another Dell +> error. I really need to look at getting coreboot on this thing. +> +> On 28 March 2015 at 10:24, Pali Rohár <pali.rohar@gmail.com> wrote: +> ... +>> config I8K +>> - tristate "Dell laptop support" +>> - select HWMON +>> + bool "Dell i8k legacy laptop support" +>> + select SENSORS_DELL_SMM +> ... +>> +config SENSORS_DELL_SMM +>> + tristate "Dell laptop SMM BIOS hwmon driver" +> +> The only change I'd suggest is not to change the tristate to bool for +> I8K. Forcing that to bool means that SENSORS_DELL_SMM can't be +> compiled as a module if the user wishes to keep i8k hanging around +> "just in case". It'll probably annoy distro kernel packagers too. +> + +Agreed. It also still needs to select HWMON (SENSORS_DELL_SMM is +only visible if HWMON is selected). + +Guenter diff --git a/a/content_digest b/N1/content_digest index 84758b2..b2c330d 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -3,8 +3,8 @@ "ref\01427540660.2334.14.camel@x220\0" "ref\0CABz95_BkAvPoBAm_tRLH6OhCk_pDfJQ8UuwfhuEYuPNt3P-PGw@mail.gmail.com\0" "From\0Guenter Roeck <linux@roeck-us.net>\0" - "Subject\0Re: [lm-sensors] [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k\0" - "Date\0Sat, 28 Mar 2015 14:13:25 +0000\0" + "Subject\0Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k\0" + "Date\0Sat, 28 Mar 2015 07:13:25 -0700\0" "To\0Steven Honeyman <stevenhoneyman@gmail.com>" " Paul Bolle <pebolle@tiscali.nl>\0" "Cc\0Pali Roh\303\241r <pali.rohar@gmail.com>" @@ -18,48 +18,71 @@ " lm-sensors@lm-sensors.org\0" "\00:1\0" "b\0" - "T24gMDMvMjgvMjAxNSAwNTo1NCBBTSwgU3RldmVuIEhvbmV5bWFuIHdyb3RlOgo+IE9uIDI4IE1h\n" - "cmNoIDIwMTUgYXQgMTE6MDQsIFBhdWwgQm9sbGUgPHBlYm9sbGVAdGlzY2FsaS5ubD4gd3JvdGU6\n" - "Cj4+IE9uIFNhdCwgMjAxNS0wMy0yOCBhdCAxMToyNCArMDEwMCwgUGFsaSBSb2jDoXIgd3JvdGU6\n" - "Cj4+PiAtLS0gYS9kcml2ZXJzL2h3bW9uL0tjb25maWcKPj4+ICsrKyBiL2RyaXZlcnMvaHdtb24v\n" - "S2NvbmZpZwo+Pj4gQEAgLTE3MDMsNiArMTcwMywxNyBAQCBjb25maWcgU0VOU09SU19VTFRSQTQ1\n" - "Cj4+PiAgICAgICAgICBUaGlzIGRyaXZlciBwcm92aWRlcyBzdXBwb3J0IGZvciB0aGUgVWx0cmE0\n" - "NSB3b3Jrc3RhdGlvbiBlbnZpcm9ubWVudGFsCj4+PiAgICAgICAgICBzZW5zb3JzLgo+Pj4KPj4+\n" - "ICtjb25maWcgU0VOU09SU19ERUxMX1NNTQo+Pj4gKyAgICAgdHJpc3RhdGUgIkRlbGwgbGFwdG9w\n" - "IFNNTSBCSU9TIGh3bW9uIGRyaXZlciIKPj4+ICsgICAgIGRlcGVuZHMgT04gWDg2Cj4+Cj4+IEhv\n" - "dyBkaWQgdGhpcyBwYXN0IHlvdXIgdGVzdGluZz8KPj4KPj4+ICsgICAgIC0tLWhlbHAtLS0KPj4+\n" - "ICsgICAgICAgVGhpcyBod21vbiBkcml2ZXIgYWRkcyBzdXBwb3J0IGZvciByZXBvcnRpbmcgdGVt\n" - "cGVyYXR1cmUgb2YgZGlmZmVyZW50Cj4+PiArICAgICAgIHNlbnNvcnMgYW5kIGNvbnRyb2xzIHRo\n" - "ZSBmYW5zIG9uIERlbGwgbGFwdG9wcyB2aWEgU3lzdGVtIE1hbmFnZW1lbnQKPj4+ICsgICAgICAg\n" - "TW9kZSBwcm92aWRlZCBieSBEZWxsIEJJT1MuCj4+PiArCj4+PiArICAgICAgIFdoZW4gb3B0aW9u\n" - "IEk4SyBpcyBhbHNvIGVuYWJsZWQgdGhpcyBkcml2ZXIgcHJvdmlkZXMgbGVnYWN5IC9wcm9jL2k4\n" - "awo+Pj4gKyAgICAgICB1c2Vyc3BhY2UgaW50ZXJmYWNlIGZvciBpOGt1dGlscyBwYWNrYWdlLgo+\n" - "Pj4gKwo+Cj4gSXQncyB3b3JraW5nIE9LIGZvciBtZSAoYWZ0ZXIgZml4aW5nIHRoZSBtaXN0YWtl\n" - "IHBvaW50ZWQgb3V0IGFib3ZlKS4KPgo+IFtMYXRpdHVkZSBlNjU0MF0KPiAtLS0tLQo+IGNvcmV0\n" - "ZW1wLWlzYS0wMDAwCj4gQWRhcHRlcjogSVNBIGFkYXB0ZXIKPiBQaHlzaWNhbCBpZCAwOiAgKzUz\n" - "LjDCsEMgIChoaWdoID0gKzg0LjDCsEMsIGNyaXQgPSArMTAwLjDCsEMpCj4gQ29yZSAwOiAgICAg\n" - "ICAgICs1MS4wwrBDICAoaGlnaCA9ICs4NC4wwrBDLCBjcml0ID0gKzEwMC4wwrBDKQo+IENvcmUg\n" - "MTogICAgICAgICArNTMuMMKwQyAgKGhpZ2ggPSArODQuMMKwQywgY3JpdCA9ICsxMDAuMMKwQykK\n" - "Pgo+IGk4ay12aXJ0dWFsLTAKPiBBZGFwdGVyOiBWaXJ0dWFsIGRldmljZQo+IFByb2Nlc3NvciBG\n" - "YW46IDMxNzEgUlBNCj4gQ1BVOiAgICAgICAgICAgICs1Mi4wwrBDCj4gQW1iaWVudDogICAgICAg\n" - "ICs0Ni4wwrBDCj4gU09ESU1NOiAgICAgICAgICs0MC4wwrBDCj4gLS0tLS0KPgo+IEFtYmllbnQg\n" - "YW5kIFNPRElNTSBsb29rIHN3YXBwZWQgLSBidXQgaXQncyBwcm9iYWJseSBqdXN0IGFub3RoZXIg\n" - "RGVsbAo+IGVycm9yLiBJIHJlYWxseSBuZWVkIHRvIGxvb2sgYXQgZ2V0dGluZyBjb3JlYm9vdCBv\n" - "biB0aGlzIHRoaW5nLgo+Cj4gT24gMjggTWFyY2ggMjAxNSBhdCAxMDoyNCwgUGFsaSBSb2jDoXIg\n" - "PHBhbGkucm9oYXJAZ21haWwuY29tPiB3cm90ZToKPiAuLi4KPj4gICBjb25maWcgSThLCj4+IC0g\n" - "ICAgICAgdHJpc3RhdGUgIkRlbGwgbGFwdG9wIHN1cHBvcnQiCj4+IC0gICAgICAgc2VsZWN0IEhX\n" - "TU9OCj4+ICsgICAgICAgYm9vbCAiRGVsbCBpOGsgbGVnYWN5IGxhcHRvcCBzdXBwb3J0Igo+PiAr\n" - "ICAgICAgIHNlbGVjdCBTRU5TT1JTX0RFTExfU01NCj4gLi4uCj4+ICtjb25maWcgU0VOU09SU19E\n" - "RUxMX1NNTQo+PiArICAgICAgIHRyaXN0YXRlICJEZWxsIGxhcHRvcCBTTU0gQklPUyBod21vbiBk\n" - "cml2ZXIiCj4KPiBUaGUgb25seSBjaGFuZ2UgSSdkIHN1Z2dlc3QgaXMgbm90IHRvIGNoYW5nZSB0\n" - "aGUgdHJpc3RhdGUgdG8gYm9vbCBmb3IKPiBJOEsuIEZvcmNpbmcgdGhhdCB0byBib29sIG1lYW5z\n" - "IHRoYXQgU0VOU09SU19ERUxMX1NNTSBjYW4ndCBiZQo+IGNvbXBpbGVkIGFzIGEgbW9kdWxlIGlm\n" - "IHRoZSB1c2VyIHdpc2hlcyB0byBrZWVwIGk4ayBoYW5naW5nIGFyb3VuZAo+ICJqdXN0IGluIGNh\n" - "c2UiLiBJdCdsbCBwcm9iYWJseSBhbm5veSBkaXN0cm8ga2VybmVsIHBhY2thZ2VycyB0b28uCj4K\n" - "CkFncmVlZC4gSXQgYWxzbyBzdGlsbCBuZWVkcyB0byBzZWxlY3QgSFdNT04gKFNFTlNPUlNfREVM\n" - "TF9TTU0gaXMKb25seSB2aXNpYmxlIGlmIEhXTU9OIGlzIHNlbGVjdGVkKS4KCkd1ZW50ZXIKCgpf\n" - "X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsbS1zZW5zb3Jz\n" - "IG1haWxpbmcgbGlzdApsbS1zZW5zb3JzQGxtLXNlbnNvcnMub3JnCmh0dHA6Ly9saXN0cy5sbS1z\n" - ZW5zb3JzLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xtLXNlbnNvcnM + "On 03/28/2015 05:54 AM, Steven Honeyman wrote:\n" + "> On 28 March 2015 at 11:04, Paul Bolle <pebolle@tiscali.nl> wrote:\n" + ">> On Sat, 2015-03-28 at 11:24 +0100, Pali Roh\303\241r wrote:\n" + ">>> --- a/drivers/hwmon/Kconfig\n" + ">>> +++ b/drivers/hwmon/Kconfig\n" + ">>> @@ -1703,6 +1703,17 @@ config SENSORS_ULTRA45\n" + ">>> This driver provides support for the Ultra45 workstation environmental\n" + ">>> sensors.\n" + ">>>\n" + ">>> +config SENSORS_DELL_SMM\n" + ">>> + tristate \"Dell laptop SMM BIOS hwmon driver\"\n" + ">>> + depends ON X86\n" + ">>\n" + ">> How did this past your testing?\n" + ">>\n" + ">>> + ---help---\n" + ">>> + This hwmon driver adds support for reporting temperature of different\n" + ">>> + sensors and controls the fans on Dell laptops via System Management\n" + ">>> + Mode provided by Dell BIOS.\n" + ">>> +\n" + ">>> + When option I8K is also enabled this driver provides legacy /proc/i8k\n" + ">>> + userspace interface for i8kutils package.\n" + ">>> +\n" + ">\n" + "> It's working OK for me (after fixing the mistake pointed out above).\n" + ">\n" + "> [Latitude e6540]\n" + "> -----\n" + "> coretemp-isa-0000\n" + "> Adapter: ISA adapter\n" + "> Physical id 0: +53.0\302\260C (high = +84.0\302\260C, crit = +100.0\302\260C)\n" + "> Core 0: +51.0\302\260C (high = +84.0\302\260C, crit = +100.0\302\260C)\n" + "> Core 1: +53.0\302\260C (high = +84.0\302\260C, crit = +100.0\302\260C)\n" + ">\n" + "> i8k-virtual-0\n" + "> Adapter: Virtual device\n" + "> Processor Fan: 3171 RPM\n" + "> CPU: +52.0\302\260C\n" + "> Ambient: +46.0\302\260C\n" + "> SODIMM: +40.0\302\260C\n" + "> -----\n" + ">\n" + "> Ambient and SODIMM look swapped - but it's probably just another Dell\n" + "> error. I really need to look at getting coreboot on this thing.\n" + ">\n" + "> On 28 March 2015 at 10:24, Pali Roh\303\241r <pali.rohar@gmail.com> wrote:\n" + "> ...\n" + ">> config I8K\n" + ">> - tristate \"Dell laptop support\"\n" + ">> - select HWMON\n" + ">> + bool \"Dell i8k legacy laptop support\"\n" + ">> + select SENSORS_DELL_SMM\n" + "> ...\n" + ">> +config SENSORS_DELL_SMM\n" + ">> + tristate \"Dell laptop SMM BIOS hwmon driver\"\n" + ">\n" + "> The only change I'd suggest is not to change the tristate to bool for\n" + "> I8K. Forcing that to bool means that SENSORS_DELL_SMM can't be\n" + "> compiled as a module if the user wishes to keep i8k hanging around\n" + "> \"just in case\". It'll probably annoy distro kernel packagers too.\n" + ">\n" + "\n" + "Agreed. It also still needs to select HWMON (SENSORS_DELL_SMM is\n" + "only visible if HWMON is selected).\n" + "\n" + Guenter -cb18c8f314733d7fa60d4eccedc04a5647d3dca3782002d3a490bc0be941ea61 +9939685927c0e5a2e2c2564c18887e30be6fe9290c62a379b315a74b0de5664b
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.