On 10/21/19 6:43 AM, Peter Maydell wrote: > In the slavio timer devcie, the ptimer TimerContext::timer is > always created by slavio_timer_init(), so there's no need to > check it for NULL; remove the single unneeded NULL check. > > This will be useful to avoid compiler/Coverity errors when > a subsequent change adds a use of t->timer before the location > we currently do the NULL check. > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > hw/timer/slavio_timer.c | 12 +++++------- > 1 file changed, 5 insertions(+), 7 deletions(-)
Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~