Передача в качестве параметра порта ввода-вывода, как? Опции

[AlexMad]

Есть набор функций работы с однопроводными устройствами,вроде

Код

#define ONEPIN       PINC // PIN register
#define ONEDDR       DDRC // Data Direction Register
#define ONEPORT      PORTC// PORT

unsigned char OneWireReset(unsigned char mask)
{
ONEPORT &= ~(mask);                 // Normal input no pull up
if (ONEPIN == 0) return 0;        // detect 0V on buss error
ONEDDR |= mask;                        // out at 0
  delay_us(500);
ONEDDR &= ~(mask);                  // Set to input
  delay_us(70);
if ((ONEPIN & mask) == 0)
   {
     delay_us(500);
   return mask;
   }
mask&=~ONEPIN;  
  delay_us(500);
return mask;
}

Работают замечательно, НО в железе есть восемь датчиков DS18b20, подключенных к одному порту (PORTC) и еще один датчик, подключенный к другому порту. Пробовал переделать функции так, чтобы рабочий порт можно было передавать, как параметр в функцию, но либо я чего-то не допер, либо это невозможно. На текущий момент предыдущим разработчиком просто сделан второй набор функций, где они работают с другим портом. На мой взгляд это не совсем правильно.

[ReAl]

В avr-gcc нормально передаются указатели на порты, думаю, что и у IAR должны передаваться, причём не удивлюсь, если там найдётся какое-то "волшебное слово" по типу __flash, которое ещё и делает это более удобным.

Для большинства портов идёт единый порядок PIN-DDR-PORT (не выполняется это, например, для порта F у меги64, который в наследство от меги103 достался как порт толькона чтение и его DDRF, PORTF улетели в memory-mapped IO, включаемое при стёртом фьюзе M103).

Учитывая это, можно (но осторожно) обойтись одним указателем.

Код

#include <avr/io.h>
#define F_CPU 8000000UL
#include <util/delay.h>

typedef struct {
    uint8_t pin;
    uint8_t ddr;
    uint8_t port;
} port_t;


uint8_t OneWireReset(volatile port_t *pport, uint8_t mask)
{
    pport->port &= ~mask;
    if( (pport->pin & mask) == 0) return 0;        // detect 0V on buss error
    pport->ddr |= mask;                        // out at 0
    _delay_ms(0.5);
    pport->ddr &= ~mask;                  // Set to input
    _delay_us(70);
    if( (pport->pin & mask) == 0) {
        _delay_ms(0.5);
        return mask;
    }
    mask &= ~pport->pin;
    _delay_ms(0.5);
    return mask;
}

uint8_t test(void) {
    return OneWireReset( (volatile port_t*)&PINB, 0x02);
}

[AlexMad]

Учитывая это, можно (но осторожно) обойтись одним указателем.

Код

#include <avr/io.h>
#define F_CPU 8000000UL
#include <util/delay.h>

typedef struct {
    uint8_t pin;
    uint8_t ddr;
    uint8_t port;
} port_t;

}

Вот тут уже ближе. Я с ходу именно так и сделал, но, толи кодевижен это не хочет понимать, толи это все-таки не получится. У меня при инициализации

Код

pin=PINB

, а также по остальным двум портам матерно ругается, что PINB должно быть константой, указатель на PINB тоже не принимает, говорит порты нельзя передать После этого я свой вопрос и задал.

А вот насчет обращения, как к памяти это интересно, попробую.

[prottoss]

А вот насчет обращения, как к памяти это интересно, попробую.

В ИАРе прокатывает вот такой код

Код

char* pPORTA = &PORTA;
*pPORTA = 0x55;

То бишь, при присвоении указателю адреса порта происходит прибавление к адресу IO 0х20