[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <BY5PR12MB49020AC690EFE6F21A81962981AA9@BY5PR12MB4902.namprd12.prod.outlook.com>
Date: Wed, 22 Feb 2023 05:28:29 +0000
From: "Datta, Shubhrajyoti" <shubhrajyoti.datta@....com>
To: "Shah, Tanmay" <tanmay.shah@....com>,
"Simek, Michal" <michal.simek@....com>,
"andersson@...nel.org" <andersson@...nel.org>,
"mathieu.poirier@...aro.org" <mathieu.poirier@...aro.org>,
"jaswinder.singh@...aro.org" <jaswinder.singh@...aro.org>,
"Levinsky, Ben" <ben.levinsky@....com>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-remoteproc@...r.kernel.org" <linux-remoteproc@...r.kernel.org>,
"Shah, Tanmay" <tanmay.shah@....com>
Subject: RE: [PATCH v3 1/3] drivers: mailbox: zynqmp: handle multiple child
nodes
[AMD Official Use Only - General]
Hi,
> -----Original Message-----
> From: Tanmay Shah <tanmay.shah@....com>
> Sent: Tuesday, February 14, 2023 2:48 AM
> To: Simek, Michal <michal.simek@....com>; andersson@...nel.org;
> mathieu.poirier@...aro.org; jaswinder.singh@...aro.org; Levinsky, Ben
> <ben.levinsky@....com>; Datta, Shubhrajyoti
> <shubhrajyoti.datta@....com>
> Cc: linux-kernel@...r.kernel.org; linux-arm-kernel@...ts.infradead.org;
> linux-remoteproc@...r.kernel.org; Shah, Tanmay
> <tanmay.shah@....com>
> Subject: [PATCH v3 1/3] drivers: mailbox: zynqmp: handle multiple child
> nodes
>
> As of now only one child node is handled by zynqmp-ipi mailbox driver.
> Upon introducing remoteproc r5 core mailbox nodes, found few
> enhancements in Xilinx zynqmp mailbox driver as following:
>
> - fix mailbox child node counts
> If child mailbox node status is disabled it causes
> crash in interrupt handler. Fix this by assigning
> only available child node during driver probe.
>
> - fix typo in IPI documentation %s/12/32/
> Xilinx IPI message buffers allows 32-byte data transfer.
> Fix documentation that says 12 bytes
>
> - fix bug in zynqmp-ipi isr handling
> Multiple IPI channels are mapped to same interrupt handler.
> Current isr implementation handles only one channel per isr.
> Fix this behavior by checking isr status bit of all child
> mailbox nodes.
It does multiple things it will be good if one patch does one stuff.
Powered by blists - more mailing lists