Skip to content

video: fbdev: Deletion of some unnecessary checks

SF Markus Elfring acpica-devel alsa-devel ath10k ath9k-devel backports batman ceph-devel cluster-devel cocci dm-devel dri-devel ecryptfs intel-gfx kernel-janitors kvm linux-acpi linux-arm-kernel linux-arm-msm linux-audit linux-bcache linux-block linux-bluetooth linux-btrfs linux-crypto linux-devicetree linux-ext4 linux-f2fs-devel linux-fbdev linux-fsdevel linux-gpio linux-hwmon linux-ia64 linux-ide linux-input linux-iommu linux-kbuild linux-media linux-mediatek linux-mm linux-mtd linux-nfs linux-nilfs linux-nvdimm linux-omap linux-parisc linux-pci linux-pm linux-ppp linux-pwm linux-raid linux-rdma linux-rockchip linux-s390 linux-samsung-soc linux-scsi linux-sctp linux-sh linux-spi linux-tegra linux-wireless linux-wpan linux-xfs linuxppc-dev lkml lustre-devel lvs-devel netdev netfilter-devel nouveau nvdimm ocfs2-devel platform-driver-x86 virtualization xen-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest8 of 9 patches 1 replyIn review

8 of 9 patches are in this archive. The cover letter announces 9.

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/9video: Deletion of checks before the function call "iounmap"drivers/video/fbdev/aty/atyfb_base.c drivers/video/fbdev/cirrusfb.c drivers/video/fbdev/controlfb.c +19 more+41 -820 replies
3/9video: One function call less in vesafb_probe() after, error detectiondrivers/video/fbdev/vesafb.c+3 -20 replies
4/9video: Add check for return value of ioremap() in cirrusfb_pci_register()drivers/video/fbdev/cirrusfb.c+7 -20 replies
5/9video: Less function calls in trident_pci_probe() after error detectiondrivers/video/fbdev/tridentfb.c+6 -30 replies
6/9video: One function call less in pvr2fb_common_init() after error detectiondrivers/video/fbdev/pvr2fb.c+6 -60 replies
7/9video: One function call less in tgafb_register() after, error detectiondrivers/video/fbdev/tgafb.c+9 -80 replies
8/9video: Less function calls in valkyriefb_init() after error detectiondrivers/video/fbdev/valkyriefb.c+21 -110 replies
9/9video: Less function calls in w100fb_probe() after error detectiondrivers/video/fbdev/w100fb.c+17 -140 replies

Reviews trailers per patch

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

Read the v1 thread

Cover letter [PATCH 0/9] video: fbdev: Deletion of some unnecessary checks

From: Markus Elfring <[email protected]>
Date: Mon, 19 Jan 2015 17:57:25 +0100

Further update suggestions were taken into account after a patch was applied from static source code analysis.

Markus Elfring (9):
fbdev: Deletion of checks before the function call "iounmap" One function call less in tdfxfb_probe() after error detection One function call less in vesafb_probe() after error detection Add check for return value of ioremap() in cirrusfb_pci_register() Less function calls in trident_pci_probe() after error detection One function call less in pvr2fb_common_init() after error detection One function call less in tgafb_register() after error detection Less function calls in valkyriefb_init() after error detection Less function calls in w100fb_probe() after error detection

 drivers/video/fbdev/aty/atyfb_base.c   | 15 +++++--------
 drivers/video/fbdev/cirrusfb.c         | 10 ++++++---
 drivers/video/fbdev/controlfb.c        |  6 ++---
 drivers/video/fbdev/efifb.c            |  3 +--
 drivers/video/fbdev/hgafb.c            |  3 +--
 drivers/video/fbdev/hpfb.c             |  3 +--
 drivers/video/fbdev/i810/i810_main.c   |  6 ++---
 drivers/video/fbdev/macfb.c            | 15 +++++--------
 drivers/video/fbdev/offb.c             |  3 +--
 drivers/video/fbdev/pvr2fb.c           | 15 ++++++-------
 drivers/video/fbdev/s1d13xxxfb.c       |  3 +--
 drivers/video/fbdev/s3fb.c             |  6 ++---
 drivers/video/fbdev/sh_mobile_hdmi.c   |  6 ++---
 drivers/video/fbdev/sh_mobile_lcdcfb.c |  3 +--
 drivers/video/fbdev/simplefb.c         |  3 +--
 drivers/video/fbdev/stifb.c            |  3 +--
 drivers/video/fbdev/tdfxfb.c           | 22 +++++++++----------
 drivers/video/fbdev/tgafb.c            | 20 ++++++++---------
 drivers/video/fbdev/tridentfb.c        | 15 +++++++------
 drivers/video/fbdev/valkyriefb.c       | 37 ++++++++++++++++++-------------
 drivers/video/fbdev/vesafb.c           | 11 +++++-----
 drivers/video/fbdev/w100fb.c           | 40 +++++++++++++++++-----------------
 22 files changed, 116 insertions(+), 132 deletions(-)
Signature · 1 line
2.2.2