diff mbox series

[05/15] hw/intc/armv7m_nvic.c: Remove out of date comment

Message ID 1501692241-23310-6-git-send-email-peter.maydell@linaro.org
State Superseded
Headers show
Series v7M: cleanups and bugfixes prior to v8M | expand

Commit Message

Peter Maydell Aug. 2, 2017, 4:43 p.m. UTC
Remove an out of date comment which says there's only one
item in the NVIC container region -- we put systick into its
own device object a while back and so now there are two
things in the container.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

---
 hw/intc/armv7m_nvic.c | 4 ----
 1 file changed, 4 deletions(-)

-- 
2.7.4

Comments

Edgar E. Iglesias Aug. 2, 2017, 5:48 p.m. UTC | #1
On Wed, Aug 02, 2017 at 05:43:51PM +0100, Peter Maydell wrote:
> Remove an out of date comment which says there's only one

> item in the NVIC container region -- we put systick into its

> own device object a while back and so now there are two

> things in the container.

> 

> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>


Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>



> ---

>  hw/intc/armv7m_nvic.c | 4 ----

>  1 file changed, 4 deletions(-)

> 

> diff --git a/hw/intc/armv7m_nvic.c b/hw/intc/armv7m_nvic.c

> index 323e2d4..2e8166a 100644

> --- a/hw/intc/armv7m_nvic.c

> +++ b/hw/intc/armv7m_nvic.c

> @@ -1036,10 +1036,6 @@ static void armv7m_nvic_realize(DeviceState *dev, Error **errp)

>       *  0xd00..0xd3c - SCS registers

>       *  0xd40..0xeff - Reserved or Not implemented

>       *  0xf00 - STIR

> -     *

> -     * At the moment there is only one thing in the container region,

> -     * but we leave it in place to allow us to pull systick out into

> -     * its own device object later.

>       */

>      memory_region_init(&s->container, OBJECT(s), "nvic", 0x1000);

>      /* The system register region goes at the bottom of the priority

> -- 

> 2.7.4

> 

>
Richard Henderson Aug. 3, 2017, 9:34 p.m. UTC | #2
On 08/02/2017 09:43 AM, Peter Maydell wrote:
> Remove an out of date comment which says there's only one

> item in the NVIC container region -- we put systick into its

> own device object a while back and so now there are two

> things in the container.

> 

> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

> ---

>  hw/intc/armv7m_nvic.c | 4 ----

>  1 file changed, 4 deletions(-)


Reviewed-by: Richard Henderson <richard.henderson@linaro.org>



r~
diff mbox series

Patch

diff --git a/hw/intc/armv7m_nvic.c b/hw/intc/armv7m_nvic.c
index 323e2d4..2e8166a 100644
--- a/hw/intc/armv7m_nvic.c
+++ b/hw/intc/armv7m_nvic.c
@@ -1036,10 +1036,6 @@  static void armv7m_nvic_realize(DeviceState *dev, Error **errp)
      *  0xd00..0xd3c - SCS registers
      *  0xd40..0xeff - Reserved or Not implemented
      *  0xf00 - STIR
-     *
-     * At the moment there is only one thing in the container region,
-     * but we leave it in place to allow us to pull systick out into
-     * its own device object later.
      */
     memory_region_init(&s->container, OBJECT(s), "nvic", 0x1000);
     /* The system register region goes at the bottom of the priority