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 <srinivas.kandagatla at linaro.org> >> --- >> 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)
--srini > Otherwise people could be switching the DTS to the new value, but the > driver won't find the clocks it's looking for. > > Thierry >
