From: Jiri Kosina <jkos...@suse.cz>

ts_format.reserved is not used anywhere yet, but the compiler generates a 
warning when the struct's (uninitialized) field is being copied around

drivers/hid/intel-ish-hid/ipc/ipc.c: In function ‘write_ipc_from_queue’:
drivers/hid/intel-ish-hid/ipc/ipc.c:316: warning: ‘ts_format.reserved’ may be 
used uninitialized in this function

Avoid this by force-initializing the field to zero.

Signed-off-by: Jiri Kosina <jkos...@suse.cz>
---
 drivers/hid/intel-ish-hid/ipc/ipc.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/hid/intel-ish-hid/ipc/ipc.c 
b/drivers/hid/intel-ish-hid/ipc/ipc.c
index 0c9ac4d..417cd07 100644
--- a/drivers/hid/intel-ish-hid/ipc/ipc.c
+++ b/drivers/hid/intel-ish-hid/ipc/ipc.c
@@ -310,6 +310,7 @@ static int write_ipc_from_queue(struct ishtp_device *dev)
                                                ((uint32_t)tv_utc.tv_usec);
                ts_format.ts1_source = HOST_SYSTEM_TIME_USEC;
                ts_format.ts2_source = HOST_UTC_TIME_USEC;
+               ts_format.reserved = 0;
 
                time_update.primary_host_time = usec_system;
                time_update.secondary_host_time = usec_utc;
-- 
1.9.2

Reply via email to