[RTEMS][PATCH v2 0/2] riscv: add freedom E310 Arty A7

[Pragnesh Patel]

Ok understood.
If i will add Freedom E310 related code in bsp/riscv/riscv directory
then there are some code changes related to "console, btimer and other
minor things" so can i add this changes under new #define let's say
#define RISCV_ENABLE_FRDME310_SUPPORT ?

I am planning to add this #define in configure.ac through
RTEMS_BSPOPTS_SET([RISCV_ENABLE_FRDME310_SUPPORT],[*],[1]), By default
it is disabled.
what's your suggestion on this?

-Pragnesh

On Wed, Sep 11, 2019 at 1:03 PM Sebastian Huber
<sebastian.huber at embedded-brains.de> wrote:
>
> On 11/09/2019 09:27, Pragnesh Patel wrote:
> > Thanks for the clarification. i am also agree with your comments but
> > let's say in future, we want to add I2C and SPI for Freedom E310 then
> > what's your suggestion?
>
> Future I2C and SPI drivers should be added to:
>
> https://git.rtems.org/rtems/tree/cpukit/dev
>
> They should be initialized in a BSP for example through the device tree.
> Currently, we lack a proper generic initialization via the device tree.
>
> --
> Sebastian Huber, embedded brains GmbH
>
> Address : Dornierstr. 4, D-82178 Puchheim, Germany
> Phone   : +49 89 189 47 41-16
> Fax     : +49 89 189 47 41-09
> E-Mail  : sebastian.huber at embedded-brains.de
> PGP     : Public key available on request.
>
> Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.