On 10/7/22 09:29, Ville Syrjälä wrote:

[...]

>>      drm_for_each_plane_mask(plane, dev, crtc_state->plane_mask)
>>              if (plane->type == DRM_PLANE_TYPE_PRIMARY)
>>                      return 0;
> 
> That is exactly what it says not to do.
> 

Ah, good to know. I re-read the kernel coding style and see now
that it only applies to single simple statements.

Better then, since not using braces even for single statements
is something that's error prone IMO, as mentioned before.

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

Reply via email to