Received: by 2002:a25:868d:0:0:0:0:0 with SMTP id z13csp3643662ybk; Tue, 19 May 2020 09:29:55 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy3zsUTurZBGcLiYL4HzvQ1xCH3LSXdMsxvhdB3w5cUQvYpo/e4HOJXi8HRrhPpovZmreCm X-Received: by 2002:a17:907:264b:: with SMTP id ar11mr28753ejc.133.1589905795594; Tue, 19 May 2020 09:29:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1589905795; cv=none; d=google.com; s=arc-20160816; b=Rw8RxtGVf0xLBVM1AEdbLmS8/+F+heIU8SMsOFHXYKvBH2Mf9KBoPxY3NtB87Owreu cYTfBkR41YCeqTmTRUZRA33t10Ap7/vqXPT+W/1RRiYJ992sWNcioewtthKra5knfr1l 2JZTTYmdWsH+dMMrA6L0BJZ+npQprM+C+zNFbGPkji47F3DFoiTgFABLNgca++SpY0W8 ce5NiHtabcPelVpi/5eAxxU8Cd0+rR8yqljxcrscHysauxajFZpygJVU6qpwRbr9OWNn eq8TpF8cr7Ef6Gv2x/yjZECEoSoCntxhc8jvJpF6/qWWsJArrFuTmXlydvAfhqcaissZ QF4A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=nM06eYScX+XNokoAV7+9FvMc5yktmL9LEFe0oZB9L9Y=; b=mRbxjC6QiUVn0zoxLuke+jIhve6p2PzTCiMxjX8ooaGo1qu5z4tmhhNNlhNeiyOkEL jpsSczvvhzZuaSGPTRjJCmg/SfInZMMHdJGQlGBsb5jS7u/Isf0qhzOABxRZB06h+6cm 348+/+GW7AeNzf7fxoxzC/QvGZsQia3uV28C9n4sF1yXd3F5yQPGq/FvAcpKmgjFl775 n5MOZ/nis89eDQ9wDf4sv4tFjD/y5BmFKkQ8M8sgeF8w5vJUeq8bBrytiyW1lEnBMMJ+ fhxXaSdr5vAgEThN40YC5qzUqxExhhH5PDmbbHGTGXX6q5+YtZ+1Z1PbRcRoLN84pZ6s NgCA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=KuOCfHiL; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id be13si6310601edb.146.2020.05.19.09.29.31; Tue, 19 May 2020 09:29:55 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=KuOCfHiL; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729335AbgESQZY (ORCPT + 99 others); Tue, 19 May 2020 12:25:24 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46536 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729041AbgESQZX (ORCPT ); Tue, 19 May 2020 12:25:23 -0400 Received: from mail-wr1-x442.google.com (mail-wr1-x442.google.com [IPv6:2a00:1450:4864:20::442]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 21BA6C08C5C0; Tue, 19 May 2020 09:25:23 -0700 (PDT) Received: by mail-wr1-x442.google.com with SMTP id l18so76658wrn.6; Tue, 19 May 2020 09:25:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=nM06eYScX+XNokoAV7+9FvMc5yktmL9LEFe0oZB9L9Y=; b=KuOCfHiLtLqcY80YA+zJqExocYmA8ZcpYRP7snwL65qx/hbvURsliycG9heOxO/fV9 3Fy8c4PR03P7tJFeX6QBgSkJqs9Ox3G8ZiJorzGDmj/Fm28CAqn80qZMPVt6IRhK5NnR XIz5wtA8TfBPLSt0N8XTk3m851XhAsb7wmGNNDnzxVgBLuv+uWSVQeCb4cFcPQmReL0o R4LqUkztD9SW7JXk6ISHBZnchT+LNFVEeoY62cVM18Yh08LQM52fQ8Amxaky1vH5WZgj kUgXE3/gIcS28w2cvagnv5hQjpJskXkFxORXhdE63CaThtyKjsF4N6l8RTMBv8P2LOmA ss4A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=nM06eYScX+XNokoAV7+9FvMc5yktmL9LEFe0oZB9L9Y=; b=qaGsh9aTlJbs21Dhzvzyg3LkSC9QHZszaOvZpYcOWM88laBfxBkR++crbVaa188DmI 7qvmncPxpINhfqCqAnIN6/bCfZmwZ9KkslsZ7pQTr4XWO3theqpWpLTW6rQ/mG9A55+c 139mLREEsCJoLZdMuyC3DtfENahMOkwlMOnr9ysp2wp0d03m0Ntny30kj1U7uId8SiGl XFEC4ARCxFSujDO91hF96l3+KYy2nKp7LbogZ2HodpyCvq01/vyOkU3kc572yFq5v+vQ ueHaPInQEGa3XWkc7EY9Fkl1avntFn66J+xYFhBgizKzN03O4AweEoLd/I7c4HzfHJt9 /6xw== X-Gm-Message-State: AOAM532mt5X0llN6pIzlKw6dGbdPRshdyB81sBh6ONfc+C0JTmj6SEek kf/7Td09/9wgbwtwVenh/RU= X-Received: by 2002:adf:dcc6:: with SMTP id x6mr26271344wrm.126.1589905521923; Tue, 19 May 2020 09:25:21 -0700 (PDT) Received: from localhost (pd9e51079.dip0.t-ipconnect.de. [217.229.16.121]) by smtp.gmail.com with ESMTPSA id l18sm206371wmj.22.2020.05.19.09.25.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 19 May 2020 09:25:20 -0700 (PDT) Date: Tue, 19 May 2020 18:25:19 +0200 From: Thierry Reding To: Christophe JAILLET Cc: ldewangan@nvidia.com, jonathanh@nvidia.com, dan.j.williams@intel.com, vkoul@kernel.org, dmaengine@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: Re: [PATCH] dmaengine: tegra210-adma: Fix an error handling path in 'tegra_adma_probe()' Message-ID: <20200519162519.GE2113674@ulmo> References: <20200516214205.276266-1-christophe.jaillet@wanadoo.fr> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="5xSkJheCpeK0RUEJ" Content-Disposition: inline In-Reply-To: <20200516214205.276266-1-christophe.jaillet@wanadoo.fr> User-Agent: Mutt/1.13.1 (2019-12-14) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --5xSkJheCpeK0RUEJ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, May 16, 2020 at 11:42:05PM +0200, Christophe JAILLET wrote: > Commit b53611fb1ce9 ("dmaengine: tegra210-adma: Fix crash during probe") > has moved some code in the probe function and reordered the error handling > path accordingly. > However, a goto has been missed. >=20 > Fix it and goto the right label if 'dma_async_device_register()' fails, so > that all resources are released. >=20 > Fixes: b53611fb1ce9 ("dmaengine: tegra210-adma: Fix crash during probe") > Signed-off-by: Christophe JAILLET > --- > drivers/dma/tegra210-adma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Acked-by: Thierry Reding --5xSkJheCpeK0RUEJ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAl7ECG8ACgkQ3SOs138+ s6HyGBAAtqw2Y28Sav7FpIgNdlgPeJGBIZ8WuLD4zojjs15UzXAuuHndZNzOR1xL dJSZvlK1T2GFmmtHmzjJJUtng+A9GDFJD8J6UeKOdpy6eLlmvtRT6hCueljrJGOb aKdGNRnFBoDZ/DqBArUILTkTgCdqHY66gacajY7p42DHGr7RxovPr/RNjzblE4Ae 7uA1CEY9Hqs3Xu7P5Aty3UhFh0cubPGCtMRnW9Y9hvyv6XSWIVnzAUwMiPJGBNLg pn3D9WeK970Ef/QEvkRCWNW4mpM1dfiHmIjbjcFRwn1NvuTI4A1MukWzS3/Qbs9u SrLIquFBkk+AtGNF5pPW/RmKOFTq+6eDaMEArLVuN6G6MNrS2xCE2mN2hb1arBAH rV/7Sc+H/FfEWjXtRSnf0JwZmSKB/rObzOyPTAzERwKDNTVU1XDuCWSjlAv9A8e/ 7TLFcFueYmMZO48jvdw1NL4u+v2kf0rWWzcyNBVeukTSHrlNvT+jI50fJ9omb0Bp RqM5CktLjENleIelSb25TSFRjMB27w703QJdRb7Njr/iDw65wrasz+q8dqGCJLJQ AFqb5peekfnhyQcHiYqd9cU+FYXnk+UJNUv3uh1T+JWZM+ec8NbDJe7v1JpTtWIY 2Yw4rtrkVHfL/mqnKkFzPLziFy72mrh7IQ1bHHx9CyTibg9B0dU= =TqZK -----END PGP SIGNATURE----- --5xSkJheCpeK0RUEJ--