acpi_power_meter: use a {RW,RO}_SENSOR_TEMPLATE macro to clean things up
authorKyle McMartin <kyle@mcmartin.ca>
Mon, 2 Apr 2012 18:19:01 +0000 (14:19 -0400)
committerGuenter Roeck <guenter.roeck@ericsson.com>
Mon, 21 May 2012 02:41:48 +0000 (19:41 -0700)
Similar to how we do PCI/USB device id structs.

Signed-off-by: Kyle McMartin <kyle@redhat.com>
Signed-off-by: Guenter Roeck <guenter.roeck@ericsson.com>
drivers/hwmon/acpi_power_meter.c

index f9111561a30ec600e0d30b79bd8ccaef313991dc..11922c8358607b7f61d5e04dcee5e4f1462641da 100644 (file)
@@ -461,128 +461,72 @@ static ssize_t show_name(struct device *dev,
        return sprintf(buf, "%s\n", ACPI_POWER_METER_NAME);
 }
 
+#define RO_SENSOR_TEMPLATE(_label, _show, _index)      \
+       {                                               \
+               .label = _label,                        \
+               .show  = _show,                         \
+               .index = _index,                        \
+       }
+
+#define RW_SENSOR_TEMPLATE(_label, _show, _set, _index)        \
+       {                                               \
+               .label = _label,                        \
+               .show  = _show,                         \
+               .set   = _set,                          \
+               .index = _index,                        \
+       }
+
 /* Sensor descriptions.  If you add a sensor, update NUM_SENSORS above! */
 static struct sensor_template meter_ro_attrs[] = {
-       {
-               .label = POWER_AVERAGE_NAME,
-               .show  = show_power,
-               .index = 0,
-       },
-       {
-               .label = "power1_accuracy",
-               .show  = show_accuracy,
-               .index = 0,
-       },
-       {       .label = "power1_average_interval_min",
-               .show  = show_val,
-               .index = 0
-       },
-       {
-               .label = "power1_average_interval_max",
-               .show  = show_val,
-               .index = 1,
-       },
-       {
-               .label = "power1_is_battery",
-               .show = show_val,
-               .index = 5,
-       },
+       RO_SENSOR_TEMPLATE(POWER_AVERAGE_NAME, show_power, 0),
+       RO_SENSOR_TEMPLATE("power1_accuracy", show_accuracy, 0),
+       RO_SENSOR_TEMPLATE("power1_average_interval_min", show_val, 0),
+       RO_SENSOR_TEMPLATE("power1_average_interval_max", show_val, 1),
+       RO_SENSOR_TEMPLATE("power1_is_battery", show_val, 5),
        {},
 };
 
 static struct sensor_template meter_rw_attrs[] = {
-       {
-               .label = POWER_AVG_INTERVAL_NAME,
-               .show  = show_avg_interval,
-               .set   = set_avg_interval,
-               .index = 0,
-       },
+       RW_SENSOR_TEMPLATE(POWER_AVG_INTERVAL_NAME, show_avg_interval,
+               set_avg_interval, 0),
        {},
 };
 
 static struct sensor_template misc_cap_attrs[] = {
-       {
-               .label = "power1_cap_min",
-               .show  = show_val,
-               .index = 2,
-       },
-       {
-               .label = "power1_cap_max",
-               .show  = show_val,
-               .index = 3,
-       },
-       {
-               .label = "power1_cap_hyst",
-               .show  = show_val,
-               .index = 4,
-       },
-       {
-               .label = POWER_ALARM_NAME,
-               .show  = show_val,
-               .index = 6,
-       },
+       RO_SENSOR_TEMPLATE("power1_cap_min", show_val, 2),
+       RO_SENSOR_TEMPLATE("power1_cap_max", show_val, 3),
+       RO_SENSOR_TEMPLATE("power1_cap_hyst", show_val, 4),
+       RO_SENSOR_TEMPLATE(POWER_ALARM_NAME, show_val, 6),
        {},
 };
 
 static struct sensor_template ro_cap_attrs[] = {
-       {
-               .label = POWER_CAP_NAME,
-               .show  = show_cap,
-               .index = 0,
-       },
+       RO_SENSOR_TEMPLATE(POWER_CAP_NAME, show_cap, 0),
        {},
 };
 
 static struct sensor_template rw_cap_attrs[] = {
-       {
-               .label = POWER_CAP_NAME,
-               .show  = show_cap,
-               .set   = set_cap,
-               .index = 0,
-       },
+       RW_SENSOR_TEMPLATE(POWER_CAP_NAME, show_cap, set_cap, 0),
        {},
 };
 
 static struct sensor_template trip_attrs[] = {
-       {
-               .label = "power1_average_min",
-               .show  = show_val,
-               .set   = set_trip,
-               .index = 7,
-       },
-       {
-               .label = "power1_average_max",
-               .show  = show_val,
-               .set   = set_trip,
-               .index = 8,
-       },
+       RW_SENSOR_TEMPLATE("power1_average_min", show_val, set_trip, 7),
+       RW_SENSOR_TEMPLATE("power1_average_max", show_val, set_trip, 8),
        {},
 };
 
 static struct sensor_template misc_attrs[] = {
-       {
-               .label = "name",
-               .show  = show_name,
-               .index = 0,
-       },
-       {
-               .label = "power1_model_number",
-               .show  = show_str,
-               .index = 0,
-       },
-       {
-               .label = "power1_oem_info",
-               .show  = show_str,
-               .index = 2,
-       },
-       {
-               .label = "power1_serial_number",
-               .show  = show_str,
-               .index = 1,
-       },
+       RO_SENSOR_TEMPLATE("name", show_name, 0),
+       RO_SENSOR_TEMPLATE("power1_model_number", show_str, 0),
+       RO_SENSOR_TEMPLATE("power1_oem_info", show_str, 2),
+       RO_SENSOR_TEMPLATE("power1_serial_number", show_str, 1),
        {},
 };
 
+#undef RO_SENSOR_TEMPLATE
+#undef RW_SENSOR_TEMPLATE
+
 /* Read power domain data */
 static void remove_domain_devices(struct acpi_power_meter_resource *resource)
 {