Re: [PATCH] drm/gma500/oaktrail_lvds: replace continue with break

On Fri, Jun 18, 2021 at 8:35 PM Colin King colin.king@canonical.com wrote:

From: Colin Ian King colin.king@canonical.com

Currently a loop scans through the connector list checking for connectors that do not match a specific criteria. The use of the continue statement is a little unintuitive and can confuse static analysis checking. Invert the criteria matching logic and use a break to terminate the loop once the first suitable connector has been found.

Thanks to Patrik Jakobsson for explaining the original intent of the code and suggesting this change.

Applied to drm-misc-next

Thanks for the patch!