evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

drivers: lmh_lite: Invalidate cache after the scm call

Browse source 
LMH driver passes a buffer to TZ for fetching information. The driver
flushes the memory before passing the buffer to TZ. Though the flush
call will clean and invalidate the cache line, it is recommended to
invalidate the cache after the scam call.

Update the LMH driver to invalidate the payload buffer after the scm
call returns.

Change-Id: I4b599f4d1e8bf174ece5503fc4ff204dcb57b423
Signed-off-by: Ram Chandrasekar <rkumbako@codeaurora.org>
This commit is contained in:
Ram Chandrasekar 2017-02-14 10:39:21 -07:00
parent 24499b633f
commit 358b1c321a
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
drivers/thermal/lmh_lite.c
View file

@ -1,4 +1,4 @@
/* Copyright (c) 2014-2016, The Linux Foundation. All rights reserved.
/* Copyright (c) 2014-2017, 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
@ -83,6 +83,8 @@
} \
/* Have barrier before reading from TZ data */ \
mb(); \
dmac_inv_range(payload, (void *)payload + \
sizeof(uint32_t) * LMH_SCM_PAYLOAD_SIZE);\
trace_lmh_event_call("GET_TYPE exit"); \
if (ret) { \
pr_err("Error in SCM v%d get type. cmd:%x err:%d\n", \
@ -332,6 +334,8 @@ static void lmh_read_and_update(struct lmh_driver_data *lmh_dat)
/* Have memory barrier before we access the TZ data */
mb();
trace_lmh_event_call("GET_INTENSITY exit");
dmac_inv_range(&payload, (void *)&payload +
sizeof(struct lmh_sensor_packet));
if (ret) {
pr_err("Error in SCM v%d read call. err:%d\n",
(is_scm_armv8()) ? 8 : 7, ret);
@ -677,6 +681,7 @@ static int lmh_get_sensor_list(void)
/* Have memory barrier before we access the TZ data */
mb();
trace_lmh_event_call("GET_SENSORS exit");
dmac_inv_range(payload, (void *)payload + buf_size);
if (ret < 0) {
pr_err("Error in SCM v%d call. err:%d\n",
(is_scm_armv8()) ? 8 : 7, ret);
@ -911,6 +916,7 @@ static int lmh_debug_read(struct lmh_debug_ops *ops, uint32_t **buf)
}
/* Have memory barrier before we access the TZ data */
mb();
dmac_inv_range(payload, (void *)payload + curr_size);
trace_lmh_event_call("GET_DEBUG_READ exit");
if (ret) {
pr_err("Error in SCM v%d get debug read. err:%d\n",
@ -977,6 +983,7 @@ static int lmh_debug_config_write(uint32_t cmd_id, uint32_t *buf, int size)
ret = scm_call2(SCM_SIP_FNID(SCM_SVC_LMH, cmd_id), &desc_arg);
/* Have memory barrier before we access the TZ data */
mb();
dmac_inv_range(payload, (void *)payload + size_bytes);
trace_lmh_event_call("CONFIG_DEBUG_WRITE exit");
if (ret) {
pr_err("Error in SCM v%d config debug read. err:%d\n",