[PATCH 35/43] drm/amdgpu/gfx6: drop all BUG()s

[Alex Deucher]

There's no need to crash the kernel for these cases.

Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
index ac90d8e9d86a8..4ba9ff7c3b3f6 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
@@ -347,7 +347,8 @@ static int gfx_v6_0_init_microcode(struct amdgpu_device 
*adev)
        case CHIP_HAINAN:
                chip_name = "hainan";
                break;
-       default: BUG();
+       default:
+               return -EINVAL;
        }
 
        err = amdgpu_ucode_request(adev, &adev->gfx.pfp_fw,
@@ -1736,7 +1737,6 @@ static void gfx_v6_0_constants_init(struct amdgpu_device 
*adev)
                gb_addr_config = HAINAN_GB_ADDR_CONFIG_GOLDEN;
                break;
        default:
-               BUG();
                break;
        }
 
@@ -2202,7 +2202,7 @@ static u64 gfx_v6_0_ring_get_wptr(struct amdgpu_ring 
*ring)
        else if (ring == &adev->gfx.compute_ring[1])
                return RREG32(mmCP_RB2_WPTR);
        else
-               BUG();
+               return 0;
 }
 
 static void gfx_v6_0_ring_set_wptr_gfx(struct amdgpu_ring *ring)
@@ -2223,8 +2223,6 @@ static void gfx_v6_0_ring_set_wptr_compute(struct 
amdgpu_ring *ring)
        } else if (ring == &adev->gfx.compute_ring[1]) {
                WREG32(mmCP_RB2_WPTR, lower_32_bits(ring->wptr));
                (void)RREG32(mmCP_RB2_WPTR);
-       } else {
-               BUG();
        }
 
 }
@@ -3310,7 +3308,6 @@ static void 
gfx_v6_0_set_compute_eop_interrupt_state(struct amdgpu_device *adev,
                }
 
        default:
-               BUG();
                break;
 
        }
-- 
2.54.0