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

Commit 84399174 authored by Paras Nagda's avatar Paras Nagda Committed by Gerrit - the friendly Code Review server
Browse files

msm: vidc_3x: Correct the arguments type to align with format string 



Correct the arguments type to align with format string(%pK) in printk
statements. Crash issue with data abort is seen because of this, use
&binfo->smem[i] instead of binfo->smem[i].

Change-Id: Ie595f402aa0427694a9e7e3477f81d626224d956
Signed-off-by: default avatarParas Nagda <pnagda@codeaurora.org>
parent dda0a79f

drivers/media/platform/msm/vidc_3x/msm_vidc.c

+4 −4
Original line number Diff line number Diff line 
// SPDX-License-Identifier: GPL-2.0-only

/*

 * Copyright (c) 2012-2020, The Linux Foundation. All rights reserved.

 * Copyright (c) 2012-2021, The Linux Foundation. All rights reserved.

 *

 * 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
@@ -533,7 +533,7 @@ int map_and_register_buf(struct msm_vidc_inst *inst, struct v4l2_buffer *b) 
		}

		dprintk(VIDC_DBG,

			"%s: [MAP] binfo = %pK, handle[%d] = %pK, device_addr = %pa, fd = %d, offset = %d, mapped = %d\n",

			__func__, binfo, i, binfo->smem[i],

			__func__, binfo, i, &binfo->smem[i],

			&binfo->device_addr[i], binfo->fd[i],

			binfo->buff_off[i], binfo->mapped[i]);

	}
@@ -586,7 +586,7 @@ int unmap_and_deregister_buf(struct msm_vidc_inst *inst, 
	for (i = 0; i < temp->num_planes; i++) {

		dprintk(VIDC_DBG,

			"%s: [UNMAP] binfo = %pK, handle[%d] = %pK, device_addr = %pa, fd = %d, offset = %d, mapped = %d\n",

			__func__, temp, i, temp->smem[i],

			__func__, temp, i, &temp->smem[i],

			&temp->device_addr[i], temp->fd[i],

			temp->buff_off[i], temp->mapped[i]);

		/*
@@ -902,7 +902,7 @@ int msm_vidc_release_buffers(void *instance, int buffer_type) 
				if (bi->mapped[i]) {

					dprintk(VIDC_DBG,

						"%s: [UNMAP] binfo = %pK, handle[%d] = %pK, device_addr = %pa, fd = %d, offset = %d, mapped = %d\n",

						__func__, bi, i, bi->smem[i],

						__func__, bi, i, &bi->smem[i],

						&bi->device_addr[i], bi->fd[i],

						bi->buff_off[i], bi->mapped[i]);

					msm_comm_smem_free(inst,