Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

media: platform: mtk-mdp3: don't use %pK through printk

In the past %pK was preferable to %p as it would not leak raw pointer
values into the kernel log.
Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
the regular %p has been improved to avoid this issue.
Furthermore, restricted pointers ("%pK") were never meant to be used
through printk(). They can still unintentionally leak raw pointers or
acquire sleeping locks in atomic contexts.

Switch to the regular pointer formatting which is safer and
easier to reason about.

Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>

authored by

Thomas Weißschuh and committed by
Hans Verkuil
5e6c9089 d5d299e7

+2 -2
+1 -1
drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c
··· 282 282 } 283 283 284 284 mdp->rproc_handle = scp_get_rproc(mdp->scp); 285 - dev_dbg(&pdev->dev, "MDP rproc_handle: %pK", mdp->rproc_handle); 285 + dev_dbg(&pdev->dev, "MDP rproc_handle: %p", mdp->rproc_handle); 286 286 287 287 mutex_init(&mdp->vpu_lock); 288 288 mutex_init(&mdp->m2m_lock);
+1 -1
drivers/media/platform/mediatek/mdp3/mtk-mdp3-vpu.c
··· 221 221 } 222 222 223 223 dev_dbg(&mdp->pdev->dev, 224 - "VPU param:%pK pa:%pad sz:%zx, work:%pK pa:%pad sz:%zx, config:%pK pa:%pad sz:%zx", 224 + "VPU param:%p pa:%pad sz:%zx, work:%p pa:%pad sz:%zx, config:%p pa:%pad sz:%zx", 225 225 vpu->param, &vpu->param_addr, vpu->param_size, 226 226 vpu->work, &vpu->work_addr, vpu->work_size, 227 227 vpu->config, &vpu->config_addr, vpu->config_size);