Re: [PATCH 3/5] gma500: fix a missing break in psb_intel_crtc_mode_set

2022-04-01 Thread Xiaomeng Tong
> On Wed, Mar 30, 2022 at 2:03 PM Xiaomeng Tong wrote: > > > > Instead of exiting the loop as expected when an entry is found, the > > list_for_each_entry() continues until the traversal is complete. It > > could result in multiple 'is_*' flags being set with true mistakely. > > > > To fix this, w

Re: [PATCH 3/5] gma500: fix a missing break in psb_intel_crtc_mode_set

2022-04-01 Thread Patrik Jakobsson
On Wed, Mar 30, 2022 at 2:03 PM Xiaomeng Tong wrote: > > Instead of exiting the loop as expected when an entry is found, the > list_for_each_entry() continues until the traversal is complete. It > could result in multiple 'is_*' flags being set with true mistakely. > > To fix this, when found the

[PATCH 3/5] gma500: fix a missing break in psb_intel_crtc_mode_set

2022-03-30 Thread Xiaomeng Tong
Instead of exiting the loop as expected when an entry is found, the list_for_each_entry() continues until the traversal is complete. It could result in multiple 'is_*' flags being set with true mistakely. To fix this, when found the entry, add a break after the switch statement. Fixes: 89c78134cc