Hi Franck, > Subject: [PATCH 1/7] firmware: imx: scu-rm: Add Resource Management APIs
Are you using an old base? This was already in tree. Regards, Peng. > > From: Franck LENORMAND <franck.lenorm...@oss.nxp.com> > > This patch adds the imx_sc_rm_is_resource_owned indicating if a specific > resource is owned by the caller. > > Signed-off-by: Franck LENORMAND <franck.lenorm...@oss.nxp.com> > --- > drivers/firmware/imx/Makefile | 2 +- > drivers/firmware/imx/rm.c | 44 +++++++++++++++++++++++ > include/linux/firmware/imx/sci.h | 1 + > include/linux/firmware/imx/svc/rm.h | 69 > +++++++++++++++++++++++++++++++++++++ > 4 files changed, 115 insertions(+), 1 deletion(-) create mode 100644 > drivers/firmware/imx/rm.c create mode 100644 > include/linux/firmware/imx/svc/rm.h > > diff --git a/drivers/firmware/imx/Makefile b/drivers/firmware/imx/Makefile > index 08bc9dd..17ea361 100644 > --- a/drivers/firmware/imx/Makefile > +++ b/drivers/firmware/imx/Makefile > @@ -1,4 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0 > obj-$(CONFIG_IMX_DSP) += imx-dsp.o > -obj-$(CONFIG_IMX_SCU) += imx-scu.o misc.o imx-scu-irq.o > +obj-$(CONFIG_IMX_SCU) += imx-scu.o misc.o imx-scu-irq.o rm.o > obj-$(CONFIG_IMX_SCU_PD) += scu-pd.o > diff --git a/drivers/firmware/imx/rm.c b/drivers/firmware/imx/rm.c new file > mode 100644 index 00000000..a397c6b > --- /dev/null > +++ b/drivers/firmware/imx/rm.c > @@ -0,0 +1,44 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2016 Freescale Semiconductor, Inc. > + * Copyright 2017-2020 NXP > + * > + * File containing client-side RPC functions for the RM service. These > + * functions are ported to clients that communicate to the SC. > + */ > + > +#include <linux/firmware/imx/svc/rm.h> > + > +struct imx_sc_msg_rm_rsrc_owned { > + struct imx_sc_rpc_msg hdr; > + u16 resource; > +} __packed __aligned(4); > + > +/** > + * imx_sc_rm_is_resource_owned() - Checks @resource is owned by current > + * partition or not > + * > + * @ipc: IPC handle > + * @resource: resource the control is associated with > + * > + * Return: > + * 0 - OK > + * < 0 - error. > + */ > +bool imx_sc_rm_is_resource_owned(struct imx_sc_ipc *ipc, u16 resource) > +{ > + struct imx_sc_msg_rm_rsrc_owned msg; > + struct imx_sc_rpc_msg *hdr = &msg.hdr; > + > + hdr->ver = IMX_SC_RPC_VERSION; > + hdr->svc = IMX_SC_RPC_SVC_RM; > + hdr->func = IMX_SC_RM_FUNC_IS_RESOURCE_OWNED; > + hdr->size = 2; > + > + msg.resource = resource; > + > + imx_scu_call_rpc(ipc, &msg, true); > + > + return hdr->func; > +} > +EXPORT_SYMBOL(imx_sc_rm_is_resource_owned); > diff --git a/include/linux/firmware/imx/sci.h > b/include/linux/firmware/imx/sci.h > index 3fa418a..3c459f5 100644 > --- a/include/linux/firmware/imx/sci.h > +++ b/include/linux/firmware/imx/sci.h > @@ -14,6 +14,7 @@ > > #include <linux/firmware/imx/svc/misc.h> #include > <linux/firmware/imx/svc/pm.h> > +#include <linux/firmware/imx/svc/rm.h> > > int imx_scu_enable_general_irq_channel(struct device *dev); int > imx_scu_irq_register_notifier(struct notifier_block *nb); diff --git > a/include/linux/firmware/imx/svc/rm.h > b/include/linux/firmware/imx/svc/rm.h > new file mode 100644 > index 00000000..9924216 > --- /dev/null > +++ b/include/linux/firmware/imx/svc/rm.h > @@ -0,0 +1,69 @@ > +/* SPDX-License-Identifier: GPL-2.0+ */ > +/* > + * Copyright (C) 2016 Freescale Semiconductor, Inc. > + * Copyright 2017-2020 NXP > + * > + * Header file containing the public API for the System Controller (SC) > + * Power Management (PM) function. This includes functions for power > +state > + * control, clock control, reset control, and wake-up event control. > + * > + * RM_SVC (SVC) Resource Management Service > + * > + * Module for the Resource Management (RM) service. > + */ > + > +#ifndef _SC_RM_API_H > +#define _SC_RM_API_H > + > +#include <linux/firmware/imx/sci.h> > + > +/* > + * This type is used to indicate RPC RM function calls. > + */ > +enum imx_sc_rm_func { > + IMX_SC_RM_FUNC_UNKNOWN = 0, > + IMX_SC_RM_FUNC_PARTITION_ALLOC = 1, > + IMX_SC_RM_FUNC_SET_CONFIDENTIAL = 31, > + IMX_SC_RM_FUNC_PARTITION_FREE = 2, > + IMX_SC_RM_FUNC_GET_DID = 26, > + IMX_SC_RM_FUNC_PARTITION_STATIC = 3, > + IMX_SC_RM_FUNC_PARTITION_LOCK = 4, > + IMX_SC_RM_FUNC_GET_PARTITION = 5, > + IMX_SC_RM_FUNC_SET_PARENT = 6, > + IMX_SC_RM_FUNC_MOVE_ALL = 7, > + IMX_SC_RM_FUNC_ASSIGN_RESOURCE = 8, > + IMX_SC_RM_FUNC_SET_RESOURCE_MOVABLE = 9, > + IMX_SC_RM_FUNC_SET_SUBSYS_RSRC_MOVABLE = 28, > + IMX_SC_RM_FUNC_SET_MASTER_ATTRIBUTES = 10, > + IMX_SC_RM_FUNC_SET_MASTER_SID = 11, > + IMX_SC_RM_FUNC_SET_PERIPHERAL_PERMISSIONS = 12, > + IMX_SC_RM_FUNC_IS_RESOURCE_OWNED = 13, > + IMX_SC_RM_FUNC_GET_RESOURCE_OWNER = 33, > + IMX_SC_RM_FUNC_IS_RESOURCE_MASTER = 14, > + IMX_SC_RM_FUNC_IS_RESOURCE_PERIPHERAL = 15, > + IMX_SC_RM_FUNC_GET_RESOURCE_INFO = 16, > + IMX_SC_RM_FUNC_MEMREG_ALLOC = 17, > + IMX_SC_RM_FUNC_MEMREG_SPLIT = 29, > + IMX_SC_RM_FUNC_MEMREG_FRAG = 32, > + IMX_SC_RM_FUNC_MEMREG_FREE = 18, > + IMX_SC_RM_FUNC_FIND_MEMREG = 30, > + IMX_SC_RM_FUNC_ASSIGN_MEMREG = 19, > + IMX_SC_RM_FUNC_SET_MEMREG_PERMISSIONS = 20, > + IMX_SC_RM_FUNC_IS_MEMREG_OWNED = 21, > + IMX_SC_RM_FUNC_GET_MEMREG_INFO = 22, > + IMX_SC_RM_FUNC_ASSIGN_PAD = 23, > + IMX_SC_RM_FUNC_SET_PAD_MOVABLE = 24, > + IMX_SC_RM_FUNC_IS_PAD_OWNED = 25, > + IMX_SC_RM_FUNC_DUMP = 27, > +}; > + > +#if IS_ENABLED(CONFIG_IMX_SCU) > +bool imx_sc_rm_is_resource_owned(struct imx_sc_ipc *ipc, u16 resource); > +#else static inline bool imx_sc_rm_is_resource_owned(struct imx_sc_ipc > +*ipc, u16 resource) { > + return true; > +} > +#endif > +#endif > -- > 2.7.4