From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Subject: Re: [PATCH -next] ASoC: tegra20-ac97: add missing clk_disable_unprepare() on error path Date: Mon, 16 Dec 2013 10:12:08 -0700 Message-ID: <52AF3468.2050203@wwwdotorg.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from avon.wwwdotorg.org (avon.wwwdotorg.org [70.85.31.133]) by alsa0.perex.cz (Postfix) with ESMTP id D2EBA261729 for ; Mon, 16 Dec 2013 18:12:12 +0100 (CET) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Wei Yongjun , lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.de, thierry.reding@gmail.com, grant.likely@linaro.org, robh+dt@kernel.org Cc: linux-tegra@vger.kernel.org, yongjun_wei@trendmicro.com.cn, alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org On 12/13/2013 10:29 PM, Wei Yongjun wrote: > From: Wei Yongjun > > Add the missing clk_disable_unprepare() before return from > tegra20_ac97_platform_probe() in the error handling case. Reviewed-by: Stephen Warren Mark, I did check that this patch doesn't conflict with any of the ASoC changes I have in the Tegra tree, so there should be no issue when you apply it. The same goes for the previous patch from Wei too, which fixes some return values from this same function.