Re: [PATCH 1/1] drm: xlnx: zynqmp: Stop the loop at lowest link rate without check

2020-07-29 Thread Hyun Kwon
Hi Daniel, Thanks for the review. On Wed, Jul 29, 2020 at 02:34:16PM -0700, Daniel Vetter wrote: > On Wed, Jul 29, 2020 at 8:21 PM Hyun Kwon wrote: > > > > The loop should exit at the lowest link rate, so break the loop > > at the lowest link rate without check. The check is always true > > beca

Re: [PATCH 1/1] drm: xlnx: zynqmp: Stop the loop at lowest link rate without check

2020-07-29 Thread Daniel Vetter
On Wed, Jul 29, 2020 at 8:21 PM Hyun Kwon wrote: > > The loop should exit at the lowest link rate, so break the loop > at the lowest link rate without check. The check is always true > because lowest link rate is smaller than current one and maximum > of current display. Otherwise, it may be seen

[PATCH 1/1] drm: xlnx: zynqmp: Stop the loop at lowest link rate without check

2020-07-29 Thread Hyun Kwon
The loop should exit at the lowest link rate, so break the loop at the lowest link rate without check. The check is always true because lowest link rate is smaller than current one and maximum of current display. Otherwise, it may be seen as the loop can potentially result in negative array offset.