mbox series

[v3,0/2] mailbox: pcc: Support platform notification for type4 and shared interrupt

Message ID 20230423110335.2679-1-lihuisong@huawei.com
Headers show
Series mailbox: pcc: Support platform notification for type4 and shared interrupt | expand

Message

lihuisong (C) April 23, 2023, 11:03 a.m. UTC
PCC supports processing platform notification for slave subspaces and
shared interrupt for multiple subspaces.

---
 -v3: update requesting shared irq code due to pcc using mbox_bind_client.
 -v2: extract cmd complete code into a function.
      unrelated types regard chan_in_use as dummy.
 -v1: using subspace type to replace comm_flow_dir in patch [1/2]
 -rfc-v3: split V2 into two patches.
 -rfc-v2: don't use platform interrupt ack register to identify if the given
      channel should respond interrupt.

Huisong Li (2):
  mailbox: pcc: Add support for platform notification handling
  mailbox: pcc: Support shared interrupt for multiple subspaces

 drivers/mailbox/pcc.c | 91 +++++++++++++++++++++++++++++++++++++------
 1 file changed, 80 insertions(+), 11 deletions(-)

Comments

Sudeep Holla April 25, 2023, 2:41 p.m. UTC | #1
Hi Rafael,

On Sun, Apr 23, 2023 at 07:03:33PM +0800, Huisong Li wrote:
> PCC supports processing platform notification for slave subspaces and
> shared interrupt for multiple subspaces.

Can you please take this via your tree ? It has been reviewed and
tested now. If it is too late for this merge window, are you happy to
pick it up if posted at -rc1 for next merge window ?
lihuisong (C) May 4, 2023, 1:30 a.m. UTC | #2
Hi Sudeep,

Can you add Reviewed-by again?
I forgot to do it when send v3.
There is no the 'Reviewed-by' tag on patchwork now.
I'm not sure if this affects patch merging.

/Huisong

在 2023/4/23 19:03, Huisong Li 写道:
> PCC supports processing platform notification for slave subspaces and
> shared interrupt for multiple subspaces.
>
> ---
>   -v3: update requesting shared irq code due to pcc using mbox_bind_client.
>   -v2: extract cmd complete code into a function.
>        unrelated types regard chan_in_use as dummy.
>   -v1: using subspace type to replace comm_flow_dir in patch [1/2]
>   -rfc-v3: split V2 into two patches.
>   -rfc-v2: don't use platform interrupt ack register to identify if the given
>        channel should respond interrupt.
>
> Huisong Li (2):
>    mailbox: pcc: Add support for platform notification handling
>    mailbox: pcc: Support shared interrupt for multiple subspaces
>
>   drivers/mailbox/pcc.c | 91 +++++++++++++++++++++++++++++++++++++------
>   1 file changed, 80 insertions(+), 11 deletions(-)
>
Sudeep Holla May 9, 2023, 9:06 a.m. UTC | #3
On Sun, Apr 23, 2023 at 07:03:33PM +0800, Huisong Li wrote:
> PCC supports processing platform notification for slave subspaces and
> shared interrupt for multiple subspaces.
>
> ---
>  -v3: update requesting shared irq code due to pcc using mbox_bind_client.
>  -v2: extract cmd complete code into a function.
>       unrelated types regard chan_in_use as dummy.
>  -v1: using subspace type to replace comm_flow_dir in patch [1/2]
>  -rfc-v3: split V2 into two patches.
>  -rfc-v2: don't use platform interrupt ack register to identify if the given
>       channel should respond interrupt.
> 
> Huisong Li (2):
>   mailbox: pcc: Add support for platform notification handling
>   mailbox: pcc: Support shared interrupt for multiple subspaces
> 

Reviewed-by: Sudeep Holla <sudeep.holla@arm.com>
lihuisong (C) May 25, 2023, 12:25 p.m. UTC | #4
Hi Rafael,

This series have been discussed, reviewed and tested.
Can you take a look at this series?

/Huisong

在 2023/4/25 22:41, Sudeep Holla 写道:
> Hi Rafael,
>
> On Sun, Apr 23, 2023 at 07:03:33PM +0800, Huisong Li wrote:
>> PCC supports processing platform notification for slave subspaces and
>> shared interrupt for multiple subspaces.
> Can you please take this via your tree ? It has been reviewed and
> tested now. If it is too late for this merge window, are you happy to
> pick it up if posted at -rc1 for next merge window ?
>