On Tue, 2023-05-02 at 17:39 +0300, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Split some overly long lines. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_fdi.c | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) It doesn't matter much, but it would be nice to say where you're splitting the long lines? If nothing else, it would at least make the commit message more unique: "drm/i915: Split some long lines in hsw_fdi_link_train()" Other than this nitpick: Reviewed-by: Luca Coelho <luciano.coelho@xxxxxxxxx> -- Cheers, Luca.