Re: [Xen-devel] [PATCH 1/4] xen: arm64: Add Basic Platform support for APM X-Gene Storm.

[Julien Grall <julien.grall@xxxxxxxxxx>]

On 09/20/2013 02:28 PM, Pranavkumar Sawargaonkar wrote:
> HI Julien,
> 
> On Fri, Sep 20, 2013 at 5:39 PM, Julien Grall <julien.grall@xxxxxxxxxx> wrote:
>> On 09/20/2013 10:52 AM, Pranavkumar Sawargaonkar wrote:
>>> This patch adds following things:
>>> Early printk support for APM X-Gene platform.
>>
>> Does the UART is compatible with 8250?
> 
> Yes it is compatible but it is ns16550.

[..]

> I think better name would be debug-ns16550.inc ??

There are some UARTs that are 8250-compatible but not
ns16550-compatible. It would be better to stay as generic as possible to
support numerous UART.

>> Also, can you divide this patch in 2 part:
>>         - Early printk support;
>>         - Initial platform support.
>>
>> Shouldn't it be only enabled on ARM64?
> 
> But then arm32 boards should be only compiled for arm32 ?

Right :). We begin to have multiple platforms that are only supported in
one architecture (arm32 or arm64). So it's good time to introduce a way
to compile only supported platform.

-- 
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel