[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/7] hw/misc/nrf51_rng: Add NRF51 random number
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 1/7] hw/misc/nrf51_rng: Add NRF51 random number generator peripheral |
Date: |
Thu, 16 Aug 2018 16:45:54 +0100 |
On 6 August 2018 at 11:01, Steffen Görtz <address@hidden> wrote:
> Add a model of the NRF51 random number generator peripheral.
> This is a simple random generator that continuously generates
> new random values after startup.
>
> Reference Manual: http://infocenter.nordicsemi.com/pdf/nRF51_RM_v3.0.pdf
>
> Signed-off-by: Steffen Görtz <address@hidden>
> ---
> hw/misc/Makefile.objs | 1 +
> hw/misc/nrf51_rng.c | 273 ++++++++++++++++++++++++++++++++++++
> include/hw/misc/nrf51_rng.h | 71 ++++++++++
> 3 files changed, 345 insertions(+)
> create mode 100644 hw/misc/nrf51_rng.c
> create mode 100644 include/hw/misc/nrf51_rng.h
> +static void update_irq(Nrf51RNGState *s)
See remark on other patch about consistency of struct naming.
> +{
> + bool irq = false;
> + irq |= s->interrupt_enabled && s->event_valrdy;
You don't need to set irq to false and then modify it;
this is equivalent to
bool irq = s->interrupt_enabled && s->event_valrdy;
> + qemu_set_irq(s->irq, irq);
> +}
> +static void nrf51_rng_reset(DeviceState *dev)
> +{
> + Nrf51RNGState *s = NRF51_RNG(dev);
This seems to be missing code to actually reset the fields ?
> +
> + rng_update_timer(s);
> +}
Otherwise the device looks good.
thanks
-- PMM
- [Qemu-devel] [PATCH 0/7] arm: nRF51 Devices and Microbit Support, Steffen Görtz, 2018/08/06
- [Qemu-devel] [PATCH 1/7] hw/misc/nrf51_rng: Add NRF51 random number generator peripheral, Steffen Görtz, 2018/08/06
- [Qemu-devel] [PATCH 7/7] hw/display/led_matrix: Add LED matrix display device, Steffen Görtz, 2018/08/06
- [Qemu-devel] [PATCH 5/7] tests/microbit-test: Add Tests for nRF51 GPIO, Steffen Görtz, 2018/08/06
- [Qemu-devel] [PATCH 2/7] hw/nvram/nrf51_nvm: Add nRF51 non-volatile memories, Steffen Görtz, 2018/08/06
- [Qemu-devel] [PATCH 4/7] hw/gpio/nrf51_gpio: Add nRF51 GPIO peripheral, Steffen Görtz, 2018/08/06