[PATCH] drm/dp: add drm_dp_link_power_down() helper

2014-12-03 Thread Jani Nikula
On Tue, 02 Dec 2014, Rob Clark wrote: > We had _power_up(), but drivers also need to be able to power down. Patch looks good, I'm just hijacking the thread to talk about the _power_up() counterpart. Sorry. ;) First, I'm not sure it's all right or sensible to read DP_SET_POWER first when the sink

[PATCH] drm/dp: add drm_dp_link_power_down() helper

2014-12-02 Thread Thierry Reding
On Tue, Dec 02, 2014 at 10:46:42AM -0500, Rob Clark wrote: > We had _power_up(), but drivers also need to be able to power down. > > Signed-off-by: Rob Clark > --- > This lets me drop edp_sink_power_state() from msm eDP code, and use > the helpers instead. > > drivers/gpu/drm/drm_dp_helper.c |

[PATCH] drm/dp: add drm_dp_link_power_down() helper

2014-12-02 Thread Alex Deucher
On Tue, Dec 2, 2014 at 10:46 AM, Rob Clark wrote: > We had _power_up(), but drivers also need to be able to power down. > > Signed-off-by: Rob Clark Reviewed-by: Alex Deucher > --- > This lets me drop edp_sink_power_state() from msm eDP code, and use > the helpers instead. > > drivers/gpu/drm

[PATCH] drm/dp: add drm_dp_link_power_down() helper

2014-12-02 Thread Rob Clark
We had _power_up(), but drivers also need to be able to power down. Signed-off-by: Rob Clark --- This lets me drop edp_sink_power_state() from msm eDP code, and use the helpers instead. drivers/gpu/drm/drm_dp_helper.c | 31 +++ include/drm/drm_dp_helper.h | 1 +