Re: [Xen-devel] [PATCH 2/3] xen/drivers/char: Don't require vpl011 for all non-x86 archs

2019-05-20 Thread Julien Grall
Hi Alistair, On 17/05/2019 23:01, Alistair Francis wrote: On Fri, May 17, 2019 at 1:46 AM Julien Grall wrote: On 16/05/2019 20:30, Alistair Francis wrote: On Thu, May 16, 2019 at 3:32 AM Jan Beulich wrote: On 16.05.19 at 02:02, wrote: Make the asm/vpl011.h dependent on the ARM

Re: [Xen-devel] [PATCH 2/3] xen/drivers/char: Don't require vpl011 for all non-x86 archs

2019-05-17 Thread Alistair Francis
On Thu, May 16, 2019 at 11:26 PM Jan Beulich wrote: > > >>> On 16.05.19 at 21:30, wrote: > > On Thu, May 16, 2019 at 3:32 AM Jan Beulich wrote: > >> > >> >>> On 16.05.19 at 02:02, wrote: > >> > Make the asm/vpl011.h dependent on the ARM architecture. > >> > >> But we only have x86 and Arm

Re: [Xen-devel] [PATCH 2/3] xen/drivers/char: Don't require vpl011 for all non-x86 archs

2019-05-17 Thread Alistair Francis
On Fri, May 17, 2019 at 1:46 AM Julien Grall wrote: > > > > On 16/05/2019 20:30, Alistair Francis wrote: > > On Thu, May 16, 2019 at 3:32 AM Jan Beulich wrote: > >> > > On 16.05.19 at 02:02, wrote: > >>> Make the asm/vpl011.h dependent on the ARM architecture. > >> > >> But we only have x86

Re: [Xen-devel] [PATCH 2/3] xen/drivers/char: Don't require vpl011 for all non-x86 archs

2019-05-17 Thread Julien Grall
On 16/05/2019 20:30, Alistair Francis wrote: On Thu, May 16, 2019 at 3:32 AM Jan Beulich wrote: On 16.05.19 at 02:02, wrote: Make the asm/vpl011.h dependent on the ARM architecture. But we only have x86 and Arm right now. A word more about your motivation would help. As the code

Re: [Xen-devel] [PATCH 2/3] xen/drivers/char: Don't require vpl011 for all non-x86 archs

2019-05-17 Thread Jan Beulich
>>> On 16.05.19 at 21:30, wrote: > On Thu, May 16, 2019 at 3:32 AM Jan Beulich wrote: >> >> >>> On 16.05.19 at 02:02, wrote: >> > Make the asm/vpl011.h dependent on the ARM architecture. >> >> But we only have x86 and Arm right now. A word more about >> your motivation would help. > > As the

Re: [Xen-devel] [PATCH 2/3] xen/drivers/char: Don't require vpl011 for all non-x86 archs

2019-05-16 Thread Alistair Francis
On Thu, May 16, 2019 at 3:32 AM Jan Beulich wrote: > > >>> On 16.05.19 at 02:02, wrote: > > Make the asm/vpl011.h dependent on the ARM architecture. > > But we only have x86 and Arm right now. A word more about > your motivation would help. As the code currently is no one can add another

Re: [Xen-devel] [PATCH 2/3] xen/drivers/char: Don't require vpl011 for all non-x86 archs

2019-05-16 Thread Jan Beulich
>>> On 16.05.19 at 02:02, wrote: > Make the asm/vpl011.h dependent on the ARM architecture. But we only have x86 and Arm right now. A word more about your motivation would help. Also I don't think your Cc list is wide enough for this change. Jan

[Xen-devel] [PATCH 2/3] xen/drivers/char: Don't require vpl011 for all non-x86 archs

2019-05-15 Thread Alistair Francis
Make the asm/vpl011.h dependent on the ARM architecture. Signed-off-by: Alistair Francis --- xen/drivers/char/console.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/drivers/char/console.c b/xen/drivers/char/console.c index 9bbcb0f57a..f840d999bc 100644 ---