Re: [PATCH v3 0/2] Fix coverity issues for AST2700

[Cédric Le Goater]

Hello Jamin

On 6/25/24 9:07 AM, Jamin Lin wrote:

change from v1:
aspeed/soc: coverity defect: DIVIDE_BY_ZERO
aspeed/sdmc: coverity defect: Control flow issues (DEADCODE)

change from v2:
add more commit log from reviewer suggestion, Cédric.

change from v3:
replace qemu_log_mask with assert dram size 0.

Jamin Lin (2):
   aspeed/soc: Fix possible divide by zero
   aspeed/sdmc: Remove extra R_MAIN_STATUS case

  hw/arm/aspeed_ast27x0.c | 2 ++
  hw/misc/aspeed_sdmc.c   | 1 -
  2 files changed, 2 insertions(+), 1 deletion(-)

LGTM.

Could you please review :

https://lore.kernel.org/qemu-devel/20240625065839.485034-1-...@redhat.com/T/#u

Thanks,

C.