All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] xen/arm: Remove GIC definitions in asm-arm/config.h
@ 2014-04-02 14:13 Julien Grall
  2014-04-02 14:32 ` Ian Campbell
  0 siblings, 1 reply; 4+ messages in thread
From: Julien Grall @ 2014-04-02 14:13 UTC (permalink / raw)
  To: xen-devel; +Cc: stefano.stabellini, Julien Grall, tim, ian.campbell

The GIC is not used anynore in assembly code.

Signed-off-by: Julien Grall <julien.grall@linaro.org>
---
 xen/include/asm-arm/config.h |   11 -----------
 1 file changed, 11 deletions(-)

diff --git a/xen/include/asm-arm/config.h b/xen/include/asm-arm/config.h
index 5b7b1a8..ef291ff 100644
--- a/xen/include/asm-arm/config.h
+++ b/xen/include/asm-arm/config.h
@@ -192,17 +192,6 @@ extern unsigned long frametable_virt_end;
 #define watchdog_disable() ((void)0)
 #define watchdog_enable()  ((void)0)
 
-#ifdef __ASSEMBLY__
-/* Board-specific: regs base address for the GIC
- * Theses constants are only intend to be used in assembly file
- * because the DT is not yet parsed.
- */
-#define GIC_DR_OFFSET 0x1000
-#define GIC_CR_OFFSET 0x2000
-#define GIC_HR_OFFSET 0x4000 /* Guess work http://lists.infradead.org/pipermail/linux-arm-kernel/2011-September/064219.html */
-#define GIC_VR_OFFSET 0x6000 /* Virtual Machine CPU interface) */
-#endif /* __ASSEMBLY__ */
-
 #endif /* __ARM_CONFIG_H__ */
 /*
  * Local variables:
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] xen/arm: Remove GIC definitions in asm-arm/config.h
  2014-04-02 14:13 [PATCH] xen/arm: Remove GIC definitions in asm-arm/config.h Julien Grall
@ 2014-04-02 14:32 ` Ian Campbell
  2014-04-08 10:34   ` Julien Grall
  0 siblings, 1 reply; 4+ messages in thread
From: Ian Campbell @ 2014-04-02 14:32 UTC (permalink / raw)
  To: Julien Grall; +Cc: xen-devel, tim, stefano.stabellini

On Wed, 2014-04-02 at 15:13 +0100, Julien Grall wrote:
> The GIC is not used anynore in assembly code.

"anymore" (will fix on commit)

> Signed-off-by: Julien Grall <julien.grall@linaro.org>

Acked-by: Ian Campbell <ian.campbell@citrix.com>

Good riddance.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] xen/arm: Remove GIC definitions in asm-arm/config.h
  2014-04-02 14:32 ` Ian Campbell
@ 2014-04-08 10:34   ` Julien Grall
  2014-04-08 15:24     ` Ian Campbell
  0 siblings, 1 reply; 4+ messages in thread
From: Julien Grall @ 2014-04-08 10:34 UTC (permalink / raw)
  To: Ian Campbell; +Cc: xen-devel, tim, stefano.stabellini

Hi Ian,

On 04/02/2014 03:32 PM, Ian Campbell wrote:
> On Wed, 2014-04-02 at 15:13 +0100, Julien Grall wrote:
>> The GIC is not used anynore in assembly code.
> 
> "anymore" (will fix on commit)
>> Signed-off-by: Julien Grall <julien.grall@linaro.org>
> 
> Acked-by: Ian Campbell <ian.campbell@citrix.com>
> 
> Good riddance.

Do you plan to commit the patch soon?

Regards,

-- 
Julien Grall

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] xen/arm: Remove GIC definitions in asm-arm/config.h
  2014-04-08 10:34   ` Julien Grall
@ 2014-04-08 15:24     ` Ian Campbell
  0 siblings, 0 replies; 4+ messages in thread
From: Ian Campbell @ 2014-04-08 15:24 UTC (permalink / raw)
  To: Julien Grall; +Cc: xen-devel, tim, stefano.stabellini

On Tue, 2014-04-08 at 11:34 +0100, Julien Grall wrote:
> Hi Ian,
> 
> On 04/02/2014 03:32 PM, Ian Campbell wrote:
> > On Wed, 2014-04-02 at 15:13 +0100, Julien Grall wrote:
> >> The GIC is not used anynore in assembly code.
> > 
> > "anymore" (will fix on commit)
> >> Signed-off-by: Julien Grall <julien.grall@linaro.org>
> > 
> > Acked-by: Ian Campbell <ian.campbell@citrix.com>
> > 
> > Good riddance.
> 
> Do you plan to commit the patch soon?

I've just done it.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-04-08 15:24 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-02 14:13 [PATCH] xen/arm: Remove GIC definitions in asm-arm/config.h Julien Grall
2014-04-02 14:32 ` Ian Campbell
2014-04-08 10:34   ` Julien Grall
2014-04-08 15:24     ` Ian Campbell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.