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=-15.8 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,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 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 2B130C433DB for ; Thu, 7 Jan 2021 12:54:47 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 9B3772336D for ; Thu, 7 Jan 2021 12:54:46 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9B3772336D 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-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=jM0sopYTDQDYp1b32QRIJIQO7guKx6SDR2/r4Q5TmKI=; b=zm979GOcoY1zlwyZ4f9L8+rQf TViC+EFCfKGqAHq4CdwPXAsNtP2r2e5wJwvQ1SnWSnMsDKY5y5x3vSyCo+J+x+Kyv5MqBArrSurko 9lp29kGYSy0MP3S9lLuzZqBEZlwA3PON6Y368338Fsup09lRdeVPKvXjF8oLXRx+WPBgbesdiDa+I wLQ4FvrO6Z1fJjgv0DSEBt8SIrPFA/91Geq1Wr7kEFAJep2xF7TWKDIhCU3gGfVDttVr2sUBLU1u+ ZxDdqFhxXzQvNkhl9C40wnx1VDGxO4OkEkukQcyN2YC4IsDrzTP06Kn/4WT5kD2jXT0Q8mLoupZvH 2bwuh2fOw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kxUo9-0005uF-5d; Thu, 07 Jan 2021 12:54:29 +0000 Received: from linux.microsoft.com ([13.77.154.182]) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kxUgB-0001ev-In; Thu, 07 Jan 2021 12:46:20 +0000 Received: from [192.168.0.114] (unknown [49.207.211.144]) by linux.microsoft.com (Postfix) with ESMTPSA id C5DF420B7192; Thu, 7 Jan 2021 04:46:02 -0800 (PST) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com C5DF420B7192 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1610023571; bh=oRvau2/egDUl+G19e+mKJcct9N7ThAjp83e0jgPJfTU=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=ElN2p/YLxSOxarXXe6EodJy3spbth4RctmOJskoyN8cmf+kXe44AeZXr/K7pWKgss jHEcRvV984of41rQRvStOiGhwCs9ciGhHnZEB7KeHM+VxG8NROIKn+aUowmt2vjSJX zmp7qqUgujmS+rW2cP1cOdEmYc9MZHqV4iaK9orY= Subject: Re: [PATCH v2 16/19] crypto: rockchip: convert tasklets to use new tasklet_setup() API To: Emil Renner Berthing , Allen Pais References: <20210107122944.16363-1-allen.lkml@gmail.com> <20210107122944.16363-13-allen.lkml@gmail.com> From: Allen Pais Message-ID: <546446ed-27e0-e74d-f290-cd8c9097b1ca@linux.microsoft.com> Date: Thu, 7 Jan 2021 18:15:56 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210107_074615_976354_41B99056 X-CRM114-Status: GOOD ( 19.81 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: giovanni.cabiddu@intel.com, alexandre.belloni@bootlin.com, aymen.sghaier@nxp.com, Heiko Stuebner , schalla@marvell.com, jamie@jamieiles.com, jesper.nilsson@axis.com, linux-samsung-soc@vger.kernel.org, herbert@gondor.apana.org.au, horia.geanta@nxp.com, k.konieczny@samsung.com, qat-linux@intel.com, krzk@kernel.org, "open list:ARM/Rockchip SoC..." , ludovic.desroches@microchip.com, thomas.lendacky@amd.com, arno@natisbad.org, vz@mleia.com, gilad@benyossef.com, gcherian@marvell.com, linux-mediatek@lists.infradead.org, lars.persson@axis.com, matthias.bgg@gmail.com, john.allen@amd.com, bbrezillon@kernel.org, nicolas.ferre@microchip.com, linux-crypto@vger.kernel.org, Romain Perier , "David S . Miller" Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org > > On Thu, 7 Jan 2021 at 13:32, Allen Pais wrote: >> >> From: Allen Pais >> >> In preparation for unconditionally passing the >> struct tasklet_struct pointer to all tasklet >> callbacks, switch to using the new tasklet_setup() >> and from_tasklet() to pass the tasklet pointer explicitly. >> >> Signed-off-by: Romain Perier >> Signed-off-by: Allen Pais >> --- >> drivers/crypto/rockchip/rk3288_crypto.c | 14 ++++++-------- >> 1 file changed, 6 insertions(+), 8 deletions(-) >> >> diff --git a/drivers/crypto/rockchip/rk3288_crypto.c b/drivers/crypto/rockchip/rk3288_crypto.c >> index 35d73061d156..af6ad9f49009 100644 >> --- a/drivers/crypto/rockchip/rk3288_crypto.c >> +++ b/drivers/crypto/rockchip/rk3288_crypto.c >> @@ -201,9 +201,9 @@ static int rk_crypto_enqueue(struct rk_crypto_info *dev, >> return ret; >> } >> >> -static void rk_crypto_queue_task_cb(unsigned long data) >> +static void rk_crypto_queue_task_cb(struct tasklet_struct *T) >> { >> - struct rk_crypto_info *dev = (struct rk_crypto_info *)data; >> + struct rk_crypto_info *dev = from_tasklet(dev, t, queue_task); > > This doesn't look right. First it's called T and then t. I wonder how > this even compiles.. > Thanks for catching it. I am not sure how I missed it and it even compiled. I will fix it. Thanks. >> struct crypto_async_request *async_req, *backlog; >> unsigned long flags; >> int err = 0; >> @@ -231,9 +231,9 @@ static void rk_crypto_queue_task_cb(unsigned long data) >> dev->complete(dev->async_req, err); >> } >> >> -static void rk_crypto_done_task_cb(unsigned long data) >> +static void rk_crypto_done_task_cb(struct tasklet_struct *t) >> { >> - struct rk_crypto_info *dev = (struct rk_crypto_info *)data; >> + struct rk_crypto_info *dev = from_tasklet(dev, t, done_task); >> >> if (dev->err) { >> dev->complete(dev->async_req, dev->err); >> @@ -389,10 +389,8 @@ static int rk_crypto_probe(struct platform_device *pdev) >> crypto_info->dev = &pdev->dev; >> platform_set_drvdata(pdev, crypto_info); >> >> - tasklet_init(&crypto_info->queue_task, >> - rk_crypto_queue_task_cb, (unsigned long)crypto_info); >> - tasklet_init(&crypto_info->done_task, >> - rk_crypto_done_task_cb, (unsigned long)crypto_info); >> + tasklet_setup(&crypto_info->queue_task, rk_crypto_queue_task_cb); >> + tasklet_setup(&crypto_info->done_task, rk_crypto_done_task_cb); >> crypto_init_queue(&crypto_info->queue, 50); >> >> crypto_info->enable_clk = rk_crypto_enable_clk; >> -- >> 2.25.1 >> >> >> _______________________________________________ >> Linux-rockchip mailing list >> Linux-rockchip@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-rockchip _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek