diff options
| author | Pan Bian <bianpan2016@163.com> | 2016-12-03 13:39:33 +0100 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-02-03 10:58:37 +0100 |
| commit | d54a90c4c19307986effa50a93e47d1df7a132a6 (patch) | |
| tree | f510943520c92ef3b9aa19855ea331a1b11ff0d5 /drivers/media/platform/sti | |
| parent | [media] mtk-vcodec: use V4L2_DEC_CMD_STOP to implement flush (diff) | |
| download | linux-d54a90c4c19307986effa50a93e47d1df7a132a6.tar.xz linux-d54a90c4c19307986effa50a93e47d1df7a132a6.zip | |
[media] media: platform: sti: return -ENOMEM on errors
Function bdisp_debugfs_create() returns 0 even on errors. So its caller
cannot detect the errors. It may be better to return "-ENOMEM" on the
exception paths.
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188801
Signed-off-by: Pan Bian <bianpan2016@163.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'drivers/media/platform/sti')
| -rw-r--r-- | drivers/media/platform/sti/bdisp/bdisp-debug.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/platform/sti/bdisp/bdisp-debug.c b/drivers/media/platform/sti/bdisp/bdisp-debug.c index 79c56356a7c7..7af66860d624 100644 --- a/drivers/media/platform/sti/bdisp/bdisp-debug.c +++ b/drivers/media/platform/sti/bdisp/bdisp-debug.c @@ -677,7 +677,7 @@ int bdisp_debugfs_create(struct bdisp_dev *bdisp) err: bdisp_debugfs_remove(bdisp); - return 0; + return -ENOMEM; } void bdisp_debugfs_remove(struct bdisp_dev *bdisp) |