diff options
author | Ankit Goyal <layog@google.com> | 2023-05-17 13:13:55 -0700 |
---|---|---|
committer | Shashank Sharma <shawshanks@google.com> | 2024-01-25 22:00:39 +0000 |
commit | 2ee7d4b9f5ea05fb777ed1224bb5932526b33bf6 (patch) | |
tree | bdf9b864a33786f7225089b31ee468f0593030fc | |
parent | 972efb1d89dcae35bd6a3b4bc304b31847bf03c9 (diff) | |
download | gpu-2ee7d4b9f5ea05fb777ed1224bb5932526b33bf6.tar.gz |
mali_kbase: add NULL checks before global_iface register dump
Based on the commit 440a991eda5dfb08e9f6ba1234a475dd8bd89325 with
further changes.
Bug: 320427777
Test: Manually tested the error path.
Signed-off-by: Shashank Sharma <shawshanks@google.com>
Change-Id: Ibb4b18ae14f8ec03379acf2b4892b212b46607c3
-rw-r--r-- | mali_kbase/csf/mali_kbase_csf_reset_gpu.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/mali_kbase/csf/mali_kbase_csf_reset_gpu.c b/mali_kbase/csf/mali_kbase_csf_reset_gpu.c index 726cee2..c722c96 100644 --- a/mali_kbase/csf/mali_kbase_csf_reset_gpu.c +++ b/mali_kbase/csf/mali_kbase_csf_reset_gpu.c @@ -268,9 +268,10 @@ void kbase_csf_debug_dump_registers(struct kbase_device *kbdev) } dev_err(kbdev->dev, " MCU DB0: %x", kbase_reg_read32(kbdev, DEBUG_MCUC_DB_VALUE_0)); - dev_err(kbdev->dev, " MCU GLB_REQ %x GLB_ACK %x", - kbase_csf_firmware_global_input_read(global_iface, GLB_REQ), - kbase_csf_firmware_global_output(global_iface, GLB_ACK)); + if (global_iface && global_iface->kbdev && global_iface->input && global_iface->output) + dev_err(kbdev->dev, " MCU GLB_REQ %x GLB_ACK %x", + kbase_csf_firmware_global_input_read(global_iface, GLB_REQ), + kbase_csf_firmware_global_output(global_iface, GLB_ACK)); } |