From 6f9a3ea228abf94116d2584c0b1e77bbe82d000f Mon Sep 17 00:00:00 2001
From: Jiamin Ma <jiamin.ma@amlogic.com>
Date: Thu, 26 Oct 2017 14:04:32 +0800
Subject: [PATCH] efuse: fix INCOMPATIBLE_CAST and NEGATIVE_RETURNS errs

PD#152879: driver defect clean up
 #299
 #300
 #320
 #321

Change-Id: I3756489ea81520ab042c74642b0483f36bff6fba
Signed-off-by: Jiamin Ma <jiamin.ma@amlogic.com>
---
 drivers/amlogic/efuse/efuse64.c | 23 +++++++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/drivers/amlogic/efuse/efuse64.c b/drivers/amlogic/efuse/efuse64.c
index 7526452d7a6253..71c6bbe1a8a0ca 100644
--- a/drivers/amlogic/efuse/efuse64.c
+++ b/drivers/amlogic/efuse/efuse64.c
@@ -309,6 +309,7 @@ ssize_t efuse_user_attr_store(char *name, const char *buf, size_t count)
 	const char *c, *s;
 	struct efusekey_info info;
 	unsigned int uint_val;
+	loff_t pos;
 
 	if (efuse_getinfo(name, &info) < 0) {
 		pr_err("%s is not found\n", name);
@@ -338,7 +339,8 @@ ssize_t efuse_user_attr_store(char *name, const char *buf, size_t count)
 		}
 	}
 
-	ret = efuse_write_usr(local_buf, info.size, (loff_t *)&(info.offset));
+	pos = ((loff_t)(info.offset)) & 0xffffffff;
+	ret = efuse_write_usr(local_buf, info.size, &pos);
 	if (ret == -1) {
 		pr_err("ERROR: efuse write user data fail!\n");
 		goto error_exit;
@@ -364,6 +366,7 @@ ssize_t efuse_user_attr_show(char *name, char *buf)
 	ssize_t ret;
 	int i;
 	struct efusekey_info info;
+	loff_t pos;
 
 	if (efuse_getinfo(name, &info) < 0) {
 		pr_err("%s is not found\n", name);
@@ -373,7 +376,8 @@ ssize_t efuse_user_attr_show(char *name, char *buf)
 	local_buf = kzalloc(sizeof(char)*(info.size), GFP_KERNEL);
 	memset(local_buf, 0, info.size);
 
-	ret = efuse_read_usr(local_buf, info.size, (loff_t *)&(info.offset));
+	pos = ((loff_t)(info.offset)) & 0xffffffff;
+	ret = efuse_read_usr(local_buf, info.size, &pos);
 	if (ret == -1) {
 		pr_err("ERROR: efuse read user data fail!\n");
 		goto error_exit;
@@ -399,7 +403,12 @@ static ssize_t userdata_show(struct class *cla,
 	loff_t offset;
 	unsigned int max_size;
 
-	max_size = efuse_get_max();
+	ret = efuse_get_max();
+	if (ret < 0) {
+		pr_err("efuse: failed to get max size\n");
+		return -1;
+	}
+	max_size = (unsigned int)ret;
 
 	op = kmalloc_array(max_size, sizeof(char), GFP_KERNEL);
 	if (!op) {
@@ -441,7 +450,13 @@ static ssize_t userdata_write(struct class *cla,
 	char *op = NULL;
 	unsigned int max_size;
 
-	max_size = efuse_get_max();
+	ret = efuse_get_max();
+	if (ret < 0) {
+		pr_err("efuse: failed to get max size\n");
+		return -1;
+	}
+	max_size = (unsigned int)ret;
+
 	op = kzalloc((sizeof(char)*max_size), GFP_KERNEL);
 	if (!op) {
 		pr_err("efuse: failed to allocate memory\n");