Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-03-01 Thread Julien Grall
On 01/03/17 10:51, Bhupinder Thakur wrote: Hi Julien, Hi Bhupinder, On 1 March 2017 at 02:52, Konrad Rzeszutek Wilk wrote: However, the pl011 ring is similar to the console ring. So maybe we should re-use the console header rather than re-inventing the wheel. Can I include a xen public

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-03-01 Thread Bhupinder Thakur
Hi Julien, On 1 March 2017 at 02:52, Konrad Rzeszutek Wilk wrote: >> However, the pl011 ring is similar to the console ring. So maybe we should >> re-use the console header rather than re-inventing the wheel. > Can I include a xen public header file (xen/include/public/io/console.h) in the xen co

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-28 Thread Konrad Rzeszutek Wilk
> However, the pl011 ring is similar to the console ring. So maybe we should > re-use the console header rather than re-inventing the wheel. That would be an excellent approach. > > Cheers, > > -- > Julien Grall ___ Xen-devel mailing list Xen-devel@l

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-28 Thread Konrad Rzeszutek Wilk
On Mon, Feb 20, 2017 at 08:53:50PM +0530, Bhupinder Thakur wrote: > Hi, > > > On 17 February 2017 at 20:59, Konrad Rzeszutek Wilk > wrote: > >> Should vpl011.h be in include/xen/public/ ? If so you need > >> a different license for that file. > >> > >> I have moved the file

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-26 Thread Julien Grall
Hi Konrad, On 02/17/2017 03:27 PM, Konrad Rzeszutek Wilk wrote: On Thu, Feb 16, 2017 at 02:34:30PM +, Julien Grall wrote: On 15/02/17 15:38, Konrad Rzeszutek Wilk wrote: On Wed, Feb 15, 2017 at 12:53:34PM +0530, Bhupinder Thakur wrote: We already have some HVM param that are x86 specific

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-20 Thread Bhupinder Thakur
Hi, On 17 February 2017 at 20:59, Konrad Rzeszutek Wilk wrote: >> Should vpl011.h be in include/xen/public/ ? If so you need >> a different license for that file. >> >> I have moved the file from the public folder and keeping it in xen/arch/arm/ > > Huh? But if this is a rin

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-17 Thread Konrad Rzeszutek Wilk
> Should vpl011.h be in include/xen/public/ ? If so you need > a different license for that file. > > I have moved the file from the public folder and keeping it in xen/arch/arm/ Huh? But if this is a ring protocol that is used by an OS that is not part of of Xen tree it needs

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-17 Thread Konrad Rzeszutek Wilk
On Thu, Feb 16, 2017 at 02:34:30PM +, Julien Grall wrote: > Hi, > > On 15/02/17 15:38, Konrad Rzeszutek Wilk wrote: > > On Wed, Feb 15, 2017 at 12:53:34PM +0530, Bhupinder Thakur wrote: > > > Hi Konrad, > > > > > > Thanks for the feedback. > > > > > > On 7 February 2017 at 00:05, Konrad Rzes

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-17 Thread Bhupinder Thakur
Hi, On 16 February 2017 at 20:04, Julien Grall wrote: > Hi, > > > On 15/02/17 15:38, Konrad Rzeszutek Wilk wrote: >> >> On Wed, Feb 15, 2017 at 12:53:34PM +0530, Bhupinder Thakur wrote: >>> >>> Hi Konrad, >>> >>> Thanks for the feedback. >>> >>> On 7 February 2017 at 00:05, Konrad Rzeszutek Wilk

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-16 Thread Julien Grall
Hi, On 15/02/17 15:38, Konrad Rzeszutek Wilk wrote: On Wed, Feb 15, 2017 at 12:53:34PM +0530, Bhupinder Thakur wrote: Hi Konrad, Thanks for the feedback. On 7 February 2017 at 00:05, Konrad Rzeszutek Wilk wrote: On Mon, Feb 06, 2017 at 11:39:08PM +0530, Bhupinder Thakur wrote: As per "VM S

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-15 Thread Konrad Rzeszutek Wilk
On Wed, Feb 15, 2017 at 12:53:34PM +0530, Bhupinder Thakur wrote: > Hi Konrad, > > Thanks for the feedback. > > On 7 February 2017 at 00:05, Konrad Rzeszutek Wilk > wrote: > > On Mon, Feb 06, 2017 at 11:39:08PM +0530, Bhupinder Thakur wrote: > >> As per "VM System Specification for ARM Processo

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-14 Thread Bhupinder Thakur
Hi Konrad, Thanks for the feedback. On 7 February 2017 at 00:05, Konrad Rzeszutek Wilk wrote: > On Mon, Feb 06, 2017 at 11:39:08PM +0530, Bhupinder Thakur wrote: >> As per "VM System Specification for ARM Processors", there is a requirement >> for Xen to support guest console >> over pl011 UAR

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-06 Thread Konrad Rzeszutek Wilk
On Mon, Feb 06, 2017 at 11:39:08PM +0530, Bhupinder Thakur wrote: > As per "VM System Specification for ARM Processors", there is a requirement > for Xen to support guest console > over pl011 UART, which is SBSA compliant. The changes in this patch implement > the pl011 emulation in Xen > and a

Re: [Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-06 Thread Julien Grall
Hi Bhupinder, Thank you for the patch. On 06/02/17 18:09, Bhupinder Thakur wrote: As per "VM System Specification for ARM Processors", there is a requirement for Xen to support guest console over pl011 UART, which is SBSA compliant. The changes in this patch implement the pl011 emulation in

[Xen-devel] [PATCH 1/1] xen/arm: Add pl011 uart support in Xen for guest domains

2017-02-06 Thread Bhupinder Thakur
As per "VM System Specification for ARM Processors", there is a requirement for Xen to support guest console over pl011 UART, which is SBSA compliant. The changes in this patch implement the pl011 emulation in Xen and a new pl011 console support in xenconsoled. Signed-off-by: Bhupinder Thakur