Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp933891pxk; Thu, 10 Sep 2020 02:31:18 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz0hoZQtxe41yuZwNUK5vGgof4Se9MvUNfQEadDdUzEpT1Sh8cStZ060EjR73awCWtJoqGQ X-Received: by 2002:a17:906:e113:: with SMTP id gj19mr8000504ejb.263.1599730278613; Thu, 10 Sep 2020 02:31:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1599730278; cv=none; d=google.com; s=arc-20160816; b=gO0iggNytCWtoOVIE27oHGMsLqQ0miIDBnHuJRca6b0h16lLALeBZ38xape+eovaCt ZBp+Y+ufy55byOMuVfw33fETG36riXxLf9oMjIG9qdBfbGI23RiNO47X/2ZbC1gBnLDo 2b9IFUSEoIv0e/l+rb+b8pZTwogoadjo4U39Q7bqRXjX7W2oRT1OsSYbMjvoqTXfwJJm yRSLztQtII5ZYluNGTRcAZX4+HRpuUZ/vsJ+3n8qdhCO6JCWo6mSbXYYDYlhPReqRFmD hKvWwPB9K1MskS+MtcWiiMc3AUGrqLMIHzED/P8tke4CUxKSDJBo8dLkp0lc8TcjsCzC 12eQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:in-reply-to :mime-version:user-agent:date:message-id:from:cc:references:to :subject; bh=KzJ8/nrJvQUcnWUpCLuJyZQCNYPA2MJS1xbHHax8L/k=; b=zwQ0wCjRKwBgj2C7i89IEpr9JclblstjxWdrGepJjzEreBvZWfeM1x5hEuXJm4W1jt 0Mp1wh/T4LaSiez7uLNOtcuY69fVdRy3IP6dxWbOqfPXTQTtVO7T2Qze+8ZXo2kaKo+d +bMZo0Fvhu+1+9gvJZlBGC6pHDCSK32MgC9TVbl7v8MOcBmiryviyEbjOBKj9VOhCCbw cQbti4DFbSDyMmpjbLYY1eOPtj2iedVhe5VKf34nM6h2MezAqqTc6VV8QLDpgV2xT9M2 TYOBWTTvLWkMvVDTJOAbmCVgBjUZIS6u+FYXO/ohM+yn4gXScAe9jeeOg93Pf30NQZBD BeFw== ARC-Authentication-Results: i=1; mx.google.com; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id b19si3120364ejd.405.2020.09.10.02.30.56; Thu, 10 Sep 2020 02:31:18 -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; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730077AbgIJJaV (ORCPT + 99 others); Thu, 10 Sep 2020 05:30:21 -0400 Received: from szxga06-in.huawei.com ([45.249.212.32]:54732 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726760AbgIJJ3G (ORCPT ); Thu, 10 Sep 2020 05:29:06 -0400 Received: from DGGEMS406-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id 3A488CCA989BC0EEEE23; Thu, 10 Sep 2020 17:29:00 +0800 (CST) Received: from [10.174.178.248] (10.174.178.248) by smtp.huawei.com (10.3.19.206) with Microsoft SMTP Server (TLS) id 14.3.487.0; Thu, 10 Sep 2020 17:28:52 +0800 Subject: Re: [PATCH] clk: qcom: lpass: Correct goto target in lpass_core_sc7180_probe() To: Stephen Boyd , , , , References: <20200827141629.101802-1-jingxiangfeng@huawei.com> <159972862952.2295844.10882678924928944990@swboyd.mtv.corp.google.com> CC: , , From: Jing Xiangfeng Message-ID: <5F59F1D3.50905@huawei.com> Date: Thu, 10 Sep 2020 17:28:51 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: <159972862952.2295844.10882678924928944990@swboyd.mtv.corp.google.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.174.178.248] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2020/9/10 17:03, Stephen Boyd wrote: > Quoting Jing Xiangfeng (2020-08-27 07:16:29) >> lpass_core_sc7180_probe() misses to call pm_clk_destroy() and >> pm_runtime_disable() in error paths. Correct goto target to fix it. >> This issue is found by code inspection. >> >> Signed-off-by: Jing Xiangfeng >> --- > > HMm.. presumably > > Fixes: edab812d802d ("clk: qcom: lpass: Add support for LPASS clock controller for SC7180") > > should be added? Ok, I will send a v2 with it. Thanks > . >