linux-kernel - Re: [PATCH 0/7] drivers/hwmon: Add On-Chip Controller (OCC) hwmon driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <7806f15f-a594-2cfb-777a-29f1e8a3fd97@roeck-us.net>
Date:   Thu, 22 Jun 2017 21:52:08 -0700
From:   Guenter Roeck <linux@...ck-us.net>
To:     Eddie James <eajames@...ux.vnet.ibm.com>,
        linux-kernel@...r.kernel.org
Cc:     linux-hwmon@...r.kernel.org, devicetree@...r.kernel.org,
        jdelvare@...e.com, mark.rutland@....com, robh+dt@...nel.org,
        gregkh@...uxfoundation.org, cbostic@...ux.vnet.ibm.com,
        jk@...abs.org, joel@....id.au, andrew@...id.au,
        "Edward A. James" <eajames@...ibm.com>
Subject: Re: [PATCH 0/7] drivers/hwmon: Add On-Chip Controller (OCC) hwmon
 driver

On 06/22/2017 03:48 PM, Eddie James wrote:
> From: "Edward A. James" <eajames@...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.
> 
> This driver has two different platform drivers as a "base" for the
> hwmon stuff, as the means of communicating with the OCC on P8 and P9 is
> completely different. For P8, the driver is an I2C client driver. For P9 the
> driver is an FSI-based OCC client driver, and uses the OCC driver in-kernel
> API.
> 
> There was a previous version of this driver that wasn't written with the
> differences in communication methods between the two versions in mind. This
> driver has been considerably simplified.
> 

I thought I did see this before.

It is customary to use "v2" in such situations, and add a change log.
You expect me to go into the two versions and compare them to figure
out what changed to evaluate if it makes sense. Do you really believe that
I have enough time to do that, and that I would be willing to spend that
time in the first place ?

Presumably you know what changed. Why not just tell me ?

Guenter

> Edward A. James (7):
>    drivers/hwmon: Add On-Chip Controller (OCC) hwmon driver
>    drivers/hwmon/occ: Add command transport method for P8 and P9
>    drivers/hwmon/occ: Parse OCC poll response
>    drivers/hwmon/occ: Add sensor types and versions
>    drivers/hwmon/occ: Add sensor attributes and register hwmon device
>    drivers/hwmon/occ: Add non-hwmon attributes
>    drivers/hwmon/occ: Add error handling
> 
>   Documentation/ABI/testing/sysfs-driver-occ-hwmon   |   77 ++
>   .../devicetree/bindings/fsi/ibm,p9-occ-hwmon.txt   |   18 +
>   .../devicetree/bindings/i2c/ibm,p8-occ-hwmon.txt   |   25 +
>   Documentation/hwmon/occ                            |   84 ++
>   drivers/hwmon/Kconfig                              |    2 +
>   drivers/hwmon/Makefile                             |    1 +
>   drivers/hwmon/occ/Kconfig                          |   28 +
>   drivers/hwmon/occ/Makefile                         |   11 +
>   drivers/hwmon/occ/common.c                         | 1242 ++++++++++++++++++++
>   drivers/hwmon/occ/common.h                         |  151 +++
>   drivers/hwmon/occ/p8_i2c.c                         |  250 ++++
>   drivers/hwmon/occ/p9_sbe.c                         |  144 +++
>   12 files changed, 2033 insertions(+)
>   create mode 100644 Documentation/ABI/testing/sysfs-driver-occ-hwmon
>   create mode 100644 Documentation/devicetree/bindings/fsi/ibm,p9-occ-hwmon.txt
>   create mode 100644 Documentation/devicetree/bindings/i2c/ibm,p8-occ-hwmon.txt
>   create mode 100644 Documentation/hwmon/occ
>   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
>