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 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/ana

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 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 configuration of link rat

[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, 2.7Gbp