From: Krzysztof Kozlowski <krzk@kernel.org>
To: "Kryštof Černý" <cleverline1mc@gmail.com>,
"Stefan Wahren" <wahrenst@gmx.net>
Cc: Rob Herring <robh@kernel.org>, Conor Dooley <conor+dt@kernel.org>,
Ben Gardner <bgardner@wabtec.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>
Subject: Re: [PATCH v3 3/3] dt-bindings: w1: ds2482: Add vcc-supply property
Date: Tue, 26 Nov 2024 08:42:18 +0100 [thread overview]
Message-ID: <6b7d93f3-ec80-403a-8536-06481aa4c088@kernel.org> (raw)
In-Reply-To: <1fe3c052-d44c-4e74-9050-4b6683e69ac3@gmail.com>
On 25/11/2024 23:39, Kryštof Černý wrote:
> Hello Stefan,
>
>> Hi Kryštof,
>>
>> [fix messageid]
>>
>>> From: Kryštof Černý <cleverline1mc@gmail.com>
>>>
>>> ds2482 has a VCC pin, accepting 2.9-5.5 V.
>>>
>>> Signed-off-by: Kryštof Černý <cleverline1mc@gmail.com>
>>
>> Sorry, I'm not able to reply from my chargebyte address.
>>
>> I'm fine with the content of this patch, but in case you need to send a
>> v4 please make this the first patch of the series.
>
> I will do it, could you point me to the documentation, where this is
> specified, so I can avoid similar mistakes in the future?
First, this is common sense: all ABI documentation must come before the
user actually starts using. Otherwise the series are not bisectable -
testing commit implementing ABI will point to errors in missing ABI
documentation.
Second, doc is submitting patches in bindings directory. Please also
read writing bindings doc, in the same place.
Best regards,
Krzysztof
next prev parent reply other threads:[~2024-11-26 7:42 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-11-24 16:21 [PATCH v3 0/3] Add support for attaching a regulator to w1: ds2482 Kryštof Černý via B4 Relay
2024-11-24 16:21 ` [PATCH v3 1/3] w1: ds2482: Add regulator support Kryštof Černý via B4 Relay
2024-11-25 15:49 ` Stefan Wahren
2024-11-24 16:21 ` [PATCH v3 2/3] w1: ds2482: Fix datasheet URL Kryštof Černý via B4 Relay
2024-11-24 16:21 ` [PATCH v3 3/3] dt-bindings: w1: ds2482: Add vcc-supply property Kryštof Černý via B4 Relay
2024-11-25 15:40 ` Stefan Wahren
2024-11-25 22:39 ` Kryštof Černý
2024-11-26 7:42 ` Krzysztof Kozlowski [this message]
2024-11-25 18:35 ` [PATCH v3 3/3] dt-bindings: w1: ds2482: Add vcc-supply propertyy Conor Dooley
2024-11-25 15:39 [PATCH v3 3/3] dt-bindings: w1: ds2482: Add vcc-supply property Stefan Wahren
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=6b7d93f3-ec80-403a-8536-06481aa4c088@kernel.org \
--to=krzk@kernel.org \
--cc=bgardner@wabtec.com \
--cc=cleverline1mc@gmail.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
--cc=wahrenst@gmx.net \
/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
all inboxes | Powered by JetHome®