From: mchehab@s-opensource.com (Mauro Carvalho Chehab)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 00/16] Make all drivers under drivers/media to build with COMPILE_TEST
Date: Thu, 5 Apr 2018 13:54:00 -0400 [thread overview]
Message-ID: <cover.1522949748.git.mchehab@s-opensource.com> (raw)
The current media policy has been for a while to only accept new drivers
that compile with COMPILE_TEST.
However, there are still several drivers under drivers/media that
doesn't build with COMPILE_TEST.
So, this series makes the existing ones also compatible with it.
Not building with COMPILE_TEST is a bad thing, for several reasons.
The main ones is that:
1) the licence the Kernel community has for Coverity only builds for
x86. So, drivers that don't build on such archtecture were likely
never tested by it.
2) That affects my per-patch handling process, with should be quick
enough to not delay my patch handling process. So, I only build for one
architecture (i386).
3) When appliying a patch, I always run two static code analyzers (W=1,
smatch and sparse). Those drivers weren't checked by me. At the end
of the day, that leads to a lower quality check for the drivers that
don't build on i386.
There are two situations on this patch series that proof the lower
quality of those drivers:
- There is a case of a driver that was added broken in 2013. Only two
years later, someone noticed and "fixed" it by markin it as BROKEN!
- 5 patches in this series (about 1/3) are just to fix build issues on
those drivers, most of them due to gcc warnings.
Mauro Carvalho Chehab (16):
omap: omap-iommu.h: allow building drivers with COMPILE_TEST
media: omap3isp: allow it to build with COMPILE_TEST
media: omap3isp/isp: remove an unused static var
media: fsl-viu: mark static functions as such
media: fsl-viu: allow building it with COMPILE_TEST
media: cec_gpio: allow building CEC_GPIO with COMPILE_TEST
media: exymos4-is: allow compile test for EXYNOS FIMC-LITE
media: mmp-camera.h: add missing platform data
media: marvel-ccic: re-enable mmp-driver build
media: mmp-driver: make two functions static
media: davinci: allow building isif code
media: davinci: allow build vpbe_display with COMPILE_TEST
media: vpbe_venc: don't store return codes if they won't be used
media: davinci: get rid of lots of kernel-doc warnings
media: omapfb_dss.h: add stubs to build with COMPILE_TEST
media: omap: allow building it with COMPILE_TEST
drivers/media/platform/Kconfig | 12 +++---
drivers/media/platform/davinci/Kconfig | 6 ++-
drivers/media/platform/davinci/isif.c | 2 -
drivers/media/platform/davinci/vpbe.c | 38 +++++++++--------
drivers/media/platform/davinci/vpbe_display.c | 21 +++++----
drivers/media/platform/davinci/vpbe_osd.c | 16 ++++---
drivers/media/platform/davinci/vpbe_venc.c | 9 ++--
drivers/media/platform/exynos4-is/Kconfig | 4 +-
drivers/media/platform/fsl-viu.c | 20 ++++++---
drivers/media/platform/marvell-ccic/Kconfig | 5 ++-
drivers/media/platform/marvell-ccic/mmp-driver.c | 4 +-
drivers/media/platform/omap/Kconfig | 6 +--
drivers/media/platform/omap3isp/isp.c | 14 +++---
include/linux/omap-iommu.h | 5 +++
include/linux/platform_data/media/mmp-camera.h | 19 +++++++++
include/video/omapfb_dss.h | 54 +++++++++++++++++++++++-
16 files changed, 162 insertions(+), 73 deletions(-)
--
2.14.3
next reply other threads:[~2018-04-05 17:54 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-05 17:54 Mauro Carvalho Chehab [this message]
2018-04-05 17:54 ` [PATCH 07/16] media: exymos4-is: allow compile test for EXYNOS FIMC-LITE Mauro Carvalho Chehab
[not found] ` <CGME20180409094946epcas1p1db108f4fcd018081c90787478004d907@epcas1p1.samsung.com>
2018-04-09 9:49 ` Sylwester Nawrocki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1522949748.git.mchehab@s-opensource.com \
--to=mchehab@s-opensource.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).