2021-02-13 12:26:09

by Orson Zhai

[permalink] [raw]
Subject: [PATCH v2 0/3] Fixes & a new supplementary feature to SPRD mailbox driver

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.

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


2021-02-26 01:59:09

by Orson Zhai

[permalink] [raw]
Subject: Re: [PATCH v2 0/3] Fixes & a new supplementary feature to SPRD mailbox driver

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
>