All of lore.kernel.org
 help / color / mirror / Atom feed
From: Orson Zhai <orsonzhai@gmail.com>
To: Rob Herring <robh+dt@kernel.org>,
	Baolin Wang <baolin.wang7@gmail.com>,
	Chunyan Zhang <zhang.lyra@gmail.com>,
	Jassi Brar <jassisinghbrar@gmail.com>
Cc: Mark Brown <broonie@kernel.org>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	Haidong Yao <yaohaidong369@gmail.com>
Subject: Re: [PATCH v2 0/3] Fixes & a new supplementary feature to SPRD mailbox driver
Date: Fri, 26 Feb 2021 09:52:37 +0800	[thread overview]
Message-ID: <20210226015237.GA2610@lenovo> (raw)
In-Reply-To: <20210213122143.19240-1-orsonzhai@gmail.com>

Hi Baolin,

On Sat, Feb 13, 2021 at 08:21:40PM +0800, Orson Zhai wrote:
> Fix a real problem fot SPRD's mailbox driver in patch 1/3.
> Add supplementary inbox support for newly added sc9863a in patch 3/3 and
> change dt bindings yaml accordingly in patch 2/3.
> 
> Changes Log:
> V2:
> - Change patches order. (Yaml go to the head of dirver)
> - Remove unnecessary initializing refcnt to zero. 
> - Add fix of possible crash caused by NULL of chan->cl. (Actually move from
>   changes to sprd-mailbox.c of patch v1)
> - Remove unnecessary "inline" for do_inbox_isr().
> - Fix yaml errors from Rob's robot checking.
> - Add sc9863a compatible string for real supplementary inbox usage. (sc9860
>   is not supported by supp-inbox)
> - Add more details to supp-inbox in commit messages.

Do you have any comments about this changes to v2?

Thanks,
Orson

> 
> Orson Zhai (3):
>   mailbox: sprd: Introduce refcnt when clients requests/free channels
>   dt-bindings: mailbox: Add interrupt-names to SPRD mailbox
>   mailbox: sprd: Add supplementary inbox support
> 
>  .../bindings/mailbox/sprd-mailbox.yaml        |  18 ++-
>  drivers/mailbox/sprd-mailbox.c                | 135 +++++++++++++-----
>  2 files changed, 117 insertions(+), 36 deletions(-)
> 
> -- 
> 2.17.1
> 

      parent reply	other threads:[~2021-02-26  1:53 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-13 12:21 [PATCH v2 0/3] Fixes & a new supplementary feature to SPRD mailbox driver Orson Zhai
2021-02-13 12:21 ` [PATCH v2 1/3] mailbox: sprd: Introduce refcnt when clients requests/free channels Orson Zhai
2021-03-07 15:23   ` Baolin Wang
2021-02-13 12:21 ` [PATCH v2 2/3] dt-bindings: mailbox: Add interrupt-names to SPRD mailbox Orson Zhai
2021-03-05 21:22   ` Rob Herring
2021-03-08  8:29     ` Orson Zhai
2021-02-13 12:21 ` [PATCH v2 3/3] mailbox: sprd: Add supplementary inbox support Orson Zhai
2021-03-07 15:23   ` Baolin Wang
2021-02-26  1:52 ` Orson Zhai [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=20210226015237.GA2610@lenovo \
    --to=orsonzhai@gmail.com \
    --cc=baolin.wang7@gmail.com \
    --cc=broonie@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jassisinghbrar@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=yaohaidong369@gmail.com \
    --cc=zhang.lyra@gmail.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 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.