[Freedreno] [PATCH 1/4] drm/msm/dsi: convert to msm_clk_get()

2017-10-17 Thread Rob Clark
We already have, as a result of upstreaming the gpu bindings, msm_clk_get() which will try to get the clock both without and with a "_clk" suffix. Use this in DSI code so we can drop the "_clk" suffix in bindings while maintaing backwards compatibility. Signed-off-by: Rob Clark --- drivers/gpu/

[Freedreno] [PATCH 2/4] drm/msm/edp: convert to msm_clk_get()

2017-10-17 Thread Rob Clark
We already have, as a result of upstreaming the gpu bindings, msm_clk_get() which will try to get the clock both without and with a "_clk" suffix. Use this in eDP code so we can drop the "_clk" suffix in bindings while maintaing backwards compatibility. Signed-off-by: Rob Clark --- drivers/gpu/

[Freedreno] [PATCH 3/4] drm/msm/hdmi: convert to msm_clk_get()

2017-10-17 Thread Rob Clark
We already have, as a result of upstreaming the gpu bindings, msm_clk_get() which will try to get the clock both without and with a "_clk" suffix. Use this in HDMI code so we can drop the "_clk" suffix in bindings while maintaing backwards compatibility. Signed-off-by: Rob Clark --- drivers/gpu

[Freedreno] [PATCH 0/4] drm/msm: msm_clk_get() all the things

2017-10-17 Thread Rob Clark
Use our msm_clk_get() helper in more places so "_clk" suffix in clock names is optional. This was added when upstreaming the GPU bindings in response to comments that clock names should not have the "_clk" suffix, and will fall back to looking up the clock with the suffix for backwards compat with

[Freedreno] [PATCH 4/4] dt-bindings: display: msm: update clk names

2017-10-17 Thread Rob Clark
Now that drm/msm is converted over to use msm_get_clk() everywhere (that matters), which handles falling back to looking for a clock with the "_clk" suffix, we can remove "_clk" from the documentation so that new dts files added do not include "_clk" in the name. Previously we were doing this for

[Freedreno] [PATCH] drm/msm/dsi: workaround for display enabled by bootloader

2017-10-17 Thread Rob Clark
Bootloader enabled display, when the driver is built-in (rather than a module loaded after CCF/genpd disable "unused" clocks/powerdomains) causes problems since the driver thinks the clocks are off, but in fact they are on. This causes (for example) clk_set_rate() to fail. A better solution would

Re: [Freedreno] [PATCH] drm/msm/dsi: workaround for display enabled by bootloader

2017-10-17 Thread Eric Anholt
Rob Clark writes: > Bootloader enabled display, when the driver is built-in (rather than a > module loaded after CCF/genpd disable "unused" clocks/powerdomains) > causes problems since the driver thinks the clocks are off, but in fact > they are on. This causes (for example) clk_set_rate() to fa

Re: [Freedreno] [PATCH] drm/msm/dsi: workaround for display enabled by bootloader

2017-10-17 Thread Rob Clark
On Tue, Oct 17, 2017 at 4:16 PM, Eric Anholt wrote: > Rob Clark writes: > >> Bootloader enabled display, when the driver is built-in (rather than a >> module loaded after CCF/genpd disable "unused" clocks/powerdomains) >> causes problems since the driver thinks the clocks are off, but in fact >>