Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Joy Zou <joy.zou@nxp.com>
To:     vkoul@kernel.org, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org, shawnguo@kernel.org,
        s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com
Cc:     shengjiu.wang@nxp.com, martink@posteo.de, dev@lynxeye.de,
        alexander.stein@ew.tq-group.com, peng.fan@nxp.com, david@ixit.cz,
        aford173@gmail.com, hongxing.zhu@nxp.com, linux-imx@nxp.com,
        dmaengine@vger.kernel.org, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: [PATCH v10 2/2] dmaengine: imx-sdma: support hdmi in sdma
Date:   Tue, 15 Nov 2022 17:38:23 +0800
Message-Id: <20221115093823.2879128-3-joy.zou@nxp.com>
In-Reply-To: <20221115093823.2879128-1-joy.zou@nxp.com>
References: <20221115093823.2879128-1-joy.zou@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?3x6cbpUI1BwZgTw213i9Fuj+UrdmGST08Ph/l+SaTBrjdQeiciFeJsC4zYz6?=
 =?us-ascii?Q?kzoFOjQ3/60w6NnQXS3H8JZRk0skPKaKwX316FtUv6+PlL2W1bvpximxFI7K?=
 =?us-ascii?Q?P5RrIehjfN/HcaO5dynSMUQHFbdruaJZfpXeY/CNJLRuo9xP8zOvepssjmo5?=
 =?us-ascii?Q?271CRY7R+kY8DKSqy6UqQk0c4xrGTTSjKnElQzEDixlolRqLUrOtx99TQOBh?=
 =?us-ascii?Q?AJg/ZW99umlaeSCkmPyWfij1kchXo8rJnUi61By/CL28JiLLLE2Xj4ciqL9k?=
 =?us-ascii?Q?4QG5ealJBXqp1xiV1cItoORpvrAKC058OWbfFNnvQBhy+XXPibd9SQSZd7mS?=
 =?us-ascii?Q?e1peshIdhQqZPKej8zbtKD4nChHB988aeWska1OpSM5m5vgQxhXeN/yNzuBx?=
 =?us-ascii?Q?/K+dSaKrqxZDOk5rhLAL3VmwwlohNx1oUimlaL5Nz9XcEiCWisrSPJHZtbZQ?=
 =?us-ascii?Q?Mp798HpGem7De4Mn9RlT7fN+6xMNNCvBtHb387X+gULUvorsSe66IlG+Mvvw?=
 =?us-ascii?Q?jDKwb7rbvt+8S9dQviRFz4qRWgpskVIXkv4EfArNITpPwyzZMcVJGSnlaf6t?=
 =?us-ascii?Q?hhjcmLzaZnwYrNc7Z/b8Rx79E2z0kL5TrzAJTD4kIdGgnMAWJcxS2TTo7FXp?=
 =?us-ascii?Q?K5nS9UpdXsQJJxHvy6JG/Zl8v1ijKSmIlLNqg01xLUpM//7+rBxorsmZkEZM?=
 =?us-ascii?Q?XSPa+U2kRSjbbEzcd97ESmx3G7OlxiL26ixYOHuumkWkrBC1lKradXCuXo82?=
 =?us-ascii?Q?Nwe+yntR4fMWn6e2+25PY3eRrLb7FMivvZw/gZUf5m+6V5WE/7fHcHmXvrKl?=
 =?us-ascii?Q?sxTFFnOR/8RllyVHaE5puEycdCTFRMrKvlBbNeMi3qOKY5gB6DT3UuDz6guE?=
 =?us-ascii?Q?6kKLPpR0oJDN8YodG4NQzFgz8xMJiu4ErAah+hO6hAZeKRwHnQPsFJrsmkRD?=
 =?us-ascii?Q?CLbPrQUs9CYJgC2eZk14L3MIgxr5YqyYE8broHKlUIfKlFFx9Qom6vAEB/Bt?=
 =?us-ascii?Q?n9M+e81LkVgV7JuAiEss0teHtqMO+mirHlzCOkOtXrxGvkuoUwbHjHc4iWaV?=
 =?us-ascii?Q?oEAeS6aUvAiDu77wynBVhggl2CtmVabZENNZm7Uqu+PAFXjR83Qidbz3du1z?=
 =?us-ascii?Q?JONuEYarimo7XHygct4SVwePpvRtyniLACfvzAHK+CoAvOpZzlojR47/gsBf?=
 =?us-ascii?Q?+TRjQcqR4pvusSdeimflgmhWYXtoT54Re5J3/kSSCYqDOdT4CaJrUAFCmKrl?=
 =?us-ascii?Q?GnXPNQBpsBm6zFEd0LbNVqpm6xJt7IoyHQn+JHfK1wNSBefkl9T40N8/LFG9?=
 =?us-ascii?Q?eY4kzX/peNdBoahL+yQ0YkUJFfM2PQQ4NBcA5dE5uRPzYzNLZeZn+OMTolWa?=
 =?us-ascii?Q?0y0oBh2Xh5HSFe6n9JNO6foBt7gbBiOPAbhzul3tVJ2+n6NsmzPuNn7KDRPb?=
 =?us-ascii?Q?XZznv22YEItRcV1lqGYGdCgGdWDWbqwVVBwGNrutR4/6UI0AsCf0hSZD2HIp?=
 =?us-ascii?Q?lDQ8e21l2h2H02wI01d6MOlTPdFd23HdQ6lWTcwSz/rr/rN2OF887KelI0PB?=
 =?us-ascii?Q?zJE69f+Hzcoz1G1ijmbVeu/6XWANf3EcqIAHngs5?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1c536565-aa0f-44bb-c15d-08dac6ecfd93
X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB5925.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Nov 2022 09:37:19.6809
 (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: sUbukpO1lgUw5kCdkjRv0Cg/kd4j9fcrNewdzPSQZSx5W/MiPuBA6Lxx3tvzfpDx
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR04MB9462
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

The hdmi script already supported in sdma firmware. So add support hdmi
in sdma driver.

The design of hdmi script is different from common script such as sai.
There is no need to config buffer descriptor for HDMI. The cyclic
capability is achieved by the hdmi script. The slave config is so simple,
only config src_addr, dts_addr and direction DMA_TRANS_NONE.

Signed-off-by: Joy Zou <joy.zou@nxp.com>
---
Changes in v7:
    modify the commit message.
---
 drivers/dma/imx-sdma.c      | 38 +++++++++++++++++++++++++++++--------
 include/linux/dma/imx-dma.h |  1 +
 2 files changed, 31 insertions(+), 8 deletions(-)

diff --git a/drivers/dma/imx-sdma.c b/drivers/dma/imx-sdma.c
index fbea5f62dd98..ab877ceeac3f 100644
--- a/drivers/dma/imx-sdma.c
+++ b/drivers/dma/imx-sdma.c
@@ -954,7 +954,10 @@ static irqreturn_t sdma_int_handler(int irq, void *dev_id)
 		desc = sdmac->desc;
 		if (desc) {
 			if (sdmac->flags & IMX_DMA_SG_LOOP) {
-				sdma_update_channel_loop(sdmac);
+				if (sdmac->peripheral_type != IMX_DMATYPE_HDMI)
+					sdma_update_channel_loop(sdmac);
+				else
+					vchan_cyclic_callback(&desc->vd);
 			} else {
 				mxc_sdma_handle_channel_normal(sdmac);
 				vchan_cookie_complete(&desc->vd);
@@ -1074,6 +1077,10 @@ static int sdma_get_pc(struct sdma_channel *sdmac,
 		per_2_emi = sdma->script_addrs->sai_2_mcu_addr;
 		emi_2_per = sdma->script_addrs->mcu_2_sai_addr;
 		break;
+	case IMX_DMATYPE_HDMI:
+		emi_2_per = sdma->script_addrs->hdmi_dma_addr;
+		sdmac->is_ram_script = true;
+		break;
 	default:
 		dev_err(sdma->dev, "Unsupported transfer type %d\n",
 			peripheral_type);
@@ -1125,11 +1132,16 @@ static int sdma_load_context(struct sdma_channel *sdmac)
 	/* Send by context the event mask,base address for peripheral
 	 * and watermark level
 	 */
-	context->gReg[0] = sdmac->event_mask[1];
-	context->gReg[1] = sdmac->event_mask[0];
-	context->gReg[2] = sdmac->per_addr;
-	context->gReg[6] = sdmac->shp_addr;
-	context->gReg[7] = sdmac->watermark_level;
+	if (sdmac->peripheral_type == IMX_DMATYPE_HDMI) {
+		context->gReg[4] = sdmac->per_addr;
+		context->gReg[6] = sdmac->shp_addr;
+	} else {
+		context->gReg[0] = sdmac->event_mask[1];
+		context->gReg[1] = sdmac->event_mask[0];
+		context->gReg[2] = sdmac->per_addr;
+		context->gReg[6] = sdmac->shp_addr;
+		context->gReg[7] = sdmac->watermark_level;
+	}
 
 	bd0->mode.command = C0_SETDM;
 	bd0->mode.status = BD_DONE | BD_WRAP | BD_EXTD;
@@ -1513,7 +1525,7 @@ static struct sdma_desc *sdma_transfer_init(struct sdma_channel *sdmac,
 	desc->sdmac = sdmac;
 	desc->num_bd = bds;
 
-	if (sdma_alloc_bd(desc))
+	if (bds && sdma_alloc_bd(desc))
 		goto err_desc_out;
 
 	/* No slave_config called in MEMCPY case, so do here */
@@ -1678,13 +1690,16 @@ static struct dma_async_tx_descriptor *sdma_prep_dma_cyclic(
 {
 	struct sdma_channel *sdmac = to_sdma_chan(chan);
 	struct sdma_engine *sdma = sdmac->sdma;
-	int num_periods = buf_len / period_len;
+	int num_periods = 0;
 	int channel = sdmac->channel;
 	int i = 0, buf = 0;
 	struct sdma_desc *desc;
 
 	dev_dbg(sdma->dev, "%s channel: %d\n", __func__, channel);
 
+	if (sdmac->peripheral_type != IMX_DMATYPE_HDMI)
+		num_periods = buf_len / period_len;
+
 	sdma_config_write(chan, &sdmac->slave_config, direction);
 
 	desc = sdma_transfer_init(sdmac, direction, num_periods);
@@ -1701,6 +1716,9 @@ static struct dma_async_tx_descriptor *sdma_prep_dma_cyclic(
 		goto err_bd_out;
 	}
 
+	if (sdmac->peripheral_type == IMX_DMATYPE_HDMI)
+		return vchan_tx_prep(&sdmac->vc, &desc->vd, flags);
+
 	while (buf < buf_len) {
 		struct sdma_buffer_descriptor *bd = &desc->bd[i];
 		int param;
@@ -1761,6 +1779,10 @@ static int sdma_config_write(struct dma_chan *chan,
 		sdmac->watermark_level |= (dmaengine_cfg->dst_maxburst << 16) &
 			SDMA_WATERMARK_LEVEL_HWML;
 		sdmac->word_size = dmaengine_cfg->dst_addr_width;
+	} else if (sdmac->peripheral_type == IMX_DMATYPE_HDMI) {
+		sdmac->per_address = dmaengine_cfg->dst_addr;
+		sdmac->per_address2 = dmaengine_cfg->src_addr;
+		sdmac->watermark_level = 0;
 	} else {
 		sdmac->per_address = dmaengine_cfg->dst_addr;
 		sdmac->watermark_level = dmaengine_cfg->dst_maxburst *
diff --git a/include/linux/dma/imx-dma.h b/include/linux/dma/imx-dma.h
index f487a4fa103a..cfec5f946e23 100644
--- a/include/linux/dma/imx-dma.h
+++ b/include/linux/dma/imx-dma.h
@@ -40,6 +40,7 @@ enum sdma_peripheral_type {
 	IMX_DMATYPE_ASRC_SP,	/* Shared ASRC */
 	IMX_DMATYPE_SAI,	/* SAI */
 	IMX_DMATYPE_MULTI_SAI,	/* MULTI FIFOs For Audio */
+	IMX_DMATYPE_HDMI,       /* HDMI Audio */
 };
 
 enum imx_dma_prio {
-- 
2.37.1