Массив портов ввода PINx, вопрос к знатокам GCC Опции

[singlskv]

Контроллеры от ATmega8 до Atmega128, и т.д.

Есть такой массив:

Код

BYTE PIN_[3]={
    (BYTE)(&PINB),
    (BYTE)(&PINF),
    (BYTE)(&PING)
  };

Доступ к нему присходит вот так:

Код

  PIN_tmp=_SFR_IO8(PIN_[0]-0x20);
  ......
  PIN_tmp=_SFR_IO8(PIN_[2]-0x20);

Все работает, более того мне даже нравится код который генерит при этом GCC,
НО, осадок остается

Собствено вопросы:

1. Есть ли в GCC другие варианты доступа к массиву PINx, без использования _SFR_IO8 ?

2. Насколько такой вариант портируем на более старые/новые реализации GCC AVR ?

3. 0x20 режет глаз, а что с этим делать ?

[singlskv]

сделайте структуру

Код

typedef struct
{
   unsigned char *pinr;
  .....
} PIN;

НЕВЕРНО. pinr, port и ddr - ДОЛЖНЫ обязательно быть volatile!!!

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

а должно ли PINx быть volatile ?

???
avr/port_def.h
....
#define _MMIO_BYTE(mem_addr) (*(volatile uint8_t *)(mem_addr))
#define _SFR_MEM8(mem_addr) _MMIO_BYTE(mem_addr)
.....

Как видете все SFR обявлены как volatile.

Обявлять структуру или массив в которых хранятся адреса портов volatile НЕТ никакой необходимости.
И даже более того вредно.
Нужно обявлять доступ к порту как volatile, то есть так как сделано в _SFR...
(*(volatile uint8_t *)(mem_addr))

[aesok]

Обявлять структуру или массив в которых хранятся адреса портов volatile НЕТ никакой необходимости.
И даже более того вредно.

В чем вред? Пример.

ADD:
Не правильно прочитал вашу фразу.

Я не предлагал обьявлять структуру как volatile, а только те члены структуры в которых храняться ссылки на порты.

Нужно обявлять доступ к порту как volatile, то есть так как сделано в _SFR...
(*(volatile uint8_t *)(mem_addr))

Главное чтобы весь доступ к портам был помечен как 'volatile'. А как - это без разницы.

Анатолий.

Сообщение отредактировал aesok - Nov 28 2006, 18:25

[singlskv]

Обявлять структуру или массив в которых хранятся адреса портов volatile НЕТ никакой необходимости.
И даже более того вредно.

В чем вред? Пример.

Вам какой результат компиляции нравится больше ?

Вот этот, с volatile при объявлении массива

Код

volatile BYTE PIN_[2]={
    (BYTE)(&PINB),
    (BYTE)(&PIND)
  };

  tmp1= *(volatile BYTE *)((WORD)PIN_[0]);
  tmp2= *(volatile BYTE *)((WORD)PIN_[0]);

результат компиляции

Код

160:        tmp1= *(volatile BYTE *)((WORD)PIN_[0]);
+0000020A:   91800100    LDS     R24,0x0100       Load direct from data space
+0000020C:   2FE8        MOV     R30,R24          Copy register
+0000020D:   27FF        CLR     R31              Clear Register
+0000020E:   8180        LDD     R24,Z+0          Load indirect with displacement
161:        tmp2= *(volatile BYTE *)((WORD)PIN_[0]);
+0000020F:   91800100    LDS     R24,0x0100       Load direct from data space
+00000211:   2FE8        MOV     R30,R24          Copy register
+00000212:   27FF        CLR     R31              Clear Register
+00000213:   8180        LDD     R24,Z+0          Load indirect with displacement

или этот, без volatile ?

Код

BYTE PIN_[2]={
    (BYTE)(&PINB),
    (BYTE)(&PIND)
  };

  tmp1= *(volatile BYTE *)((WORD)PIN_[0]);
  tmp2= *(volatile BYTE *)((WORD)PIN_[0]);

результат компиляции

Код

160:        tmp1= *(volatile BYTE *)((WORD)PIN_[0]);
+0000020A:   91800100    LDS     R24,0x0100       Load direct from data space
+0000020C:   2FE8        MOV     R30,R24          Copy register
+0000020D:   27FF        CLR     R31              Clear Register
+0000020E:   8180        LDD     R24,Z+0          Load indirect with displacement
161:        tmp2= *(volatile BYTE *)((WORD)PIN_[0]);
+0000020F:   8180        LDD     R24,Z+0          Load indirect with displacement

[aesok]

Созрело 2 решения:

1. Ваше.
BYTE PIN_[2]={
(BYTE)(&PINB),
(BYTE)(&PIND)
};

tmp1= *(volatile BYTE *)((WORD)PIN_[0]);
tmp2= *(volatile BYTE *)((WORD)PIN_[0]);

Минусы: Не забывать писать "*(volatile..." при каждом обращении к порту!!!
Плюсы: Экономия 1 байта RAM на каждий порт.

2. Моё.

struct __port {
volatile uint8_t *port;
unsigned char pin;
};


struct __port ports[3] = {
{&PINB, 1<<PINB5},
{&PINF, 1<<PINF2},
{&PING, 1<<PING1}
};


PIN_tmp=*(ports[0].port);
tmp=PIN_tmp & ports[0].pin;

Минусы: Потеря 1 байта RAM на каждий порт.
Плюсы: Более понятная и постая запись. (моё личное мнение)



Анатолий.