Message ID | 20230620201221.2580428-4-Frank.Li@nxp.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp3924252vqr; Tue, 20 Jun 2023 13:35:33 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ7hdcE++hdmjvCFPm3ZLf3pDBMm8JBx/Gnzkj6Kt/yT2KMtnoM4EekkJk9/Otof3+Lr0MXB X-Received: by 2002:a05:6a20:3241:b0:116:d935:f742 with SMTP id hm1-20020a056a20324100b00116d935f742mr12821047pzc.20.1687293332693; Tue, 20 Jun 2023 13:35:32 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1687293332; cv=pass; d=google.com; s=arc-20160816; b=JffgEREdRJGw0WjfVVgiOdo7k54ZG+tOJlBaM+mKm6+2g4/cJQ4Wl2oVA0rZcpPkpy 3l6We7TJ4+SxXqbRijuBV78/wPwdeZtSdMJAghshHypL63w9Mqj3t/AaFSFELteip2GC mB09gS004bHcP3GxcbVtfZBESI8ljxXR7Nv5wdVUdmfW8rDeSLebgoYfM5rR9Z4jvTgS WWJ5yTxCHyeSlv41/7AQJtP6UhOH/a01SCDVnaiGxHei5Gt0VksI3uPI1q7YJyFJ/Fg+ sxxVs9OjzUgLwi8XB3/G6zZHLSEc88L3zuft6qy5+rjjGiLzI8e1xh2XDwVbJaCdqvCX rK9w== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:content-transfer-encoding :references:in-reply-to:message-id:date:subject:to:from :dkim-signature; bh=QI8n0XPwXsaaH6UZQG+Zvq7pxrcY++lIzGOz4LgaYtE=; b=p+yhA0Yo0XKoOhRtHNjj2JFE8c2WBm6lJaTojZ+I+tJRIZAt0btNsC0SeOjd3SaANV bBDSmm7jtCEcBIeTSdf92Jt9am+3cmsFJK6dlfFqkqDccBKzy33gJtsiziRZSxmja4F6 AA1vn/rRKEhUR3PST/aCu1BPuDVkchjzess/Qtl9YwNKqtVYpaeg7/bIW2HVrJ+a85XF 7AsbS/uFGK73zKsUTpxRMroBT2K929PM9OTHcViNkMbJ2ky/88cZOioaZgQ3E9uMBdn1 QScJEVMBxt9zyT87kLVsVU9JEH+ho5dkBkdOyX1EUIh/fG2p+ar+SiJjci95ENv7CCxp YaWg== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@nxp.com header.s=selector2 header.b=gEdhM43X; arc=pass (i=1 spf=pass spfdomain=nxp.com dkim=pass dkdomain=nxp.com dmarc=pass fromdomain=nxp.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nxp.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id y63-20020a638a42000000b00542b188b326si2360486pgd.876.2023.06.20.13.35.19; Tue, 20 Jun 2023 13:35:32 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@nxp.com header.s=selector2 header.b=gEdhM43X; arc=pass (i=1 spf=pass spfdomain=nxp.com dkim=pass dkdomain=nxp.com dmarc=pass fromdomain=nxp.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nxp.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230143AbjFTUND (ORCPT <rfc822;maxin.john@gmail.com> + 99 others); Tue, 20 Jun 2023 16:13:03 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49868 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229894AbjFTUMu (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Tue, 20 Jun 2023 16:12:50 -0400 Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-db3eur04on2045.outbound.protection.outlook.com [40.107.6.45]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E0F001728; Tue, 20 Jun 2023 13:12:47 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dkSRnZnW14J30Vc3ugyJLPTHqljCce/qfnpJdGLwBoiM109pt2RW9UNNTVAAAP/stQx3cDuYPUgbJ1vhK+hx8K6LXOFcNhm5BsbJRxYc4td8il2cnsVQ0BmWiQ9ulVra5nE/Qi+o7LoezFz7qIUbVY7Bw06NjeQyw6bLlS9VIhhYSLNcGjMb5hek3KrPAVDWXgIQltCbzHc+gSJSbf0rhhIghLw5z8wC1WQZDreeNucPB4AE6XI4vcNMw2+IQ571Tjh7BL+FG9Ev19ltLEASp9Y+xR8x/2u1ffzTHGMV0oou0S/IkQsQAvwmP1WfjFLUCKpw/LJo4Vih1JpKHcQfrA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=QI8n0XPwXsaaH6UZQG+Zvq7pxrcY++lIzGOz4LgaYtE=; b=KRelEUIE+PzuzANgVEZUyCW10xJ/SB893ob0Cd9fdLzCT++DQrKXw2zoeV5G6GwoTGNbacmdhuDxK5qI4BUsfv7udnRRjp0KHGE4ra3lonLQWc1JoPlCk7fh+g5tLmausUXPQ+ivHbUIvECPnG1/tvEWdhovipIUv0mSsFqq09+hqoaGqaMpIY9mJjwe6ynELb6i9e0W8hmjjLVkPq2G/b6uey0ivWdJxtyg5fC+F1+ZOPh7Re3lbJ8qw3hPYFIBHVdr54yRZN4OY8mnqnZYf4jnya8qvHXP71HMHeTXHMrFzJcucHnV0/L6W629b6D3j+zioW8hCqq7ilR/FNynfg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=QI8n0XPwXsaaH6UZQG+Zvq7pxrcY++lIzGOz4LgaYtE=; b=gEdhM43XWUua0sHQsgGwua9MakxmThjBZZfgLbJoeqcqEop6TP7p6NwXZKBgDiu92nKRxB8aBFVl1OOtxIUkIn5HO1HTZLh8KZklE4vL/e7hfIjfUF+owiv/wBoepTr6RB9qK1dC2T0muvQkHcV4B2ooTNwgs+Mut5unHZ/gbiY= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM6PR04MB4838.eurprd04.prod.outlook.com (2603:10a6:20b:4::16) by DU2PR04MB8903.eurprd04.prod.outlook.com (2603:10a6:10:2e2::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6500.37; Tue, 20 Jun 2023 20:12:45 +0000 Received: from AM6PR04MB4838.eurprd04.prod.outlook.com ([fe80::4a2a:262e:415f:e41c]) by AM6PR04MB4838.eurprd04.prod.outlook.com ([fe80::4a2a:262e:415f:e41c%7]) with mapi id 15.20.6500.036; Tue, 20 Jun 2023 20:12:45 +0000 From: Frank Li <Frank.Li@nxp.com> To: vkoul@kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, dmaengine@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, peng.fan@nxp.com, joy.zou@nxp.com, shenwei.wang@nxp.com, imx@lists.linux.dev Subject: [PATCH v9 03/13] dmaengine: fsl-edma: clean up fsl_edma_irq_exit() Date: Tue, 20 Jun 2023 16:12:11 -0400 Message-Id: <20230620201221.2580428-4-Frank.Li@nxp.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20230620201221.2580428-1-Frank.Li@nxp.com> References: <20230620201221.2580428-1-Frank.Li@nxp.com> Content-Transfer-Encoding: 8bit Content-Type: text/plain X-ClientProxiedBy: SJ0PR13CA0086.namprd13.prod.outlook.com (2603:10b6:a03:2c4::31) To AM6PR04MB4838.eurprd04.prod.outlook.com (2603:10a6:20b:4::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM6PR04MB4838:EE_|DU2PR04MB8903:EE_ X-MS-Office365-Filtering-Correlation-Id: 46027cf8-060f-4c5a-b22a-08db71cab626 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Gp+ttxQT7vjRgM7LSFlpsxScnooCIra+bYVo9WknjuRoaqS9ujqr6nskwNATjc2ImsZtRb+P96zmf1E/8socbNwhbe+6HtXR+JU/C+d7zjq5FZaSwdzy+eH29fLjEWjpBhvpO+sRxVmKrf+t7TM44vowO2d3mULrWoVEFVf49OycLn7ckFQwtCWkjFwVbplU0EUUnbiyolPSEErWR0BJmZEPvD0UJWcjVuk6elcUsY63IwDSqPA4DctjyfR+azpUEbe629E/n1RraBV3ebG5QvZ4KYtkNy4umVTncSJ3tMrN830cMeYqTLoln7y+SNdTkD3TD3EzF+1ZapGa9lBuDGYA1L0spi21pnfraWP+CI+0REhw+MZZb4QO6iWpxw7Yzq36A0IcRrpaQABuNBcbnLt8RiQ+U5e8v8O7Zr2Wwd5n/AmLzq0vTtUALVsTX8ZoqDinXx0NG1bx0NlvN2shE0re3CWNT3+WNA5hoPy2QHzqYFYibZEOfH4RtKoK4u57Ts5+ysp6Mc52yN3/pLTeI5VJe5OZhEo+3Sf5nHs4k9/JBBy6mgFiMN5xadWwYE9Q1mKnH7zlTv88wPP4/6mf/RHmA7jDOXGkpx/tY9Zst9hO7O9qEAfj7vp8oF3xoVw5i4SI6INFeIWtPoTYK+2VqA== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM6PR04MB4838.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(346002)(39860400002)(136003)(376002)(396003)(366004)(451199021)(83380400001)(66556008)(316002)(41300700001)(66946007)(66476007)(36756003)(1076003)(6506007)(2906002)(8676002)(8936002)(5660300002)(86362001)(38100700002)(38350700002)(921005)(2616005)(6666004)(478600001)(6512007)(26005)(186003)(52116002)(6486002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: 6boJ1Z38BV27ugACHumHqICt7/Ah43SJogu+iGbvrc/blcl4PSO8kpZyG8XdqhL3pR1RmMVMiBA4jy+3CLrp1Z81wVXHOplmqdfYN4fH6UgaTdxcFH8Z44dfieNlQvpBWXLmIHBQk2Vj8MA3ThnAhPdYlPtCI1h5UTjEE70TYiYS+xFRS63qhmpCVZcjjtSX+mmxW6RilQ+m6jgU+oIzeCVh6V/JCxsIQZ0DilMpzTXleijcNG55KvrevECHoIoVwjwvqAXcB29pIyhv+8kq5LqTsCdTqIN1Zni3X6qXYQ/fK7ReAt2luXRrCPIUJXredN9Xak4nGKiXlIuZ8SHQ31xjpxZfqEuWpsacU1QJBi3auD+vKPr/pEbaWAwyM8aeGUyGZnojy0akC2YtpuauAAec7rj9dV5TmjYlphgD2yLS1lb5cCJoJVnf9oPD0WiKtPxwsAJPJwFSRP6HlonIeDLELbYiw5w2l0EEzLhQGcIQSm4qeCzIAgz1F9SHp3DiGVmm0KRpMFYv+aD/0hVI/+kk2YkmKzID67XNe+pUp9MNe8f2MTLSkOv36xPhbqqfpSvN2DgWXl9s7vDz+J05rUWdYviouxnP1cv8KfOiQ11BvM+tn/FgQsk+Sg12v4AHeZ9in2DflC7Gg0/6b/TBye5OU1wwHaIb38812+J9gMeYSzdKrf5OUd4TR5UdPxN2eyIXGE+2E8mgrEd056rJU1HW9mouj2toP6fDxR/xbJZdri9tvJDU/eQeI7lPwbp7y8N1mEQO2gX8wFVP64pzQggJYF75Rn4eKOO/z9HL/dxW1ZB61OBzrkUi/eZJ7012baamW0SEV57iZ9Yv+xeORVD7l6xmvBwh5cR0e2TRwKkS0YGIM6Aayamtrn3M2FoypslYB85s/+t3QE47f1fG3GBktp9la6KE98KAIOWLuE16+N1B5sMlNb5IPz4yyKWI/o4qxHkyzc+M439Q4IxbFWVqCQpzCNW0HGXs+4TtnkF/hYhRonMzdRglHpcUVtAacSymg6EqQkM6jP9gJVnSRYFb9kXmW7PJ+LfjJk4Vedma5hrFrIZ4uw6XUbhbbIr1Z8CMwAkOwzSkN8+tNnfr9w3HbzwPyr8uhpSl77IsA86QjrwHpnV9yjXenzZavF2JVNW+Mc5wDCPQxRPVRuqfjzL2ddSE6HksBs2lrUTQdl4koSHSo+BTV0F4MjAo5IdqxHlijlheO7x/WLBxNcHNGeAziWnBW1VrGoBSUBykgJ0WaultUpxH0/xSLw7seHyxm6PAAXja5/WcSCks46bvYoMxekuXg3R+cWuXN0fFLT8KeL4Q0i6K5x4VrBF5AycT0JMRQJlbK3SXXJQGjlO7OJs0Hwom+Z0Oy3rgUTu3U6CZUylHmKHDVw+gUkFseL+o/AQFjYXTF53uJ8cixnaceIHNUVEE42LO9AAzQgA0/89aB+iDoNiM75BDtlNadd4FQjHGcrQEBntuIR6bk7YxO7ieTBzplTX/Cpimx9olW0oviIC95DE1eAV15EEr+ig4TMA7HTDlRIBmMNUpOd9Hw3Qpp9Ydupb0WQpFxmxwiuciDd+Sg4vLVE2agHly60w5 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 46027cf8-060f-4c5a-b22a-08db71cab626 X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB4838.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Jun 2023 20:12:45.7839 (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: uaVPJbTCu0FXqJZoPz106TEX/YQqC2da9rKSU6s7gYGj+BuifqzgUIiGEBUdVr75nTs47l73EnC36gHmPcFeVQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR04MB8903 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,T_SCC_BODY_TEXT_LINE, URIBL_BLOCKED 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 X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1769255293861618779?= X-GMAIL-MSGID: =?utf-8?q?1769255293861618779?= |
Series |
dmaengine: edma: add freescale edma v3 support
|
|
Commit Message
Frank Li
June 20, 2023, 8:12 p.m. UTC
In the probe function, devm_request_irq is now being used to automatically manage IRQ. It eliminates the need for manual IRQ freeing during removal. Signed-off-by: Frank Li <Frank.Li@nxp.com> Reviewed-by: Peng Fan <peng.fan@nxp.com> --- drivers/dma/fsl-edma-main.c | 12 ------------ 1 file changed, 12 deletions(-)
Comments
On 20-06-23, 16:12, Frank Li wrote: > In the probe function, devm_request_irq is now being used to automatically > manage IRQ. It eliminates the need for manual IRQ freeing during removal. > > Signed-off-by: Frank Li <Frank.Li@nxp.com> > Reviewed-by: Peng Fan <peng.fan@nxp.com> > --- > drivers/dma/fsl-edma-main.c | 12 ------------ > 1 file changed, 12 deletions(-) > > diff --git a/drivers/dma/fsl-edma-main.c b/drivers/dma/fsl-edma-main.c > index e40769666e39..16305bb9a2ef 100644 > --- a/drivers/dma/fsl-edma-main.c > +++ b/drivers/dma/fsl-edma-main.c > @@ -216,17 +216,6 @@ fsl_edma2_irq_init(struct platform_device *pdev, > return 0; > } > > -static void fsl_edma_irq_exit( > - struct platform_device *pdev, struct fsl_edma_engine *fsl_edma) > -{ > - if (fsl_edma->txirq == fsl_edma->errirq) { > - devm_free_irq(&pdev->dev, fsl_edma->txirq, fsl_edma); > - } else { > - devm_free_irq(&pdev->dev, fsl_edma->txirq, fsl_edma); > - devm_free_irq(&pdev->dev, fsl_edma->errirq, fsl_edma); > - } > -} > - > static void fsl_disable_clocks(struct fsl_edma_engine *fsl_edma, int nr_clocks) > { > int i; > @@ -427,7 +416,6 @@ static int fsl_edma_remove(struct platform_device *pdev) > struct device_node *np = pdev->dev.of_node; > struct fsl_edma_engine *fsl_edma = platform_get_drvdata(pdev); > > - fsl_edma_irq_exit(pdev, fsl_edma); I would not agree, this code is a failsafe to ensure that irq is quiesced and cant fire again while we are unrolling which also triggers a tasklet We are better of with this change > fsl_edma_cleanup_vchan(&fsl_edma->dma_dev); > of_dma_controller_free(np); > dma_async_device_unregister(&fsl_edma->dma_dev); > -- > 2.34.1
diff --git a/drivers/dma/fsl-edma-main.c b/drivers/dma/fsl-edma-main.c index e40769666e39..16305bb9a2ef 100644 --- a/drivers/dma/fsl-edma-main.c +++ b/drivers/dma/fsl-edma-main.c @@ -216,17 +216,6 @@ fsl_edma2_irq_init(struct platform_device *pdev, return 0; } -static void fsl_edma_irq_exit( - struct platform_device *pdev, struct fsl_edma_engine *fsl_edma) -{ - if (fsl_edma->txirq == fsl_edma->errirq) { - devm_free_irq(&pdev->dev, fsl_edma->txirq, fsl_edma); - } else { - devm_free_irq(&pdev->dev, fsl_edma->txirq, fsl_edma); - devm_free_irq(&pdev->dev, fsl_edma->errirq, fsl_edma); - } -} - static void fsl_disable_clocks(struct fsl_edma_engine *fsl_edma, int nr_clocks) { int i; @@ -427,7 +416,6 @@ static int fsl_edma_remove(struct platform_device *pdev) struct device_node *np = pdev->dev.of_node; struct fsl_edma_engine *fsl_edma = platform_get_drvdata(pdev); - fsl_edma_irq_exit(pdev, fsl_edma); fsl_edma_cleanup_vchan(&fsl_edma->dma_dev); of_dma_controller_free(np); dma_async_device_unregister(&fsl_edma->dma_dev);