Skip to content

Fix sparse/smatch errors on non-x86 drivers

Mauro Carvalho Chehab linux-media 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest21 patches 6 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/21media: davinci_vpfe: remove useless checks from ipipedrivers/staging/media/davinci_vpfe/dm365_ipipe.c drivers/staging/media/davinci_vpfe/dm365_ipipe_hw.c+7 -300 replies
2/21media: dm365_ipipe: remove an unused vardrivers/staging/media/davinci_vpfe/dm365_ipipe.c+0 -20 replies
3/21media: davinci_vpfe: fix vpfe_ipipe_init() error handlingdrivers/staging/media/davinci_vpfe/dm365_ipipe.c+8 -82 replies
4/21media: davinci_vpfe: mark __iomem as suchdrivers/staging/media/davinci_vpfe/dm365_ipipeif.c+6 -60 replies
5/21media: davinci_vpfe: get rid of an unused var at dm365_isif.cdrivers/staging/media/davinci_vpfe/dm365_isif.c+2 -70 replies
6/21media: davinci_vpfe: vpfe_video: remove an unused vardrivers/staging/media/davinci_vpfe/vpfe_video.c+1 -30 replies
7/21media: davinci_vpfe: don't use kernel-doc markup for simple commentsdrivers/staging/media/davinci_vpfe/vpfe_mc_capture.c+1 -10 replies
8/21media: davinci_vpfe: fix a typo for "default"drivers/staging/media/davinci_vpfe/dm365_resizer.c+2 -20 replies
9/21media: davinci_vpfe: cleanup ipipe_[g|s]_config logicdrivers/staging/media/davinci_vpfe/dm365_ipipe.c+50 -480 replies
10/21media: davinci_vpfe: fix __user annotationsdrivers/staging/media/davinci_vpfe/dm365_ipipe.c drivers/staging/media/davinci_vpfe/dm365_ipipeif.c drivers/staging/media/davinci_vpfe/dm365_resizer.c+12 -140 replies
11/21media: si470x: fix __be16 annotationsdrivers/media/radio/si470x/radio-si470x-i2c.c+3 -30 replies
12/21media: isif: reorder a statement to match coding styledrivers/media/platform/davinci/isif.c+1 -10 replies
13/21media: davinci: fix an inconsistent identdrivers/media/platform/davinci/vpbe_osd.c+3 -20 replies
14/21media: mmp-driver: add needed __iomem marks to power_regsdrivers/media/platform/marvell-ccic/mmp-driver.c+1 -10 replies
15/21media: vpbe_display: properly handle error casedrivers/media/platform/davinci/vpbe_display.c+2 -20 replies
16/21media: vpbe_display: get rid of warningsdrivers/media/platform/davinci/vpbe_display.c+4 -40 replies
17/21media: ispstat: use %p to print the address of a bufferdrivers/media/platform/omap3isp/ispstat.c+2 -22 replies
18/21media: isppreview: fix __user annotationsdrivers/media/platform/omap3isp/isppreview.c+1 -12 replies
19/21media: fsl-viu: use %p to print pointersdrivers/media/platform/fsl-viu.c+6 -110 replies
20/21media: fsl-viu: fix __iomem annotationsdrivers/media/platform/fsl-viu.c+11 -150 replies
21/21media: omap_vout: fix wrong identingdrivers/media/platform/omap/omap_vout.c+7 -80 replies

Reviews trailers per patch

2 of 21 patches have a Reviewed-by or Acked-by.

PatchLaurent Pinchart
1/21
2/21
3/21
4/21
5/21
6/21
7/21
8/21
9/21
10/21
11/21
12/21
13/21
14/21
15/21
16/21
17/21R-b R-b
18/21R-b R-b
19/21
20/21
21/21

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 00/21] Fix sparse/smatch errors on non-x86 drivers

After applying this patch series:

    https://www.mail-archive.com/[email protected]/msg128829.html

Which allows to build all drivers with COMPILE_TEST on x86, my scripts can finally check for errors/warnings on those drivers.

This patch series correct those warnings.

Mauro Carvalho Chehab (21):
media: davinci_vpfe: remove useless checks from ipipe
media: dm365_ipipe: remove an unused var
media: davinci_vpfe: fix vpfe_ipipe_init() error handling
media: davinci_vpfe: mark __iomem as such
media: davinci_vpfe: get rid of an unused var at dm365_isif.c
media: davinci_vpfe: vpfe_video: remove an unused var
media: davinci_vpfe: don't use kernel-doc markup for simple comments
media: davinci_vpfe: fix a typo for "default"
media: davinci_vpfe: cleanup ipipe_[g|s]_config logic
media: davinci_vpfe: fix __user annotations
media: si470x: fix __be16 annotations
media: isif: reorder a statement to match coding style
media: davinci: fix an inconsistent ident
media: mmp-driver: add needed __iomem marks to power_regs
media: vpbe_display: properly handle error case
media: vpbe_display: get rid of warnings
media: ispstat: use %p to print the address of a buffer
media: isppreview: fix __user annotations
media: fsl-viu: use %p to print pointers
media: fsl-viu: fix __iomem annotations
media: omap_vout: fix wrong identing

 drivers/media/platform/davinci/isif.c              |   2 +-
 drivers/media/platform/davinci/vpbe_display.c      |  12 +-
 drivers/media/platform/davinci/vpbe_osd.c          |   5 +-
 drivers/media/platform/fsl-viu.c                   |  43 +++----
 drivers/media/platform/marvell-ccic/mmp-driver.c   |   2 +-
 drivers/media/platform/omap/omap_vout.c            |  15 ++-
 drivers/media/platform/omap3isp/isppreview.c       |   2 +-
 drivers/media/platform/omap3isp/ispstat.c          |   4 +-
 drivers/media/radio/si470x/radio-si470x-i2c.c      |   6 +-
 drivers/staging/media/davinci_vpfe/dm365_ipipe.c   | 131 ++++++++++-----------
 .../staging/media/davinci_vpfe/dm365_ipipe_hw.c    |  19 +--
 drivers/staging/media/davinci_vpfe/dm365_ipipeif.c |  14 +--
 drivers/staging/media/davinci_vpfe/dm365_isif.c    |   9 +-
 drivers/staging/media/davinci_vpfe/dm365_resizer.c |  13 +-
 .../staging/media/davinci_vpfe/vpfe_mc_capture.c   |   2 +-
 drivers/staging/media/davinci_vpfe/vpfe_video.c    |   4 +-
 16 files changed, 121 insertions(+), 162 deletions(-)
Signature · 1 line
2.14.3