diff options
author | Vasyl Gomonovych <gomonovych@gmail.com> | 2017-11-28 16:40:27 -0500 |
---|---|---|
committer | Michael S. Tsirkin <mst@redhat.com> | 2018-01-30 18:47:34 -0500 |
commit | 0a9e63aa397711cb7dd8a5c3db33448740a0b844 (patch) | |
tree | d4f8b8e74b031b2bbe3de1bb1a7769d22c4419be | |
parent | c2c9f9bc5b830cbb63d6674b0d51a7773e54c90c (diff) |
firmware: Use PTR_ERR_OR_ZERO()
Fix ptr_ret.cocci warnings:
drivers/firmware/efi/efi.c:610:8-14: WARNING: PTR_ERR_OR_ZERO can be used
Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR
Generated by: scripts/coccinelle/api/ptr_ret.cocci
Signed-off-by: Vasyl Gomonovych <gomonovych@gmail.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Acked-by: Gabriel Somlo <somlo@cmu.edu>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
-rw-r--r-- | drivers/firmware/qemu_fw_cfg.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/firmware/qemu_fw_cfg.c b/drivers/firmware/qemu_fw_cfg.c index deb483064f53..a41b572eeeb1 100644 --- a/drivers/firmware/qemu_fw_cfg.c +++ b/drivers/firmware/qemu_fw_cfg.c | |||
@@ -694,10 +694,8 @@ static int fw_cfg_cmdline_set(const char *arg, const struct kernel_param *kp) | |||
694 | */ | 694 | */ |
695 | fw_cfg_cmdline_dev = platform_device_register_simple("fw_cfg", | 695 | fw_cfg_cmdline_dev = platform_device_register_simple("fw_cfg", |
696 | PLATFORM_DEVID_NONE, res, processed); | 696 | PLATFORM_DEVID_NONE, res, processed); |
697 | if (IS_ERR(fw_cfg_cmdline_dev)) | ||
698 | return PTR_ERR(fw_cfg_cmdline_dev); | ||
699 | 697 | ||
700 | return 0; | 698 | return PTR_ERR_OR_ZERO(fw_cfg_cmdline_dev); |
701 | } | 699 | } |
702 | 700 | ||
703 | static int fw_cfg_cmdline_get(char *buf, const struct kernel_param *kp) | 701 | static int fw_cfg_cmdline_get(char *buf, const struct kernel_param *kp) |