From: Rob Herring <robh+dt@kernel.org>
To: Viresh Kumar <viresh.kumar@linaro.org>
Cc: Jassi Brar <jaswinder.singh@linaro.org>,
morten_bp@live.dk, Tushar Khandelwal <Tushar.Khandelwal@arm.com>,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
Vincent Guittot <vincent.guittot@linaro.org>,
devicetree@vger.kernel.org, Sudeep Holla <Sudeep.Holla@arm.com>,
Frank Rowand <frowand.list@gmail.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH V4] dt-bindings: mailbox : arm,mhuv2: Add bindings
Date: Tue, 13 Oct 2020 06:55:19 -0500 [thread overview]
Message-ID: <CAL_Jsq+Cmr=92+7aSoNpCXdCTHysvVqU2mn7=GjrUp-7Wn1QNA@mail.gmail.com> (raw)
In-Reply-To: <61ca14fc441f92c1e7994e5bebae5c49811a3050.1602563406.git.viresh.kumar@linaro.org>
On Mon, Oct 12, 2020 at 11:32 PM Viresh Kumar <viresh.kumar@linaro.org> wrote:
>
> This patch adds device tree binding for ARM Message Handling Unit (MHU)
> controller version 2.
>
> Based on earlier work by Morten Borup Petersen.
>
> Co-developed-by: Tushar Khandelwal <tushar.khandelwal@arm.com>
> Signed-off-by: Tushar Khandelwal <tushar.khandelwal@arm.com>
> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
>
> ---
> V3->V4:
> - Made interrupts property compulsory for receiver
> - Added interrupts for both nodes in example
> - Fixed min/max items for protocols property
> - Don't add -tx/rx to node's name.
>
> V2->V3:
> - compatible is changed to arm-mhuv2-tx/rx. Later version of MHUv2
> have an interrupt in the sender mode as well and we can't find if a
> mailbox is sender or receiver just based on an interrupt anymore. We
> needed a better way.
>
> - arm-mhuv2-mode is renamed to arm,mhuv2-protocols to fit the purpose
> better.
>
> - Minor formatting otherwise.
>
> Morten: Please let me know if you want me to add your signoff here from
> some other email id as you no longer work with ARM.
> ---
> .../bindings/mailbox/arm,mhuv2.yaml | 215 ++++++++++++++++++
> 1 file changed, 215 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mailbox/arm,mhuv2.yaml
Reviewed-by: Rob Herring <robh@kernel.org>
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh+dt@kernel.org>
To: Viresh Kumar <viresh.kumar@linaro.org>
Cc: devicetree@vger.kernel.org,
Vincent Guittot <vincent.guittot@linaro.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Jassi Brar <jaswinder.singh@linaro.org>,
morten_bp@live.dk, Tushar Khandelwal <Tushar.Khandelwal@arm.com>,
Sudeep Holla <Sudeep.Holla@arm.com>,
Frank Rowand <frowand.list@gmail.com>,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH V4] dt-bindings: mailbox : arm,mhuv2: Add bindings
Date: Tue, 13 Oct 2020 06:55:19 -0500 [thread overview]
Message-ID: <CAL_Jsq+Cmr=92+7aSoNpCXdCTHysvVqU2mn7=GjrUp-7Wn1QNA@mail.gmail.com> (raw)
In-Reply-To: <61ca14fc441f92c1e7994e5bebae5c49811a3050.1602563406.git.viresh.kumar@linaro.org>
On Mon, Oct 12, 2020 at 11:32 PM Viresh Kumar <viresh.kumar@linaro.org> wrote:
>
> This patch adds device tree binding for ARM Message Handling Unit (MHU)
> controller version 2.
>
> Based on earlier work by Morten Borup Petersen.
>
> Co-developed-by: Tushar Khandelwal <tushar.khandelwal@arm.com>
> Signed-off-by: Tushar Khandelwal <tushar.khandelwal@arm.com>
> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
>
> ---
> V3->V4:
> - Made interrupts property compulsory for receiver
> - Added interrupts for both nodes in example
> - Fixed min/max items for protocols property
> - Don't add -tx/rx to node's name.
>
> V2->V3:
> - compatible is changed to arm-mhuv2-tx/rx. Later version of MHUv2
> have an interrupt in the sender mode as well and we can't find if a
> mailbox is sender or receiver just based on an interrupt anymore. We
> needed a better way.
>
> - arm-mhuv2-mode is renamed to arm,mhuv2-protocols to fit the purpose
> better.
>
> - Minor formatting otherwise.
>
> Morten: Please let me know if you want me to add your signoff here from
> some other email id as you no longer work with ARM.
> ---
> .../bindings/mailbox/arm,mhuv2.yaml | 215 ++++++++++++++++++
> 1 file changed, 215 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mailbox/arm,mhuv2.yaml
Reviewed-by: Rob Herring <robh@kernel.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-10-13 11:56 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-09 11:42 [PATCH V3] dt-bindings: mailbox: arm,mhuv2: Add bindings Viresh Kumar
2020-10-09 11:42 ` Viresh Kumar
2020-10-12 19:16 ` Rob Herring
2020-10-12 19:16 ` Rob Herring
2020-10-13 4:32 ` [PATCH V4] dt-bindings: mailbox : " Viresh Kumar
2020-10-13 4:32 ` Viresh Kumar
2020-10-13 11:55 ` Rob Herring [this message]
2020-10-13 11:55 ` Rob Herring
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='CAL_Jsq+Cmr=92+7aSoNpCXdCTHysvVqU2mn7=GjrUp-7Wn1QNA@mail.gmail.com' \
--to=robh+dt@kernel.org \
--cc=Sudeep.Holla@arm.com \
--cc=Tushar.Khandelwal@arm.com \
--cc=devicetree@vger.kernel.org \
--cc=frowand.list@gmail.com \
--cc=jaswinder.singh@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=morten_bp@live.dk \
--cc=vincent.guittot@linaro.org \
--cc=viresh.kumar@linaro.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.