hw/misc: Don't special case RESET_TYPE_COLD in npcm7xx_clk, gcr
authorPeter Maydell <peter.maydell@linaro.org>
Fri, 12 Apr 2024 16:08:04 +0000 (17:08 +0100)
committerPeter Maydell <peter.maydell@linaro.org>
Thu, 25 Apr 2024 09:21:06 +0000 (10:21 +0100)
commit1e0f2b38ac104ec3606750cce847cc9d8e4f66ac
treea71e4bc64668da22cfe27c4c629124f368d7c578
parenta6819c1bd0b76d7fa0c8f4ae19e4e80dd61a1502
hw/misc: Don't special case RESET_TYPE_COLD in npcm7xx_clk, gcr

The npcm7xx_clk and npcm7xx_gcr device reset methods look at
the ResetType argument and only handle RESET_TYPE_COLD,
producing a warning if another reset type is passed. This
is different from how every other three-phase-reset method
we have works, and makes it difficult to add new reset types.

A better pattern is "assume that any reset type you don't know
about should be handled like RESET_TYPE_COLD"; switch these
devices to do that. Then adding a new reset type will only
need to touch those devices where its behaviour really needs
to be different from the standard cold reset.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Luc Michel <luc.michel@amd.com>
Message-id: 20240412160809.1260625-2-peter.maydell@linaro.org
hw/misc/npcm7xx_clk.c
hw/misc/npcm7xx_gcr.c