Message-Id: <65651d7d-7c29-4cc5-7ebf-b9d33fed7500@linux.vnet.ibm.com>
Date:   Mon, 12 Nov 2018 10:25:19 -0600
From:   Eddie James <eajames@...ux.vnet.ibm.com>
To:     Guenter Roeck <linux@...ck-us.net>,
        Eddie James <eajames@...ux.ibm.com>
Cc:     linux-kernel@...r.kernel.org, linux-hwmon@...r.kernel.org,
        devicetree@...r.kernel.org, jdelvare@...e.com,
        linux-doc@...r.kernel.org, gregkh@...uxfoundation.org,
        robh+dt@...nel.org, mark.rutland@....com, rdunlap@...radead.org,
        benh@...nel.crashing.org
Subject: Re: [PATCH v6 00/10] hwmon and fsi: Add On-Chip Controller Driver



On 11/09/2018 05:03 PM, Guenter Roeck wrote:
> On Thu, Nov 08, 2018 at 03:05:19PM -0600, Eddie James wrote:
>> From: Eddie James <eajames@...ux.vnet.ibm.com>
>>
>> This series adds a hwmon driver to support the OCC on POWER8 and POWER9
>> processors. The OCC is an embedded processor that provides realtime power and
>> thermal monitoring and management.
>>
>> The series also adds a "bus" driver to handle atomic communication between the
>> service processor and the OCC on a POWER9 chip. This communication takes place
>> over FSI bus to the SBE (Self-Boot engine) FIFO, which in turn communicates
>> with the OCC. The driver for the SBEFIFO is already available as an FSI client
>> driver.
>>
>> For POWER8 OCCs, communication between the service processor and the OCC is
>> achieved over I2C bus.
>>
> I am not entirely happy with the series - there are still lots of proprietary
> attributes, and I would have preferred the use of the _info API at this point -
> but this has taken long enough. Series applied to hwmon-next. Please send any
> fixes as follow-up patches.

Thanks a lot Guenter, appreciate your help getting here.

Eddie

>
> Thanks,
> Guenter
>
>> Changes since v5:
>>   * Makefile fix when compiling both P8 and P9 versions
>>   * Spelling fix in hwmon doc
>>   * Added an additional sentence for P9 binding doc to explain that OCC isn't
>>     an FSI slave device.
>>
>> Changes since v4:
>>   * Make the hwmon attributes conform almost completely to standard names and
>>     values. The only exception is powerX_cap_user and powerX_cap_user_source.
>>   * Improve hwmon documentation.
>>   * Add ibm,p9-occ dt documentation.
>>
>> Changes since v3:
>>   * Add the FSI OCC driver.
>>   * Pull the sysfs attribute code into it's own file for cleanliness.
>>   * Various fixes for attribute creation and integer overflow.
>>
>> Changes since v2:
>>   * Add sysfs_notify for the error and throttling attributes when change is
>>     detected.
>>   * Removed occs_present counting of devices bound.
>>   * Improved remove() of P9 driver to avoid bad behavior with relation to OCC
>>     driver when unbound.
>>   * Added default cases (return EINVAL) for all sensor show functions.
>>   * Added temperature fault sensor.
>>   * Added back dt binding documentation for P9 to address checkpatch warning.
>>   * Added occs_present attribute from the poll response.
>>
>> Changes since v1:
>>   * Remove wait loop in P9 code, as that is now handled by FSI OCC driver.
>>   * Removed dt binding documentation for P9, FSI OCC driver will probe OCC hwmon
>>     driver automatically.
>>   * Moved OCC response code definitions to the OCC include file.
>>   * Fixed includes.
>>   * Changed some structure fields to __beXX as that is what they are.
>>   * Changed some errnos.
>>   * Removed some dev_err().
>>   * Refactored P8 code a bit to use #defined addresses and magic values, and
>>     changed "goto retry" to a loop.
>>   * Refactored error handling a bit.
>>
>> Eddie James (10):
>>    dt-bindings: fsi: Add P9 OCC device documentation
>>    fsi: Add On-Chip Controller (OCC) driver
>>    Documentation: hwmon: Add OCC documentation
>>    dt-bindings: i2c: Add P8 OCC hwmon device documentation
>>    hwmon: Add On-Chip Controller (OCC) hwmon driver
>>    hwmon (occ): Add command transport method for P8 and P9
>>    hwmon (occ): Parse OCC poll response
>>    hwmon (occ): Add sensor types and versions
>>    hwmon (occ): Add sensor attributes and register hwmon device
>>    hwmon (occ): Add sysfs attributes for additional OCC data
>>
>>   .../devicetree/bindings/fsi/ibm,p9-occ.txt         |   16 +
>>   .../devicetree/bindings/i2c/ibm,p8-occ-hwmon.txt   |   25 +
>>   Documentation/hwmon/occ                            |  112 ++
>>   drivers/fsi/Kconfig                                |   10 +
>>   drivers/fsi/Makefile                               |    1 +
>>   drivers/fsi/fsi-occ.c                              |  599 +++++++++++
>>   drivers/hwmon/Kconfig                              |    2 +
>>   drivers/hwmon/Makefile                             |    1 +
>>   drivers/hwmon/occ/Kconfig                          |   31 +
>>   drivers/hwmon/occ/Makefile                         |    5 +
>>   drivers/hwmon/occ/common.c                         | 1098 ++++++++++++++++++++
>>   drivers/hwmon/occ/common.h                         |  128 +++
>>   drivers/hwmon/occ/p8_i2c.c                         |  255 +++++
>>   drivers/hwmon/occ/p9_sbe.c                         |  106 ++
>>   drivers/hwmon/occ/sysfs.c                          |  188 ++++
>>   include/linux/fsi-occ.h                            |   25 +
>>   16 files changed, 2602 insertions(+)
>>   create mode 100644 Documentation/devicetree/bindings/fsi/ibm,p9-occ.txt
>>   create mode 100644 Documentation/devicetree/bindings/i2c/ibm,p8-occ-hwmon.txt
>>   create mode 100644 Documentation/hwmon/occ
>>   create mode 100644 drivers/fsi/fsi-occ.c
>>   create mode 100644 drivers/hwmon/occ/Kconfig
>>   create mode 100644 drivers/hwmon/occ/Makefile
>>   create mode 100644 drivers/hwmon/occ/common.c
>>   create mode 100644 drivers/hwmon/occ/common.h
>>   create mode 100644 drivers/hwmon/occ/p8_i2c.c
>>   create mode 100644 drivers/hwmon/occ/p9_sbe.c
>>   create mode 100644 drivers/hwmon/occ/sysfs.c
>>   create mode 100644 include/linux/fsi-occ.h
>>
>> -- 
>> 1.8.3.1
>>

Powered by blists - more mailing lists