diff options
| author | dev-harsh1998 <dev-harsh1998@hotmail.com> | 2019-03-19 10:12:12 +0530 |
|---|---|---|
| committer | Davide Garberi <dade.garberi@gmail.com> | 2022-07-27 18:59:12 +0200 |
| commit | 205ac8bfe6b022e6cca31842a612fa2d9eae2566 (patch) | |
| tree | 6a09d7d2f5af3ea916118569f9a852c3510791e4 | |
| parent | e1fa5d805cf31b9f903ec018e523be689f3dd60f (diff) | |
shenqi_nv: drop the useless dump_smem function
* The dump_smem function more like a debug function which was implemented
by the oem for debugging purposes but after the recent cleanup up
DD3Boh/android_kernel_zuk_msm8996@4c947e4ab0bee7f79b3ab65132cf93749fb249b4 we now have proper fail checks so there is no need to have this function.
* Kill it with fire and drop the expense cost of calling it, also cleanup whitespaces while we are at it.
Signed-off-by: dev-harsh1998 <dev-harsh1998@hotmail.com>
| -rw-r--r-- | drivers/soc/qcom/shenqi_nv.c | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/drivers/soc/qcom/shenqi_nv.c b/drivers/soc/qcom/shenqi_nv.c index 70cc90a3951d..b2addc3e179a 100644 --- a/drivers/soc/qcom/shenqi_nv.c +++ b/drivers/soc/qcom/shenqi_nv.c @@ -21,12 +21,6 @@ struct smem_nv { static struct smem_nv *psmem_nv = NULL; -void dump_smem(struct smem_nv *buf) -{ - print_hex_dump(KERN_ALERT, "wifi : ", DUMP_PREFIX_NONE, 16, 1, - buf->nv_wifi, NV_WIFI_ADDR_SIZE, false); -} - static int smem_read_nv(void) { struct smem_nv *buf; @@ -44,13 +38,12 @@ static int smem_read_nv(void) } memcpy(psmem_nv, buf, sizeof(struct smem_nv)); - dump_smem(psmem_nv); return 0; } static long dump_wifi_addr(struct file *filp, char __user *buf, - size_t count, loff_t *f_pos) + size_t count, loff_t *f_pos) { loff_t pos = *f_pos; int ret; @@ -61,7 +54,7 @@ static long dump_wifi_addr(struct file *filp, char __user *buf, return ret; } - if (!psmem_nv) + if (!psmem_nv) { printk(KERN_ERR "Could not get smem for wlan mac nv\n"); return 0; |