Received: by 2002:a25:31c3:0:0:0:0:0 with SMTP id x186csp3905016ybx; Mon, 4 Nov 2019 04:48:45 -0800 (PST) X-Google-Smtp-Source: APXvYqxXXDjjfziepSdSqKI4tbI09RFweE2wXms48RFgjT0Yv/fBl5hv7uQC+BT/JB78quor0V32 X-Received: by 2002:a50:f987:: with SMTP id q7mr26270811edn.205.1572871725092; Mon, 04 Nov 2019 04:48:45 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1572871725; cv=none; d=google.com; s=arc-20160816; b=KHD1uUIkI6zaI6uQxY8CHoG2/CCH08dKLCQ05eeTjAMXACRRy1yXK7tgcrH7bFCERa DUtz44tUe7k6XVVsx1c+o4c0zVxsqnkge9Qx1nxBYfSR+uzP6ANHkiXItnvS7cW9o/vy pT1wD3aH7ZbQ1G1myjVtbKhdrasSbO8p/LX6m9qSln6N3kBEGNRNhd+rzXfgZ7aTNs/h DeSyVF1T978BmsZMvZ4iADmcrUBloXFUIQ8FO/uUatrgxwWamEyBZb2jksQ9P2/JPsVa 7G5cDnkwtZFzXsLV1xrTC/9xpFZGbLQtoELZrzmqsXRvuwEVJJ4EOkxraFWDpzJjnQp6 lPRg== 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=nQW/bkzEewczaxT+3uYr2EI926apxCfUzfvK6W9flNM=; b=e8fcMupJPuOEXddc0yN0HiIDli6emSUPKR8MXVwAwZmpRImcmJj2jndgDNXC6eq05X Q+7ReMxu9mvaZTdKhEURwBun2VorZOzcaKn0DvShKQvqvn417295G5KxwMFGayZpcyC1 pbCrfiWN5/iknBQdp+/u5ENBOdDvEGvyL6avXd51ZUb7N9ZPxke3r6jPcjuAUoFQCKgj hnuklfIp56izwmoSiBRCa7XMSK9iTLcpIQxFwuRHQR0LH7fUGzhTapAOvNGSMcDTzJsU rk/7kNPfQi6UBg0PPqqzy5FN+L3zvjvuRBtUCfebWdmhQ/MDg/CXXwLRfNSWRYmyLkyI kQ9w== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=eX9YNLda; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id z44si7864349edz.211.2019.11.04.04.48.21; Mon, 04 Nov 2019 04:48:45 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=fail header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=eX9YNLda; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728955AbfKDMrX (ORCPT + 99 others); Mon, 4 Nov 2019 07:47:23 -0500 Received: from heliosphere.sirena.org.uk ([172.104.155.198]:46862 "EHLO heliosphere.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728396AbfKDMrX (ORCPT ); Mon, 4 Nov 2019 07:47:23 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sirena.org.uk; s=20170815-heliosphere; h=In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=nQW/bkzEewczaxT+3uYr2EI926apxCfUzfvK6W9flNM=; b=eX9YNLdaHWXApFC5WH3YUJSoz duye5V5aJxyEa0X2sRrWdWgQsDTo4uhgIwg2UK8SaMVGp11RGSodOlIl07eRtejb2BzATT5SOEPtv lq9Hkne174NX0ErvZCPnbeJU5n9yGScoQnxfQfhkJYoxb2ct+DWNG8cLD4lKkgleLNPE4=; Received: from cpc102320-sgyl38-2-0-cust46.18-2.cable.virginm.net ([82.37.168.47] helo=ypsilon.sirena.org.uk) by heliosphere.sirena.org.uk with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1iRblP-0002Yo-Ij; Mon, 04 Nov 2019 12:47:19 +0000 Received: by ypsilon.sirena.org.uk (Postfix, from userid 1000) id F2AE1274301E; Mon, 4 Nov 2019 12:47:18 +0000 (GMT) Date: Mon, 4 Nov 2019 12:47:18 +0000 From: Mark Brown To: Palmer Dabbelt Cc: hslester96@gmail.com, Paul Walmsley , linux-spi@vger.kernel.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] spi: sifive: disable clk when probe fails and remove Message-ID: <20191104124718.GC5238@sirena.co.uk> References: <20191101121745.13413-1-hslester96@gmail.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="96YOpH+ONegL0A3E" Content-Disposition: inline In-Reply-To: X-Cookie: This page intentionally left blank. User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --96YOpH+ONegL0A3E Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Nov 01, 2019 at 04:07:29PM -0700, Palmer Dabbelt wrote: > On Fri, 01 Nov 2019 05:17:45 PDT (-0700), hslester96@gmail.com wrote: > > The driver forgets to disable and unprepare clk when probe fails and > > remove. > > Add the calls to fix the problem. > Reviewed-by: Palmer Dabbelt Please delete unneeded context from mails when replying. Doing this makes it much easier to find your reply in the message, helping ensure it won't be missed by people scrolling through the irrelevant quoted material. --96YOpH+ONegL0A3E Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAl3AHdYACgkQJNaLcl1U h9Bxbwf+PS/3tt1r3nRdDUeTgiw12asFTlFdosGPaCdJ0zCp0XBMPgFZBia9R16z QBrOHOQ5BL+W7FEEOh9hfPpvdVPzgygwAip2n2uHEWrOZ7pt7PWoGj2ybu0JDXTu rPhwWNvY36hCVKElOiixQeR43pU7yKJZzPoWVTxJgGarWaE03ngE0jEvQUqtsVMB aI0LNK076Dt9geC7wg5FFSppQTVv6c9PvpTehaS5hKvY00hfUS3i9uFgYsXjuggz TYsKknIrHD0gm0Q1bCfGK7u6C3YkbyPs6uj1j2xZLh5isMgVokBkrlpkbpMtKl+i PPxqmUKox9ymgSDfmnZ1yGneuS3qVg== =2j2y -----END PGP SIGNATURE----- --96YOpH+ONegL0A3E--