Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753605AbbHJNmz (ORCPT ); Mon, 10 Aug 2015 09:42:55 -0400 Received: from mail-wi0-f171.google.com ([209.85.212.171]:34577 "EHLO mail-wi0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752277AbbHJNmx (ORCPT ); Mon, 10 Aug 2015 09:42:53 -0400 Message-ID: <55C8AA54.7040806@linaro.org> Date: Mon, 10 Aug 2015 14:42:44 +0100 From: Srinivas Kandagatla User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Thierry Reding CC: dri-devel@lists.freedesktop.org, robdclark@gmail.com, David Airlie , linux-kernel@vger.kernel.org, Rob Herring , linux-arm-msm@vger.kernel.org Subject: Re: [PATCH RFC 4/5] drm/msm/hdmi: deprecate non standard clock-names References: <1439207923-30812-1-git-send-email-srinivas.kandagatla@linaro.org> <1439207989-30986-1-git-send-email-srinivas.kandagatla@linaro.org> <20150810124947.GD1262@ulmo.nvidia.com> <55C8A497.9050600@linaro.org> <20150810133350.GC7302@ulmo.nvidia.com> In-Reply-To: <20150810133350.GC7302@ulmo.nvidia.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3129 Lines: 77 On 10/08/15 14:33, Thierry Reding wrote: > On Mon, Aug 10, 2015 at 02:18:15PM +0100, Srinivas Kandagatla wrote: >> >> >> On 10/08/15 13:49, Thierry Reding wrote: >>> On Mon, Aug 10, 2015 at 12:59:49PM +0100, Srinivas Kandagatla wrote: >>>> This patch updates the bindings to discourage the usage of non standard >>>> clock names, this will help in projects focused on upstreaming. >>>> >>>> These deprecated properties are still supported but will be remove over >>>> the time. >>>> >>>> Signed-off-by: Srinivas Kandagatla >>>> --- >>>> Documentation/devicetree/bindings/drm/msm/hdmi.txt | 16 ++++++++-------- >>>> 1 file changed, 8 insertions(+), 8 deletions(-) >>>> >>>> diff --git a/Documentation/devicetree/bindings/drm/msm/hdmi.txt b/Documentation/devicetree/bindings/drm/msm/hdmi.txt >>>> index 6dc202e..6fbfdd8 100644 >>>> --- a/Documentation/devicetree/bindings/drm/msm/hdmi.txt >>>> +++ b/Documentation/devicetree/bindings/drm/msm/hdmi.txt >>>> @@ -12,16 +12,16 @@ Required properties: >>>> - clocks: device clocks >>>> - clock-names: Corresponding name for each entry in the clocks property. >>>> for "qcom,hdmi-tx-8960" compatible names should be >>>> - "core_clk" >>>> - "master_iface_clk" >>>> - "slave_iface_clk" >>>> + "core_clk" is deprecated, use "core" instead >>>> + "master_iface_clk" is deprecated, use "master_iface" instead >>>> + "slave_iface_clk" is deprecated, use "slave_iface" instead >>>> >>>> for "qcom,hdmi-tx-8084" and "qcom,hdmi-tx-8074" compatible names should be >>>> - "extp_clk" >>>> - "alt_iface_clk" >>>> - "iface_clk" >>>> - "core_clk" >>>> - "mdp_core_clk" >>>> + "extp_clk" is deprecated, use "extp" instead >>>> + "alt_iface_clk" is deprecated, use "alt_iface" intstead >>>> + "iface_clk" is deprecated, use "iface" instead >>>> + "core_clk" is deprecated, use "core" instead >>>> + "mdp_core_clk" is deprecated, use "mdp_core" instead >>> >>> Shouldn't there be a driver counterpart of this to accept the new names? >> Driver changes are in this same series "[PATCH RFC 5/5] drm/msm/hdmi: remove >> _clk suffix from clock names"(https://lkml.org/lkml/2015/8/10/453) > > I don't have that patch in my inbox. It looks to be doing things > backwards (look up the deprecated name first). I think it should be: > > clk = devm_clk_get(dev, id); > if (IS_ERR(clk)) { If the clock controller is not ready yet, it would return EPROBE DEFER, which gets dropped here, as a result the driver would not be probed again. Probably both of the error codes needs be checked before returning. > char clk_name[32]; > > snprintf(clk_name, sizeof(clk_name), "%s_clk", id); > clk = devm_clk_get(dev, clk_name); > if (IS_ERR(clk)) > return clk; > } > > Also note how I've dropped the ERR_CAST(), that's not useful here > because you aren't actually casting, but simply returning clk. > > Thierry > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/