* [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv @ 2018-08-30 13:17 Bin Meng 2018-09-03 9:50 ` Bin Meng 2018-09-06 13:45 ` [U-Boot] " Tom Rini 0 siblings, 2 replies; 9+ messages in thread From: Bin Meng @ 2018-08-30 13:17 UTC (permalink / raw) To: u-boot This updates travis configuration to use kernel.org pre-built toolchain for riscv. Signed-off-by: Bin Meng <bmeng.cn@gmail.com> --- .travis.yml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/.travis.yml b/.travis.yml index 4db629d..95cfa5b 100644 --- a/.travis.yml +++ b/.travis.yml @@ -87,9 +87,8 @@ before_script: fi - if [[ "${TOOLCHAIN}" == "powerpc" ]]; then ./tools/buildman/buildman --fetch-arch powerpc; fi - if [[ "${TOOLCHAIN}" == "riscv" ]]; then - wget https://github.com/andestech/prebuilt/releases/download/20180530/riscv64-unknown-linux-gnu.tar.gz && - tar -C /tmp -xf riscv64-unknown-linux-gnu.tar.gz && - echo -e "\n[toolchain-prefix]\nriscv = /tmp/riscv64-unknown-linux-gnu/bin/riscv64-unknown-linux-gnu-" >> ~/.buildman; + ./tools/buildman/buildman --fetch-arch riscv64; + echo -e "\n[toolchain-alias]\nriscv = riscv64" >> ~/.buildman; fi - if [[ "${QEMU_TARGET}" != "" ]]; then git clone git://git.qemu.org/qemu.git /tmp/qemu; -- 2.7.4 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv 2018-08-30 13:17 [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv Bin Meng @ 2018-09-03 9:50 ` Bin Meng 2018-09-03 19:23 ` Tom Rini 2018-09-06 13:45 ` [U-Boot] " Tom Rini 1 sibling, 1 reply; 9+ messages in thread From: Bin Meng @ 2018-09-03 9:50 UTC (permalink / raw) To: u-boot Hi Tom, On Thu, Aug 30, 2018 at 9:12 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > This updates travis configuration to use kernel.org pre-built > toolchain for riscv. > > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> > --- > > .travis.yml | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > Since this is travis-ci changes, can we take this for v2018.09? I've verified that it works well for current riscv build on travis-ci.org. Regards, Bin ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv 2018-09-03 9:50 ` Bin Meng @ 2018-09-03 19:23 ` Tom Rini 2018-09-04 1:27 ` Bin Meng [not found] ` <752D002CFF5D0F4FA35C0100F1D73F3F6BCB9FC7@ATCPCS16.andestech.com> 0 siblings, 2 replies; 9+ messages in thread From: Tom Rini @ 2018-09-03 19:23 UTC (permalink / raw) To: u-boot On Mon, Sep 03, 2018 at 05:50:39PM +0800, Bin Meng wrote: > Hi Tom, > > On Thu, Aug 30, 2018 at 9:12 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > This updates travis configuration to use kernel.org pre-built > > toolchain for riscv. > > > > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> > > --- > > > > .travis.yml | 5 ++--- > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > Since this is travis-ci changes, can we take this for v2018.09? I've > verified that it works well for current riscv build on travis-ci.org. Seeing as things are still working before this change I would like to see an ack from the custodian, thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180903/f051bcd3/attachment.sig> ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv 2018-09-03 19:23 ` Tom Rini @ 2018-09-04 1:27 ` Bin Meng [not found] ` <752D002CFF5D0F4FA35C0100F1D73F3F6BCB9FC7@ATCPCS16.andestech.com> 1 sibling, 0 replies; 9+ messages in thread From: Bin Meng @ 2018-09-04 1:27 UTC (permalink / raw) To: u-boot Hi Rick, On Tue, Sep 4, 2018 at 3:23 AM Tom Rini <trini@konsulko.com> wrote: > > On Mon, Sep 03, 2018 at 05:50:39PM +0800, Bin Meng wrote: > > Hi Tom, > > > > On Thu, Aug 30, 2018 at 9:12 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > > > This updates travis configuration to use kernel.org pre-built > > > toolchain for riscv. > > > > > > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> > > > --- > > > > > > .travis.yml | 5 ++--- > > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > > > > Since this is travis-ci changes, can we take this for v2018.09? I've > > verified that it works well for current riscv build on travis-ci.org. > > Seeing as things are still working before this change I would like to > see an ack from the custodian, thanks! Are you OK with this? Regards, Bin ^ permalink raw reply [flat|nested] 9+ messages in thread
[parent not found: <752D002CFF5D0F4FA35C0100F1D73F3F6BCB9FC7@ATCPCS16.andestech.com>]
* [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv [not found] ` <752D002CFF5D0F4FA35C0100F1D73F3F6BCB9FC7@ATCPCS16.andestech.com> @ 2018-09-04 5:36 ` Rick Chen 2018-09-04 8:56 ` Bin Meng 0 siblings, 1 reply; 9+ messages in thread From: Rick Chen @ 2018-09-04 5:36 UTC (permalink / raw) To: u-boot > From: Tom Rini [mailto:trini at konsulko.com] > Sent: Tuesday, September 04, 2018 3:24 AM > To: Bin Meng > Cc: U-Boot Mailing List; Rick Jian-Zhi Chen(陳建志) > Subject: Re: [PATCH] travis: Use kernel.org pre-built toolchain for riscv > > On Mon, Sep 03, 2018 at 05:50:39PM +0800, Bin Meng wrote: > > Hi Tom, > > > > On Thu, Aug 30, 2018 at 9:12 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > > > This updates travis configuration to use kernel.org pre-built > > > toolchain for riscv. > > > > > > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> > > > --- > > > > > > .travis.yml | 5 ++--- > > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > > > > Since this is travis-ci changes, can we take this for v2018.09? I've > > verified that it works well for current riscv build on travis-ci.org. > > Seeing as things are still working before this change I would like to see an ack > from the custodian, thanks! > > -- > Tom Hi Bin I think lt is a positive way to use kernel.org pre-built toolchain for code coverage. And very agree with that. But there is a problem, it maybe encounter in the future. If I or someone try to add a private csr which the public toolchain may not recognize it. How shall it be overcomed ? Do you have any ideas ? Rick ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv 2018-09-04 5:36 ` Rick Chen @ 2018-09-04 8:56 ` Bin Meng 2018-09-06 1:43 ` Bin Meng 0 siblings, 1 reply; 9+ messages in thread From: Bin Meng @ 2018-09-04 8:56 UTC (permalink / raw) To: u-boot Hi Rick, On Tue, Sep 4, 2018 at 1:36 PM Rick Chen <rickchen36@gmail.com> wrote: > > > From: Tom Rini [mailto:trini at konsulko.com] > > Sent: Tuesday, September 04, 2018 3:24 AM > > To: Bin Meng > > Cc: U-Boot Mailing List; Rick Jian-Zhi Chen(陳建志) > > Subject: Re: [PATCH] travis: Use kernel.org pre-built toolchain for riscv > > > > On Mon, Sep 03, 2018 at 05:50:39PM +0800, Bin Meng wrote: > > > Hi Tom, > > > > > > On Thu, Aug 30, 2018 at 9:12 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > > > > > This updates travis configuration to use kernel.org pre-built > > > > toolchain for riscv. > > > > > > > > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> > > > > --- > > > > > > > > .travis.yml | 5 ++--- > > > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > > > > > > > Since this is travis-ci changes, can we take this for v2018.09? I've > > > verified that it works well for current riscv build on travis-ci.org. > > > > Seeing as things are still working before this change I would like > to see an ack > > from the custodian, thanks! > > > > -- > > Tom > > Hi Bin > > I think lt is a positive way to use kernel.org pre-built toolchain for > code coverage. > And very agree with that. > > But there is a problem, it maybe encounter in the future. > If I or someone try to add a private csr which the public toolchain > may not recognize it. I assume what you said "private" csr means "vendor-specific" csr that is not defined by the risc-v ISA? > How shall it be overcomed ? Do you have any ideas ? I believe this shall be implemented properly by the toolchain itself. Or we hardcode the instruction for such "private" csr if there is no good solution from the toolchain side. Having a custom toolchain seems not a good idea. Regards, Bin ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv 2018-09-04 8:56 ` Bin Meng @ 2018-09-06 1:43 ` Bin Meng 2018-09-06 1:53 ` Rick Chen 0 siblings, 1 reply; 9+ messages in thread From: Bin Meng @ 2018-09-06 1:43 UTC (permalink / raw) To: u-boot Hi Rick, On Tue, Sep 4, 2018 at 4:56 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > Hi Rick, > > On Tue, Sep 4, 2018 at 1:36 PM Rick Chen <rickchen36@gmail.com> wrote: > > > > > From: Tom Rini [mailto:trini at konsulko.com] > > > Sent: Tuesday, September 04, 2018 3:24 AM > > > To: Bin Meng > > > Cc: U-Boot Mailing List; Rick Jian-Zhi Chen(陳建志) > > > Subject: Re: [PATCH] travis: Use kernel.org pre-built toolchain for riscv > > > > > > On Mon, Sep 03, 2018 at 05:50:39PM +0800, Bin Meng wrote: > > > > Hi Tom, > > > > > > > > On Thu, Aug 30, 2018 at 9:12 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > > > > > > > This updates travis configuration to use kernel.org pre-built > > > > > toolchain for riscv. > > > > > > > > > > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> > > > > > --- > > > > > > > > > > .travis.yml | 5 ++--- > > > > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > > > > > > > > > > Since this is travis-ci changes, can we take this for v2018.09? I've > > > > verified that it works well for current riscv build on travis-ci.org. > > > > > > Seeing as things are still working before this change I would like > > to see an ack > > > from the custodian, thanks! > > > > > > -- > > > Tom > > > > Hi Bin > > > > I think lt is a positive way to use kernel.org pre-built toolchain for > > code coverage. > > And very agree with that. > > > > But there is a problem, it maybe encounter in the future. > > If I or someone try to add a private csr which the public toolchain > > may not recognize it. > > I assume what you said "private" csr means "vendor-specific" csr that > is not defined by the risc-v ISA? > > > How shall it be overcomed ? Do you have any ideas ? > > I believe this shall be implemented properly by the toolchain itself. > Or we hardcode the instruction for such "private" csr if there is no > good solution from the toolchain side. Having a custom toolchain seems > not a good idea. I did not see a confirmative response on whether this patch is OK. Can you please comment, and if no issue I hope this can catch up the train of upcoming release :) Regards, Bin ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv 2018-09-06 1:43 ` Bin Meng @ 2018-09-06 1:53 ` Rick Chen 0 siblings, 0 replies; 9+ messages in thread From: Rick Chen @ 2018-09-06 1:53 UTC (permalink / raw) To: u-boot Bin Meng <bmeng.cn@gmail.com> 於 2018年9月6日 週四 上午9:43寫道: > > Hi Rick, > > On Tue, Sep 4, 2018 at 4:56 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > Hi Rick, > > > > On Tue, Sep 4, 2018 at 1:36 PM Rick Chen <rickchen36@gmail.com> wrote: > > > > > > > From: Tom Rini [mailto:trini at konsulko.com] > > > > Sent: Tuesday, September 04, 2018 3:24 AM > > > > To: Bin Meng > > > > Cc: U-Boot Mailing List; Rick Jian-Zhi Chen(陳建志) > > > > Subject: Re: [PATCH] travis: Use kernel.org pre-built toolchain for riscv > > > > > > > > On Mon, Sep 03, 2018 at 05:50:39PM +0800, Bin Meng wrote: > > > > > Hi Tom, > > > > > > > > > > On Thu, Aug 30, 2018 at 9:12 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > > > > > > > > > This updates travis configuration to use kernel.org pre-built > > > > > > toolchain for riscv. > > > > > > > > > > > > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> > > > > > > --- > > > > > > > > > > > > .travis.yml | 5 ++--- > > > > > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > > > > > > > > > > > > > Since this is travis-ci changes, can we take this for v2018.09? I've > > > > > verified that it works well for current riscv build on travis-ci.org. > > > > > > > > Seeing as things are still working before this change I would like > > > to see an ack > > > > from the custodian, thanks! > > > > > > > > -- > > > > Tom > > > > > > Hi Bin > > > > > > I think lt is a positive way to use kernel.org pre-built toolchain for > > > code coverage. > > > And very agree with that. > > > > > > But there is a problem, it maybe encounter in the future. > > > If I or someone try to add a private csr which the public toolchain > > > may not recognize it. > > > > I assume what you said "private" csr means "vendor-specific" csr that > > is not defined by the risc-v ISA? > > > > > How shall it be overcomed ? Do you have any ideas ? > > > > I believe this shall be implemented properly by the toolchain itself. > > Or we hardcode the instruction for such "private" csr if there is no > > good solution from the toolchain side. Having a custom toolchain seems > > not a good idea. Hi Bin Agree with you, maybe I shall consider to isolate the vendor-specific code with CONFIG_XXX. > > I did not see a confirmative response on whether this patch is OK. Can > you please comment, and if no issue I hope this can catch up the train > of upcoming release :) > Hi Tom I think it is OK to pull this change into the master. Hope everything goes fine :) Rick > Regards, > Bin ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] travis: Use kernel.org pre-built toolchain for riscv 2018-08-30 13:17 [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv Bin Meng 2018-09-03 9:50 ` Bin Meng @ 2018-09-06 13:45 ` Tom Rini 1 sibling, 0 replies; 9+ messages in thread From: Tom Rini @ 2018-09-06 13:45 UTC (permalink / raw) To: u-boot On Thu, Aug 30, 2018 at 06:17:06AM -0700, Bin Meng wrote: > This updates travis configuration to use kernel.org pre-built > toolchain for riscv. > > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> Applied to u-boot/master, thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180906/ff0253d1/attachment.sig> ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-09-06 13:45 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-08-30 13:17 [U-Boot] [PATCH] travis: Use kernel.org pre-built toolchain for riscv Bin Meng 2018-09-03 9:50 ` Bin Meng 2018-09-03 19:23 ` Tom Rini 2018-09-04 1:27 ` Bin Meng [not found] ` <752D002CFF5D0F4FA35C0100F1D73F3F6BCB9FC7@ATCPCS16.andestech.com> 2018-09-04 5:36 ` Rick Chen 2018-09-04 8:56 ` Bin Meng 2018-09-06 1:43 ` Bin Meng 2018-09-06 1:53 ` Rick Chen 2018-09-06 13:45 ` [U-Boot] " Tom Rini
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.