Hi, Any update on this patch. - Pragnesh
-----Original Message----- From: Sebastian Huber <sebastian.hu...@embedded-brains.de> Sent: 11 October 2019 11:56 To: Pragnesh Patel <pragnesh.pa...@sifive.com>; Hesham Almatary <hesham.almat...@cl.cam.ac.uk> Cc: rtems-de...@rtems.org <devel@rtems.org> Subject: Re: [PATCH v4] riscv: add freedom E310 Arty A7 bsp On 11/10/2019 08:18, Pragnesh Patel wrote: >>> RISCV_LINKCMD([RISCV_RAM_REGION_BEGIN],[begin of the RAM region >>> for linker command file (default is 0x70000000 for 64-bit with >>> -mcmodel=medlow and 0x80000000 for all >>> other)],[${RISCV_RAM_REGION_BEGIN_DEFAULT}]) >>> -RISCV_LINKCMD([RISCV_RAM_REGION_SIZE],[size of the RAM region for >>> linker command file (default 64MiB)],[0x04000000]) >>> +RISCV_LINKCMD([RISCV_RAM_REGION_SIZE],[size of the RAM region for >>> +linker command file (default is 256 MiB for frdme310arty and 64 MiB >>> +for all other)],[${RISCV_RAM_REGION_SIZE_DEFAULT}]) >>> >> No need for this change? > Arty A7 100T has a 256 MB of RAM. So, do you want me to make > RISCV_RAM_REGION_SIZE to default 64 MB for frdme310arty? Since we have now a BSP variant for this board I think it makes sense to have all RAM available by default. -- 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.hu...@embedded-brains.de PGP : Public key available on request. Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG. _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel