staging: iio: adc: fix multiple checkpatch issues
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v4 Latest9 patches 10 repliesIn review
- v39 patches 19 repliesSuperseded
- v29 patches 7 repliesSuperseded
Earlier versions · 1
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v1 | 10 patches | 8 replies | Superseded |
Changes in v4 from the cover letter
- reworked patch 2/9 in order to follow kernel coding style
- changed patch 5/9 in order to not break user space
- reworked patch 8/9 because abbreviations are not considered camel case
- changed patch 9/9 by moving block comment above the macro def
Patches in v4 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/9 | staging: iio: adc: use kernel types u32 and u64drivers/ | +15 | 1 reply | |
| 2/9 | staging: iio: adc: properly indent to match open paranthesisdrivers/ drivers/ drivers/ +7 more | +249 | 1 reply | |
| 3/9 | staging: iio: adc: remove explicit comparison to NULLdrivers/ drivers/ drivers/ | +6 | 1 reply | |
| 4/9 | staging: iio: adc: remove space after castdrivers/ drivers/ drivers/ +1 more | +7 | 1 reply | |
| 5/9 | staging: iio: adc: add spaces around binary operatorsdrivers/ | +1 | 1 reply | |
| 6/9 | staging: iio: adc: add blank line after declarationsdrivers/ drivers/ | +2 | 1 reply | |
| 7/9 | staging: iio: adc: remove multiple blank linesdrivers/ drivers/ drivers/ +1 more | +0 | 1 reply | |
| 8/9 | staging: iio: adc: remove the use of CamelCasedrivers/ drivers/ drivers/ | +8 | 1 reply | |
| 9/9 | staging: iio: adc: properly indent block commentsdrivers/ | +25 | 2 replies |
Reviews trailers per patch
8 of 9 patches have a Reviewed-by or Acked-by.
| Patch | Daniel Baluta |
|---|---|
| 1/9 | R-b |
| 2/9 | R-b |
| 3/9 | R-b |
| 4/9 | R-b |
| 5/9 | R-b |
| 6/9 | R-b |
| 7/9 | R-b |
| 8/9 | R-b |
| 9/9 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v4 0/9] staging: iio: adc: fix multiple checkpatch issues
This patchset fixes multiple checkpatch issues.
Changes in v2:
- removed commit 09/10 from previous version since it introduced build warnings
Changes in v3:
- reworked patch 2/9 so that it does not introduce any other checkpatch warnings
- changed the cover letter description in order to keep up with the other updates
Changes in v4:
- reworked patch 2/9 in order to follow kernel coding style
- changed patch 5/9 in order to not break user space
- reworked patch 8/9 because abbreviations are not considered camel case
- changed patch 9/9 by moving block comment above the macro def
Ioana Ciornei (9):
staging: iio: adc: use kernel types u32 and u64
staging: iio: adc: properly indent to match open paranthesis
staging: iio: adc: remove explicit comparison to NULL
staging: iio: adc: remove space after cast
staging: iio: adc: add spaces around binary operators
staging: iio: adc: add blank line after declarations
staging: iio: adc: remove multiple blank lines
staging: iio: adc: remove the use of CamelCase
staging: iio: adc: properly indent block comments
drivers/staging/iio/adc/ad7192.c | 97 ++++++++++-------- drivers/staging/iio/adc/ad7192.h | 2 +- drivers/staging/iio/adc/ad7280a.c | 178 ++++++++++++++++---------------- drivers/staging/iio/adc/ad7606_core.c | 23 +++-- drivers/staging/iio/adc/ad7606_par.c | 15 +-- drivers/staging/iio/adc/ad7606_spi.c | 5 +- drivers/staging/iio/adc/ad7780.c | 13 +-- drivers/staging/iio/adc/ad7816.c | 53 +++++----- drivers/staging/iio/adc/lpc32xx_adc.c | 14 +-- drivers/staging/iio/adc/mxs-lradc.c | 185 +++++++++++++++++++--------------- drivers/staging/iio/adc/spear_adc.c | 4 +- 11 files changed, 313 insertions(+), 276 deletions(-)
Signature · 1 line
2.1.4