evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

msm: ipa3: Fix uninitialized variables access

Browse source 
Kernel compilation warning is generated due to
these uninitialized variables access. This change
fixes them by initializing the variables.

Change-Id: Ifae04fe5e4d1b9a0c2c399a3c6746c38dea5c4cc
CRs-fixed: 2004062
Signed-off-by: Ghanim Fodi <gfodi@codeaurora.org>
This commit is contained in:
Ghanim Fodi 2017-02-26 21:45:59 +02:00 • committed by Gerrit - the friendly Code Review server
parent 541fe938d2
commit 3e6f65c965
3 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
drivers/platform/msm/ipa/ipa_v3/ipa_client.c
View file

@ -1777,7 +1777,8 @@ dealloc_chan_fail:
int ipa3_xdci_suspend(u32 ul_clnt_hdl, u32 dl_clnt_hdl, int ipa3_xdci_suspend(u32 ul_clnt_hdl, u32 dl_clnt_hdl,
bool should_force_clear, u32 qmi_req_id, bool is_dpl) bool should_force_clear, u32 qmi_req_id, bool is_dpl)
{ {
struct ipa3_ep_context *ul_ep, *dl_ep; struct ipa3_ep_context *ul_ep = NULL;
struct ipa3_ep_context *dl_ep;
int result = -EFAULT; int result = -EFAULT;
u32 source_pipe_bitmask = 0; u32 source_pipe_bitmask = 0;
bool dl_data_pending = true; bool dl_data_pending = true;

2
drivers/platform/msm/ipa/ipa_v3/ipa_dp.c
View file

@ -337,7 +337,7 @@ int ipa3_send_one(struct ipa3_sys_context *sys, struct ipa3_desc *desc,
int result; int result;
u16 sps_flags = SPS_IOVEC_FLAG_EOT; u16 sps_flags = SPS_IOVEC_FLAG_EOT;
dma_addr_t dma_address; dma_addr_t dma_address;
u16 len; u16 len = 0;
u32 mem_flag = GFP_ATOMIC; u32 mem_flag = GFP_ATOMIC;
if (unlikely(!in_atomic)) if (unlikely(!in_atomic))

6
drivers/platform/msm/ipa/ipa_v3/ipa_uc.c
View file

@ -1,4 +1,4 @@
/* Copyright (c) 2012-2016, The Linux Foundation. All rights reserved. /* Copyright (c) 2012-2017, The Linux Foundation. All rights reserved.
* *
* This program is free software; you can redistribute it and/or modify * This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License version 2 and * it under the terms of the GNU General Public License version 2 and
@ -498,7 +498,7 @@ static int ipa3_uc_send_cmd_64b_param(u32 cmd_lo, u32 cmd_hi, u32 opcode,
{ {
int index; int index;
union IpaHwCpuCmdCompletedResponseData_t uc_rsp; union IpaHwCpuCmdCompletedResponseData_t uc_rsp;
unsigned long flags; unsigned long flags = 0;
int retries = 0; int retries = 0;
send_cmd_lock: send_cmd_lock:
@ -775,7 +775,7 @@ int ipa3_uc_send_cmd(u32 cmd, u32 opcode, u32 expected_status,
void ipa3_uc_register_handlers(enum ipa3_hw_features feature, void ipa3_uc_register_handlers(enum ipa3_hw_features feature,
struct ipa3_uc_hdlrs *hdlrs) struct ipa3_uc_hdlrs *hdlrs)
{ {
unsigned long flags; unsigned long flags = 0;
if (0 > feature || IPA_HW_FEATURE_MAX <= feature) { if (0 > feature || IPA_HW_FEATURE_MAX <= feature) {
IPAERR("Feature %u is invalid, not registering hdlrs\n", IPAERR("Feature %u is invalid, not registering hdlrs\n",