OSDN Git Service

soc: qcom: hab: unify variable type to avoid implicit conversion
authorYao Jiang <yaojia@codeaurora.org>
Mon, 4 Mar 2019 03:54:54 +0000 (11:54 +0800)
committerYao Jiang <yaojia@codeaurora.org>
Wed, 13 Mar 2019 04:52:20 +0000 (12:52 +0800)
correcting the type of sizebytes and total_size to size_t,it can avoid
implicit conversion which maybe cause some unknown issue.

Change-Id: I1c1cc9e92ca2920cc49685edef0d928dd67454c7
Signed-off-by: Yao Jiang <yaojia@codeaurora.org>
drivers/soc/qcom/hab/ghs_comm.c
drivers/soc/qcom/hab/qvm_comm.c

index 825f33a..1fdb166 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (c) 2018, The Linux Foundation. All rights reserved.
+/* Copyright (c) 2018-2019, The Linux Foundation. All rights reserved.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 2 and
@@ -39,7 +39,7 @@ int physical_channel_send(struct physical_channel *pchan,
                struct hab_header *header,
                void *payload)
 {
-       int sizebytes = HAB_HEADER_GET_SIZE(*header);
+       size_t sizebytes = HAB_HEADER_GET_SIZE(*header);
        struct ghs_vdev *dev  = (struct ghs_vdev *)pchan->hyp_data;
        GIPC_Result result;
        uint8_t *msg;
index 25d0208..d5332bc 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (c) 2016-2018, The Linux Foundation. All rights reserved.
+/* Copyright (c) 2016-2019, The Linux Foundation. All rights reserved.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 2 and
@@ -42,9 +42,9 @@ int physical_channel_send(struct physical_channel *pchan,
                struct hab_header *header,
                void *payload)
 {
-       int sizebytes = HAB_HEADER_GET_SIZE(*header);
+       size_t sizebytes = HAB_HEADER_GET_SIZE(*header);
        struct qvm_channel *dev  = (struct qvm_channel *)pchan->hyp_data;
-       int total_size = sizeof(*header) + sizebytes;
+       size_t total_size = sizeof(*header) + sizebytes;
 
        if (total_size > dev->pipe_ep->tx_info.sh_buf->size)
                return -EINVAL; /* too much data for ring */