From: Eddie James <eajames@linux.vnet.ibm.com>
To: Guenter Roeck <linux@roeck-us.net>, Eddie James <eajames@linux.ibm.com>
Cc: linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org,
devicetree@vger.kernel.org, jdelvare@suse.com,
linux-doc@vger.kernel.org, gregkh@linuxfoundation.org,
robh+dt@kernel.org, mark.rutland@arm.com, rdunlap@infradead.org,
benh@kernel.crashing.org
Subject: Re: [PATCH v6 00/10] hwmon and fsi: Add On-Chip Controller Driver
Date: Mon, 12 Nov 2018 10:25:19 -0600 [thread overview]
Message-ID: <65651d7d-7c29-4cc5-7ebf-b9d33fed7500@linux.vnet.ibm.com> (raw)
In-Reply-To: <20181109230354.GA31326@roeck-us.net>
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@linux.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
>>
prev parent reply other threads:[~2018-11-13 2:21 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-08 21:05 [PATCH v6 00/10] hwmon and fsi: Add On-Chip Controller Driver Eddie James
2018-11-08 21:05 ` [PATCH v6 01/10] dt-bindings: fsi: Add P9 OCC device documentation Eddie James
2018-11-08 21:05 ` [PATCH v6 02/10] fsi: Add On-Chip Controller (OCC) driver Eddie James
2018-11-08 21:05 ` [PATCH v6 03/10] Documentation: hwmon: Add OCC documentation Eddie James
2018-11-08 21:05 ` [PATCH v6 04/10] dt-bindings: i2c: Add P8 OCC hwmon device documentation Eddie James
2018-11-08 21:05 ` [PATCH v6 05/10] hwmon: Add On-Chip Controller (OCC) hwmon driver Eddie James
2018-11-08 21:05 ` [PATCH v6 06/10] hwmon (occ): Add command transport method for P8 and P9 Eddie James
2018-11-08 21:05 ` [PATCH v6 07/10] hwmon (occ): Parse OCC poll response Eddie James
2018-11-08 21:05 ` [PATCH v6 08/10] hwmon (occ): Add sensor types and versions Eddie James
2018-11-08 21:05 ` [PATCH v6 09/10] hwmon (occ): Add sensor attributes and register hwmon device Eddie James
2018-11-08 21:05 ` [PATCH v6 10/10] hwmon (occ): Add sysfs attributes for additional OCC data Eddie James
2018-11-09 23:03 ` [PATCH v6 00/10] hwmon and fsi: Add On-Chip Controller Driver Guenter Roeck
2018-11-12 16:25 ` Eddie James [this message]
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=65651d7d-7c29-4cc5-7ebf-b9d33fed7500@linux.vnet.ibm.com \
--to=eajames@linux.vnet.ibm.com \
--cc=benh@kernel.crashing.org \
--cc=devicetree@vger.kernel.org \
--cc=eajames@linux.ibm.com \
--cc=gregkh@linuxfoundation.org \
--cc=jdelvare@suse.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=mark.rutland@arm.com \
--cc=rdunlap@infradead.org \
--cc=robh+dt@kernel.org \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).