Re: [PATCH 0/2] Trivial double space style fix

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 07 Aug 2024, Andi Shyti <andi.shyti@xxxxxxxxxxxxxxx> wrote:
> Hi,
>
> I was bothered a few times to see the double space in the
> parameters list so that I decided to replace once and for all all
> the double spaces that are not added on purpose to maintain some
> visual alignment in the code.
>
> Checkpatch doesn't have any rules for that because, indeed, this
> is used also to put in columns parameters in repetitive function
> calls or declarations. Nevertheless, it doesn't mean it's
> stylistically correct.

Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx>

>
> Thanks,
> Andi
>
> Andi Shyti (2):
>   drm/i915: Replace double blank with single blank after comma in gem/ and gt/
>   drm/i915: Replace double blank with single blank after comma
>
>  drivers/gpu/drm/i915/display/i9xx_wm.c               | 2 +-
>  drivers/gpu/drm/i915/display/intel_ddi.c             | 2 +-
>  drivers/gpu/drm/i915/display/intel_display.c         | 2 +-
>  drivers/gpu/drm/i915/display/intel_display_debugfs.c | 2 +-
>  drivers/gpu/drm/i915/display/intel_dp_hdcp.c         | 2 +-
>  drivers/gpu/drm/i915/gem/i915_gem_object_types.h     | 2 +-
>  drivers/gpu/drm/i915/gt/selftest_migrate.c           | 2 +-
>  drivers/gpu/drm/i915/gt/uc/intel_uc.c                | 2 +-
>  drivers/gpu/drm/i915/gvt/trace.h                     | 2 +-
>  drivers/gpu/drm/i915/i915_driver.c                   | 6 +++---
>  drivers/gpu/drm/i915/i915_sysfs.c                    | 4 ++--
>  drivers/gpu/drm/i915/selftests/intel_memory_region.c | 2 +-
>  12 files changed, 15 insertions(+), 15 deletions(-)

-- 
Jani Nikula, Intel



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux