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

Commit 2d4ec481 authored Feb 07, 2019 by Jordan Crouse

msm: ipa: Fix compiler warnings



Add a default label for a switch() statement to catch unused
enum values and fix a destination copy error in ipa_ut_framework.c.

Fixes: 859783a1 ("msm: ipa: Add snapshot of IPA driver")
Change-Id: Ic0dedbad65985fc24d9dda90c6be115be770f631
Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org>

parent 1cbccef3

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

+2 −1

Original line number	Diff line number	Diff line
		// SPDX-License-Identifier: GPL-2.0-only
		/*
		* Copyright (c) 2018, The Linux Foundation. All rights reserved.
		* Copyright (c) 2018-2019, The Linux Foundation. All rights reserved.
		*/

		#include <linux/dma-mapping.h>
		@@ -902,6 +902,7 @@ static void imp_mhi_status_cb(struct mhi_device *mhi_dev, enum MHI_CB mhi_cb)

		case MHI_CB_EE_RDDM:
		case MHI_CB_PENDING_DATA:
		default:
		IMP_ERR("unexpected event %d\n", mhi_cb);
		break;
		}

drivers/platform/msm/ipa/test/ipa_ut_framework.c

+2 −2

Original line number	Diff line number	Diff line
		// SPDX-License-Identifier: GPL-2.0-only
		/*
		* Copyright (c) 2017-2018, The Linux Foundation. All rights reserved.
		* Copyright (c) 2017-2019, The Linux Foundation. All rights reserved.
		*/

		#include <linux/mutex.h>
		@@ -852,7 +852,7 @@ static ssize_t ipa_ut_dbgfs_enable_write(struct file *file,

		IPA_UT_DBG("Entry\n");

		if (sizeof(lcl_buf) < count + 1) {
		if (count >= sizeof(lcl_buf)) {
		IPA_UT_ERR("No enough space\n");
		return -E2BIG;
		}