On 7/29/19 4:58 AM, Liviu Dudau wrote: >> case MW_RESTART: >> drm_writeback_signal_completion(&malidp->mw_connector, 0); >> - /* fall through to a new start */ > > It's a shame that the compiler throws a warning here, it would've been really > useful to keep the hint that going back to a new start is intentional without > having to type another comment. > You don't have to remove the rest of the comment. For a one-line comment this works just fine: /* fall through - to a new start */ -- Gustavo