From: David Marchand <david.marchand@redhat.com>
To: Thierry Herbelot <thierry.herbelot@6wind.com>
Cc: "Juraj Linkeš" <juraj.linkes@pantheon.tech>,
"dev@dpdk.org" <dev@dpdk.org>,
"Thomas Monjalon" <thomas@monjalon.net>,
"Honnappa Nagarahalli" <honnappa.nagarahalli@arm.com>,
"Ruifeng Wang" <ruifeng.wang@arm.com>
Subject: Re: [dpdk-dev] [PATCH] doc/cross_build: update the list supported Arm SoCs
Date: Thu, 29 Apr 2021 11:17:39 +0200 [thread overview]
Message-ID: <CAJFAV8zSY4kZnjJ9YOtFzUA9e3qR4ECoN0HrPAApNCczrwjYZg@mail.gmail.com> (raw)
In-Reply-To: <810b1f06f19c4421889395959904d7d9@pantheon.tech>
On Thu, Apr 29, 2021 at 11:12 AM Juraj Linkeš
<juraj.linkes@pantheon.tech> wrote:
> > -----Original Message-----
> > From: Thierry Herbelot <thierry.herbelot@6wind.com>
> > Sent: Thursday, April 29, 2021 9:54 AM
> > To: dev@dpdk.org
> > Cc: Thierry Herbelot <thierry.herbelot@6wind.com>; Thomas Monjalon
> > <thomas@monjalon.net>; Juraj Linkeš <juraj.linkes@pantheon.tech>; Honnappa
> > Nagarahalli <honnappa.nagarahalli@arm.com>; Ruifeng Wang
> > <ruifeng.wang@arm.com>
> > Subject: [PATCH] doc/cross_build: update the list supported Arm SoCs
> >
> > Align the list of supported targets with config/arm/meson.build
> >
> > Fixes: f2340c858046 ("config/arm: add platform config option")
> >
>
> I believe this also fixes the patches that introduced the kunpeng and cn10k soc. f2340c858046 was merged first (with n2 missing in the docs), then 7cf32a22b240 and 1b4c86a721c9 added the kunpeng and cn10k configs without adding to the docs.
>
> With that said,
> Reviewed-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
This patch is wrong.
It fixes the qualcomm doc part while it should go in:
http://patchwork.dpdk.org/project/dpdk/patch/20210429080644.11423-1-thierry.herbelot@6wind.com/
It fixes kunpeng and cn10k, so the fixes line is wrong.
Please drop it, I already submitted a different fix for the latter.
--
David Marchand
prev parent reply other threads:[~2021-04-29 9:17 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-29 7:54 Thierry Herbelot
2021-04-29 9:12 ` Juraj Linkeš
2021-04-29 9:17 ` David Marchand [this message]
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=CAJFAV8zSY4kZnjJ9YOtFzUA9e3qR4ECoN0HrPAApNCczrwjYZg@mail.gmail.com \
--to=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=honnappa.nagarahalli@arm.com \
--cc=juraj.linkes@pantheon.tech \
--cc=ruifeng.wang@arm.com \
--cc=thierry.herbelot@6wind.com \
--cc=thomas@monjalon.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).