From: Guenter Roeck <linux@roeck-us.net>
To: "Tilki, Ibrahim" <Ibrahim.Tilki@analog.com>,
"jdelvare@suse.com" <jdelvare@suse.com>
Cc: "linux-hwmon@vger.kernel.org" <linux-hwmon@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH RESEND v6 3/4] dt-bindings: hwmon: Add bindings for max31760
Date: Thu, 8 Sep 2022 09:18:11 -0700 [thread overview]
Message-ID: <50319404-1b09-93ea-dbd5-b840167a4819@roeck-us.net> (raw)
In-Reply-To: <DM8PR03MB6247E3770CF8C2A20A47E45E96409@DM8PR03MB6247.namprd03.prod.outlook.com>
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@analog.com>
>>
>> This time you dropped the maintainers from Cc:. Why ?
>>
>> Guenter
>>
>
> I thought I need to send the patch to devicetree list <devicetree@vger.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@analog.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";
>>> + };
>>> + };
next prev parent reply other threads:[~2022-09-08 16:18 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-08 8:23 [PATCH RESEND v6 0/4] hwmon: Add max31760 fan speed controller Ibrahim Tilki
2022-09-08 8:23 ` [PATCH RESEND v6 1/4] drivers: hwmon: Add max31760 fan speed controller driver Ibrahim Tilki
2022-09-08 14:50 ` kernel test robot
2022-09-08 21:08 ` kernel test robot
2022-09-08 8:23 ` [PATCH RESEND v6 2/4] docs: hwmon: add max31760 documentation Ibrahim Tilki
2022-09-08 8:23 ` [PATCH RESEND v6 3/4] dt-bindings: hwmon: Add bindings for max31760 Ibrahim Tilki
2022-09-08 15:25 ` Guenter Roeck
2022-09-08 15:46 ` Tilki, Ibrahim
2022-09-08 16:18 ` Guenter Roeck [this message]
2022-09-08 8:23 ` [PATCH RESEND v6 4/4] MAINTAINERS: Add maintainer for hwmon/max31760 Ibrahim Tilki
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=50319404-1b09-93ea-dbd5-b840167a4819@roeck-us.net \
--to=linux@roeck-us.net \
--cc=Ibrahim.Tilki@analog.com \
--cc=devicetree@vger.kernel.org \
--cc=jdelvare@suse.com \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-kernel@vger.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).