Re: [PATCH v2 0/5] staging: Fix parameters alignment

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

 



On Mon, Mar 04, 2024 at 12:40:01PM +0000, Dorine Tipo wrote:
> Hello!
> 
> This V2 patch series addresses inconsistencies in parameter alignment with
> open parantheses, enhancing readability and maintainability of the code.
> The patch series includes updates to multiple files in the staging
> directory. Each patch focuses on specific functions or sections of code to
> ensure comprehensive coverage of the alignment improvements.
> 
> Patch 1 fixes the parameter alignment in cvmx_spi_restart_interface() in
> octeon-stubs.h in the octeon driver
> Patches 2-6 fix the alignment of parameters in various functions inside
> card.c in the vt6655 driver
> 
> Changelog:
> - V2: Modified subjects to be unique for each patch, as requested by
>   greg k-h
> - Fixed the warnings in the patches as suggested by Philipp
> 
> 
> Dorine Tipo (5):
>   Staging: octeon: Match parenthesis alignment

Do the octeon patch as a separate patchset.  It's a totally different
driver.

>   staging: vt6655: Correct parameters alignment
>   staging: vt6655: Align parameters with parenthesis
>   staging: vt6655: Match open parenthesis alignment
>   staging: vt6655: Fix open parenthesis alignment

Combine all these patches into one patch.  The rule is "one thing per
patch"  not "one line per patch".

regards,
dan carpenter





[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux