Re: [PATCH v3 05/14] drm: bridge/analogix_dp: fix link_rate lane_count bug

2015-08-20 Thread Yakir Yang
Hi Jingoo, On 08/20/2015 02:22 AM, Jingoo Han wrote: On 2015. 8. 19., at PM 11:50, Yakir Yang y...@rock-chips.com wrote: link_rate and lane_count already configed in analogix_dp_set_link_train(), s/configed/configured Also, the commit name such as fix ... bug is not good. How about

Re: [PATCH v3 05/14] drm: bridge/analogix_dp: fix link_rate lane_count bug

2015-08-20 Thread Jingoo Han
On 2015. 8. 19., at PM 11:50, Yakir Yang y...@rock-chips.com wrote: link_rate and lane_count already configed in analogix_dp_set_link_train(), s/configed/configured Also, the commit name such as fix ... bug is not good. How about following? drm: bridge/analogix_dp: remove duplicate

[PATCH v3 05/14] drm: bridge/analogix_dp: fix link_rate lane_count bug

2015-08-19 Thread Yakir Yang
link_rate and lane_count already configed in analogix_dp_set_link_train(), so we don't need to config those repeatly after training finished, just remove them out. Beside Display Port 1.2 already support 5.4Gbps link rate, the maximum sets would change from {1.62Gbps, 2.7Gbps} to {1.62Gbps,