Skip to content

staging: fbtft: Fixes some issues found with checkpatch.pl

Leonardo Brás lkml 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest15 patches 6 repliesIn review
  2. v110 of 11 patches 3 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/15staging: fbtft: Changes udelay(n) to usleep_range(n, n + x). - Styledrivers/staging/fbtft/fb_agm1264k-fl.c drivers/staging/fbtft/fb_ra8875.c drivers/staging/fbtft/fb_tinylcd.c +2 more+15 -161 reply
2/15staging: fbtft: Puts macro arguments in parenthesis to avoid precedence issues - Styledrivers/staging/fbtft/fb_hx8347d.c drivers/staging/fbtft/fb_ili9163.c drivers/staging/fbtft/fb_ili9320.c +7 more+22 -150 replies
3/15staging: fbtft: Fixes some alignment issues - Styledrivers/staging/fbtft/fb_ssd1289.c drivers/staging/fbtft/fb_ssd1351.c drivers/staging/fbtft/fb_st7735r.c +9 more+106 -820 replies
4/15staging: fbtft: Fixes some lines long lines (>80) - Styledrivers/staging/fbtft/fb_ssd1306.c drivers/staging/fbtft/fb_ssd1331.c drivers/staging/fbtft/fb_st7735r.c +3 more+88 -321 reply
5/15staging: fbtft: A bit more information on dev_err.drivers/staging/fbtft/fb_ssd1351.c+4 -20 replies
6/15staging: fbtft: Changes gamma table to define.drivers/staging/fbtft/fbtft_device.c+5 -80 replies
7/15staging: fbtft: Removes one nesting level to help readability - Styledrivers/staging/fbtft/flexfb.c+15 -120 replies
8/15staging: fbtft: Adjust some empty-line problems - Styledrivers/staging/fbtft/fb_s6d02a1.c drivers/staging/fbtft/fb_s6d1121.c drivers/staging/fbtft/fb_ssd1289.c +1 more+2 -30 replies
9/15staging: fbtft: Erases some repetitive usage of function name - Styledrivers/staging/fbtft/fb_sh1106.c drivers/staging/fbtft/fb_ssd1306.c drivers/staging/fbtft/fb_ssd1325.c +3 more+6 -60 replies
10/15staging: fbtft: Fixes some defines styles - Styledrivers/staging/fbtft/fbtft-bus.c drivers/staging/fbtft/fbtft-core.c drivers/staging/fbtft/fbtft.h+12 -82 replies
11/15staging: fbtft: Includes description to mutex and spinlock - Styledrivers/staging/fbtft/fbtft.h+2 -21 reply
12/15staging: fbtft: Add spaces around / - Styledrivers/staging/fbtft/fbtft-core.c+1 -10 replies
13/15staging: fbtft: Corrects long index line - Styledrivers/staging/fbtft/fbtft-sysfs.c+4 -10 replies
14/15staging: fbtft: Replaces (1 << n) for macro BIT(n) - Styledrivers/staging/fbtft/fbtft.h+28 -280 replies
15/15staging: fbtft: Replaces custom debug macro with dev_dbg().drivers/staging/fbtft/fbtft.h+5 -121 reply

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [PATCH v2 00/15] staging: fbtft: Fixes some issues found with checkpatch.pl

Fixes some issues found with checkpatch.pl, and other issues found browsing the code.

Leonardo Brás (15):
staging: fbtft: Changes udelay(n) to usleep_range(n, n + x). - Style
staging: fbtft: Puts macro arguments in parenthesis to avoid
precedence issues - Style
staging: fbtft: Fixes some alignment issues - Style
staging: fbtft: Fixes some lines long lines (>80) - Style
staging: fbtft: A bit more information on dev_err.
staging: fbtft: Changes gamma table to define.
staging: fbtft: Removes one nesting level to help readability - Style
staging: fbtft: Adjust some empty-line problems - Style
staging: fbtft: Erases some repetitive usage of function name - Style
staging: fbtft: Fixes some defines styles - Style
staging: fbtft: Includes description to mutex and spinlock - Style
staging: fbtft: Add spaces around / - Style
staging: fbtft: Corrects long index line - Style
staging: fbtft: Replaces (1 << n) for macro BIT(n) - Style
staging: fbtft: Replaces custom debug macro with dev_dbg().

 drivers/staging/fbtft/fb_agm1264k-fl.c |   2 +-
 drivers/staging/fbtft/fb_hx8347d.c     |   2 +-
 drivers/staging/fbtft/fb_ili9163.c     |   2 +-
 drivers/staging/fbtft/fb_ili9320.c     |   2 +-
 drivers/staging/fbtft/fb_ili9325.c     |   2 +-
 drivers/staging/fbtft/fb_ili9341.c     |   2 +-
 drivers/staging/fbtft/fb_ra8875.c      |   4 +-
 drivers/staging/fbtft/fb_s6d02a1.c     |   1 -
 drivers/staging/fbtft/fb_s6d1121.c     |   3 +-
 drivers/staging/fbtft/fb_sh1106.c      |   2 +-
 drivers/staging/fbtft/fb_ssd1289.c     |   9 +-
 drivers/staging/fbtft/fb_ssd1306.c     |   5 +-
 drivers/staging/fbtft/fb_ssd1325.c     |   2 +-
 drivers/staging/fbtft/fb_ssd1331.c     |   5 +-
 drivers/staging/fbtft/fb_ssd1351.c     |  39 +++++---
 drivers/staging/fbtft/fb_st7735r.c     |  21 +++--
 drivers/staging/fbtft/fb_st7789v.c     |  13 ++-
 drivers/staging/fbtft/fb_tinylcd.c     |   2 +-
 drivers/staging/fbtft/fb_uc1611.c      |   2 +-
 drivers/staging/fbtft/fb_upd161704.c   |  19 ++--
 drivers/staging/fbtft/fb_watterott.c   |  30 +++---
 drivers/staging/fbtft/fbtft-bus.c      |  27 +++---
 drivers/staging/fbtft/fbtft-core.c     |  53 ++++++-----
 drivers/staging/fbtft/fbtft-io.c       |  17 ++--
 drivers/staging/fbtft/fbtft-sysfs.c    |   9 +-
 drivers/staging/fbtft/fbtft.h          | 122 ++++++++++++++-----------
 drivers/staging/fbtft/fbtft_device.c   |  40 ++++----
 drivers/staging/fbtft/flexfb.c         |  45 +++++----
 28 files changed, 271 insertions(+), 211 deletions(-)
Signature · 1 line
2.18.0