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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 09A8CC433EF for ; Tue, 7 Jun 2022 09:13:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238808AbiFGJNy (ORCPT ); Tue, 7 Jun 2022 05:13:54 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35924 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232209AbiFGJNu (ORCPT ); Tue, 7 Jun 2022 05:13:50 -0400 Received: from mailgw02.mediatek.com (unknown [210.61.82.184]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E45F669CC2; Tue, 7 Jun 2022 02:13:48 -0700 (PDT) X-UUID: 0fa794990b414e1890f80e81b3df8836-20220607 X-CID-P-RULE: Release_Ham X-CID-O-INFO: VERSION:1.1.5,REQID:adedc9e7-406b-48db-b14e-8ba1509dafe0,OB:0,LO B:0,IP:0,URL:0,TC:0,Content:0,EDM:0,RT:0,SF:45,FILE:0,RULE:Release_Ham,ACT ION:release,TS:45 X-CID-INFO: VERSION:1.1.5,REQID:adedc9e7-406b-48db-b14e-8ba1509dafe0,OB:0,LOB: 0,IP:0,URL:0,TC:0,Content:0,EDM:0,RT:0,SF:45,FILE:0,RULE:Release_Ham,ACTIO N:release,TS:45 X-CID-META: VersionHash:2a19b09,CLOUDID:31deece4-2ba2-4dc1-b6c5-11feb6c769e0,C OID:ebc12f442421,Recheck:0,SF:28|17|19|48,TC:nil,Content:0,EDM:-3,IP:nil,U RL:0,File:nil,QS:0,BEC:nil X-UUID: 0fa794990b414e1890f80e81b3df8836-20220607 Received: from mtkcas11.mediatek.inc [(172.21.101.40)] by mailgw02.mediatek.com (envelope-from ) (Generic MTA with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 1283919427; Tue, 07 Jun 2022 17:13:44 +0800 Received: from mtkcas10.mediatek.inc (172.21.101.39) by mtkmbs11n2.mediatek.inc (172.21.101.187) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.2.792.3; Tue, 7 Jun 2022 17:13:42 +0800 Received: from mtksdccf07 (172.21.84.99) by mtkcas10.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Tue, 7 Jun 2022 17:13:42 +0800 Message-ID: Subject: Re: [PATCH v19 1/6] soc: mediatek: mutex: add common interface for modules setting From: CK Hu To: Moudy Ho , Mauro Carvalho Chehab , Rob Herring , Matthias Brugger , Krzysztof Kozlowski , Hans Verkuil CC: Chun-Kuang Hu , Rob Landley , Laurent Pinchart , , , , , , Alexandre Courbot , , , , , Benjamin Gaignard , AngeloGioacchino Del Regno , daoyuan huang , Ping-Hsun Wu , , , , , , , , Date: Tue, 7 Jun 2022 17:13:42 +0800 In-Reply-To: <20220531055224.19280-2-moudy.ho@mediatek.com> References: <20220531055224.19280-1-moudy.ho@mediatek.com> <20220531055224.19280-2-moudy.ho@mediatek.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.28.5-0ubuntu0.18.04.2 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-MTK: N Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Moudy: On Tue, 2022-05-31 at 13:52 +0800, Moudy Ho wrote: > In order to allow multiple modules to operate MUTEX hardware through > a common interfrace, two flexible indexes "mtk_mutex_mod_index" and > "mtk_mutex_sof_index" need to be added to replace original component > ID so that like DDP and MDP can add their own MOD table or SOF > settings independently. > > In addition, 2 generic interface "mtk_mutex_write_mod" and > "mtk_mutex_write_sof" have been added, which is expected to replace > the "mtk_mutex_add_comp" and "mtk_mutex_remove_comp" pair originally > dedicated to DDP in the future. > > Signed-off-by: Moudy Ho > Reviewed-by: AngeloGioacchino Del Regno < > angelogioacchino.delregno@collabora.com> > Reviewed-by: Rex-BC Chen > --- > drivers/soc/mediatek/mtk-mutex.c | 59 > ++++++++++++++++++++++++++ > include/linux/soc/mediatek/mtk-mutex.h | 26 ++++++++++++ > 2 files changed, 85 insertions(+) > > diff --git a/drivers/soc/mediatek/mtk-mutex.c > b/drivers/soc/mediatek/mtk-mutex.c > index 981d56967e7a..a3b054b33826 100644 > --- a/drivers/soc/mediatek/mtk-mutex.c > +++ b/drivers/soc/mediatek/mtk-mutex.c > @@ -185,6 +185,7 @@ struct mtk_mutex_data { > const unsigned int *mutex_sof; > const unsigned int mutex_mod_reg; > const unsigned int mutex_sof_reg; > + const unsigned int *mutex_table_mod; > const bool no_clk; > }; > > @@ -606,6 +607,64 @@ void mtk_mutex_release(struct mtk_mutex *mutex) > } > EXPORT_SYMBOL_GPL(mtk_mutex_release); > > +int mtk_mutex_write_mod(struct mtk_mutex *mutex, > + enum mtk_mutex_mod_index idx, bool clear) > +{ > + struct mtk_mutex_ctx *mtx = container_of(mutex, struct > mtk_mutex_ctx, > + mutex[mutex->id]); > + unsigned int reg; > + unsigned int offset; > + > + WARN_ON(&mtx->mutex[mutex->id] != mutex); > + > + if (idx < MUTEX_MOD_IDX_MDP_RDMA0 || > + idx >= MUTEX_MOD_IDX_MAX) { > + dev_err(mtx->dev, "Not supported MOD table index : %d", > idx); > + return -EINVAL; > + } > + > + offset = DISP_REG_MUTEX_MOD(mtx->data->mutex_mod_reg, > + mutex->id); > + reg = readl_relaxed(mtx->regs + offset); > + > + if (clear) > + reg &= ~BIT(mtx->data->mutex_table_mod[idx]); > + else > + reg |= BIT(mtx->data->mutex_table_mod[idx]); > + > + writel_relaxed(reg, mtx->regs + offset); > + > + return 0; > +} > +EXPORT_SYMBOL_GPL(mtk_mutex_write_mod); > + > +int mtk_mutex_write_sof(struct mtk_mutex *mutex, > + enum mtk_mutex_sof_index idx, bool clear) > +{ > + struct mtk_mutex_ctx *mtx = container_of(mutex, struct > mtk_mutex_ctx, > + mutex[mutex->id]); > + unsigned int val; > + > + WARN_ON(&mtx->mutex[mutex->id] != mutex); > + > + if (idx < MUTEX_SOF_IDX_SINGLE_MODE || > + idx >= MUTEX_SOF_IDX_MAX) { > + dev_err(mtx->dev, "Not supported SOF index : %d", idx); > + return -EINVAL; > + } > + > + if (clear) > + val = MUTEX_SOF_SINGLE_MODE; This equal to 'idx = MUTEX_SOF_SINGLE_MODE', so 'clear' is not necessary. Regards, CK > + else > + val = idx; > + > + writel_relaxed(val, mtx->regs + > + DISP_REG_MUTEX_SOF(mtx->data->mutex_sof_reg, > mutex->id)); > + > + return 0; > +} > +EXPORT_SYMBOL_GPL(mtk_mutex_write_sof); > + > static int mtk_mutex_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > diff --git a/include/linux/soc/mediatek/mtk-mutex.h > b/include/linux/soc/mediatek/mtk-mutex.h > index 6fe4ffbde290..829231523e3f 100644 > --- a/include/linux/soc/mediatek/mtk-mutex.h > +++ b/include/linux/soc/mediatek/mtk-mutex.h > @@ -10,6 +10,26 @@ struct regmap; > struct device; > struct mtk_mutex; > > +enum mtk_mutex_mod_index { > + /* MDP table index */ > + MUTEX_MOD_IDX_MDP_RDMA0, > + MUTEX_MOD_IDX_MDP_RSZ0, > + MUTEX_MOD_IDX_MDP_RSZ1, > + MUTEX_MOD_IDX_MDP_TDSHP0, > + MUTEX_MOD_IDX_MDP_WROT0, > + MUTEX_MOD_IDX_MDP_WDMA, > + MUTEX_MOD_IDX_MDP_AAL0, > + MUTEX_MOD_IDX_MDP_CCORR0, > + > + MUTEX_MOD_IDX_MAX /* ALWAYS keep at the end */ > +}; > + > +enum mtk_mutex_sof_index { > + MUTEX_SOF_IDX_SINGLE_MODE, > + > + MUTEX_SOF_IDX_MAX /* ALWAYS keep at the end */ > +}; > + > struct mtk_mutex *mtk_mutex_get(struct device *dev); > int mtk_mutex_prepare(struct mtk_mutex *mutex); > void mtk_mutex_add_comp(struct mtk_mutex *mutex, > @@ -22,5 +42,11 @@ void mtk_mutex_unprepare(struct mtk_mutex *mutex); > void mtk_mutex_put(struct mtk_mutex *mutex); > void mtk_mutex_acquire(struct mtk_mutex *mutex); > void mtk_mutex_release(struct mtk_mutex *mutex); > +int mtk_mutex_write_mod(struct mtk_mutex *mutex, > + enum mtk_mutex_mod_index idx, > + bool clear); > +int mtk_mutex_write_sof(struct mtk_mutex *mutex, > + enum mtk_mutex_sof_index idx, > + bool clear); > > #endif /* MTK_MUTEX_H */