linux-kernel - Re: [PATCH RESEND v6 3/4] dt-bindings: hwmon: Add bindings for max31760

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: <50319404-1b09-93ea-dbd5-b840167a4819@roeck-us.net>
Date:   Thu, 8 Sep 2022 09:18:11 -0700
From:   Guenter Roeck <linux@...ck-us.net>
To:     "Tilki, Ibrahim" <Ibrahim.Tilki@...log.com>,
        "jdelvare@...e.com" <jdelvare@...e.com>
Cc:     "linux-hwmon@...r.kernel.org" <linux-hwmon@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH RESEND v6 3/4] dt-bindings: hwmon: Add bindings for
 max31760

On 9/8/22 08:46, Tilki, Ibrahim wrote:
> 
>> On 9/8/22 01:23, Ibrahim Tilki wrote:
>>> Adding bindings for Analog Devices MAX31760 Fan-Speed Controller
>>>
>>> Signed-off-by: Ibrahim Tilki <Ibrahim.Tilki@...log.com>
>>
>> This time you dropped the maintainers from Cc:. Why ?
>>
>> Guenter
>>
> 
> I thought I need to send the patch to devicetree list <devicetree@...r.kernel.org>
> instead of sending directly to maintainers as per Rob's reply.

I don't think Rob said "instead" in his e-mail. He asked you to send
the patches to the recipients listed by the get_maintainers script.

> Should I add them to Cc as well and resend again?
> 

You'll wee if/when he replies. Note that you got a 0-day report,
so you'll have to send another version of your series anyway
with the reported problem fixed.

Thanks,
Guenter

> Ibrahim
> 
> 
>>> ---
>>>    .../bindings/hwmon/adi,max31760.yaml          | 44 +++++++++++++++++++
>>>    1 file changed, 44 insertions(+)
>>>    create mode 100644
>>> Documentation/devicetree/bindings/hwmon/adi,max31760.yaml
>>>
>>> diff --git a/Documentation/devicetree/bindings/hwmon/adi,max31760.yaml
>>> b/Documentation/devicetree/bindings/hwmon/adi,max31760.yaml
>>> new file mode 100644
>>> index 000000000..003ec1317
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/hwmon/adi,max31760.yaml
>>> @@ -0,0 +1,44 @@
>>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
>>> +---
>>> +$id:
>>> +https://urldefense.com/v3/__http://devicetree.org/schemas/hwmon/adi,m
>>> +ax31760.yaml*__;Iw!!A3Ni8CS0y2Y!4jxXOqgSL2JDA_zhVxa1gDg-X3GvtxnISuyIJ
>>> +0fz_0Blfkx3tzqwICgIVHzIxrv7-7nDoCZqZgYJ0OtFP8A$
>>> +$schema:
>>> +https://urldefense.com/v3/__http://devicetree.org/meta-schemas/core.y
>>> +aml*__;Iw!!A3Ni8CS0y2Y!4jxXOqgSL2JDA_zhVxa1gDg-X3GvtxnISuyIJ0fz_0Blfk
>>> +x3tzqwICgIVHzIxrv7-7nDoCZqZgYJ5GeDd5g$
>>> +
>>> +title: Analog Devices MAX31760 Fan-Speed Controller
>>> +
>>> +maintainers:
>>> +  - Ibrahim Tilki <Ibrahim.Tilki@...log.com>
>>> +
>>> +description: |
>>> +  Analog Devices MAX31760 Fan-Speed Controller
>>> +  https://datasheets.maximintegrated.com/en/ds/MAX31760.pdf
>>> +
>>> +properties:
>>> +  compatible:
>>> +    enum:
>>> +      - adi,max31760
>>> +
>>> +  reg:
>>> +    description: |
>>> +      I2C address of slave device.
>>> +    items:
>>> +      minimum: 0x50
>>> +      maximum: 0x57
>>> +
>>> +required:
>>> +  - compatible
>>> +  - reg
>>> +
>>> +additionalProperties: false
>>> +
>>> +examples:
>>> +  - |
>>> +    i2c0 {
>>> +        #address-cells = <1>;
>>> +        #size-cells = <0>;
>>> +
>>> +        max31760@50 {
>>> +                reg = <0x50>;
>>> +                compatible = "adi,max31760";
>>> +        };
>>> +    };