[PATCH v2 0/2] Add adm1281 support

Jose Ramon San Buenaventura posted 2 patches 1 year, 9 months ago
There is a newer version of this series
.../devicetree/bindings/hwmon/adi,adm1275.yaml     |  4 +++-
Documentation/hwmon/adm1275.rst                    | 14 +++++++++++---
drivers/hwmon/pmbus/Kconfig                        |  4 ++--
drivers/hwmon/pmbus/adm1275.c                      | 11 +++++++++--
4 files changed, 25 insertions(+), 8 deletions(-)
[PATCH v2 0/2] Add adm1281 support
Posted by Jose Ramon San Buenaventura 1 year, 9 months ago
This patch removes the extra case added in the adm1275_read_byte_data
for STATUS_CML reads. Upon checking, the reads for the STATUS_CML register
is already handled in the pmbus_core. 

It was also clarified and agreed upon that any other actionable steps
involving the STATUS_CML error flags should be added in the pmbus_core
and not on the specific chip driver.

Jose Ramon San Buenaventura (2):
  dt-bindings: hwmon: adm1275: add adm1281
  hwmon: pmbus: adm1275: add adm1281 support

 .../devicetree/bindings/hwmon/adi,adm1275.yaml     |  4 +++-
 Documentation/hwmon/adm1275.rst                    | 14 +++++++++++---
 drivers/hwmon/pmbus/Kconfig                        |  4 ++--
 drivers/hwmon/pmbus/adm1275.c                      | 11 +++++++++--
 4 files changed, 25 insertions(+), 8 deletions(-)


base-commit: 96fca68c4fbf77a8185eb10f7557e23352732ea2
-- 
2.39.2
Re: [PATCH v2 0/2] Add adm1281 support
Posted by Krzysztof Kozlowski 1 year, 9 months ago
On 22/04/2024 04:51, Jose Ramon San Buenaventura wrote:
> This patch removes the extra case added in the adm1275_read_byte_data
> for STATUS_CML reads. Upon checking, the reads for the STATUS_CML register
> is already handled in the pmbus_core. 
> 
> It was also clarified and agreed upon that any other actionable steps
> involving the STATUS_CML error flags should be added in the pmbus_core
> and not on the specific chip driver.
> 

Where is the changelog? It's v2, so what happened here?

Best regards,
Krzysztof
Re: [PATCH v2 0/2] Add adm1281 support
Posted by Guenter Roeck 1 year, 9 months ago
On 4/21/24 20:28, Krzysztof Kozlowski wrote:
> On 22/04/2024 04:51, Jose Ramon San Buenaventura wrote:
>> This patch removes the extra case added in the adm1275_read_byte_data
>> for STATUS_CML reads. Upon checking, the reads for the STATUS_CML register
>> is already handled in the pmbus_core.
>>
>> It was also clarified and agreed upon that any other actionable steps
>> involving the STATUS_CML error flags should be added in the pmbus_core
>> and not on the specific chip driver.
>>
> 
> Where is the changelog? It's v2, so what happened here?
> 

Same question.

Guenter