From: "Cédric Le Goater" <clg@fr.ibm.com> To: lm-sensors@lm-sensors.org Cc: "Stewart Smith" <stewart@linux.vnet.ibm.com>, "Cédric Le Goater" <clg@fr.ibm.com>, "Jean Delvare" <jdelvare@suse.de>, "Neelesh Gupta" <neelegup@linux.vnet.ibm.com>, skiboot@lists.ozlabs.org, linuxppc-dev@lists.ozlabs.org, "Guenter Roeck" <linux@roeck-us.net> Subject: [PATCH 0/5] hwmon: (ibmpowernv) remove dependency on OPAL index Date: Wed, 18 Mar 2015 16:47:40 +0100 [thread overview] Message-ID: <1426693665-10797-1-git-send-email-clg@fr.ibm.com> (raw) In-Reply-To: <1423117857-32759-1-git-send-email-clg@fr.ibm.com> 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. Cheers, C. Cédric Le Goater (5): hwmon: (ibmpowernv) replace AMBIENT_TEMP by TEMP hwmon: (ibmpowernv) add a get_sensor_type() routine hwmon: (ibmpowernv) add a convert_opal_attr_name() routine hwmon: (ibmpowernv) change create_hwmon_attr_name() prototype hwmon: (ibmpowernv) do not use the OPAL index for hwmon attribute names drivers/hwmon/ibmpowernv.c | 122 +++++++++++++++++++++++++++++--------------- 1 file changed, 81 insertions(+), 41 deletions(-) -- 1.7.10.4
WARNING: multiple messages have this Message-ID (diff)
From: "Cédric Le Goater" <clg@fr.ibm.com> To: lm-sensors@lm-sensors.org Cc: "Stewart Smith" <stewart@linux.vnet.ibm.com>, "Cédric Le Goater" <clg@fr.ibm.com>, "Jean Delvare" <jdelvare@suse.de>, "Neelesh Gupta" <neelegup@linux.vnet.ibm.com>, skiboot@lists.ozlabs.org, linuxppc-dev@lists.ozlabs.org, "Guenter Roeck" <linux@roeck-us.net> Subject: [lm-sensors] [PATCH 0/5] hwmon: (ibmpowernv) remove dependency on OPAL index Date: Wed, 18 Mar 2015 15:47:40 +0000 [thread overview] Message-ID: <1426693665-10797-1-git-send-email-clg@fr.ibm.com> (raw) In-Reply-To: <1423117857-32759-1-git-send-email-clg@fr.ibm.com> SGVsbG8gIQoKVGhlIGN1cnJlbnQgaW1wbGVtZW50YXRpb24gb2YgdGhlIGRyaXZlciB1c2VzIGFu IGluZGV4IGZvciB0aGUgaHdtb24gCmF0dHJpYnV0ZSB3aGljaCBpcyBleHRyYWN0ZWQgZnJvbSB0 aGUgZGV2aWNlIG5vZGUgbmFtZS4gVGhpcyBpbmRleAppcyBjYWxjdWxhdGVkIGJ5IHRoZSBPUEFM IGZpcm13YXJlIGFuZCBpdHMgdXNhZ2UgY3JlYXRlcyBhIGRlcGVuZGVuY3kgCndpdGggdGhlIGRy aXZlciB3aGljaCBtYWtlcyBjaGFuZ2VzIGEgbGl0dGxlIG1vcmUgY29tcGxleCBpbiBPUEFMLgoK VGhpcyBwYXRjaHNldCBjaGFuZ2VzIHRoZSBpYm1wb3dlcm52IGNvZGUgdG8gdXNlIGl0cyBvd24g aW5kZXguIEl0IApzdGFydHMgd2l0aCBhIGZldyBjbGVhbnVwcywgbW9zdGx5IGNvZGUgc2h1ZmZs aW5nIGFyb3VuZCB0aGUgY3JlYXRpb24gCm9mIHRoZSBod21vbiBzeXNmcyBhdHRyaWJ1dGVzIGFu ZCBjb21wbGV0ZXMgYnkgcmVtb3ZpbmcgdGhlIGRlcGVuZGVuY3kuCgpJdCBhbHNvIHByZXBhcmVz IGdyb3VuZCBmb3IgZnV0dXJlIE9QQUwgY2hhbmdlcyA6ICAKCiAgIGh0dHBzOi8vbGlzdHMub3ps YWJzLm9yZy9waXBlcm1haWwvc2tpYm9vdC8yMDE1LU1hcmNoLzAwMDYzOS5odG1sCgp3aGljaCB3 aWxsIGJlIGFkZHJlc3NlZCBpbiBhIG90aGVyIHNtYWxsIHBhdGNoc2V0LgoKClRoZSBwYXRjaGVz IGFyZSBiYXNlZCBvbiBMaW51eCA0LjAuMC1yYzQgYW5kIHdlcmUgdGVzdGVkIG9uIElCTSBQb3dl ciAKYW5kIE9wZW4gUG93ZXIgc3lzdGVtcyBydW5uaW5nIFRydXN0eS4gCgpDaGVlcnMsCgpDLgoK CkPDqWRyaWMgTGUgR29hdGVyICg1KToKICBod21vbjogKGlibXBvd2VybnYpIHJlcGxhY2UgQU1C SUVOVF9URU1QIGJ5IFRFTVAKICBod21vbjogKGlibXBvd2VybnYpIGFkZCBhIGdldF9zZW5zb3Jf dHlwZSgpIHJvdXRpbmUKICBod21vbjogKGlibXBvd2VybnYpIGFkZCBhIGNvbnZlcnRfb3BhbF9h dHRyX25hbWUoKSByb3V0aW5lCiAgaHdtb246IChpYm1wb3dlcm52KSBjaGFuZ2UgY3JlYXRlX2h3 bW9uX2F0dHJfbmFtZSgpIHByb3RvdHlwZQogIGh3bW9uOiAoaWJtcG93ZXJudikgZG8gbm90IHVz ZSB0aGUgT1BBTCBpbmRleCBmb3IgaHdtb24gYXR0cmlidXRlCiAgICBuYW1lcwoKIGRyaXZlcnMv aHdtb24vaWJtcG93ZXJudi5jIHwgIDEyMiArKysrKysrKysrKysrKysrKysrKysrKysrKysrKy0t LS0tLS0tLS0tLS0tLQogMSBmaWxlIGNoYW5nZWQsIDgxIGluc2VydGlvbnMoKyksIDQxIGRlbGV0 aW9ucygtKQoKLS0gCjEuNy4xMC40CgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX18KbG0tc2Vuc29ycyBtYWlsaW5nIGxpc3QKbG0tc2Vuc29yc0BsbS1zZW5z b3JzLm9yZwpodHRwOi8vbGlzdHMubG0tc2Vuc29ycy5vcmcvbWFpbG1hbi9saXN0aW5mby9sbS1z ZW5zb3Jz
next prev parent reply other threads:[~2015-03-18 15:47 UTC|newest] Thread overview: 96+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <1423117857-32759-1-git-send-email-clg@fr.ibm.com> 2015-02-20 15:07 ` [RFC PATCH 0/3] hwmon: (ibmpowernv) add DTS support Cédric Le Goater 2015-02-20 15:07 ` [lm-sensors] " Cédric Le Goater 2015-02-20 16:52 ` Guenter Roeck 2015-02-20 16:52 ` [lm-sensors] " Guenter Roeck 2015-02-20 20:15 ` Cedric Le Goater 2015-02-20 20:15 ` [lm-sensors] " Cedric Le Goater 2015-02-20 23:52 ` Guenter Roeck 2015-02-20 23:52 ` [lm-sensors] " Guenter Roeck 2015-02-21 7:14 ` Cedric Le Goater 2015-02-21 7:14 ` [lm-sensors] " Cedric Le Goater 2015-02-21 11:03 ` Guenter Roeck 2015-02-21 11:03 ` [lm-sensors] " Guenter Roeck 2015-02-23 10:54 ` Cedric Le Goater 2015-02-23 10:54 ` [lm-sensors] " Cedric Le Goater 2015-02-20 15:07 ` [RFC PATCH 1/3] powerpc/powernv: Check OPAL sensor calls exist Cédric Le Goater 2015-02-20 15:07 ` [lm-sensors] " Cédric Le Goater 2015-02-20 16:53 ` Guenter Roeck 2015-02-20 16:53 ` [lm-sensors] " Guenter Roeck 2015-02-20 20:18 ` Cedric Le Goater 2015-02-20 20:18 ` [lm-sensors] " Cedric Le Goater 2015-02-24 4:54 ` Michael Ellerman 2015-02-24 4:54 ` [lm-sensors] " Michael Ellerman 2015-02-25 17:28 ` Cedric Le Goater 2015-02-25 17:28 ` [lm-sensors] " Cedric Le Goater 2015-02-20 15:07 ` [RFC PATCH 2/3] powerpc/powernv: handle OPAL_SUCCESS return in opal_sensor_read Cédric Le Goater 2015-02-20 15:07 ` [lm-sensors] " Cédric Le Goater 2015-02-20 15:07 ` [RFC PATCH 3/3] hwmon: (ibmpowernv) add DTS support Cédric Le Goater 2015-02-20 15:07 ` [lm-sensors] " Cédric Le Goater 2015-03-18 15:47 ` Cédric Le Goater [this message] 2015-03-18 15:47 ` [lm-sensors] [PATCH 0/5] hwmon: (ibmpowernv) remove dependency on OPAL index Cédric Le Goater 2015-03-19 4:05 ` Guenter Roeck 2015-03-19 4:05 ` [lm-sensors] " Guenter Roeck 2015-03-18 15:47 ` [PATCH 1/5] hwmon: (ibmpowernv) replace AMBIENT_TEMP by TEMP Cédric Le Goater 2015-03-18 15:47 ` [lm-sensors] " Cédric Le Goater 2015-03-18 15:47 ` [PATCH 2/5] hwmon: (ibmpowernv) add a get_sensor_type() routine Cédric Le Goater 2015-03-18 15:47 ` [lm-sensors] " Cédric Le Goater 2015-03-18 15:47 ` [PATCH 3/5] hwmon: (ibmpowernv) add a convert_opal_attr_name() routine Cédric Le Goater 2015-03-18 15:47 ` [lm-sensors] " Cédric Le Goater 2015-03-19 3:58 ` Guenter Roeck 2015-03-19 3:58 ` [lm-sensors] " Guenter Roeck 2015-03-18 15:47 ` [PATCH 4/5] hwmon: (ibmpowernv) change create_hwmon_attr_name() prototype Cédric Le Goater 2015-03-18 15:47 ` [lm-sensors] " Cédric Le Goater 2015-03-19 4:02 ` Guenter Roeck 2015-03-19 4:02 ` [lm-sensors] " Guenter Roeck 2015-03-18 15:47 ` [PATCH 5/5] hwmon: (ibmpowernv) do not use the OPAL index for hwmon attribute names Cédric Le Goater 2015-03-18 15:47 ` [lm-sensors] " Cédric Le Goater 2015-03-19 17:44 ` [PATCH v2 0/5] hwmon: (ibmpowernv) remove dependency on OPAL index Cédric Le Goater 2015-03-19 17:44 ` [lm-sensors] " Cédric Le Goater 2015-03-20 15:26 ` Guenter Roeck 2015-03-20 15:26 ` [lm-sensors] " Guenter Roeck 2015-03-20 16:52 ` Cedric Le Goater 2015-03-20 16:52 ` [lm-sensors] " Cedric Le Goater 2015-04-01 10:15 ` [PATCH 0/4] hwmon: (ibmpowernv) add DTS support Cédric Le Goater 2015-04-01 10:15 ` [lm-sensors] " Cédric Le Goater 2015-04-01 10:15 ` [PATCH 1/4] hwmon: (ibmpowernv) add a helper routine create_hwmon_attr Cédric Le Goater 2015-04-01 10:15 ` [lm-sensors] " Cédric Le Goater 2015-04-01 10:15 ` [PATCH 2/4] hwmon: (ibmpowernv) add support for the new device tree Cédric Le Goater 2015-04-01 10:15 ` [lm-sensors] " Cédric Le Goater 2015-04-01 10:15 ` [PATCH 3/4] hwmon: (ibmpowernv) add a label attribute Cédric Le Goater 2015-04-01 10:15 ` [lm-sensors] " Cédric Le Goater 2015-04-01 10:15 ` [PATCH 4/4] hwmon: (ibmpowernv) pretty print labels Cédric Le Goater 2015-04-01 10:15 ` [lm-sensors] " Cédric Le Goater 2015-04-03 15:49 ` Guenter Roeck 2015-04-03 15:49 ` [lm-sensors] " Guenter Roeck 2015-04-07 14:42 ` Cedric Le Goater 2015-04-07 14:42 ` [lm-sensors] " Cedric Le Goater 2015-04-07 14:45 ` Cédric Le Goater 2015-04-07 14:45 ` [lm-sensors] " Cédric Le Goater 2015-04-07 16:44 ` Guenter Roeck 2015-04-07 16:44 ` [lm-sensors] " Guenter Roeck 2015-04-07 18:03 ` Cedric Le Goater 2015-04-07 18:03 ` [lm-sensors] " Cedric Le Goater 2015-04-07 19:22 ` Guenter Roeck 2015-04-07 19:22 ` [lm-sensors] " Guenter Roeck 2015-04-08 6:57 ` Cedric Le Goater 2015-04-08 6:57 ` [lm-sensors] " Cedric Le Goater 2015-04-07 20:22 ` [Skiboot] " Benjamin Herrenschmidt 2015-04-07 20:22 ` [lm-sensors] " Benjamin Herrenschmidt 2015-03-19 17:44 ` [PATCH v2 1/5] hwmon: (ibmpowernv) replace AMBIENT_TEMP by TEMP Cédric Le Goater 2015-03-19 17:44 ` [lm-sensors] " Cédric Le Goater 2015-03-19 17:44 ` [PATCH v2 2/5] hwmon: (ibmpowernv) add a get_sensor_type() routine Cédric Le Goater 2015-03-19 17:44 ` [lm-sensors] " Cédric Le Goater 2015-03-19 17:44 ` [PATCH v2 3/5] hwmon: (ibmpowernv) add a convert_opal_attr_name() routine Cédric Le Goater 2015-03-19 17:44 ` [lm-sensors] " Cédric Le Goater 2015-03-19 17:44 ` [PATCH v2 4/5] hwmon: (ibmpowernv) change create_hwmon_attr_name() prototype Cédric Le Goater 2015-03-19 17:44 ` [lm-sensors] " Cédric Le Goater 2015-03-20 8:06 ` Cedric Le Goater 2015-03-20 8:06 ` [lm-sensors] " Cedric Le Goater 2015-03-20 15:27 ` Guenter Roeck 2015-03-20 15:27 ` [lm-sensors] " Guenter Roeck 2015-03-19 17:44 ` [PATCH v2 5/5] hwmon: (ibmpowernv) do not use the OPAL index for hwmon attribute names Cédric Le Goater 2015-03-19 17:44 ` [lm-sensors] " Cédric Le Goater 2015-04-08 15:20 [PATCH 2/4] hwmon: (ibmpowernv) add support for the new device tree Guenter Roeck 2015-04-08 15:20 ` [lm-sensors] " Guenter Roeck 2015-04-08 16:06 ` Cedric Le Goater 2015-04-08 16:06 ` [lm-sensors] " Cedric Le Goater
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1426693665-10797-1-git-send-email-clg@fr.ibm.com \ --to=clg@fr.ibm.com \ --cc=jdelvare@suse.de \ --cc=linux@roeck-us.net \ --cc=linuxppc-dev@lists.ozlabs.org \ --cc=lm-sensors@lm-sensors.org \ --cc=neelegup@linux.vnet.ibm.com \ --cc=skiboot@lists.ozlabs.org \ --cc=stewart@linux.vnet.ibm.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.