From: Kuldeep Singh <singh.kuldeep87k@gmail.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Cc: Mark Brown <broonie@kernel.org>,
linux-arm-kernel@lists.infradead.org, linux-spi@vger.kernel.org,
devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
Robin Murphy <robin.murphy@arm.com>,
Linus Walleij <linus.walleij@linaro.org>
Subject: Re: [PATCH 1/2] dt-bindings: spi: Update clocks property for ARM pl022
Date: Tue, 8 Mar 2022 10:51:53 +0530 [thread overview]
Message-ID: <20220308052153.GA67357@9a2d8922b8f1> (raw)
In-Reply-To: <f157651b-07c7-43ee-372a-81bd5d615d5d@canonical.com>
On Mon, Mar 07, 2022 at 10:55:52PM +0100, Krzysztof Kozlowski wrote:
> On 07/03/2022 21:37, Kuldeep Singh wrote:
> > Add missing minItems property to clocks in ARM pl022 bindings.
> >
> > This helps in resolving below warnings:
> > clocks: [[4]] is too short
> > clock-names: ['apb_pclk'] is too short
> >
> > Signed-off-by: Kuldeep Singh <singh.kuldeep87k@gmail.com>
> > ---
> > Documentation/devicetree/bindings/spi/spi-pl022.yaml | 1 +
> > 1 file changed, 1 insertion(+)
> >
>
> I saw these series in the past... please do not send the same version again.
>
> You need to version your series (git format-patch -v2 -2) and add
> changelog in cover letter (or for small patches after '---' in commits).
Ahh no. I completely missed out updating series number before sending.
Actually I have been struggling to get mutt client setup with gmail
account to send replies but it doesn't seem to work properly. Gmail
labels has been hardest part. In between these hurdles, I rolled out
patches and missed out this important thing.
Kindly consider these 2 patches as deprecated.
>
> How did you resolve the comments that the DTS might be wrong? It looks
> like you simply resent it...
Few broadcom boards(stingray and ns2) require DT updation for which I
sent patches and received Florain's blessings. Those patches are now
available in github.com/Broadcom/stblinux, devicetree-arm64
There are several other updations(in realview, versatile etc.) required
which I couldn't mention before as they are part of different tree.
prev parent reply other threads:[~2022-03-08 5:22 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-07 20:37 [PATCH 1/2] dt-bindings: spi: Update clocks property for ARM pl022 Kuldeep Singh
2022-03-07 20:37 ` [PATCH 2/2] dt-bindings: spi: Update clock-names " Kuldeep Singh
2022-03-07 21:55 ` [PATCH 1/2] dt-bindings: spi: Update clocks " Krzysztof Kozlowski
2022-03-08 5:21 ` Kuldeep Singh [this message]
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=20220308052153.GA67357@9a2d8922b8f1 \
--to=singh.kuldeep87k@gmail.com \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzysztof.kozlowski@canonical.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-spi@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=robin.murphy@arm.com \
/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).