Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Received-SPF: Pass (protection.outlook.com: domain of xilinx.com designates
 149.199.62.198 as permitted sender) receiver=protection.outlook.com;
 client-ip=149.199.62.198; helo=xsj-pvapexch01.xlnx.xilinx.com;
Envelope-to: linux-arm-kernel@lists.infradead.org,
 linux-kernel@vger.kernel.org,
 zou_wei@huawei.com
Subject: Re: [PATCH v3 0/6] Add Xilinx Event Management Driver
To:     Abhyuday Godhasara <abhyuday.godhasara@xilinx.com>,
        <michal.simek@xilinx.com>
CC:     <rajan.vaja@xilinx.com>, <manish.narani@xilinx.com>,
        <zou_wei@huawei.com>, <lakshmi.sai.krishna.potthuri@xilinx.com>,
        <wendy.liang@xilinx.com>, <jliang@xilinx.com>,
        <linux-kernel@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>
References: <20210702105144.28457-1-abhyuday.godhasara@xilinx.com>
From:   Michal Simek <michal.simek@xilinx.com>
Message-ID: <438b398c-a901-7568-11e0-cd9bf302343f@xilinx.com>
Date:   Wed, 25 Aug 2021 09:23:52 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.13.0
MIME-Version: 1.0
In-Reply-To: <20210702105144.28457-1-abhyuday.godhasara@xilinx.com>
Content-Type: text/plain; charset="utf-8"
Content-Language: en-US
Content-Transfer-Encoding: 7bit
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Aug 2021 07:23:56.4489
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: 171514df-5371-4b90-6b2f-08d967994cc5
X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c;Ip=[149.199.62.198];Helo=[xsj-pvapexch01.xlnx.xilinx.com]
X-MS-Exchange-CrossTenant-AuthSource: SN1NAM02FT0052.eop-nam02.prod.protection.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR02MB4722
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi,

On 7/2/21 12:51 PM, Abhyuday Godhasara wrote:
> This Linux driver provides support to subscribe error/event notification
> and receive notification from firmware for error/event and forward event
> notification to subscribed driver via registered callback.
> 
> All types of events like power and error will be handled from single place
> as part of event management driver.
> 
> Changes in v3:
> - Update the commit message.
> 
> Changes in v2:
> - Removed updated copyright year from unchanged files.
> - make sgi_num as module parameter for event management driver.
> - Use same object for error detection and printing.
> 
> Abhyuday Godhasara (5):
>   firmware: xilinx: add register notifier in zynqmp firmware
>   firmware: xilinx: add macros of node ids for error event
>   drivers: soc: xilinx: add xilinx event management driver
>   firmware: xilinx: instantiate xilinx event manager driver
>   driver: soc: xilinx: register for power events in zynqmp power driver
> 
> Rajan Vaja (1):
>   firmware: xilinx: export the feature check of zynqmp firmware
> 
>  MAINTAINERS                                 |   6 +
>  drivers/firmware/xilinx/zynqmp.c            |  40 +-
>  drivers/soc/xilinx/Kconfig                  |  10 +
>  drivers/soc/xilinx/Makefile                 |   1 +
>  drivers/soc/xilinx/xlnx_event_manager.c     | 600 ++++++++++++++++++++
>  drivers/soc/xilinx/zynqmp_power.c           |  54 +-
>  include/linux/firmware/xlnx-event-manager.h |  36 ++
>  include/linux/firmware/xlnx-zynqmp.h        |  33 +-
>  8 files changed, 771 insertions(+), 9 deletions(-)
>  create mode 100644 drivers/soc/xilinx/xlnx_event_manager.c
>  create mode 100644 include/linux/firmware/xlnx-event-manager.h
> 

This patch is in conflict with 2b9fc773c31b ("drivers: firmware: Add PDI
load API support") which is in Greg tree.
Can you please rebase it on the top of it and CC Greg on that version?
Would be the best if this will go via his tree because it depends on
that patch.

For the whole series
Acked-by: Michal Simek <michal.simek@xilinx.com>

Thanks,
Michal