+-- On Thu, 16 Jul 2020, Peter Maydell wrote --+ | > +static void imx7_digprog_write(void *opaque, hwaddr addr, | > + uint64_t data, unsigned size) | > +{ | > + qemu_log_mask(LOG_UNIMP, "%s not implemented\n", __func__); | > +} | | This covers a single register (DIGPROG) which is read-only (it returns a | silicon revision number). So this is not a LOG_UNIMP, but a LOG_GUEST_ERROR: | | qemu_log_mask(LOG_GUEST_ERROR, "Guest write to read-only | ANALOG_DIGPROG register\n");
Should this be g_assert_not_reached() in that case? Thank you. -- Prasad J Pandit / Red Hat Product Security Team 8685 545E B54C 486B C6EB 271E E285 8B5A F050 DE8D