Peter Maydell <peter.mayd...@linaro.org> writes:
> process_its_cmd() returns a bool, like all the other process_ functions.
> However we were putting its return value into 'res', not 'result',
> which meant we would ignore it when deciding whether to continue
> or stall the command queue. Fix the typo.
Arguably having to generic result types is confusing. Naming things is
hard but maybe it should be a clear:
MemTxResult memtx_res = MEMTX_OK;
ItsCmdResult its_res = CMD_CONTINUE;
?
Anyway:
Reviewed-by: Alex Bennée <alex.ben...@linaro.org>
--
Alex Bennée
