From: "Emilio G. Cota" <cota@braap.org>
To: Alistair Francis <alistair.francis@xilinx.com>
Cc: qemu-devel@nongnu.org, peter.maydell@linaro.org,
alistair23@gmail.com, edgar.iglesias@xilinx.com,
edgar.iglesias@gmail.com, ehabkost@redhat.com, f4bug@amsat.org
Subject: Re: [Qemu-devel] [PATCH for 2.11 v2 2/2] xlnx-zcu102: Add an info message deprecating the EP108
Date: Fri, 10 Nov 2017 13:37:41 -0500 [thread overview]
Message-ID: <20171110183741.GA23011@flamenco> (raw)
In-Reply-To: <20171110180722.GA17844@flamenco>
On Fri, Nov 10, 2017 at 13:07:22 -0500, Emilio G. Cota wrote:
> On Thu, Nov 09, 2017 at 13:49:35 -0800, Alistair Francis wrote:
> > The EP108 was an early access development board that is no longer used.
> > Add an info message to convert any users to the ZCU102 instead. On QEMU
> > they are both identical.
> >
> > This patch also updated the qemu-doc.texi file to indicate that the
> > EP108 has been deprecated.
> >
> > Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
>
> Reviewed-by: Emilio G. Cota <cota@braap.org>
>
> Note to potential reviewers: This patch is meant to be applied after the
> patch that updates ep108's max_cpus from 1 to NR_APUS + NR_RPUS, for
> otherwise the message added here won't be seen unless -smp == 1.
> This other patch is already in target-arm.next:
> https://lists.gnu.org/archive/html/qemu-devel/2017-10/msg06537.html
Correction: that patch does updates the zcu102, not the ep108.
I think we should update max_cpus for ep108 as well.
Emilio
next prev parent reply other threads:[~2017-11-10 18:37 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-09 21:49 [Qemu-devel] [PATCH for 2.11 v2 0/2] Xilinx ZCU102 fixes for 2.11 Alistair Francis
2017-11-09 21:49 ` [Qemu-devel] [PATCH for 2.11 v2 2/2] xlnx-zcu102: Add an info message deprecating the EP108 Alistair Francis
2017-11-10 18:07 ` Emilio G. Cota
2017-11-10 18:37 ` Emilio G. Cota [this message]
[not found] ` <e23ac66f6ca0ac1b1bf89350e2b3a36a13e52957.1510263731.git.alistair.francis@xilinx.com>
2017-11-10 18:12 ` [Qemu-devel] [PATCH for 2.11 v2 1/2] xlnx-zynqmp: Properly support the smp command line option Emilio G. Cota
2017-11-10 18:52 ` [Qemu-devel] [PATCH for 2.11 v2 0/2] Xilinx ZCU102 fixes for 2.11 Emilio G. Cota
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171110183741.GA23011@flamenco \
--to=cota@braap.org \
--cc=alistair.francis@xilinx.com \
--cc=alistair23@gmail.com \
--cc=edgar.iglesias@gmail.com \
--cc=edgar.iglesias@xilinx.com \
--cc=ehabkost@redhat.com \
--cc=f4bug@amsat.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.