Re: [PATCH] bsps/riscv: Rename riscv_generic to riscv32_generic and add compiler flags

2018-05-28 Thread Hesham Almatary
On Mon, May 28, 2018 at 12:01 PM, Sebastian Huber wrote: > On 28/05/18 12:02, Hesham Almatary wrote: >> >> On Mon, May 28, 2018 at 10:55 AM, Sebastian Huber >> wrote: >>> >>> On 28/05/18 11:47, Hesham Almatary wrote: This makes it explicitly 32-bit. Compiler flags are needed if built wi

Re: [PATCH] bsps/riscv: Rename riscv_generic to riscv32_generic and add compiler flags

2018-05-28 Thread Sebastian Huber
On 28/05/18 12:02, Hesham Almatary wrote: On Mon, May 28, 2018 at 10:55 AM, Sebastian Huber wrote: On 28/05/18 11:47, Hesham Almatary wrote: This makes it explicitly 32-bit. Compiler flags are needed if built with a non riscv32-* toolchain (e.g. riscv64-* with multilib). I would like to rena

Re: [PATCH] bsps/riscv: Rename riscv_generic to riscv32_generic and add compiler flags

2018-05-28 Thread Hesham Almatary
On Mon, May 28, 2018 at 10:55 AM, Sebastian Huber wrote: > On 28/05/18 11:47, Hesham Almatary wrote: >> >> This makes it explicitly 32-bit. Compiler flags are needed if built with >> a non riscv32-* toolchain (e.g. riscv64-* with multilib). > > > I would like to rename the BSPs to use the ISA name

Re: [PATCH] bsps/riscv: Rename riscv_generic to riscv32_generic and add compiler flags

2018-05-28 Thread Hesham Almatary
This should allow building riscv32_generic with riscv64-rtems* toolchain (or just riscv-rtems*). I tested that. On Mon, May 28, 2018 at 10:47 AM, Hesham Almatary wrote: > This makes it explicitly 32-bit. Compiler flags are needed if built with > a non riscv32-* toolchain (e.g. riscv64-* with mult

Re: [PATCH] bsps/riscv: Rename riscv_generic to riscv32_generic and add compiler flags

2018-05-28 Thread Sebastian Huber
On 28/05/18 11:47, Hesham Almatary wrote: This makes it explicitly 32-bit. Compiler flags are needed if built with a non riscv32-* toolchain (e.g. riscv64-* with multilib). I would like to rename the BSPs to use the ISA name, e.g. rv32imac, rv64imafdc, etc. I also would like to add support for

[PATCH] bsps/riscv: Rename riscv_generic to riscv32_generic and add compiler flags

2018-05-28 Thread Hesham Almatary
This makes it explicitly 32-bit. Compiler flags are needed if built with a non riscv32-* toolchain (e.g. riscv64-* with multilib). --- bsps/riscv/riscv_generic/config/riscv32_generic.cfg | 7 +++ bsps/riscv/riscv_generic/config/riscv_generic.cfg | 7 --- 2 files changed, 7 insertions(+),