From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 24998C54EE9 for ; Wed, 28 Sep 2022 17:32:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=bcfiVLh3t0b+RPrd7gNKA8yOzA2fMPdc5ZxcKs9+DAI=; b=IPTu7DLvz8CT81 If+/qEZYK0RwNwTtXLDiZ9rjbgzmXS8bzrZGteMX8WH8pc0gIsz39yx+hU5QAQ/clJ8h+/yNW1Ukm omjbD6PHgOBoWdkyfSFZONa9d6aSViBEDbHzS1T12yzHnv6TeaUT/HEuHLBrbDZpPFLHvvvZ+rjEj AzR2J5XYVQyy03Ecnsd7V4DkHG1/+s8mhdNT1xSwK0yyYgEH/bK+stIQ0YbgzV1tKWm7pc6PHCkz9 ny2eoee+G1aEl0BY8nUuVqY/UrzFNS0/DKzHmI7E7RufZ9JQ8Q/RkaVUEequycM1iwk+H7Kc6Wio8 2ECFYZE5eAO/ecpEqFUA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1odatn-00HOEW-Vf; Wed, 28 Sep 2022 17:31:08 +0000 Received: from mail-pf1-x431.google.com ([2607:f8b0:4864:20::431]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1odati-00HOCQ-TJ for linux-arm-kernel@lists.infradead.org; Wed, 28 Sep 2022 17:31:05 +0000 Received: by mail-pf1-x431.google.com with SMTP id a80so13133056pfa.4 for ; Wed, 28 Sep 2022 10:30:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date; bh=7SWYiaNKhmtQ3unxY0hqTaTEitWRnJQ84ZlYxegZMW0=; b=zFXwTuD0zvolx0yhxmJHqDe0tV3PoaGUydiZZUunr1f1U6xKteTtDLNmm0npdDHHhW M+5tpot5nePGujc6tvEVkXqMXFEGkb0rxmmcrr1JIjkpIEbkMFPRUbJVrzag1Dw+gHxW l0UgRv4cKSVNiv/apd+GO5lVTCHanWWrghsth24xDupihXXmNpL/hAwK2d9mS2qS8UaZ E5saFJoYXt8qLsHGvP/7SAVaHyw4OxnOYQoxgVnxkRbOMQsEyzyCx9dz72b6nYVM/f8n PNde/3lCPxw6o24+COME0BUTzQTLoyW5VczhRIgwfEP+DAvnm5bGLUag0OzR05DQSwQo FfoA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date; bh=7SWYiaNKhmtQ3unxY0hqTaTEitWRnJQ84ZlYxegZMW0=; b=sliTSy8KgjJOdd8d9fEQQ95UEkAXRhBPiOpqYhKYJdykexFeERURlwLH2Bq6JgfXYj Vpjs3SbPUrIvJad4t6NkPx9yq4NQy/pXA1JQeMdAX3e9+zaTZjDfyhtBjJ4IkBhjGUSZ tFLMD+NXDyQatkkMO2E1SXdm/dkjje0404ixXnp9PBrYAUsq+Vx1i4BN09j3lYi/sNJr T8VC42D1k053wOIJU8ytkdNBQFkK6k5QY0g6zNJSa7P0tBA2Dhik+uDw+mvSzSnCC0QJ sBLWgH7roKIB5N/0vfaC49jQC0yB+TeZjFoLGBsyqO3IcM2QZs5mNrIIcb8rxu5WToYy goAA== X-Gm-Message-State: ACrzQf0wiqJ3gAN9eYihDn4LFzGyFQ1+0eBPo4rd88xZsARYr2hCTcl4 phKnuGbJI+Ijl9hIzFtqt8YOpg== X-Google-Smtp-Source: AMsMyM7bRCSYw6DeUGvv/veAx1EPKfBAlxoP2JDdiafIdTOMoI049J4O0xHmgVdk/kUflWf6ekpN5w== X-Received: by 2002:a63:3e4a:0:b0:439:246d:e681 with SMTP id l71-20020a633e4a000000b00439246de681mr24051046pga.424.1664386259140; Wed, 28 Sep 2022 10:30:59 -0700 (PDT) Received: from p14s (S0106889e681aac74.cg.shawcable.net. [68.147.0.187]) by smtp.gmail.com with ESMTPSA id v11-20020a17090a088b00b001f22647cb56sm1813582pjc.27.2022.09.28.10.30.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 28 Sep 2022 10:30:57 -0700 (PDT) Date: Wed, 28 Sep 2022 11:30:54 -0600 From: Mathieu Poirier To: Peng Fan Cc: Peng Fan , "robh+dt@kernel.org" , "krzysztof.kozlowski+dt@linaro.org" , "shawnguo@kernel.org" , "s.hauer@pengutronix.de" , "kernel@pengutronix.de" , "festevam@gmail.com" , dl-linux-imx , "linux-remoteproc@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , Bjorn Andersson Subject: Re: [PATCH V5 3/6] remoteproc: imx_rproc: support attaching to i.MX8QXP M4 Message-ID: <20220928173054.GC2990524@p14s> References: <20220824011023.1493050-1-peng.fan@oss.nxp.com> <20220824011023.1493050-4-peng.fan@oss.nxp.com> <20220926232127.GB2817947@p14s> <20220927175649.GB2883698@p14s> <64c6bdc2-583d-a2d0-f8b8-c4487f8a4d97@oss.nxp.com> <65b7224d-d3eb-4513-d733-ec781864fb7b@oss.nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <65b7224d-d3eb-4513-d733-ec781864fb7b@oss.nxp.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220928_103102_980616_94232E2D X-CRM114-Status: GOOD ( 59.57 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Sep 28, 2022 at 06:01:45PM +0800, Peng Fan wrote: > Hi Mathieu, > = > On 9/28/2022 3:39 PM, Peng Fan wrote: > > = > > = > > On 9/28/2022 1:56 AM, Mathieu Poirier wrote: > > > On Tue, Sep 27, 2022 at 02:48:02AM +0000, Peng Fan wrote: > > > > Hi Mathieu, > > > > = > > > > Thanks for reviewing this patchset. > > > > > Subject: Re: [PATCH V5 3/6] remoteproc: imx_rproc: support attach= ing to > > > > > i.MX8QXP M4 > > > > > = > > > > > On Wed, Aug 24, 2022 at 09:10:20AM +0800, Peng Fan (OSS) wrote: > > > > > > From: Peng Fan > > > > > > = > > > > > > When M4 is kicked by SCFW, M4 runs in its own hardware partitio= n, > > > > > > Linux could only do IPC with M4, it could not start, > > > > > > stop, update image. > > > > > > = > > > > > > We disable recovery reboot when M4 is managed by SCFW, because > > > > > > remoteproc core still not support M4 auto-recovery without load= ing > > > > > > image. > > > > > > = > > > > > > Signed-off-by: Peng Fan > > > > > > --- > > > > > > =A0 drivers/remoteproc/imx_rproc.c | 108 > > > > > > ++++++++++++++++++++++++++++++++- > > > > > > =A0 1 file changed, 107 insertions(+), 1 deletion(-) > > > > > > = > > > > > > diff --git a/drivers/remoteproc/imx_rproc.c > > > > > > b/drivers/remoteproc/imx_rproc.c index 7cc4fd207e2d..bcba74e900= 20 > > > > > > 100644 > > > > > > --- a/drivers/remoteproc/imx_rproc.c > > > > > > +++ b/drivers/remoteproc/imx_rproc.c > > > > > > @@ -6,6 +6,7 @@ > > > > > > =A0 #include > > > > > > =A0 #include > > > > > > =A0 #include > > > > > > +#include > > > > > > =A0 #include > > > > > > =A0 #include > > > > > > =A0 #include > > > > > > @@ -59,6 +60,8 @@ > > > > > > =A0 #define IMX_SIP_RPROC_STARTED=A0=A0=A0=A0=A0=A0=A0 0x01 > > > > > > =A0 #define IMX_SIP_RPROC_STOP=A0=A0=A0=A0=A0=A0=A0 0x02 > > > > > > = > > > > > > +#define IMX_SC_IRQ_GROUP_REBOOTED=A0=A0=A0 5 > > > > > > + > > > > > > =A0 /** > > > > > > =A0=A0 * struct imx_rproc_mem - slim internal memory structure > > > > > > =A0=A0 * @cpu_addr: MPU virtual address of the memory region @@= -89,6 > > > > > > +92,10 @@ struct imx_rproc { > > > > > > =A0=A0=A0=A0=A0 struct work_struct=A0=A0=A0=A0=A0=A0=A0 rproc_w= ork; > > > > > > =A0=A0=A0=A0=A0 struct workqueue_struct=A0=A0=A0=A0=A0=A0=A0 *w= orkqueue; > > > > > > =A0=A0=A0=A0=A0 void __iomem=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 *= rsc_table; > > > > > > +=A0=A0=A0 struct imx_sc_ipc=A0=A0=A0=A0=A0=A0=A0 *ipc_handle; > > > > > > +=A0=A0=A0 struct notifier_block=A0=A0=A0=A0=A0=A0=A0 rproc_nb; > > > > > > +=A0=A0=A0 u32=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 rpr= oc_pt;=A0=A0=A0 /* partition id */ > > > > > > +=A0=A0=A0 u32=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 rsr= c_id;=A0=A0=A0 /* resource id */ > > > > > > =A0 }; > > > > > > = > > > > > > =A0 static const struct imx_rproc_att > > > > > > imx_rproc_att_imx93[] =3D { @@ -117,6 > > > > > > +124,18 @@ static const struct imx_rproc_att imx_rproc_att_imx9= 3[] =3D { > > > > > > =A0=A0=A0=A0=A0 { 0xD0000000, 0xa0000000, 0x10000000, 0 },=A0 }; > > > > > > = > > > > > > +static const struct imx_rproc_att imx_rproc_att_imx8qxp[] =3D { > > > > > > +=A0=A0=A0 { 0x08000000, 0x08000000, 0x10000000, 0 }, > > > > > > +=A0=A0=A0 /* TCML/U */ > > > > > > +=A0=A0=A0 { 0x1FFE0000, 0x34FE0000, 0x00040000, ATT_OWN | ATT_= IOMEM }, > > > > > > +=A0=A0=A0 /* OCRAM(Low 96KB) */ > > > > > > +=A0=A0=A0 { 0x21000000, 0x00100000, 0x00018000, 0 }, > > > > > > +=A0=A0=A0 /* OCRAM */ > > > > > > +=A0=A0=A0 { 0x21100000, 0x00100000, 0x00040000, 0 }, > > > > > > +=A0=A0=A0 /* DDR (Data) */ > > > > > > +=A0=A0=A0 { 0x80000000, 0x80000000, 0x60000000, 0 }, }; > > > > > > + > > > > > > =A0 static const struct imx_rproc_att imx_rproc_att_imx8mn[] = =3D { > > > > > > =A0=A0=A0=A0=A0 /* dev addr , sys addr=A0 , size=A0=A0=A0=A0=A0= =A0=A0 , flags */ > > > > > > =A0=A0=A0=A0=A0 /* ITCM=A0=A0 */ > > > > > > @@ -255,6 +274,12 @@ static const struct imx_rproc_dcfg > > > > > imx_rproc_cfg_imx8mq =3D { > > > > > > =A0=A0=A0=A0=A0 .method=A0=A0=A0=A0=A0=A0=A0 =3D IMX_RPROC_MMIO, > > > > > > =A0 }; > > > > > > = > > > > > > +static const struct imx_rproc_dcfg imx_rproc_cfg_imx8qxp =3D { > > > > > > +=A0=A0=A0 .att=A0=A0=A0=A0=A0=A0=A0 =3D imx_rproc_att_imx8qxp, > > > > > > +=A0=A0=A0 .att_size=A0=A0=A0 =3D ARRAY_SIZE(imx_rproc_att_imx8= qxp), > > > > > > +=A0=A0=A0 .method=A0=A0=A0=A0=A0=A0=A0 =3D IMX_RPROC_SCU_API, > > > > > > +}; > > > > > > + > > > > > > =A0 static const struct imx_rproc_dcfg imx_rproc_cfg_imx8ulp = =3D { > > > > > > =A0=A0=A0=A0=A0 .att=A0=A0=A0=A0=A0=A0=A0 =3D imx_rproc_att_imx= 8ulp, > > > > > > =A0=A0=A0=A0=A0 .att_size=A0=A0=A0 =3D ARRAY_SIZE(imx_rproc_att= _imx8ulp), > > > > > > @@ -680,6 +705,37 @@ static void imx_rproc_free_mbox(struct rpr= oc > > > > > *rproc) > > > > > > =A0=A0=A0=A0=A0 mbox_free_channel(priv->rx_ch); > > > > > > =A0 } > > > > > > = > > > > > > +static void imx_rproc_put_scu(struct rproc *rproc) { > > > > > > +=A0=A0=A0 struct imx_rproc *priv =3D rproc->priv; > > > > > > +=A0=A0=A0 const struct imx_rproc_dcfg *dcfg =3D priv->dcfg; > > > > > > + > > > > > > +=A0=A0=A0 if (dcfg->method !=3D IMX_RPROC_SCU_API) > > > > > > +=A0=A0=A0=A0=A0=A0=A0 return; > > > > > > + > > > > > > +=A0=A0=A0 if (!imx_sc_rm_is_resource_owned(priv->ipc_handle, > > > > > > priv->rsrc_id)) > > > > > > +=A0=A0=A0=A0=A0=A0=A0 return; > > > > > > + > > > > > > +=A0=A0=A0 imx_scu_irq_group_enable(IMX_SC_IRQ_GROUP_REBOOTED, > > > > > BIT(priv->rproc_pt), false); > > > > > > +=A0=A0=A0 imx_scu_irq_unregister_notifier(&priv->rproc_nb); > > > > > > +} > > > > > > + > > > > > > +static int imx_rproc_partition_notify(struct notifier_block *n= b, > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0= =A0 unsigned long event, void *group) { > > > > > > +=A0=A0=A0 struct imx_rproc *priv =3D container_of(nb, struct i= mx_rproc, > > > > > > +rproc_nb); > > > > > > + > > > > > > +=A0=A0=A0 /* Ignore other irqs */ > > > > > > +=A0=A0=A0 if (!((event & BIT(priv->rproc_pt)) && (*(u8 *)group= =3D=3D > > > > > IMX_SC_IRQ_GROUP_REBOOTED))) > > > > > > +=A0=A0=A0=A0=A0=A0=A0 return 0; > > > > > > + > > > > > > +=A0=A0=A0 rproc_report_crash(priv->rproc, RPROC_WATCHDOG); > > > > > > + > > > > > > +=A0=A0=A0 pr_info("Partition%d reset!\n", priv->rproc_pt); > > > > > > + > > > > > > +=A0=A0=A0 return 0; > > > > > > +} > > > > > > + > > > > > > =A0 static int imx_rproc_detect_mode(struct imx_rproc *priv)=A0= { > > > > > > =A0=A0=A0=A0=A0 struct regmap_config config =3D { .name =3D "im= x-rproc" }; @@ -689,6 > > > > > > +745,7 @@ static int imx_rproc_detect_mode(struct imx_rproc *pr= iv) > > > > > > =A0=A0=A0=A0=A0 struct arm_smccc_res res; > > > > > > =A0=A0=A0=A0=A0 int ret; > > > > > > =A0=A0=A0=A0=A0 u32 val; > > > > > > +=A0=A0=A0 u8 pt; > > > > > > = > > > > > > =A0=A0=A0=A0=A0 switch (dcfg->method) { > > > > > > =A0=A0=A0=A0=A0 case IMX_RPROC_NONE: > > > > > > @@ -699,6 +756,51 @@ static int imx_rproc_detect_mode(struct > > > > > imx_rproc *priv) > > > > > > =A0=A0=A0=A0=A0=A0=A0=A0=A0 if (res.a0) > > > > > > =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 priv->rproc->state =3D = RPROC_DETACHED; > > > > > > =A0=A0=A0=A0=A0=A0=A0=A0=A0 return 0; > > > > > > +=A0=A0=A0 case IMX_RPROC_SCU_API: > > > > > > +=A0=A0=A0=A0=A0=A0=A0 ret =3D imx_scu_get_handle(&priv->ipc_ha= ndle); > > > > > > +=A0=A0=A0=A0=A0=A0=A0 if (ret) > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 return ret; > > > > > > +=A0=A0=A0=A0=A0=A0=A0 ret =3D of_property_read_u32(dev->of_nod= e, "fsl,resource-id", > > > > > &priv->rsrc_id); > > > > > > +=A0=A0=A0=A0=A0=A0=A0 if (ret) { > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 dev_err(dev, "No fsl,resourc= e-id property\n"); > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 return ret; > > > > > > +=A0=A0=A0=A0=A0=A0=A0 } > > > > > > + > > > > > > +=A0=A0=A0=A0=A0=A0=A0 /* > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0 * If Mcore resource is not owned by A= core partition, It is > > > > > kicked by ROM, > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0 * and Linux could only do IPC with Mc= ore and nothing else. > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0 */ > > > > > > +=A0=A0=A0=A0=A0=A0=A0 if (imx_sc_rm_is_resource_owned(priv->ip= c_handle, priv- > > > > > > rsrc_id)) > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 return 0; > > > > > = > > > > > If imx_sc_rm_is_resource_owned() return '1' than the remote > > > > > processor is > > > > > under Linux's control and what follows below is not needed.=A0 > > > > > That is also > > > > > coherent with the comment in [1]. > > > > = > > > > Case 1: If M4 is owned by Linux, here directly return 0. > > > > Case 2: If M4 is not owned by Linux, the following code after > > > > this line will set > > > > state as RPROC_DETACHED. > > > = > > > I understand that part. > > > = > > > > = > > > > Patch 3/6(this patch) is only to support case 2. > > > > Patch 4/6 is to support case 1. > > > > = > > > = > > > Let's leave the subsequent patches alone for now. > > > = > > > > > = > > > > > That is in contrast with what is happening in > > > > > imx_rproc_put_scu(). There, if > > > > > the remote processor is _not_ owned by Linux than the condition r= eturns > > > > > without calling imx_scu_irq_group_enable() and > > > > > imx_scu_irq_unregister_notifier().=A0 That seems to be a bug. > > > > = > > > > No. The two functions only needed when M4 is in a separate > > > > hardware partition. > > > > = > > > > The scu irq is only needed when M4 is out of linux control and need= some > > > > notification such as M4 is reset by SCU(System Control Unit). > > > > That linux got > > > > notification that M4 is reset by SCU. > > > = > > > I also understand that part. > > > = > > > What I am underlining here is that when the M4 is independent, functi= on > > > imx_scu_irq_register_notifier() and imx_scu_irq_group_enable() are > > > called but > > > their cleanup equivalent are not called in imx_rproc_put_scu() > > > because of the '!' > > > in the if() statement. > > = > > you are right, this is bug in my side. It should be as below based on > > patch 3/6. > > = > > diff --git a/drivers/remoteproc/imx_rproc.c > > b/drivers/remoteproc/imx_rproc.c > > index bcba74e90020..a56aecae00c6 100644 > > --- a/drivers/remoteproc/imx_rproc.c > > +++ b/drivers/remoteproc/imx_rproc.c > > @@ -713,7 +713,7 @@ static void imx_rproc_put_scu(struct rproc *rproc) > > =A0=A0=A0=A0=A0=A0=A0 if (dcfg->method !=3D IMX_RPROC_SCU_API) > > =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 return; > > = > > -=A0=A0=A0=A0=A0=A0 if (!imx_sc_rm_is_resource_owned(priv->ipc_handle, = priv->rsrc_id)) > > +=A0=A0=A0=A0=A0=A0 if (imx_sc_rm_is_resource_owned(priv->ipc_handle, p= riv->rsrc_id)) Indeed, which raises questions about how this patchset was tested. And it = is not the first time we touch base on that. > > =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 return; > > = > > Thanks for detailed reviewing. > = > If you are fine with this change, I could send out V6. Anyway, I'll wait > to see if you have other comments in this patchset. > I am out of time for this patchset and as such will not provide more commen= ts on this revision. > Thanks, > Peng. > = > > = > > Thanks, > > Peng. > > = > > > = > > > > = > > > > > = > > > > > There is also a problem in patch 4/6 associated to that. > > > > = > > > > If the upper explanation eliminate your concern, "a problem in > > > > patch 4/6" should > > > > not be a problem. > > > > = > > > > When M4 is owned by Linux, Linux need handle the power domain. > > > > If M4 is not owned > > > > by Linux, SCU firmware will handle the power domain, and Linux > > > > has no permission to > > > > touch that. > > > > = > > > > Thanks > > > > Peng > > > > = > > > > > = > > > > > Thanks, > > > > > Mathieu > > > > > = > > > > > = > > > > > [1]. > > > > > https://eur01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F= %2Felixir > > > > > .bootlin.com%2Flinux%2Fv6.0- > > > > > rc7%2Fsource%2Fdrivers%2Ffirmware%2Fimx%2Frm.c%23L24&data=3D0 > > > > > 5%7C01%7Cpeng.fan%40nxp.com%7Cbe679e9a409a48b834b908daa015d92 > > > > > c%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637998312946913 > > > > > 710%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2lu > > > > > MzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=3D > > > > > JDRvoDGGgEiSmbhj3410V2DNxamZbDmMS0U2GvBnI74%3D&reserved > > > > > =3D0 > > > > > = > > > > > > + > > > > > > +=A0=A0=A0=A0=A0=A0=A0 priv->rproc->state =3D RPROC_DETACHED; > > > > > > +=A0=A0=A0=A0=A0=A0=A0 priv->rproc->recovery_disabled =3D true; > > > > > > + > > > > > > +=A0=A0=A0=A0=A0=A0=A0 /* Get partition id and enable irq in SC= FW */ > > > > > > +=A0=A0=A0=A0=A0=A0=A0 ret =3D imx_sc_rm_get_resource_owner(pri= v->ipc_handle, > > > > > priv->rsrc_id, &pt); > > > > > > +=A0=A0=A0=A0=A0=A0=A0 if (ret) { > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 dev_err(dev, "not able to ge= t resource owner\n"); > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 return ret; > > > > > > +=A0=A0=A0=A0=A0=A0=A0 } > > > > > > + > > > > > > +=A0=A0=A0=A0=A0=A0=A0 priv->rproc_pt =3D pt; > > > > > > +=A0=A0=A0=A0=A0=A0=A0 priv->rproc_nb.notifier_call =3D imx_rpr= oc_partition_notify; > > > > > > + > > > > > > +=A0=A0=A0=A0=A0=A0=A0 ret =3D imx_scu_irq_register_notifier(&p= riv->rproc_nb); > > > > > > +=A0=A0=A0=A0=A0=A0=A0 if (ret) { > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 dev_warn(dev, "register scu = notifier failed.\n"); > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 return ret; > > > > > > +=A0=A0=A0=A0=A0=A0=A0 } > > > > > > + > > > > > > +=A0=A0=A0=A0=A0=A0=A0 ret =3D > > > > > imx_scu_irq_group_enable(IMX_SC_IRQ_GROUP_REBOOTED, BIT(priv- > > > > > > rproc_pt), > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0= =A0=A0=A0=A0=A0=A0 true); > > > > > > +=A0=A0=A0=A0=A0=A0=A0 if (ret) { > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 imx_scu_irq_unregister_notif= ier(&priv->rproc_nb); > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 dev_warn(dev, "Enable irq fa= iled.\n"); > > > > > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 return ret; > > > > > > +=A0=A0=A0=A0=A0=A0=A0 } > > > > > > + > > > > > > +=A0=A0=A0=A0=A0=A0=A0 return 0; > > > > > > =A0=A0=A0=A0=A0 default: > > > > > > =A0=A0=A0=A0=A0=A0=A0=A0=A0 break; > > > > > > =A0=A0=A0=A0=A0 } > > > > > > @@ -803,7 +905,7 @@ static int imx_rproc_probe(struct platform_= device > > > > > > *pdev) > > > > > > = > > > > > > =A0=A0=A0=A0=A0 ret =3D imx_rproc_clk_enable(priv); > > > > > > =A0=A0=A0=A0=A0 if (ret) > > > > > > -=A0=A0=A0=A0=A0=A0=A0 goto err_put_mbox; > > > > > > +=A0=A0=A0=A0=A0=A0=A0 goto err_put_scu; > > > > > > = > > > > > > =A0=A0=A0=A0=A0 INIT_WORK(&priv->rproc_work, imx_rproc_vq_work); > > > > > > = > > > > > > @@ -820,6 +922,8 @@ static int imx_rproc_probe(struct platform_= device > > > > > > *pdev) > > > > > > = > > > > > > =A0 err_put_clk: > > > > > > =A0=A0=A0=A0=A0 clk_disable_unprepare(priv->clk); > > > > > > +err_put_scu: > > > > > > +=A0=A0=A0 imx_rproc_put_scu(rproc); > > > > > > =A0 err_put_mbox: > > > > > > =A0=A0=A0=A0=A0 imx_rproc_free_mbox(rproc); > > > > > > =A0 err_put_wkq: > > > > > > @@ -837,6 +941,7 @@ static int imx_rproc_remove(struct > > > > > platform_device > > > > > > *pdev) > > > > > > = > > > > > > =A0=A0=A0=A0=A0 clk_disable_unprepare(priv->clk); > > > > > > =A0=A0=A0=A0=A0 rproc_del(rproc); > > > > > > +=A0=A0=A0 imx_rproc_put_scu(rproc); > > > > > > =A0=A0=A0=A0=A0 imx_rproc_free_mbox(rproc); > > > > > > =A0=A0=A0=A0=A0 destroy_workqueue(priv->workqueue); > > > > > > =A0=A0=A0=A0=A0 rproc_free(rproc); > > > > > > @@ -852,6 +957,7 @@ static const struct of_device_id > > > > > imx_rproc_of_match[] =3D { > > > > > > =A0=A0=A0=A0=A0 { .compatible =3D "fsl,imx8mm-cm4", .data =3D > > > > > &imx_rproc_cfg_imx8mq }, > > > > > > =A0=A0=A0=A0=A0 { .compatible =3D "fsl,imx8mn-cm7", .data =3D > > > > > &imx_rproc_cfg_imx8mn }, > > > > > > =A0=A0=A0=A0=A0 { .compatible =3D "fsl,imx8mp-cm7", .data =3D > > > > > &imx_rproc_cfg_imx8mn }, > > > > > > +=A0=A0=A0 { .compatible =3D "fsl,imx8qxp-cm4", .data =3D > > > > > &imx_rproc_cfg_imx8qxp }, > > > > > > =A0=A0=A0=A0=A0 { .compatible =3D "fsl,imx8ulp-cm33", .data =3D > > > > > &imx_rproc_cfg_imx8ulp }, > > > > > > =A0=A0=A0=A0=A0 { .compatible =3D "fsl,imx93-cm33", .data =3D > > > > > > &imx_rproc_cfg_imx93 }, > > > > > > =A0=A0=A0=A0=A0 {}, > > > > > > -- = > > > > > > 2.37.1 > > > > > > = _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel