All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: Guo Ren <guoren@kernel.org>
Cc: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>,
	arnout@mind.be, linux-csky@vger.kernel.org,
	buildroot@buildroot.org, Guo Ren <ren_guo@c-sky.com>
Subject: Re: [PATCH V3 01/10] arch/csky: Use VDSP instead of DSP
Date: Wed, 29 May 2019 10:38:24 +0200	[thread overview]
Message-ID: <20190529103824.58560e4e@windsurf.home> (raw)
In-Reply-To: <CAJF2gTT5N6ST9Ph-L+bX2WtNx4ENu0wQ2uQ63D=3C-Zqoit+3w@mail.gmail.com>

Hello,

On Wed, 29 May 2019 16:12:25 +0800
Guo Ren <guoren@kernel.org> wrote:

> > Not just two patches: also a better commit log for the DSP -> VDSP
> > change.  
> 
> I want these two patches:
> 
> 1. Remove DSP, because it's no use in current buildroot.
> 2. Add VDSP support

OK, but then please split in yet another patch the fact of using
arch.mk.csky to build the GCC_TARGET_CPU variable.

> Here is my commit log for Introduce VDSP feature:
> 
> VDSP is C-SKY enhanced extension instruction set for SIMD, AI and DSP
> operation. It could be support by abiv2 (ck807, ck810, ck860).
> 
> Is that OK?

Yes. But please add a proper commit log in the patch removing the DSP
option.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

WARNING: multiple messages have this Message-ID (diff)
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH V3 01/10] arch/csky: Use VDSP instead of DSP
Date: Wed, 29 May 2019 10:38:24 +0200	[thread overview]
Message-ID: <20190529103824.58560e4e@windsurf.home> (raw)
In-Reply-To: <CAJF2gTT5N6ST9Ph-L+bX2WtNx4ENu0wQ2uQ63D=3C-Zqoit+3w@mail.gmail.com>

Hello,

On Wed, 29 May 2019 16:12:25 +0800
Guo Ren <guoren@kernel.org> wrote:

> > Not just two patches: also a better commit log for the DSP -> VDSP
> > change.  
> 
> I want these two patches:
> 
> 1. Remove DSP, because it's no use in current buildroot.
> 2. Add VDSP support

OK, but then please split in yet another patch the fact of using
arch.mk.csky to build the GCC_TARGET_CPU variable.

> Here is my commit log for Introduce VDSP feature:
> 
> VDSP is C-SKY enhanced extension instruction set for SIMD, AI and DSP
> operation. It could be support by abiv2 (ck807, ck810, ck860).
> 
> Is that OK?

Yes. But please add a proper commit log in the patch removing the DSP
option.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2019-05-29  8:38 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-29  7:18 [PATCH V3 00/10] Update arch/csky for buildroot guoren
2019-05-29  7:18 ` [Buildroot] " guoren at kernel.org
2019-05-29  7:18 ` [PATCH V3 01/10] arch/csky: Use VDSP instead of DSP guoren
2019-05-29  7:18   ` [Buildroot] " guoren at kernel.org
2019-05-29  7:21   ` Thomas Petazzoni
2019-05-29  7:21     ` [Buildroot] " Thomas Petazzoni
2019-05-29  7:30     ` Guo Ren
2019-05-29  7:30       ` [Buildroot] " Guo Ren
2019-05-29  7:56       ` Thomas Petazzoni
2019-05-29  7:56         ` [Buildroot] " Thomas Petazzoni
2019-05-29  8:12         ` Guo Ren
2019-05-29  8:12           ` [Buildroot] " Guo Ren
2019-05-29  8:38           ` Thomas Petazzoni [this message]
2019-05-29  8:38             ` Thomas Petazzoni
2019-05-29  9:02             ` Guo Ren
2019-05-29  9:02               ` [Buildroot] " Guo Ren
2019-05-29  9:06               ` Guo Ren
2019-05-29  9:06                 ` [Buildroot] " Guo Ren
2019-05-29  9:23               ` Thomas Petazzoni
2019-05-29  9:23                 ` [Buildroot] " Thomas Petazzoni
2019-05-29  9:45                 ` Guo Ren
2019-05-29  9:45                   ` [Buildroot] " Guo Ren
2019-05-29 11:58                   ` Thomas Petazzoni
2019-05-29 11:58                     ` [Buildroot] " Thomas Petazzoni
2019-05-29 12:34                     ` Guo Ren
2019-05-29 12:34                       ` [Buildroot] " Guo Ren
2019-05-29  7:18 ` [PATCH V3 02/10] arch/csky: Add ck860 supported guoren
2019-05-29  7:18   ` [Buildroot] " guoren at kernel.org
2019-05-29  7:18 ` [PATCH V3 03/10] arch/csky: Add FLOAT_ABI compiler options guoren
2019-05-29  7:18   ` [Buildroot] " guoren at kernel.org
2019-05-29  7:23   ` Thomas Petazzoni
2019-05-29  7:23     ` [Buildroot] " Thomas Petazzoni
2019-05-29  7:30     ` Guo Ren
2019-05-29  7:30       ` [Buildroot] " Guo Ren
2019-05-29  7:57       ` Thomas Petazzoni
2019-05-29  7:57         ` [Buildroot] " Thomas Petazzoni
2019-05-29  8:30         ` Guo Ren
2019-05-29  8:30           ` [Buildroot] " Guo Ren
2019-05-29  8:39           ` Thomas Petazzoni
2019-05-29  8:39             ` [Buildroot] " Thomas Petazzoni
2019-05-29  8:53             ` Guo Ren
2019-05-29  8:53               ` [Buildroot] " Guo Ren
2019-05-29  7:39     ` Guo Ren
2019-05-29  7:39       ` [Buildroot] " Guo Ren
2019-05-29  7:18 ` [PATCH V3 04/10] arch/csky: Add ABI variable for toolchain build guoren
2019-05-29  7:18   ` [Buildroot] " guoren at kernel.org
2019-05-29  7:18 ` [PATCH V3 05/10] package/binutils: Add C-SKY support guoren
2019-05-29  7:18   ` [Buildroot] " guoren at kernel.org
2019-05-29  7:18 ` [PATCH V3 06/10] package/gcc: " guoren
2019-05-29  7:18   ` [Buildroot] " guoren at kernel.org
2019-05-29  7:18 ` [PATCH V3 07/10] package/gdb: " guoren
2019-05-29  7:18   ` [Buildroot] " guoren at kernel.org
2019-05-29  7:18 ` [PATCH V3 08/10] package/glibc: " guoren
2019-05-29  7:18   ` [Buildroot] " guoren at kernel.org
2019-05-29  7:18 ` [PATCH V3 09/10] configs/qemu_cskyXXX_virt: new defconfig guoren
2019-05-29  7:18   ` [Buildroot] " guoren at kernel.org
2019-05-29  7:18 ` [PATCH V3 10/10] arch/csky: Enable csky toolchain build guoren
2019-05-29  7:18   ` [Buildroot] " guoren at kernel.org

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=20190529103824.58560e4e@windsurf.home \
    --to=thomas.petazzoni@bootlin.com \
    --cc=arnout@mind.be \
    --cc=buildroot@buildroot.org \
    --cc=guoren@kernel.org \
    --cc=linux-csky@vger.kernel.org \
    --cc=ren_guo@c-sky.com \
    --cc=thomas.de_schampheleire@nokia.com \
    /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.