* [PATCH V4] hwmon: (ibmpowernv) Add highest/lowest attributes to sensors
@ 2017-05-06 15:27 Shilpasri G Bhat
2017-05-14 15:47 ` Guenter Roeck
0 siblings, 1 reply; 2+ messages in thread
From: Shilpasri G Bhat @ 2017-05-06 15:27 UTC (permalink / raw)
To: jdelvare, linux, paulus, mpe
Cc: linux-hwmon, linuxppc-dev, linux-kernel, svaidy, ego,
akshay.adiga, andrew, clg, Shilpasri G Bhat
OCC provides historical minimum and maximum value for the sensor
readings. This patch exports them as highest and lowest attributes
for the inband sensors copied by OCC to main memory.
Signed-off-by: Shilpasri G Bhat <shilpa.bhat@linux.vnet.ibm.com>
---
Changes from V3:
- Removed extra variable 'hwmon_index'
- Retain the initialization of sensor_data for *_input attribute in
create_device_attrs(), otherwise all the sensors from different
classes will numbered continuously.
drivers/hwmon/ibmpowernv.c | 69 +++++++++++++++++++++++++++++++++++++++++-----
1 file changed, 62 insertions(+), 7 deletions(-)
diff --git a/drivers/hwmon/ibmpowernv.c b/drivers/hwmon/ibmpowernv.c
index 6d2e660..862b832 100644
--- a/drivers/hwmon/ibmpowernv.c
+++ b/drivers/hwmon/ibmpowernv.c
@@ -287,6 +287,7 @@ static int populate_attr_groups(struct platform_device *pdev)
opal = of_find_node_by_path("/ibm,opal/sensors");
for_each_child_of_node(opal, np) {
const char *label;
+ int len;
if (np->name == NULL)
continue;
@@ -298,10 +299,14 @@ static int populate_attr_groups(struct platform_device *pdev)
sensor_groups[type].attr_count++;
/*
- * add a new attribute for labels
+ * add attributes for labels, min and max
*/
if (!of_property_read_string(np, "label", &label))
sensor_groups[type].attr_count++;
+ if (of_find_property(np, "sensor-data-min", &len))
+ sensor_groups[type].attr_count++;
+ if (of_find_property(np, "sensor-data-max", &len))
+ sensor_groups[type].attr_count++;
}
of_node_put(opal);
@@ -337,6 +342,41 @@ static void create_hwmon_attr(struct sensor_data *sdata, const char *attr_name,
sdata->dev_attr.show = show;
}
+static void populate_sensor(struct sensor_data *sdata, int od, int hd, int sid,
+ const char *attr_name, enum sensors type,
+ const struct attribute_group *pgroup,
+ ssize_t (*show)(struct device *dev,
+ struct device_attribute *attr,
+ char *buf))
+{
+ sdata->id = sid;
+ sdata->type = type;
+ sdata->opal_index = od;
+ sdata->hwmon_index = hd;
+ create_hwmon_attr(sdata, attr_name, show);
+ pgroup->attrs[sensor_groups[type].attr_count++] = &sdata->dev_attr.attr;
+}
+
+static char *get_max_attr(enum sensors type)
+{
+ switch (type) {
+ case POWER_INPUT:
+ return "input_highest";
+ default:
+ return "highest";
+ }
+}
+
+static char *get_min_attr(enum sensors type)
+{
+ switch (type) {
+ case POWER_INPUT:
+ return "input_lowest";
+ default:
+ return "lowest";
+ }
+}
+
/*
* Iterate through the device tree for each child of 'sensors' node, create
* a sysfs attribute file, the file is named by translating the DT node name
@@ -417,16 +457,31 @@ static int create_device_attrs(struct platform_device *pdev)
* attribute. They are related to the same
* sensor.
*/
- sdata[count].type = type;
- sdata[count].opal_index = sdata[count - 1].opal_index;
- sdata[count].hwmon_index = sdata[count - 1].hwmon_index;
make_sensor_label(np, &sdata[count], label);
+ populate_sensor(&sdata[count], opal_index,
+ sdata[count - 1].hwmon_index,
+ sensor_id, "label", type, pgroups[type],
+ show_label);
+ count++;
+ }
- create_hwmon_attr(&sdata[count], "label", show_label);
+ if (!of_property_read_u32(np, "sensor-data-max", &sensor_id)) {
+ attr_name = get_max_attr(type);
+ populate_sensor(&sdata[count], opal_index,
+ sdata[count - 1].hwmon_index,
+ sensor_id, attr_name, type,
+ pgroups[type], show_sensor);
+ count++;
+ }
- pgroups[type]->attrs[sensor_groups[type].attr_count++] =
- &sdata[count++].dev_attr.attr;
+ if (!of_property_read_u32(np, "sensor-data-min", &sensor_id)) {
+ attr_name = get_min_attr(type);
+ populate_sensor(&sdata[count], opal_index,
+ sdata[count - 1].hwmon_index,
+ sensor_id, attr_name, type,
+ pgroups[type], show_sensor);
+ count++;
}
}
--
1.8.3.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH V4] hwmon: (ibmpowernv) Add highest/lowest attributes to sensors
2017-05-06 15:27 [PATCH V4] hwmon: (ibmpowernv) Add highest/lowest attributes to sensors Shilpasri G Bhat
@ 2017-05-14 15:47 ` Guenter Roeck
0 siblings, 0 replies; 2+ messages in thread
From: Guenter Roeck @ 2017-05-14 15:47 UTC (permalink / raw)
To: Shilpasri G Bhat, jdelvare, paulus, mpe
Cc: linux-hwmon, linuxppc-dev, linux-kernel, svaidy, ego,
akshay.adiga, andrew, clg
On 05/06/2017 08:27 AM, Shilpasri G Bhat wrote:
> OCC provides historical minimum and maximum value for the sensor
> readings. This patch exports them as highest and lowest attributes
> for the inband sensors copied by OCC to main memory.
>
> Signed-off-by: Shilpasri G Bhat <shilpa.bhat@linux.vnet.ibm.com>
> ---
> Changes from V3:
> - Removed extra variable 'hwmon_index'
> - Retain the initialization of sensor_data for *_input attribute in
> create_device_attrs(), otherwise all the sensors from different
> classes will numbered continuously.
>
> drivers/hwmon/ibmpowernv.c | 69 +++++++++++++++++++++++++++++++++++++++++-----
> 1 file changed, 62 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/hwmon/ibmpowernv.c b/drivers/hwmon/ibmpowernv.c
> index 6d2e660..862b832 100644
> --- a/drivers/hwmon/ibmpowernv.c
> +++ b/drivers/hwmon/ibmpowernv.c
> @@ -287,6 +287,7 @@ static int populate_attr_groups(struct platform_device *pdev)
> opal = of_find_node_by_path("/ibm,opal/sensors");
> for_each_child_of_node(opal, np) {
> const char *label;
> + int len;
>
> if (np->name == NULL)
> continue;
> @@ -298,10 +299,14 @@ static int populate_attr_groups(struct platform_device *pdev)
> sensor_groups[type].attr_count++;
>
> /*
> - * add a new attribute for labels
> + * add attributes for labels, min and max
> */
> if (!of_property_read_string(np, "label", &label))
> sensor_groups[type].attr_count++;
> + if (of_find_property(np, "sensor-data-min", &len))
> + sensor_groups[type].attr_count++;
> + if (of_find_property(np, "sensor-data-max", &len))
> + sensor_groups[type].attr_count++;
len isn't used, so you can just pass NULL and avoid the unnecessary variable.
> }
>
> of_node_put(opal);
> @@ -337,6 +342,41 @@ static void create_hwmon_attr(struct sensor_data *sdata, const char *attr_name,
> sdata->dev_attr.show = show;
> }
>
> +static void populate_sensor(struct sensor_data *sdata, int od, int hd, int sid,
> + const char *attr_name, enum sensors type,
> + const struct attribute_group *pgroup,
> + ssize_t (*show)(struct device *dev,
> + struct device_attribute *attr,
> + char *buf))
> +{
> + sdata->id = sid;
> + sdata->type = type;
> + sdata->opal_index = od;
> + sdata->hwmon_index = hd;
> + create_hwmon_attr(sdata, attr_name, show);
> + pgroup->attrs[sensor_groups[type].attr_count++] = &sdata->dev_attr.attr;
> +}
> +
> +static char *get_max_attr(enum sensors type)
> +{
> + switch (type) {
> + case POWER_INPUT:
> + return "input_highest";
> + default:
> + return "highest";
> + }
> +}
> +
> +static char *get_min_attr(enum sensors type)
> +{
> + switch (type) {
> + case POWER_INPUT:
> + return "input_lowest";
> + default:
> + return "lowest";
> + }
> +}
> +
> /*
> * Iterate through the device tree for each child of 'sensors' node, create
> * a sysfs attribute file, the file is named by translating the DT node name
> @@ -417,16 +457,31 @@ static int create_device_attrs(struct platform_device *pdev)
> * attribute. They are related to the same
> * sensor.
> */
> - sdata[count].type = type;
> - sdata[count].opal_index = sdata[count - 1].opal_index;
> - sdata[count].hwmon_index = sdata[count - 1].hwmon_index;
>
> make_sensor_label(np, &sdata[count], label);
> + populate_sensor(&sdata[count], opal_index,
> + sdata[count - 1].hwmon_index,
> + sensor_id, "label", type, pgroups[type],
> + show_label);
> + count++;
> + }
>
> - create_hwmon_attr(&sdata[count], "label", show_label);
> + if (!of_property_read_u32(np, "sensor-data-max", &sensor_id)) {
> + attr_name = get_max_attr(type);
> + populate_sensor(&sdata[count], opal_index,
> + sdata[count - 1].hwmon_index,
> + sensor_id, attr_name, type,
> + pgroups[type], show_sensor);
> + count++;
> + }
>
> - pgroups[type]->attrs[sensor_groups[type].attr_count++] =
> - &sdata[count++].dev_attr.attr;
> + if (!of_property_read_u32(np, "sensor-data-min", &sensor_id)) {
> + attr_name = get_min_attr(type);
> + populate_sensor(&sdata[count], opal_index,
> + sdata[count - 1].hwmon_index,
> + sensor_id, attr_name, type,
> + pgroups[type], show_sensor);
> + count++;
> }
> }
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-05-14 15:47 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-06 15:27 [PATCH V4] hwmon: (ibmpowernv) Add highest/lowest attributes to sensors Shilpasri G Bhat
2017-05-14 15:47 ` Guenter Roeck
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).