diff options
| author | Neil Roberts <[email protected]> | 2020-07-05 15:45:19 +0200 |
|---|---|---|
| committer | Alejandro Piñeiro <[email protected]> | 2020-07-07 21:40:16 +0200 |
| commit | b921d17aa69af0a02639461d8c5d829ebb0d0349 (patch) | |
| tree | f0e481635acd1eb28edb0ff68093be691e915742 /src | |
| parent | ec4d3def165307d6d419c50a21c24f4c3c15e9a4 (diff) | |
broadcom/qpu: set VC5_QPU_RADDR_A out of the switch at _pack_branch
Detected after mesa added Wimplicit-fallthrough project wide.
Reviewed-by: Eric Anholt <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5769>
Diffstat (limited to 'src')
| -rw-r--r-- | src/broadcom/qpu/qpu_pack.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/broadcom/qpu/qpu_pack.c b/src/broadcom/qpu/qpu_pack.c index e70daf55a9f..7502bbfb9d6 100644 --- a/src/broadcom/qpu/qpu_pack.c +++ b/src/broadcom/qpu/qpu_pack.c @@ -1452,14 +1452,14 @@ v3d_qpu_instr_pack_branch(const struct v3d_device_info *devinfo, *packed_instr |= QPU_SET_FIELD(instr->branch.offset >> 24, VC5_QPU_BRANCH_ADDR_HIGH); + default: + break; + } - case V3D_QPU_BRANCH_DEST_REGFILE: + if (instr->branch.bdi == V3D_QPU_BRANCH_DEST_REGFILE || + instr->branch.bdu == V3D_QPU_BRANCH_DEST_REGFILE) { *packed_instr |= QPU_SET_FIELD(instr->branch.raddr_a, VC5_QPU_RADDR_A); - break; - - default: - break; } return true; |