Дальнейший вопрос с отображенным в память интерфейсом - PullRequest
Новая
       1

Дальнейший вопрос с отображенным в память интерфейсом

1 голос
/ 12 ноября 2010

У меня все еще есть некоторые проблемы с моим кодом c, который связан с устройством отображения памяти.В настоящий момент я объявляю адресное пространство для регистров, которые я записываю, как энергозависимый указатель, и я записываю в них данные, как показано ниже: 

volatile unsigned int *wr_register = (int *) 0x40000000;
volatile unsigned int *c_register =  (int *) 0x40000100;
...

main{

  *wr_register = 0x01234567;

  *c_register = 0x01234567;    
  *(c_register+1) = 0x89abcdef;  

}

Это работает более или менее нормально.Однако я хотел бы иметь определенные функции чтения и записи, которые взаимодействуют с регистрами, отображаемыми в памяти.Поэтому в идеале это выглядело бы примерно так: 

const unsigned int wr_register = 0x40000000;
const unsigned int c_register  = 0x40000100;

function write_REG(unsigned int address, int offset, int data)
{
    (unsigned int*) (address + offset) = data;
}

main{

  *write_REG(0x40000000, 0, 0x01234567);

  *write_REG(0x40000100, 0, 0x01234567);
  *write_REG(0x40000100, 1, 0x89abcdef);  

}

Если честно, я еще не пробовал, но мне интересно, может ли кто-нибудь сказать мне, если это правильный способ сделать это?

РЕДАКТИРОВАТЬ: Может быть, это полезно для кого-то еще, здесь у меня есть моя функция, и они, кажется, работают.Большое спасибо за полезные комментарии! 

void reg_write(unsigned int address, int offset, int data)
{
    *((volatile unsigned int*)address + offset) = data;
}

int reg_read(unsigned int address, int offset)
{
    return(*((volatile unsigned int*)address + offset));
}

Большое спасибо

Ответы [ 2 ]

1 голос
/ 12 ноября 2010

С вашим кодом довольно много проблем:

  1. Я полагаю, вы имели в виду void, где вы написали function.
  2. Вы также должны сделать указатель внутри функции равным volatile.
  3. Вы должны разыменовать указатель перед записью данных. * должен быть внутри функции, а не на месте вызова (*write_REG), как сейчас - это будет ошибка компиляции.
  4. Вы должны добавить смещение к указателю, а не к адресу. Это потому, что смещение 1 должно быть следующим int, которое может быть, скажем, на расстоянии 4 байта, но добавление его к адресу добавит только 1 байт.

Ваша исправленная функция должна выглядеть следующим образом: 

void write_REG(unsigned int address, int offset, int data)
{
    *((volatile unsigned int*)address + offset) = data;
}

и вы бы назвали это как: 

write_REG(0x40000000, 0, 0x01234567);
0 голосов
/ 12 ноября 2010

Это было бы прекрасно ИМХО.Иногда я использую такие макросы, как: 

#define WR_REG     *(volatile unsigned int*)0x40000000

Это позволяет использовать регистры в виде переменных типа: 

WR_REG = 0x12345678;
...