[PATCH] Modify raspberrypi mm_config_table to map GPIO and registers.

[Chris Johns]

Hesham Moustafa wrote:
>
> On Mon, Oct 14, 2013 at 4:02 PM, Sebastian Huber
> <sebastian.huber at embedded-brains.de
> <mailto:sebastian.huber at embedded-brains.de>> wrote:
>
>     On 2013-10-14 15:34, Hesham AL-Matary wrote:
>
>         diff --git
>         a/c/src/lib/libbsp/arm/__raspberrypi/startup/mm_config___table.c
>         b/c/src/lib/libbsp/arm/__raspberrypi/startup/mm_config___table.c
>         index 5436a76..2b748b2 100644
>         --- a/c/src/lib/libbsp/arm/__raspberrypi/startup/mm_config___table.c
>         +++ b/c/src/lib/libbsp/arm/__raspberrypi/startup/mm_config___table.c
>         @@ -42,7 +42,7 @@ bsp_mm_config_table[] = {
>              }, {
>                .begin = (uint32_t) bsp_section_text_begin,
>                .end = (uint32_t) bsp_section_text_end,
>         -    .flags = ARMV7_MMU_CODE_CACHED
>         +    .flags = ARMV7_MMU_READ_WRITE
>
>
>     What is the problem with the cache?  Did the cache previously work?
>
> There is a problem with applying a Read-Only permissions in general to this
> area for RaspberryPi BSP.
>

What is the reason ?

>
>
>              }, {
>                .begin = (uint32_t) bsp_section_rodata_begin,
>                .end = (uint32_t) bsp_section_rodata_end,
>         @@ -63,8 +63,12 @@ bsp_mm_config_table[] = {
>                .begin = (uint32_t) bsp_section_stack_begin,
>                .end = (uint32_t) bsp_section_stack_end,
>                .flags = MMU_DATA_READ_WRITE
>         +  }, {
>         +    .begin = 0x20000000,
>         +    .end = 0x20FFFFFF,
>         +    .flags = ARMV7_MMU_DATA_READ_WRITE
>
>
>     What is this for an area?
>
> It's for Raspberry GPIO pins and other registers (defined at
> raspbberrypi.h).
>

Should this be ARMV7_MMU_DEVICE and not ARMV7_MMU_DATA_READ_WRITE ?

Chris