On Thu, Jun 2, 2022 at 10:16 PM Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote: > > Hi all, > > In commit > > 8caad14e7224 ("drm/msm/dpu: Fix pointer dereferenced before checking") > > Fixes tag > > Fixes: d7d0e73f7de33 ("drm/msm/dpu: introduce the dpu_encoder_phys_* for > > has these problem(s): > > - Subject has leading but no trailing parentheses > - Subject has leading but no trailing quotes > > Fixes tags should not be truncated or split across more than 1 line. So: > > Fixes: d7d0e73f7de3 ("drm/msm/dpu: introduce the dpu_encoder_phys_* for writeback") Hmm, checkpatch seems to not catch this (unless it was a recent addition that landed after what msm-fixes is based on. will the truncated subject confuse the scripts that look for patches to backport to stable, ie. do we *really* have to rewrite history to fix this? BR, -R