diff options
author | Chengguang Xu <cgxu519@gmail.com> | 2019-05-02 20:15:05 +0800 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-05-24 20:50:36 +0200 |
commit | 7ef0b1524417743e6861490420225affe451486b (patch) | |
tree | cd11afb441a7cd2abc48bc1a0860482a540e4abd /drivers/firmware/google/framebuffer-coreboot.c | |
parent | 17aa207e6e08b0bcf92cf7b473fae9a70e325dbe (diff) |
chardev: set variable ret to -EBUSY before checking minor range overlap
When allocating dynamic major, the minor range overlap check
in __register_chrdev_region() will not fail, so actually
there is no real case to passing non negative error code to
caller. However, set variable ret to -EBUSY before checking
minor range overlap will avoid false-positive warning from
code analyzing tool(like Smatch) and also make the code more
easy to understand.
Suggested-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Chengguang Xu <cgxu519@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/firmware/google/framebuffer-coreboot.c')
0 files changed, 0 insertions, 0 deletions