diff for duplicates of <550A4B03.5070306@roeck-us.net> diff --git a/a/1.txt b/N1/1.txt index 3c66b4d..d74702f 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,21 +1,29 @@ -T24gMDMvMTgvMjAxNSAwODo0NyBBTSwgQ8OpZHJpYyBMZSBHb2F0ZXIgd3JvdGU6Cj4gSGVsbG8g -IQo+Cj4gVGhlIGN1cnJlbnQgaW1wbGVtZW50YXRpb24gb2YgdGhlIGRyaXZlciB1c2VzIGFuIGlu -ZGV4IGZvciB0aGUgaHdtb24KPiBhdHRyaWJ1dGUgd2hpY2ggaXMgZXh0cmFjdGVkIGZyb20gdGhl -IGRldmljZSBub2RlIG5hbWUuIFRoaXMgaW5kZXgKPiBpcyBjYWxjdWxhdGVkIGJ5IHRoZSBPUEFM -IGZpcm13YXJlIGFuZCBpdHMgdXNhZ2UgY3JlYXRlcyBhIGRlcGVuZGVuY3kKPiB3aXRoIHRoZSBk -cml2ZXIgd2hpY2ggbWFrZXMgY2hhbmdlcyBhIGxpdHRsZSBtb3JlIGNvbXBsZXggaW4gT1BBTC4K -Pgo+IFRoaXMgcGF0Y2hzZXQgY2hhbmdlcyB0aGUgaWJtcG93ZXJudiBjb2RlIHRvIHVzZSBpdHMg -b3duIGluZGV4LiBJdAo+IHN0YXJ0cyB3aXRoIGEgZmV3IGNsZWFudXBzLCBtb3N0bHkgY29kZSBz -aHVmZmxpbmcgYXJvdW5kIHRoZSBjcmVhdGlvbgo+IG9mIHRoZSBod21vbiBzeXNmcyBhdHRyaWJ1 -dGVzIGFuZCBjb21wbGV0ZXMgYnkgcmVtb3ZpbmcgdGhlIGRlcGVuZGVuY3kuCj4KPiBJdCBhbHNv -IHByZXBhcmVzIGdyb3VuZCBmb3IgZnV0dXJlIE9QQUwgY2hhbmdlcyA6Cj4KPiAgICAgaHR0cHM6 -Ly9saXN0cy5vemxhYnMub3JnL3BpcGVybWFpbC9za2lib290LzIwMTUtTWFyY2gvMDAwNjM5Lmh0 -bWwKPgo+IHdoaWNoIHdpbGwgYmUgYWRkcmVzc2VkIGluIGEgb3RoZXIgc21hbGwgcGF0Y2hzZXQu -Cj4KPgo+IFRoZSBwYXRjaGVzIGFyZSBiYXNlZCBvbiBMaW51eCA0LjAuMC1yYzQgYW5kIHdlcmUg -dGVzdGVkIG9uIElCTSBQb3dlcgo+IGFuZCBPcGVuIFBvd2VyIHN5c3RlbXMgcnVubmluZyBUcnVz -dHkuCj4KCkkgY29tbWVudGVkIG9uIHR3byBvZiB0aGUgcGF0Y2hlczsgdGhlIG90aGVycyBhcmUg -b2suCgpQbGVhc2UgcmUtc2VuZCB0aGUgZW50aXJlIHNlcmllcyBhZnRlciBhZGRyZXNzaW5nIG15 -IGNvbW1lbnRzLgoKVGhhbmtzLApHdWVudGVyCgoKCl9fX19fX19fX19fX19fX19fX19fX19fX19f -X19fX19fX19fX19fX19fX19fX19fCmxtLXNlbnNvcnMgbWFpbGluZyBsaXN0CmxtLXNlbnNvcnNA -bG0tc2Vuc29ycy5vcmcKaHR0cDovL2xpc3RzLmxtLXNlbnNvcnMub3JnL21haWxtYW4vbGlzdGlu -Zm8vbG0tc2Vuc29ycw= +On 03/18/2015 08:47 AM, Cédric Le Goater wrote: +> Hello ! +> +> The current implementation of the driver uses an index for the hwmon +> attribute which is extracted from the device node name. This index +> is calculated by the OPAL firmware and its usage creates a dependency +> with the driver which makes changes a little more complex in OPAL. +> +> This patchset changes the ibmpowernv code to use its own index. It +> starts with a few cleanups, mostly code shuffling around the creation +> of the hwmon sysfs attributes and completes by removing the dependency. +> +> It also prepares ground for future OPAL changes : +> +> https://lists.ozlabs.org/pipermail/skiboot/2015-March/000639.html +> +> which will be addressed in a other small patchset. +> +> +> The patches are based on Linux 4.0.0-rc4 and were tested on IBM Power +> and Open Power systems running Trusty. +> + +I commented on two of the patches; the others are ok. + +Please re-send the entire series after addressing my comments. + +Thanks, +Guenter diff --git a/a/content_digest b/N1/content_digest index 102cc84..92c953d 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,8 +1,8 @@ "ref\01423117857-32759-1-git-send-email-clg@fr.ibm.com\0" "ref\01426693665-10797-1-git-send-email-clg@fr.ibm.com\0" "From\0Guenter Roeck <linux@roeck-us.net>\0" - "Subject\0Re: [lm-sensors] [PATCH 0/5] hwmon: (ibmpowernv) remove dependency on OPAL index\0" - "Date\0Thu, 19 Mar 2015 04:05:23 +0000\0" + "Subject\0Re: [PATCH 0/5] hwmon: (ibmpowernv) remove dependency on OPAL index\0" + "Date\0Wed, 18 Mar 2015 21:05:23 -0700\0" "To\0C\303\251dric Le Goater <clg@fr.ibm.com>" " lm-sensors@lm-sensors.org\0" "Cc\0Stewart Smith <stewart@linux.vnet.ibm.com>" @@ -12,26 +12,34 @@ " Jean Delvare <jdelvare@suse.de>\0" "\00:1\0" "b\0" - "T24gMDMvMTgvMjAxNSAwODo0NyBBTSwgQ8OpZHJpYyBMZSBHb2F0ZXIgd3JvdGU6Cj4gSGVsbG8g\n" - "IQo+Cj4gVGhlIGN1cnJlbnQgaW1wbGVtZW50YXRpb24gb2YgdGhlIGRyaXZlciB1c2VzIGFuIGlu\n" - "ZGV4IGZvciB0aGUgaHdtb24KPiBhdHRyaWJ1dGUgd2hpY2ggaXMgZXh0cmFjdGVkIGZyb20gdGhl\n" - "IGRldmljZSBub2RlIG5hbWUuIFRoaXMgaW5kZXgKPiBpcyBjYWxjdWxhdGVkIGJ5IHRoZSBPUEFM\n" - "IGZpcm13YXJlIGFuZCBpdHMgdXNhZ2UgY3JlYXRlcyBhIGRlcGVuZGVuY3kKPiB3aXRoIHRoZSBk\n" - "cml2ZXIgd2hpY2ggbWFrZXMgY2hhbmdlcyBhIGxpdHRsZSBtb3JlIGNvbXBsZXggaW4gT1BBTC4K\n" - "Pgo+IFRoaXMgcGF0Y2hzZXQgY2hhbmdlcyB0aGUgaWJtcG93ZXJudiBjb2RlIHRvIHVzZSBpdHMg\n" - "b3duIGluZGV4LiBJdAo+IHN0YXJ0cyB3aXRoIGEgZmV3IGNsZWFudXBzLCBtb3N0bHkgY29kZSBz\n" - "aHVmZmxpbmcgYXJvdW5kIHRoZSBjcmVhdGlvbgo+IG9mIHRoZSBod21vbiBzeXNmcyBhdHRyaWJ1\n" - "dGVzIGFuZCBjb21wbGV0ZXMgYnkgcmVtb3ZpbmcgdGhlIGRlcGVuZGVuY3kuCj4KPiBJdCBhbHNv\n" - "IHByZXBhcmVzIGdyb3VuZCBmb3IgZnV0dXJlIE9QQUwgY2hhbmdlcyA6Cj4KPiAgICAgaHR0cHM6\n" - "Ly9saXN0cy5vemxhYnMub3JnL3BpcGVybWFpbC9za2lib290LzIwMTUtTWFyY2gvMDAwNjM5Lmh0\n" - "bWwKPgo+IHdoaWNoIHdpbGwgYmUgYWRkcmVzc2VkIGluIGEgb3RoZXIgc21hbGwgcGF0Y2hzZXQu\n" - "Cj4KPgo+IFRoZSBwYXRjaGVzIGFyZSBiYXNlZCBvbiBMaW51eCA0LjAuMC1yYzQgYW5kIHdlcmUg\n" - "dGVzdGVkIG9uIElCTSBQb3dlcgo+IGFuZCBPcGVuIFBvd2VyIHN5c3RlbXMgcnVubmluZyBUcnVz\n" - "dHkuCj4KCkkgY29tbWVudGVkIG9uIHR3byBvZiB0aGUgcGF0Y2hlczsgdGhlIG90aGVycyBhcmUg\n" - "b2suCgpQbGVhc2UgcmUtc2VuZCB0aGUgZW50aXJlIHNlcmllcyBhZnRlciBhZGRyZXNzaW5nIG15\n" - "IGNvbW1lbnRzLgoKVGhhbmtzLApHdWVudGVyCgoKCl9fX19fX19fX19fX19fX19fX19fX19fX19f\n" - "X19fX19fX19fX19fX19fX19fX19fCmxtLXNlbnNvcnMgbWFpbGluZyBsaXN0CmxtLXNlbnNvcnNA\n" - "bG0tc2Vuc29ycy5vcmcKaHR0cDovL2xpc3RzLmxtLXNlbnNvcnMub3JnL21haWxtYW4vbGlzdGlu\n" - Zm8vbG0tc2Vuc29ycw= + "On 03/18/2015 08:47 AM, C\303\251dric Le Goater wrote:\n" + "> Hello !\n" + ">\n" + "> The current implementation of the driver uses an index for the hwmon\n" + "> attribute which is extracted from the device node name. This index\n" + "> is calculated by the OPAL firmware and its usage creates a dependency\n" + "> with the driver which makes changes a little more complex in OPAL.\n" + ">\n" + "> This patchset changes the ibmpowernv code to use its own index. It\n" + "> starts with a few cleanups, mostly code shuffling around the creation\n" + "> of the hwmon sysfs attributes and completes by removing the dependency.\n" + ">\n" + "> It also prepares ground for future OPAL changes :\n" + ">\n" + "> https://lists.ozlabs.org/pipermail/skiboot/2015-March/000639.html\n" + ">\n" + "> which will be addressed in a other small patchset.\n" + ">\n" + ">\n" + "> The patches are based on Linux 4.0.0-rc4 and were tested on IBM Power\n" + "> and Open Power systems running Trusty.\n" + ">\n" + "\n" + "I commented on two of the patches; the others are ok.\n" + "\n" + "Please re-send the entire series after addressing my comments.\n" + "\n" + "Thanks,\n" + Guenter -7d77a430f73f70d96b592389f74a7cc878d29253bbfc195869e339c81e3f31bf +4660207a0e1ebaedc7daf5c4fa9f1cd47f81546d818f84e6a5b1e6cf7c568d0d
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.