On Tue, 24 Nov 2015, Maarten Lankhorst <[email protected]>
wrote:
> +#define drm_for_each_encoder_mask(encoder, dev, encoder_mask) \
> + list_for_each_entry((encoder), &(dev)->mode_config.encoder_list, head) \
> + if ((encoder_mask) & (1 << drm_encoder_index(encoder)))
How about
if (!((encoder_mask) & (1 << drm_encoder_index(encoder)))); else
to avoid dangling else problems?
This inspired me to write [1].
BR,
Jani.
[1]
http://patchwork.freedesktop.org/patch/msgid/[email protected]
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
