[XEN,v3,01/11] xen: arm: fix indentation of struct vtimer
diff mbox series

Message ID 20191115200115.44890-2-stewart.hildebrand@dornerworks.com
State New
Headers show
Series
  • xen: arm: context switch vtimer PPI state
Related show

Commit Message

Stewart Hildebrand Nov. 15, 2019, 8:01 p.m. UTC
From: Ian Campbell <ian.campbell@citrix.com>

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Reviewed-by: Julien Grall <julien.grall@citrix.com> [1]
Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> [2]

[1] https://lists.xenproject.org/archives/html/xen-devel/2015-11/msg00985.html
[2] https://lists.xenproject.org/archives/html/xen-devel/2015-12/msg02646.html

---
v3:
  * Rebase (no conflicts)
  * Add Reviewed-by and Acked-by from a few years ago
---
 xen/include/asm-arm/domain.h | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Julien Grall Nov. 23, 2019, 6:40 p.m. UTC | #1
Hi,

On 15/11/2019 20:01, Stewart Hildebrand wrote:
> From: Ian Campbell <ian.campbell@citrix.com>
> 
> Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> Reviewed-by: Julien Grall <julien.grall@citrix.com> [1]
> Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> [2]

Regardless the complexity of the patch, this was reviewed nearly 4 years 
ago and therefore review tags may be stall even if the reviewers are the 
same.

Indeed, the code base has changed quite a lot and some of the patches 
may require modifications to fit the direction of the project.

Anyway, for this patch:

Acked-by: Julien Grall <julien.grall@xen.org>

I will commit it in my branch for-next/4.14 and will merge it once 4.13 
has been released.

Cheers,
Julien Grall Nov. 23, 2019, 6:45 p.m. UTC | #2
On 23/11/2019 18:40, Julien Grall wrote:
> Hi,
> 
> On 15/11/2019 20:01, Stewart Hildebrand wrote:
>> From: Ian Campbell <ian.campbell@citrix.com>
>>
>> Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
>> Reviewed-by: Julien Grall <julien.grall@citrix.com> [1]
>> Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> [2]
> 
> Regardless the complexity of the patch, this was reviewed nearly 4 years 
> ago and therefore review tags may be stall even if the reviewers are the 
> same.
> 
> Indeed, the code base has changed quite a lot and some of the patches 
> may require modifications to fit the direction of the project.
> 
> Anyway, for this patch:
> 
> Acked-by: Julien Grall <julien.grall@xen.org>

Sorry, it should be julien@xen.org. Still getting use of my new e-mail.

> 
> I will commit it in my branch for-next/4.14 and will merge it once 4.13 
> has been released.
> 
> Cheers,
>

Patch
diff mbox series

diff --git a/xen/include/asm-arm/domain.h b/xen/include/asm-arm/domain.h
index 86ebdd2bcf..f3f3fb7d7f 100644
--- a/xen/include/asm-arm/domain.h
+++ b/xen/include/asm-arm/domain.h
@@ -35,11 +35,11 @@  enum domain_type {
 #define is_domain_direct_mapped(d) ((d) == hardware_domain)
 
 struct vtimer {
-        struct vcpu *v;
-        int irq;
-        struct timer timer;
-        uint32_t ctl;
-        uint64_t cval;
+    struct vcpu *v;
+    int irq;
+    struct timer timer;
+    uint32_t ctl;
+    uint64_t cval;
 };
 
 struct arch_domain