On Fri, Feb 2, 2018 at 12:24 PM, Philippe Mathieu-Daudé <f4...@amsat.org> wrote: > 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?
Yep, fixed in V2 which I'll send out next week. Alistair > >> >> 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) >>