OSDN Git Service

drm: xlnx: zynqmp: Use switch - case for link rate downshift
authorHyun Kwon <hyun.kwon@xilinx.com>
Wed, 29 Jul 2020 23:30:45 +0000 (16:30 -0700)
committerHyun Kwon <hyun.kwon@xilinx.com>
Sat, 1 Aug 2020 01:56:10 +0000 (01:56 +0000)
commit70c8b4b8ab3714fb2d075be22aec05ae9c9a6a09
treef72aa230f6f051d2760b30645eb0c2c090adc266
parent2d889db7626d2b6d67f402e2478b11f9a49b44e3
drm: xlnx: zynqmp: Use switch - case for link rate downshift

Use switch - case to downshift from the current link rate. It's a small
loop now, so fine to be replaced with switch - case. With a loop, it is
confusing and hard to follow as reported below.

The patch d76271d22694: "drm: xlnx: DRM/KMS driver for Xilinx ZynqMP
DisplayPort Subsystem" from Jul 7, 2018, leads to the following
static checker warning:

drivers/gpu/drm/xlnx/zynqmp_dp.c:594 zynqmp_dp_mode_configure()
error: iterator underflow 'bws' (-1)-2

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Hyun Kwon <hyun.kwon@xilinx.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Link: https://patchwork.freedesktop.org/patch/msgid/1596065445-4630-1-git-send-email-hyun.kwon@xilinx.com
drivers/gpu/drm/xlnx/zynqmp_dp.c