<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 12, 2013 at 2:55 PM, Gedare Bloom <span dir="ltr"><<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">On Thu, Sep 12, 2013 at 2:59 AM, Hesham AL-Matary<br>

<<a href="mailto:heshamelmatary@gmail.com">heshamelmatary@gmail.com</a>> wrote:<br>
> ---<br>
>  c/src/lib/libbsp/shared/include/mm.h              | 36 +++++++++++++++++++++++<br>
>  c/src/lib/libbsp/shared/src/no_memorymanagement.c | 21 +++++++++++++<br>
>  2 files changed, 57 insertions(+)<br>
>  create mode 100644 c/src/lib/libbsp/shared/include/mm.h<br>
>  create mode 100644 c/src/lib/libbsp/shared/src/no_memorymanagement.c<br>
><br>
> diff --git a/c/src/lib/libbsp/shared/include/mm.h b/c/src/lib/libbsp/shared/include/mm.h<br>
> new file mode 100644<br>
> index 0000000..f2ac71c<br>
> --- /dev/null<br>
> +++ b/c/src/lib/libbsp/shared/include/mm.h<br>
> @@ -0,0 +1,36 @@<br>
> +/*<br>
> + * Copyright (c) 2013 Gedare Bloom.<br>
> + *<br>
> + * The license and distribution terms for this file may be<br>
> + * found in the file LICENSE in this distribution or at<br>
> + * <a href="http://www.rtems.com/license/LICENSE" target="_blank">http://www.rtems.com/license/LICENSE</a>.<br>
> + */<br>
> +<br>
> +#ifndef __LIBBSP_MM_H<br>
> +#define __LIBBSP_MM_H<br>
> +<br>
> +#include <stdint.h><br>
> +#include <stdlib.h><br>
> +<br>
> +#ifdef __cplusplus<br>
> +extern "C" {<br>
> +#endif<br>
> +<br>
> +typedef struct {<br>
> +  uint32_t begin;<br>
> +  uint32_t end;<br>
> +  uint32_t flags;<br>
> +} mm_init_start_config;<br>
</div></div>Should begin/end be uintptr_t? This structure appears to be unused<br>
here, so perhaps it should be left out?<br>
<div><div class="h5"><br></div></div></blockquote><div>I copied the struct definition from arm-cp15-start.h so that mm_config_table.c</div><div>can include it and recognize such a type.</div><div> I can't include arm-cp15-start.h (which it has an exact definition)</div>
<div>to mm_config_table.c because there will be a multiple-definition </div><div>compilation error (as arm-cp15-start.h implements some routines) because</div><div>mm.c also includes it.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div><div class="h5">
> +<br>
> +void bsp_memory_management_initialize(void);<br>
> +<br>
> +void bsp_memory_management_set_attributes(<br>
> +  uintptr_t base,<br>
> +  size_t size,<br>
> +  uint32_t attr<br>
> +);<br>
> +<br>
> +#ifdef __cplusplus<br>
> +}<br>
> +#endif<br>
> +#endif<br>
> diff --git a/c/src/lib/libbsp/shared/src/no_memorymanagement.c b/c/src/lib/libbsp/shared/src/no_memorymanagement.c<br>
> new file mode 100644<br>
> index 0000000..1a5fc63<br>
> --- /dev/null<br>
> +++ b/c/src/lib/libbsp/shared/src/no_memorymanagement.c<br>
> @@ -0,0 +1,21 @@<br>
> +/*<br>
> + * Copyright (c) 2013 Hesham AL-Matary<br>
> + * Copyright (c) 2013 Gedare Bloom.<br>
> + *<br>
> + * The license and distribution terms for this file may be<br>
> + * found in the file LICENSE in this distribution or at<br>
> + * <a href="http://www.rtems.com/license/LICENSE" target="_blank">http://www.rtems.com/license/LICENSE</a>.<br>
> + */<br>
> +<br>
> +#include <rtems.h><br>
> +#include <libbsp/mm.h><br>
> +<br>
> +void bsp_memory_management_initialize( void ) { }<br>
> +<br>
> +void bsp_memory_management_set_attributes(<br>
> +  uintptr_t base,<br>
> +  size_t size,<br>
> +  uint32_t flags<br>
> +)<br>
> +{<br>
> +}<br>
> --<br>
> 1.8.3.1<br>
><br>
</div></div>> _______________________________________________<br>
> rtems-devel mailing list<br>
> <a href="mailto:rtems-devel@rtems.org">rtems-devel@rtems.org</a><br>
> <a href="http://www.rtems.org/mailman/listinfo/rtems-devel" target="_blank">http://www.rtems.org/mailman/listinfo/rtems-devel</a><br>
</blockquote></div><br></div></div>