kernel/drivers/net/ethernet/sfc/mcdi_mon.c

   1 /****************************************************************************
   2  * Driver for Solarflare network controllers and boards
   3  * Copyright 2011-2013 Solarflare Communications Inc.
   4  *
   5  * This program is free software; you can redistribute it and/or modify it
   6  * under the terms of the GNU General Public License version 2 as published
   7  * by the Free Software Foundation, incorporated herein by reference.
   8  */
   9
  10 #include <linux/bitops.h>
  11 #include <linux/slab.h>
  12 #include <linux/hwmon.h>
  13 #include <linux/stat.h>
  14
  15 #include "net_driver.h"
  16 #include "mcdi.h"
  17 #include "mcdi_pcol.h"
  18 #include "nic.h"
  19
  20 enum efx_hwmon_type {
  21         EFX_HWMON_UNKNOWN,
  22         EFX_HWMON_TEMP,         /* temperature */
  23         EFX_HWMON_COOL,         /* cooling device, probably a heatsink */
  24         EFX_HWMON_IN,           /* voltage */
  25         EFX_HWMON_CURR,         /* current */
  26         EFX_HWMON_POWER,        /* power */
  27         EFX_HWMON_TYPES_COUNT
  28 };
  29
  30 static const char *const efx_hwmon_unit[EFX_HWMON_TYPES_COUNT] = {
  31         [EFX_HWMON_TEMP]  = " degC",
  32         [EFX_HWMON_COOL]  = " rpm", /* though nonsense for a heatsink */
  33         [EFX_HWMON_IN]    = " mV",
  34         [EFX_HWMON_CURR]  = " mA",
  35         [EFX_HWMON_POWER] = " W",
  36 };
  37
  38 static const struct {
  39         const char *label;
  40         enum efx_hwmon_type hwmon_type;
  41         int port;
  42 } efx_mcdi_sensor_type[] = {
  43 #define SENSOR(name, label, hwmon_type, port)                           \
  44         [MC_CMD_SENSOR_##name] = { label, EFX_HWMON_ ## hwmon_type, port }
  45         SENSOR(CONTROLLER_TEMP,         "Controller board temp.",   TEMP,  -1),
  46         SENSOR(PHY_COMMON_TEMP,         "PHY temp.",                TEMP,  -1),
  47         SENSOR(CONTROLLER_COOLING,      "Controller heat sink",     COOL,  -1),
  48         SENSOR(PHY0_TEMP,               "PHY temp.",                TEMP,  0),
  49         SENSOR(PHY0_COOLING,            "PHY heat sink",            COOL,  0),
  50         SENSOR(PHY1_TEMP,               "PHY temp.",                TEMP,  1),
  51         SENSOR(PHY1_COOLING,            "PHY heat sink",            COOL,  1),
  52         SENSOR(IN_1V0,                  "1.0V supply",              IN,    -1),
  53         SENSOR(IN_1V2,                  "1.2V supply",              IN,    -1),
  54         SENSOR(IN_1V8,                  "1.8V supply",              IN,    -1),
  55         SENSOR(IN_2V5,                  "2.5V supply",              IN,    -1),
  56         SENSOR(IN_3V3,                  "3.3V supply",              IN,    -1),
  57         SENSOR(IN_12V0,                 "12.0V supply",             IN,    -1),
  58         SENSOR(IN_1V2A,                 "1.2V analogue supply",     IN,    -1),
  59         SENSOR(IN_VREF,                 "Ref. voltage",             IN,    -1),
  60         SENSOR(OUT_VAOE,                "AOE FPGA supply",          IN,    -1),
  61         SENSOR(AOE_TEMP,                "AOE FPGA temp.",           TEMP,  -1),
  62         SENSOR(PSU_AOE_TEMP,            "AOE regulator temp.",      TEMP,  -1),
  63         SENSOR(PSU_TEMP,                "Controller regulator temp.",
  64                                                                     TEMP,  -1),
  65         SENSOR(FAN_0,                   "Fan 0",                    COOL,  -1),
  66         SENSOR(FAN_1,                   "Fan 1",                    COOL,  -1),
  67         SENSOR(FAN_2,                   "Fan 2",                    COOL,  -1),
  68         SENSOR(FAN_3,                   "Fan 3",                    COOL,  -1),
  69         SENSOR(FAN_4,                   "Fan 4",                    COOL,  -1),
  70         SENSOR(IN_VAOE,                 "AOE input supply",         IN,    -1),
  71         SENSOR(OUT_IAOE,                "AOE output current",       CURR,  -1),
  72         SENSOR(IN_IAOE,                 "AOE input current",        CURR,  -1),
  73         SENSOR(NIC_POWER,               "Board power use",          POWER, -1),
  74         SENSOR(IN_0V9,                  "0.9V supply",              IN,    -1),
  75         SENSOR(IN_I0V9,                 "0.9V supply current",      CURR,  -1),
  76         SENSOR(IN_I1V2,                 "1.2V supply current",      CURR,  -1),
  77         SENSOR(IN_0V9_ADC,              "0.9V supply (ext. ADC)",   IN,    -1),
  78         SENSOR(CONTROLLER_2_TEMP,       "Controller board temp. 2", TEMP,  -1),
  79         SENSOR(VREG_INTERNAL_TEMP,      "Regulator die temp.",      TEMP,  -1),
  80         SENSOR(VREG_0V9_TEMP,           "0.9V regulator temp.",     TEMP,  -1),
  81         SENSOR(VREG_1V2_TEMP,           "1.2V regulator temp.",     TEMP,  -1),
  82         SENSOR(CONTROLLER_VPTAT,
  83                               "Controller PTAT voltage (int. ADC)", IN,    -1),
  84         SENSOR(CONTROLLER_INTERNAL_TEMP,
  85                                  "Controller die temp. (int. ADC)", TEMP,  -1),
  86         SENSOR(CONTROLLER_VPTAT_EXTADC,
  87                               "Controller PTAT voltage (ext. ADC)", IN,    -1),
  88         SENSOR(CONTROLLER_INTERNAL_TEMP_EXTADC,
  89                                  "Controller die temp. (ext. ADC)", TEMP,  -1),
  90         SENSOR(AMBIENT_TEMP,            "Ambient temp.",            TEMP,  -1),
  91         SENSOR(AIRFLOW,                 "Air flow raw",             IN,    -1),
  92         SENSOR(VDD08D_VSS08D_CSR,       "0.9V die (int. ADC)",      IN,    -1),
  93         SENSOR(VDD08D_VSS08D_CSR_EXTADC, "0.9V die (ext. ADC)",     IN,    -1),
  94         SENSOR(HOTPOINT_TEMP,  "Controller board temp. (hotpoint)", TEMP,  -1),
  95 #undef SENSOR
  96 };
  97
  98 static const char *const sensor_status_names[] = {
  99         [MC_CMD_SENSOR_STATE_OK] = "OK",
 100         [MC_CMD_SENSOR_STATE_WARNING] = "Warning",
 101         [MC_CMD_SENSOR_STATE_FATAL] = "Fatal",
 102         [MC_CMD_SENSOR_STATE_BROKEN] = "Device failure",
 103         [MC_CMD_SENSOR_STATE_NO_READING] = "No reading",
 104 };
 105
 106 void efx_mcdi_sensor_event(struct efx_nic *efx, efx_qword_t *ev)
 107 {
 108         unsigned int type, state, value;
 109         enum efx_hwmon_type hwmon_type = EFX_HWMON_UNKNOWN;
 110         const char *name = NULL, *state_txt, *unit;
 111
 112         type = EFX_QWORD_FIELD(*ev, MCDI_EVENT_SENSOREVT_MONITOR);
 113         state = EFX_QWORD_FIELD(*ev, MCDI_EVENT_SENSOREVT_STATE);
 114         value = EFX_QWORD_FIELD(*ev, MCDI_EVENT_SENSOREVT_VALUE);
 115
 116         /* Deal gracefully with the board having more drivers than we
 117          * know about, but do not expect new sensor states. */
 118         if (type < ARRAY_SIZE(efx_mcdi_sensor_type)) {
 119                 name = efx_mcdi_sensor_type[type].label;
 120                 hwmon_type = efx_mcdi_sensor_type[type].hwmon_type;
 121         }
 122         if (!name)
 123                 name = "No sensor name available";
 124         EFX_BUG_ON_PARANOID(state >= ARRAY_SIZE(sensor_status_names));
 125         state_txt = sensor_status_names[state];
 126         EFX_BUG_ON_PARANOID(hwmon_type >= EFX_HWMON_TYPES_COUNT);
 127         unit = efx_hwmon_unit[hwmon_type];
 128         if (!unit)
 129                 unit = "";
 130
 131         netif_err(efx, hw, efx->net_dev,
 132                   "Sensor %d (%s) reports condition '%s' for value %d%s\n",
 133                   type, name, state_txt, value, unit);
 134 }
 135
 136 #ifdef CONFIG_SFC_MCDI_MON
 137
 138 struct efx_mcdi_mon_attribute {
 139         struct device_attribute dev_attr;
 140         unsigned int index;
 141         unsigned int type;
 142         enum efx_hwmon_type hwmon_type;
 143         unsigned int limit_value;
 144         char name[12];
 145 };
 146
 147 static int efx_mcdi_mon_update(struct efx_nic *efx)
 148 {
 149         struct efx_mcdi_mon *hwmon = efx_mcdi_mon(efx);
 150         MCDI_DECLARE_BUF(inbuf, MC_CMD_READ_SENSORS_EXT_IN_LEN);
 151         int rc;
 152
 153         MCDI_SET_QWORD(inbuf, READ_SENSORS_EXT_IN_DMA_ADDR,
 154                        hwmon->dma_buf.dma_addr);
 155         MCDI_SET_DWORD(inbuf, READ_SENSORS_EXT_IN_LENGTH, hwmon->dma_buf.len);
 156
 157         rc = efx_mcdi_rpc(efx, MC_CMD_READ_SENSORS,
 158                           inbuf, sizeof(inbuf), NULL, 0, NULL);
 159         if (rc == 0)
 160                 hwmon->last_update = jiffies;
 161         return rc;
 162 }
 163
 164 static int efx_mcdi_mon_get_entry(struct device *dev, unsigned int index,
 165                                   efx_dword_t *entry)
 166 {
 167         struct efx_nic *efx = dev_get_drvdata(dev->parent);
 168         struct efx_mcdi_mon *hwmon = efx_mcdi_mon(efx);
 169         int rc;
 170
 171         BUILD_BUG_ON(MC_CMD_READ_SENSORS_OUT_LEN != 0);
 172
 173         mutex_lock(&hwmon->update_lock);
 174
 175         /* Use cached value if last update was < 1 s ago */
 176         if (time_before(jiffies, hwmon->last_update + HZ))
 177                 rc = 0;
 178         else
 179                 rc = efx_mcdi_mon_update(efx);
 180
 181         /* Copy out the requested entry */
 182         *entry = ((efx_dword_t *)hwmon->dma_buf.addr)[index];
 183
 184         mutex_unlock(&hwmon->update_lock);
 185
 186         return rc;
 187 }
 188
 189 static ssize_t efx_mcdi_mon_show_value(struct device *dev,
 190                                        struct device_attribute *attr,
 191                                        char *buf)
 192 {
 193         struct efx_mcdi_mon_attribute *mon_attr =
 194                 container_of(attr, struct efx_mcdi_mon_attribute, dev_attr);
 195         efx_dword_t entry;
 196         unsigned int value, state;
 197         int rc;
 198
 199         rc = efx_mcdi_mon_get_entry(dev, mon_attr->index, &entry);
 200         if (rc)
 201                 return rc;
 202
 203         state = EFX_DWORD_FIELD(entry, MC_CMD_SENSOR_VALUE_ENTRY_TYPEDEF_STATE);
 204         if (state == MC_CMD_SENSOR_STATE_NO_READING)
 205                 return -EBUSY;
 206
 207         value = EFX_DWORD_FIELD(entry, MC_CMD_SENSOR_VALUE_ENTRY_TYPEDEF_VALUE);
 208
 209         switch (mon_attr->hwmon_type) {
 210         case EFX_HWMON_TEMP:
 211                 /* Convert temperature from degrees to milli-degrees Celsius */
 212                 value *= 1000;
 213                 break;
 214         case EFX_HWMON_POWER:
 215                 /* Convert power from watts to microwatts */
 216                 value *= 1000000;
 217                 break;
 218         default:
 219                 /* No conversion needed */
 220                 break;
 221         }
 222
 223         return sprintf(buf, "%u\n", value);
 224 }
 225
 226 static ssize_t efx_mcdi_mon_show_limit(struct device *dev,
 227                                        struct device_attribute *attr,
 228                                        char *buf)
 229 {
 230         struct efx_mcdi_mon_attribute *mon_attr =
 231                 container_of(attr, struct efx_mcdi_mon_attribute, dev_attr);
 232         unsigned int value;
 233
 234         value = mon_attr->limit_value;
 235
 236         switch (mon_attr->hwmon_type) {
 237         case EFX_HWMON_TEMP:
 238                 /* Convert temperature from degrees to milli-degrees Celsius */
 239                 value *= 1000;
 240                 break;
 241         case EFX_HWMON_POWER:
 242                 /* Convert power from watts to microwatts */
 243                 value *= 1000000;
 244                 break;
 245         default:
 246                 /* No conversion needed */
 247                 break;
 248         }
 249
 250         return sprintf(buf, "%u\n", value);
 251 }
 252
 253 static ssize_t efx_mcdi_mon_show_alarm(struct device *dev,
 254                                        struct device_attribute *attr,
 255                                        char *buf)
 256 {
 257         struct efx_mcdi_mon_attribute *mon_attr =
 258                 container_of(attr, struct efx_mcdi_mon_attribute, dev_attr);
 259         efx_dword_t entry;
 260         int state;
 261         int rc;
 262
 263         rc = efx_mcdi_mon_get_entry(dev, mon_attr->index, &entry);
 264         if (rc)
 265                 return rc;
 266
 267         state = EFX_DWORD_FIELD(entry, MC_CMD_SENSOR_VALUE_ENTRY_TYPEDEF_STATE);
 268         return sprintf(buf, "%d\n", state != MC_CMD_SENSOR_STATE_OK);
 269 }
 270
 271 static ssize_t efx_mcdi_mon_show_label(struct device *dev,
 272                                        struct device_attribute *attr,
 273                                        char *buf)
 274 {
 275         struct efx_mcdi_mon_attribute *mon_attr =
 276                 container_of(attr, struct efx_mcdi_mon_attribute, dev_attr);
 277         return sprintf(buf, "%s\n",
 278                        efx_mcdi_sensor_type[mon_attr->type].label);
 279 }
 280
 281 static void
 282 efx_mcdi_mon_add_attr(struct efx_nic *efx, const char *name,
 283                       ssize_t (*reader)(struct device *,
 284                                         struct device_attribute *, char *),
 285                       unsigned int index, unsigned int type,
 286                       unsigned int limit_value)
 287 {
 288         struct efx_mcdi_mon *hwmon = efx_mcdi_mon(efx);
 289         struct efx_mcdi_mon_attribute *attr = &hwmon->attrs[hwmon->n_attrs];
 290
 291         strlcpy(attr->name, name, sizeof(attr->name));
 292         attr->index = index;
 293         attr->type = type;
 294         if (type < ARRAY_SIZE(efx_mcdi_sensor_type))
 295                 attr->hwmon_type = efx_mcdi_sensor_type[type].hwmon_type;
 296         else
 297                 attr->hwmon_type = EFX_HWMON_UNKNOWN;
 298         attr->limit_value = limit_value;
 299         sysfs_attr_init(&attr->dev_attr.attr);
 300         attr->dev_attr.attr.name = attr->name;
 301         attr->dev_attr.attr.mode = S_IRUGO;
 302         attr->dev_attr.show = reader;
 303         hwmon->group.attrs[hwmon->n_attrs++] = &attr->dev_attr.attr;
 304 }
 305
 306 int efx_mcdi_mon_probe(struct efx_nic *efx)
 307 {
 308         unsigned int n_temp = 0, n_cool = 0, n_in = 0, n_curr = 0, n_power = 0;
 309         struct efx_mcdi_mon *hwmon = efx_mcdi_mon(efx);
 310         MCDI_DECLARE_BUF(inbuf, MC_CMD_SENSOR_INFO_EXT_IN_LEN);
 311         MCDI_DECLARE_BUF(outbuf, MC_CMD_SENSOR_INFO_OUT_LENMAX);
 312         unsigned int n_pages, n_sensors, n_attrs, page;
 313         size_t outlen;
 314         char name[12];
 315         u32 mask;
 316         int rc, i, j, type;
 317
 318         /* Find out how many sensors are present */
 319         n_sensors = 0;
 320         page = 0;
 321         do {
 322                 MCDI_SET_DWORD(inbuf, SENSOR_INFO_EXT_IN_PAGE, page);
 323
 324                 rc = efx_mcdi_rpc(efx, MC_CMD_SENSOR_INFO, inbuf, sizeof(inbuf),
 325                                   outbuf, sizeof(outbuf), &outlen);
 326                 if (rc)
 327                         return rc;
 328                 if (outlen < MC_CMD_SENSOR_INFO_OUT_LENMIN)
 329                         return -EIO;
 330
 331                 mask = MCDI_DWORD(outbuf, SENSOR_INFO_OUT_MASK);
 332                 n_sensors += hweight32(mask & ~(1 << MC_CMD_SENSOR_PAGE0_NEXT));
 333                 ++page;
 334         } while (mask & (1 << MC_CMD_SENSOR_PAGE0_NEXT));
 335         n_pages = page;
 336
 337         /* Don't create a device if there are none */
 338         if (n_sensors == 0)
 339                 return 0;
 340
 341         rc = efx_nic_alloc_buffer(
 342                 efx, &hwmon->dma_buf,
 343                 n_sensors * MC_CMD_SENSOR_VALUE_ENTRY_TYPEDEF_LEN,
 344                 GFP_KERNEL);
 345         if (rc)
 346                 return rc;
 347
 348         mutex_init(&hwmon->update_lock);
 349         efx_mcdi_mon_update(efx);
 350
 351         /* Allocate space for the maximum possible number of
 352          * attributes for this set of sensors:
 353          * value, min, max, crit, alarm and label for each sensor.
 354          */
 355         n_attrs = 6 * n_sensors;
 356         hwmon->attrs = kcalloc(n_attrs, sizeof(*hwmon->attrs), GFP_KERNEL);
 357         if (!hwmon->attrs) {
 358                 rc = -ENOMEM;
 359                 goto fail;
 360         }
 361         hwmon->group.attrs = kcalloc(n_attrs + 1, sizeof(struct attribute *),
 362                                      GFP_KERNEL);
 363         if (!hwmon->group.attrs) {
 364                 rc = -ENOMEM;
 365                 goto fail;
 366         }
 367
 368         for (i = 0, j = -1, type = -1; ; i++) {
 369                 enum efx_hwmon_type hwmon_type;
 370                 const char *hwmon_prefix;
 371                 unsigned hwmon_index;
 372                 u16 min1, max1, min2, max2;
 373
 374                 /* Find next sensor type or exit if there is none */
 375                 do {
 376                         type++;
 377
 378                         if ((type % 32) == 0) {
 379                                 page = type / 32;
 380                                 j = -1;
 381                                 if (page == n_pages)
 382                                         goto hwmon_register;
 383
 384                                 MCDI_SET_DWORD(inbuf, SENSOR_INFO_EXT_IN_PAGE,
 385                                                page);
 386                                 rc = efx_mcdi_rpc(efx, MC_CMD_SENSOR_INFO,
 387                                                   inbuf, sizeof(inbuf),
 388                                                   outbuf, sizeof(outbuf),
 389                                                   &outlen);
 390                                 if (rc)
 391                                         goto fail;
 392                                 if (outlen < MC_CMD_SENSOR_INFO_OUT_LENMIN) {
 393                                         rc = -EIO;
 394                                         goto fail;
 395                                 }
 396
 397                                 mask = (MCDI_DWORD(outbuf,
 398                                                    SENSOR_INFO_OUT_MASK) &
 399                                         ~(1 << MC_CMD_SENSOR_PAGE0_NEXT));
 400
 401                                 /* Check again for short response */
 402                                 if (outlen <
 403                                     MC_CMD_SENSOR_INFO_OUT_LEN(hweight32(mask))) {
 404                                         rc = -EIO;
 405                                         goto fail;
 406                                 }
 407                         }
 408                 } while (!(mask & (1 << type % 32)));
 409                 j++;
 410
 411                 if (type < ARRAY_SIZE(efx_mcdi_sensor_type)) {
 412                         hwmon_type = efx_mcdi_sensor_type[type].hwmon_type;
 413
 414                         /* Skip sensors specific to a different port */
 415                         if (hwmon_type != EFX_HWMON_UNKNOWN &&
 416                             efx_mcdi_sensor_type[type].port >= 0 &&
 417                             efx_mcdi_sensor_type[type].port !=
 418                             efx_port_num(efx))
 419                                 continue;
 420                 } else {
 421                         hwmon_type = EFX_HWMON_UNKNOWN;
 422                 }
 423
 424                 switch (hwmon_type) {
 425                 case EFX_HWMON_TEMP:
 426                         hwmon_prefix = "temp";
 427                         hwmon_index = ++n_temp; /* 1-based */
 428                         break;
 429                 case EFX_HWMON_COOL:
 430                         /* This is likely to be a heatsink, but there
 431                          * is no convention for representing cooling
 432                          * devices other than fans.
 433                          */
 434                         hwmon_prefix = "fan";
 435                         hwmon_index = ++n_cool; /* 1-based */
 436                         break;
 437                 default:
 438                         hwmon_prefix = "in";
 439                         hwmon_index = n_in++; /* 0-based */
 440                         break;
 441                 case EFX_HWMON_CURR:
 442                         hwmon_prefix = "curr";
 443                         hwmon_index = ++n_curr; /* 1-based */
 444                         break;
 445                 case EFX_HWMON_POWER:
 446                         hwmon_prefix = "power";
 447                         hwmon_index = ++n_power; /* 1-based */
 448                         break;
 449                 }
 450
 451                 min1 = MCDI_ARRAY_FIELD(outbuf, SENSOR_ENTRY,
 452                                         SENSOR_INFO_ENTRY, j, MIN1);
 453                 max1 = MCDI_ARRAY_FIELD(outbuf, SENSOR_ENTRY,
 454                                         SENSOR_INFO_ENTRY, j, MAX1);
 455                 min2 = MCDI_ARRAY_FIELD(outbuf, SENSOR_ENTRY,
 456                                         SENSOR_INFO_ENTRY, j, MIN2);
 457                 max2 = MCDI_ARRAY_FIELD(outbuf, SENSOR_ENTRY,
 458                                         SENSOR_INFO_ENTRY, j, MAX2);
 459
 460                 if (min1 != max1) {
 461                         snprintf(name, sizeof(name), "%s%u_input",
 462                                  hwmon_prefix, hwmon_index);
 463                         efx_mcdi_mon_add_attr(
 464                                 efx, name, efx_mcdi_mon_show_value, i, type, 0);
 465
 466                         if (hwmon_type != EFX_HWMON_POWER) {
 467                                 snprintf(name, sizeof(name), "%s%u_min",
 468                                          hwmon_prefix, hwmon_index);
 469                                 efx_mcdi_mon_add_attr(
 470                                         efx, name, efx_mcdi_mon_show_limit,
 471                                         i, type, min1);
 472                         }
 473
 474                         snprintf(name, sizeof(name), "%s%u_max",
 475                                  hwmon_prefix, hwmon_index);
 476                         efx_mcdi_mon_add_attr(
 477                                 efx, name, efx_mcdi_mon_show_limit,
 478                                 i, type, max1);
 479
 480                         if (min2 != max2) {
 481                                 /* Assume max2 is critical value.
 482                                  * But we have no good way to expose min2.
 483                                  */
 484                                 snprintf(name, sizeof(name), "%s%u_crit",
 485                                          hwmon_prefix, hwmon_index);
 486                                 efx_mcdi_mon_add_attr(
 487                                         efx, name, efx_mcdi_mon_show_limit,
 488                                         i, type, max2);
 489                         }
 490                 }
 491
 492                 snprintf(name, sizeof(name), "%s%u_alarm",
 493                          hwmon_prefix, hwmon_index);
 494                 efx_mcdi_mon_add_attr(
 495                         efx, name, efx_mcdi_mon_show_alarm, i, type, 0);
 496
 497                 if (type < ARRAY_SIZE(efx_mcdi_sensor_type) &&
 498                     efx_mcdi_sensor_type[type].label) {
 499                         snprintf(name, sizeof(name), "%s%u_label",
 500                                  hwmon_prefix, hwmon_index);
 501                         efx_mcdi_mon_add_attr(
 502                                 efx, name, efx_mcdi_mon_show_label, i, type, 0);
 503                 }
 504         }
 505
 506 hwmon_register:
 507         hwmon->groups[0] = &hwmon->group;
 508         hwmon->device = hwmon_device_register_with_groups(&efx->pci_dev->dev,
 509                                                           KBUILD_MODNAME, NULL,
 510                                                           hwmon->groups);
 511         if (IS_ERR(hwmon->device)) {
 512                 rc = PTR_ERR(hwmon->device);
 513                 goto fail;
 514         }
 515
 516         return 0;
 517
 518 fail:
 519         efx_mcdi_mon_remove(efx);
 520         return rc;
 521 }
 522
 523 void efx_mcdi_mon_remove(struct efx_nic *efx)
 524 {
 525         struct efx_mcdi_mon *hwmon = efx_mcdi_mon(efx);
 526
 527         if (hwmon->device)
 528                 hwmon_device_unregister(hwmon->device);
 529         kfree(hwmon->attrs);
 530         kfree(hwmon->group.attrs);
 531         efx_nic_free_buffer(efx, &hwmon->dma_buf);
 532 }
 533
 534 #endif /* CONFIG_SFC_MCDI_MON */