aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c
diff options
context:
space:
mode:
authorNam Cao <[email protected]>2025-07-27 17:31:13 +0000
committerSteven Rostedt (Google) <[email protected]>2025-07-28 14:39:34 +0000
commit3cfb9c1a7db091f9030826b897d07ddce1591e7f (patch)
treef8ec7059a912f742103e902a6e133affad298d02 /drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c
parentrv: Fix wrong type cast in monitors_show() (diff)
downloadkernel-3cfb9c1a7db091f9030826b897d07ddce1591e7f.tar.gz
kernel-3cfb9c1a7db091f9030826b897d07ddce1591e7f.zip
rv: Fix wrong type cast in reactors_show() and monitor_reactor_show()
Argument 'p' of reactors_show() and monitor_reactor_show() is not a pointer to struct rv_reactor, it is actually a pointer to the list_head inside struct rv_reactor. Therefore it's wrong to cast 'p' to struct rv_reactor *. This wrong type cast has been there since the beginning. But it still worked because the list_head was the first field in struct rv_reactor_def. This is no longer true since commit 3d3c376118b5 ("rv: Merge struct rv_reactor_def into struct rv_reactor") moved the list_head, and this wrong type cast became a functional problem. Properly use container_of() instead. Cc: Masami Hiramatsu <[email protected]> Cc: Mathieu Desnoyers <[email protected]> Cc: Gabriele Monaco <[email protected]> Link: https://lore.kernel.org/b4febbd6844311209e4c8768b65d508b81bd8c9b.1753625621.git.namcao@linutronix.de Fixes: 3d3c376118b5 ("rv: Merge struct rv_reactor_def into struct rv_reactor") Signed-off-by: Nam Cao <[email protected]> Signed-off-by: Steven Rostedt (Google) <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_isp.c')
0 files changed, 0 insertions, 0 deletions