diff options
| author | Srinivas Kandagatla <srinivas.kandagatla@linaro.org> | 2018-05-17 17:03:53 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2018-05-17 18:11:23 +0100 |
| commit | e744619d056e6ec5cd9db38e55511a22dc463f89 (patch) | |
| tree | 24eeea45bef61c80ef0043d650c2bd38a7ff7d63 /drivers/soc | |
| parent | 81dd1c5dcf510a2104b3468e9c4884f85ef1f644 (diff) | |
| download | linux-e744619d056e6ec5cd9db38e55511a22dc463f89.tar.bz2 | |
soc: qcom: apr: fix invalid msg_type check
Removed invalid msg_type check.
This also fixes below static checker warning:
apr.c:95:35: warning: comparison is always true due to limited range of
data type [-Wtype-limits]
warn: always true condition '(msg_type != 69864) => (0-u16max != 69864)'
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/soc')
| -rw-r--r-- | drivers/soc/qcom/apr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/soc/qcom/apr.c b/drivers/soc/qcom/apr.c index 97f3622da535..57af8a537332 100644 --- a/drivers/soc/qcom/apr.c +++ b/drivers/soc/qcom/apr.c @@ -92,7 +92,7 @@ static int apr_callback(struct rpmsg_device *rpdev, void *buf, } msg_type = APR_HDR_FIELD_MT(hdr->hdr_field); - if (msg_type >= APR_MSG_TYPE_MAX && msg_type != APR_BASIC_RSP_RESULT) { + if (msg_type >= APR_MSG_TYPE_MAX) { dev_err(apr->dev, "APR: Wrong message type: %d\n", msg_type); return -EINVAL; } |