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;
From:   Peng Fan <peng.fan@nxp.com>
To:     "Peng Fan (OSS)" <peng.fan@oss.nxp.com>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>
CC:     "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "festevam@gmail.com" <festevam@gmail.com>,
        dl-linux-imx <linux-imx@nxp.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH] firmware: imx: add get resource owner api
Thread-Topic: [PATCH] firmware: imx: add get resource owner api
Thread-Index: AQHYBpp88Wk7iU3DuUWNyL9elrFSa6x1Dtlg
Date:   Wed, 26 Jan 2022 08:23:44 +0000
Message-ID: <DU0PR04MB9417B5E817D6A3FB162030BE88209@DU0PR04MB9417.eurprd04.prod.outlook.com>
References: <20220111032147.342012-1-peng.fan@oss.nxp.com>
In-Reply-To: <20220111032147.342012-1-peng.fan@oss.nxp.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?ma4RMASesJ9uTz0GFZ8vVv5hHgPTocQx0NpSiJ96FaWCJBQquwxDXeek5Az+?=
 =?us-ascii?Q?VUakWhREq3tZoInQyI0tpD0Z9ZmSpvp5r3VW8f4sew6HgFDDakEO59+7xX0H?=
 =?us-ascii?Q?QAyuZeUM24GWUqj+FJ1f8IGYdveVioRqzpNgGnOF5HGiuyo/C5uSerTkjLIZ?=
 =?us-ascii?Q?e+sNNAWS2Ekjo8UPL63wgenC5eVBpNWBYgfeUiT+UMcpXlD+ONW4Tv1QUKvu?=
 =?us-ascii?Q?YqL979TZcO/ECUTqgg0/GJ80oYVWCpFlRrRsrj9Hfl7wy9XT83V49fh4ORJK?=
 =?us-ascii?Q?SXU/jajI/DU8uQAC07DSG/A6HAYjLB9tzkweVqJ9y86Ke9FAPR2RQ5tOmFBc?=
 =?us-ascii?Q?BK/fvXvB/B0nl3/j9EbW6TqsDZdylxpDJYTZr3rybZFu0ATqNHkxt2BsTcdm?=
 =?us-ascii?Q?EXcEb+JMXhn4NiudSpgcUN2Q80MNZTIsRNaHH9+8PREYwNl6+J9fsKGnnoeN?=
 =?us-ascii?Q?hnyA9TdySIlvfvBHDcPuAaUEN1XA0rBndYaKpGPXLwJiBBFlyp5bMbLBalpg?=
 =?us-ascii?Q?SBsYOLLCJXw8/rT9Yxld3w0ujXCN2FCx++kPi639fAQJODrCDha+NwjtEEda?=
 =?us-ascii?Q?XkhnayUk9Za0dWmKsN+b2KXB/Ps/5qQ0rXeaadeQn3OBibPqMdROqBimFs60?=
 =?us-ascii?Q?8A031jVv+JExpMpf6fKi0SeG21ae2tk5BnDHFeFuRqBa0LAGYdsbV5laPr00?=
 =?us-ascii?Q?m3DKyeulQjBNMDiOJalqMIafzL6acdtxv2V6jvzU1CCIMswQj0s0v0giF1S4?=
 =?us-ascii?Q?9v2Y4J01QIXRgZ8tmD68lfHOs8rUBbqhtFGeoF6x+SHIv682H976jFQlMvQO?=
 =?us-ascii?Q?ZNJ37aQzDxR31WuFHjrjk1KBn40E+jy/78tG/8g8PG+WzjxIgG8U8y6o8Okp?=
 =?us-ascii?Q?CWfZRnA4GlPQnUJEpzNXzLz/XiXa+hyuh9NaFy8cSR+CIKtKx8THCGhJJP5f?=
 =?us-ascii?Q?dNUfLEJ5PFtm3Ishn0DxnJr5hn7hMDnY4vPdt3MubmEPVn6pubBUswAKIu+G?=
 =?us-ascii?Q?zoL0Y4OslqJS3ieVq7zkK3szZ4rbhXh4C7FqvVytkNNAI/hZUera6AqVVjN7?=
 =?us-ascii?Q?hnhv9jwCtX1pGXa+j1IpkNeCOSP24HdAi5x4KKJ3/CPx7NjEoR2Dclf+zncU?=
 =?us-ascii?Q?w2qxIBIOVe2c4ZTQKKcpX2ZzmCSJpkftYhcSgJUbEcKn0LnJkgYFnZw11t4u?=
 =?us-ascii?Q?LlAbvnMHTTSz4UxtlVEkL15eVYbX6n4AJIdyTMBTevp7mWDnssQdpyv4BpdQ?=
 =?us-ascii?Q?+E+kM+QXKxjL2KUuoj8WNfGjz+hTYciLo5DCTEGeb2IiCQUwoPXfaTt6gl4J?=
 =?us-ascii?Q?SI63+WD4f3iCIqB8NiZ6/Y8q3WnyVkGlGtTsXbDMGNZbW+2iJsZbIYqJRbVk?=
 =?us-ascii?Q?nz2Mp+U/83GSzUzw4t0Rbe2vJiee0+KpIPOfR2Z8mjmhfglD1fNuGP/2YFCv?=
 =?us-ascii?Q?WXvTHfh33OTrjbqj1JJZwZbl9OLhngKDfjWM1bFi+O37RbA+ERTJf06ArjTC?=
 =?us-ascii?Q?eYOBT5XXjOpOMJ+3xWDSu+sABkOcIRgYnBL7cOJmkgLRJTe4345eR64noGRT?=
 =?us-ascii?Q?XzDOq4uJ7cy/cU1mkzN5r9mwt5t9n39iQTCMxy1L3DtHg4JYSiqUM4Ryvx5O?=
 =?us-ascii?Q?xQ=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bf985beb-f038-4f53-3ee9-08d9e0a52aca
X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Jan 2022 08:23:44.0339
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: FLW06P49o/2Z1ssHNCO2x5T00wRcVCxOZuajINkSykd0psuOg/R36lVzFwWpaVlin3pDUknjeqbmy9gMtLSM2A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR04MB5526
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> Subject: [PATCH] firmware: imx: add get resource owner api

Ping.. we need this API for remoteproc partition case.

Thanks,
Peng.

>=20
> From: Peng Fan <peng.fan@nxp.com>
>=20
> Add resource owner management API, this API could be used to check
> whether M4 is under control of Linux.
>=20
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>  drivers/firmware/imx/rm.c           | 45
> +++++++++++++++++++++++++++++
>  include/linux/firmware/imx/svc/rm.h |  5 ++++
>  2 files changed, 50 insertions(+)
>=20
> diff --git a/drivers/firmware/imx/rm.c b/drivers/firmware/imx/rm.c index
> a12db6ff323b..3c3605f98123 100644
> --- a/drivers/firmware/imx/rm.c
> +++ b/drivers/firmware/imx/rm.c
> @@ -43,3 +43,48 @@ bool imx_sc_rm_is_resource_owned(struct imx_sc_ipc
> *ipc, u16 resource)
>  	return hdr->func;
>  }
>  EXPORT_SYMBOL(imx_sc_rm_is_resource_owned);
> +
> +/*
> + * This function get @resource partition number
> + *
> + * @param[in]     ipc         IPC handle
> + * @param[in]     resource    resource the control is associated with
> + * @param[out]    pt          pointer to return the partition number
> + *
> + * @return Returns 0 for success and < 0 for errors.
> + */
> +struct imx_sc_msg_rm_get_resource_owner {
> +	struct imx_sc_rpc_msg hdr;
> +	union {
> +		struct {
> +			u16 resource;
> +		} req;
> +		struct {
> +			u8 val;
> +		} resp;
> +	} data;
> +} __packed __aligned(4);
> +
> +int imx_sc_rm_get_resource_owner(struct imx_sc_ipc *ipc, u16 resource,
> +u8 *pt) {
> +	struct imx_sc_msg_rm_get_resource_owner msg;
> +	struct imx_sc_rpc_msg *hdr =3D &msg.hdr;
> +	int ret;
> +
> +	hdr->ver =3D IMX_SC_RPC_VERSION;
> +	hdr->svc =3D IMX_SC_RPC_SVC_RM;
> +	hdr->func =3D IMX_SC_RM_FUNC_GET_RESOURCE_OWNER;
> +	hdr->size =3D 2;
> +
> +	msg.data.req.resource =3D resource;
> +
> +	ret =3D imx_scu_call_rpc(ipc, &msg, true);
> +	if (ret)
> +		return ret;
> +
> +	if (pt)
> +		*pt =3D msg.data.resp.val;
> +
> +	return 0;
> +}
> +EXPORT_SYMBOL(imx_sc_rm_get_resource_owner);
> diff --git a/include/linux/firmware/imx/svc/rm.h
> b/include/linux/firmware/imx/svc/rm.h
> index 456b6a59d29b..ff481b23ea36 100644
> --- a/include/linux/firmware/imx/svc/rm.h
> +++ b/include/linux/firmware/imx/svc/rm.h
> @@ -59,11 +59,16 @@ enum imx_sc_rm_func {
>=20
>  #if IS_ENABLED(CONFIG_IMX_SCU)
>  bool imx_sc_rm_is_resource_owned(struct imx_sc_ipc *ipc, u16 resource);
> +int imx_sc_rm_get_resource_owner(struct imx_sc_ipc *ipc, u16 resource,
> +u8 *pt);
>  #else
>  static inline bool
>  imx_sc_rm_is_resource_owned(struct imx_sc_ipc *ipc, u16 resource)  {
>  	return true;
>  }
> +static inline int imx_sc_rm_get_resource_owner(struct imx_sc_ipc *ipc,
> +u16 resource, u8 *pt) {
> +	return -ENOTSUPP;
> +}
>  #endif
>  #endif
> --
> 2.25.1