Commit c9eaee0c authored by Dan Carpenter's avatar Dan Carpenter Committed by Kalle Valo

rtw88: Fix an error code in rtw_debugfs_set_rsvd_page()

The sscanf() function returns the number of matches (0 or 1 in this
case).  It doesn't return error codes.  We should return -EINVAL if the
string is invalid

Fixes: c376c1fc ("rtw88: add h2c command in debugfs")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/YE8nmatMDBDDWkjq@mwanda
parent 4517f811
...@@ -271,7 +271,7 @@ static ssize_t rtw_debugfs_set_rsvd_page(struct file *filp, ...@@ -271,7 +271,7 @@ static ssize_t rtw_debugfs_set_rsvd_page(struct file *filp,
if (num != 2) { if (num != 2) {
rtw_warn(rtwdev, "invalid arguments\n"); rtw_warn(rtwdev, "invalid arguments\n");
return num; return -EINVAL;
} }
debugfs_priv->rsvd_page.page_offset = offset; debugfs_priv->rsvd_page.page_offset = offset;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment