From 8816d3dac87ab56c5df702f0c25dd997f7c8b884 Mon Sep 17 00:00:00 2001 From: Ryan Lattrel Date: Wed, 18 Mar 2020 10:57:58 -0500 Subject: [PATCH] sec_mmi: Fix default prop reset on resume When the post resume runs it stores the values set in init if the values were updated when the chip was on instead of the new values. Copy the new values to the buffer regardless of the state. Change-Id: I6ab9bf709746f2ac67eee786c4c5f36e9fcb1b72 Signed-off-by: Ryan Lattrel Reviewed-on: https://gerrit.mot.com/1516061 SME-Granted: SME Approvals Granted SLTApproved: Slta Waiver Tested-by: Jira Key Reviewed-by: Konstantin Makariev Submit-Approved: Jira Key --- drivers/input/touchscreen/sec_mmi/sec_ts_mmi_class.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/input/touchscreen/sec_mmi/sec_ts_mmi_class.c b/drivers/input/touchscreen/sec_mmi/sec_ts_mmi_class.c index e84ce3255f79..792b7ecc06b2 100644 --- a/drivers/input/touchscreen/sec_mmi/sec_ts_mmi_class.c +++ b/drivers/input/touchscreen/sec_mmi/sec_ts_mmi_class.c @@ -231,9 +231,9 @@ static ssize_t sec_mmi_suppression_store(struct device *dev, buffer = (unsigned char)value; dev_dbg(dev, "%s: program value 0x%02x\n", __func__, (unsigned int)buffer); + ts->suppression_data = buffer; if (ts->power_status == SEC_TS_STATE_POWER_OFF) { dev_dbg(dev, "%s: power off state\n", __func__); - ts->suppression_data = buffer; return size; } @@ -306,9 +306,9 @@ static ssize_t sec_mmi_pill_region_store(struct device *dev, (unsigned int)buffer[2], (unsigned int)buffer[3], (unsigned int)buffer[4]); + memcpy(ts->pill_region_data, buffer, sizeof(ts->pill_region_data)); if (ts->power_status == SEC_TS_STATE_POWER_OFF) { dev_dbg(dev, "%s: power off state\n", __func__); - memcpy(ts->pill_region_data, buffer, sizeof(ts->pill_region_data)); return size; } @@ -386,9 +386,9 @@ static ssize_t sec_mmi_hold_distance_store(struct device *dev, buffer = (unsigned char)value; dev_dbg(dev, "%s: program value 0x%02x\n", __func__, (unsigned int)buffer); + ts->hold_distance_data = buffer; if (ts->power_status == SEC_TS_STATE_POWER_OFF) { dev_dbg(dev, "%s: power off state\n", __func__); - ts->hold_distance_data = buffer; return size; }