radeonsi: fix range checking for state regs
end value is exclusive, but in practice we shouldn't hit this. Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Christian König <deathsimple@vodafone.de>
This commit is contained in:
committed by
Christian König
parent
f1a3de5e9d
commit
918e302a19
@@ -57,17 +57,18 @@ void si_pm4_set_reg(struct si_pm4_state *state, unsigned reg, uint32_t val)
|
||||
{
|
||||
unsigned opcode;
|
||||
|
||||
if (reg >= SI_CONFIG_REG_OFFSET && reg <= SI_CONFIG_REG_END) {
|
||||
if (reg >= SI_CONFIG_REG_OFFSET && reg < SI_CONFIG_REG_END) {
|
||||
opcode = PKT3_SET_CONFIG_REG;
|
||||
reg -= SI_CONFIG_REG_OFFSET;
|
||||
|
||||
} else if (reg >= SI_SH_REG_OFFSET && reg <= SI_SH_REG_END) {
|
||||
} else if (reg >= SI_SH_REG_OFFSET && reg < SI_SH_REG_END) {
|
||||
opcode = PKT3_SET_SH_REG;
|
||||
reg -= SI_SH_REG_OFFSET;
|
||||
|
||||
} else if (reg >= SI_CONTEXT_REG_OFFSET && reg <= SI_CONTEXT_REG_END) {
|
||||
} else if (reg >= SI_CONTEXT_REG_OFFSET && reg < SI_CONTEXT_REG_END) {
|
||||
opcode = PKT3_SET_CONTEXT_REG;
|
||||
reg -= SI_CONTEXT_REG_OFFSET;
|
||||
|
||||
} else {
|
||||
R600_ERR("Invalid register offset %08x!\n", reg);
|
||||
return;
|
||||
|
||||
Reference in New Issue
Block a user