From: "Jeremy C. Andrus" <jeremya@xxxxxxxxxxxxxxx>
The "InternalLimits" sysfs entry used the "Limit_Register_Count" show
function. This has been changed to the correct "InternalLimits" function.
The rmi_f09_remove function tries to kfree u8 values which could never
hold memory pointers. I have removed the offending code, but would
appreciate feedback from one more knowledgeable than myself.
Signed-off-by: Jeremy C. Andrus <jeremya@xxxxxxxxxxxxxxx>
Change-Id: I54ca675af8eae19f7dfb3494d531baaab0281dfb
Cc: Stephen Warren <swarren@xxxxxxxxxxxxx>
Cc: Olof Johansson <olof@xxxxxxxxx>
---
drivers/input/touchscreen/rmi4/rmi_f09.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/input/touchscreen/rmi4/rmi_f09.c b/drivers/input/touchscreen/rmi4/rmi_f09.c
index 0ec980d..f1f0d01 100644
--- a/drivers/input/touchscreen/rmi4/rmi_f09.c
+++ b/drivers/input/touchscreen/rmi4/rmi_f09.c
@@ -107,7 +107,7 @@ static struct device_attribute attrs[] = {
__ATTR(HostTestEn, RMI_RW_ATTR,
rmi_f09_HostTestEn_show, rmi_f09_HostTestEn_store),
__ATTR(InternalLimits, RMI_RO_ATTR,
- rmi_f09_Limit_Register_Count_show, rmi_store_error),
+ rmi_f09_InternalLimits_show, rmi_store_error),
__ATTR(Result_Register_Count, RMI_RO_ATTR,
rmi_f09_Result_Register_Count_show, rmi_store_error),
};
@@ -169,11 +169,6 @@ error_exit:
static void rmi_f09_remove(struct rmi_function_container *fc)
{
- struct rmi_fn_09_data *data = fc->data;
- if (data) {
- kfree(data->query.Limit_Register_Count);
- kfree(data->query.f09_bist_query1);
- }
kfree(fc->data);
}
--
1.8.1
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
