Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 42ad67bf authored by Mohammed Javid's avatar Mohammed Javid
Browse files

msm: ipa: Fix wrong usage and cleanup code 



Couple of code cleanup:
 - Initialize the return variable before using it.
 - Remove unnecessary NULL check.

Change-Id: I8e63cb95ae99d1656143ae4b1d130f92890bb3c5
Signed-off-by: default avatarMohammed Javid <mjavid@codeaurora.org>
parent d98d9ee8

drivers/platform/msm/ipa/ipa_v2/ipa_hdr.c

+1 −2
Original line number Diff line number Diff line
@@ -649,7 +649,6 @@ static int __ipa_add_hdr_proc_ctx(struct ipa_hdr_proc_ctx_add *proc_ctx, 
	return 0;



ipa_insert_failed:

	if (offset)

	list_move(&offset->link,

		&htbl->head_free_offset_list[offset->bin]);

	entry->offset_entry = NULL;

drivers/platform/msm/ipa/ipa_v2/ipa_rt.c

+1 −1
Original line number Diff line number Diff line
@@ -1389,7 +1389,7 @@ int ipa2_put_rt_tbl(u32 rt_tbl_hdl) 
{

	struct ipa_rt_tbl *entry;

	enum ipa_ip_type ip = IPA_IP_MAX;

	int result;

	int result = 0;



	mutex_lock(&ipa_ctx->lock);

	entry = ipa_id_find(rt_tbl_hdl);

drivers/platform/msm/ipa/ipa_v3/ipa_hdr.c

+1 −2
Original line number Diff line number Diff line
@@ -424,7 +424,6 @@ static int __ipa_add_hdr_proc_ctx(struct ipa_hdr_proc_ctx_add *proc_ctx, 
	return 0;



ipa_insert_failed:

	if (offset)

	list_move(&offset->link,

		&htbl->head_free_offset_list[offset->bin]);

	entry->offset_entry = NULL;

drivers/platform/msm/ipa/ipa_v3/ipa_rt.c

+1 −1
Original line number Diff line number Diff line
@@ -1470,7 +1470,7 @@ int ipa3_put_rt_tbl(u32 rt_tbl_hdl) 
{

	struct ipa3_rt_tbl *entry;

	enum ipa_ip_type ip = IPA_IP_MAX;

	int result;

	int result = 0;



	mutex_lock(&ipa3_ctx->lock);

	entry = ipa3_id_find(rt_tbl_hdl);