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 X-Spam-Level: X-Spam-Status: No, score=-14.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3114CC48BCD for ; Wed, 9 Jun 2021 14:40:11 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id E754E61364 for ; Wed, 9 Jun 2021 14:40:10 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E754E61364 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.microsoft.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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=JpD27MtqpefSMQTuE5/Eay2WeFUIpxynaAzLOFzrJLo=; b=MCbTJQLfHYgyTZ IE49dJE02bSdrzinpTbacbwlGbd8WR2ACKmdDcA62K2uSPs60mjHGR+aBEjpFjHcnaBlsYW0Q6ECd HqXj+XFnYsxTai3KyDOHIpQE8OQrVHF1reV7FZfIhWFwxsxhT59GAc09VNefV+YiYJF1HL9qK7uBw h6svUWbiEO1DYf7jT7yaU0ykYxx/3ph7bVm2Wa/ROG0mRmoKJo024hRhMgBGvvBwTwoOhFs7mL40W IegOOJ7O6bO55k3t03gQstKGB3ya4BBoy3ITFT/m5ooVX5wgWS/I0k4KbJQWHjpsFOuXH0sHICWwN 6pTHe7KFFkFp/FbZnYdw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1lqzKy-00ELoX-0A; Wed, 09 Jun 2021 14:37:44 +0000 Received: from linux.microsoft.com ([13.77.154.182]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1lqynU-00EBM6-Jk for linux-arm-kernel@lists.infradead.org; Wed, 09 Jun 2021 14:03:10 +0000 Received: from sequoia (162-237-133-238.lightspeed.rcsntx.sbcglobal.net [162.237.133.238]) by linux.microsoft.com (Postfix) with ESMTPSA id 5998420B83C2; Wed, 9 Jun 2021 07:03:05 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 5998420B83C2 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1623247386; bh=IRts58tl+eYwjKkNGvvrg6lIfNUrj21VJzDxSCoORkE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=AxS4OoK9mboxHZ5xicd4/Qun7iZD4pccHsdXfoPMd5KALbDOcn2QWC4kjIEPYLxLM 0Vqn2UWjdo2R1UhbbIUk6xI0p5633RrMb4t/F/0qEPXfsBjNUeLVB4EMT3KT8l2zGv 65VJP9lwfb2UtrPydcyPkY+9ptQkAOg+sFrf/zX0= Date: Wed, 9 Jun 2021 09:03:03 -0500 From: Tyler Hicks To: Jens Wiklander Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, op-tee@lists.trustedfirmware.org, Sumit Garg , Herbert Xu , Sakkinen , Sasha Levin , Thirupathaiah Annapureddy , Vikas Gupta , "David S . Miller" Subject: Re: [PATCH 1/7] tee: remove unused tee_shm_pool_alloc_res_mem() Message-ID: <20210609140303.GE4910@sequoia> References: <20210609102324.2222332-1-jens.wiklander@linaro.org> <20210609102324.2222332-2-jens.wiklander@linaro.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210609102324.2222332-2-jens.wiklander@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210609_070308_739483_A958A2E6 X-CRM114-Status: GOOD ( 19.75 ) 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="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 2021-06-09 12:23:18, Jens Wiklander wrote: > None of the drivers in the TEE subsystem uses > tee_shm_pool_alloc_res_mem() so remove the function. > > Signed-off-by: Jens Wiklander Reviewed-by: Tyler Hicks Tyler > --- > drivers/tee/tee_shm_pool.c | 56 -------------------------------------- > include/linux/tee_drv.h | 30 -------------------- > 2 files changed, 86 deletions(-) > > diff --git a/drivers/tee/tee_shm_pool.c b/drivers/tee/tee_shm_pool.c > index fcbb461fc59c..a9f9d50fd181 100644 > --- a/drivers/tee/tee_shm_pool.c > +++ b/drivers/tee/tee_shm_pool.c > @@ -47,62 +47,6 @@ static const struct tee_shm_pool_mgr_ops pool_ops_generic = { > .destroy_poolmgr = pool_op_gen_destroy_poolmgr, > }; > > -/** > - * tee_shm_pool_alloc_res_mem() - Create a shared memory pool from reserved > - * memory range > - * @priv_info: Information for driver private shared memory pool > - * @dmabuf_info: Information for dma-buf shared memory pool > - * > - * Start and end of pools will must be page aligned. > - * > - * Allocation with the flag TEE_SHM_DMA_BUF set will use the range supplied > - * in @dmabuf, others will use the range provided by @priv. > - * > - * @returns pointer to a 'struct tee_shm_pool' or an ERR_PTR on failure. > - */ > -struct tee_shm_pool * > -tee_shm_pool_alloc_res_mem(struct tee_shm_pool_mem_info *priv_info, > - struct tee_shm_pool_mem_info *dmabuf_info) > -{ > - struct tee_shm_pool_mgr *priv_mgr; > - struct tee_shm_pool_mgr *dmabuf_mgr; > - void *rc; > - > - /* > - * Create the pool for driver private shared memory > - */ > - rc = tee_shm_pool_mgr_alloc_res_mem(priv_info->vaddr, priv_info->paddr, > - priv_info->size, > - 3 /* 8 byte aligned */); > - if (IS_ERR(rc)) > - return rc; > - priv_mgr = rc; > - > - /* > - * Create the pool for dma_buf shared memory > - */ > - rc = tee_shm_pool_mgr_alloc_res_mem(dmabuf_info->vaddr, > - dmabuf_info->paddr, > - dmabuf_info->size, PAGE_SHIFT); > - if (IS_ERR(rc)) > - goto err_free_priv_mgr; > - dmabuf_mgr = rc; > - > - rc = tee_shm_pool_alloc(priv_mgr, dmabuf_mgr); > - if (IS_ERR(rc)) > - goto err_free_dmabuf_mgr; > - > - return rc; > - > -err_free_dmabuf_mgr: > - tee_shm_pool_mgr_destroy(dmabuf_mgr); > -err_free_priv_mgr: > - tee_shm_pool_mgr_destroy(priv_mgr); > - > - return rc; > -} > -EXPORT_SYMBOL_GPL(tee_shm_pool_alloc_res_mem); > - > struct tee_shm_pool_mgr *tee_shm_pool_mgr_alloc_res_mem(unsigned long vaddr, > phys_addr_t paddr, > size_t size, > diff --git a/include/linux/tee_drv.h b/include/linux/tee_drv.h > index 54269e47ac9a..53b9b2a13a87 100644 > --- a/include/linux/tee_drv.h > +++ b/include/linux/tee_drv.h > @@ -272,36 +272,6 @@ static inline void tee_shm_pool_mgr_destroy(struct tee_shm_pool_mgr *poolm) > poolm->ops->destroy_poolmgr(poolm); > } > > -/** > - * struct tee_shm_pool_mem_info - holds information needed to create a shared > - * memory pool > - * @vaddr: Virtual address of start of pool > - * @paddr: Physical address of start of pool > - * @size: Size in bytes of the pool > - */ > -struct tee_shm_pool_mem_info { > - unsigned long vaddr; > - phys_addr_t paddr; > - size_t size; > -}; > - > -/** > - * tee_shm_pool_alloc_res_mem() - Create a shared memory pool from reserved > - * memory range > - * @priv_info: Information for driver private shared memory pool > - * @dmabuf_info: Information for dma-buf shared memory pool > - * > - * Start and end of pools will must be page aligned. > - * > - * Allocation with the flag TEE_SHM_DMA_BUF set will use the range supplied > - * in @dmabuf, others will use the range provided by @priv. > - * > - * @returns pointer to a 'struct tee_shm_pool' or an ERR_PTR on failure. > - */ > -struct tee_shm_pool * > -tee_shm_pool_alloc_res_mem(struct tee_shm_pool_mem_info *priv_info, > - struct tee_shm_pool_mem_info *dmabuf_info); > - > /** > * tee_shm_pool_free() - Free a shared memory pool > * @pool: The shared memory pool to free > -- > 2.31.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel