[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] ARM: add Calxeda Midway platform

On Tue, 2013-08-20 at 20:01 +0100, Julien Grall wrote:
> On 20 August 2013 16:03, Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote:
> > On Tue, 2013-08-13 at 17:13 +0200, Andre Przywara wrote:
> >> Calxeda Midway is an ARMv7 server platform with Cortex-A15 cores.
> >> The peripheral side has many similarities with the machine known as
> >> Highbank.
> >> Add Calxeda Midway to the list of supported platforms to avoid a
> >> warning on boot and provide the proper reset method.
> >>
> >> Signed-off-by: Andre Przywara <andre.przywara@xxxxxxxxxx>
> >
> > acked + applied, thanks.
> >
> >
> Few mails before, I asked Andre to change one thing in his patch.
> He planned to send a new version later.

Sorry, I skimmed the thread but missed your request for a change.
(trimming quotes would be a massive help here)

Andre, can you send an incremental patch please?

Julien suggested replacing "static const char const *" with "const char
* const", was the lack of "static" in the replacement deliberate?

Julien suggested that the other platform code was similarly broken,
could fix that at the same time too?


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.