Hi Alistair, On 02/02/2018 04:38 PM, Alistair Francis wrote: > Ensure that the post write hook is called during reset. This allows us > to rely on the post write functions instead of having to call them from > the reset() function.
This makes sens. Can you add a comment about this new behavior in include/hw/register.h? > > Signed-off-by: Alistair Francis <alistair.fran...@xilinx.com> with comment: Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> > --- > > hw/core/register.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/hw/core/register.c b/hw/core/register.c > index 900294b9c4..0741a1af32 100644 > --- a/hw/core/register.c > +++ b/hw/core/register.c > @@ -159,13 +159,21 @@ uint64_t register_read(RegisterInfo *reg, uint64_t re, > const char* prefix, > > void register_reset(RegisterInfo *reg) > { > + const RegisterAccessInfo *ac; > + > g_assert(reg); > > if (!reg->data || !reg->access) { > return; > } > > + ac = reg->access; > + > register_write_val(reg, reg->access->reset); > + > + if (ac->post_write) { > + ac->post_write(reg, reg->access->reset); > + } > } > > void register_init(RegisterInfo *reg) >