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

[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) и еще один датчик, подключенный к другому порту. Пробовал переделать функции так, чтобы рабочий порт можно было передавать, как параметр в функцию, но либо я чего-то не допер, либо это невозможно. На текущий момент предыдущим разработчиком просто сделан второй набор функций, где они работают с другим портом. На мой взгляд это не совсем правильно.

[rezident]

2 ReAl, в сообщении #6 вызывает сомнение, должен ли быть только указатель на структуру port_t типом volatile или все-таки все члены структуры должны быть volatile?
ИМХО более правильно было бы

Код

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

[ReAl]

должен ли быть только указатель на структуру port_t типом volatile или все-таки все члены структуры должны быть volatile?

volatile там не указатель, а тот объект, на который он указывает.
Ведь не
port_t * volatile pport;
написано.
Т.е. вся структура (все её поля) - volatile. Так же, как все они const, если вся структура const.
Вместо этого приписывать кавлификатор персонально к полям имеет смысл только тогда, когда часть полей volatile, часть - нет.

Собственно, квалификатор нужно было вообще в typedef включить и везде использовать просто port_t *

Код

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

[singlskv]

Собственно, квалификатор нужно было вообще в typedef включить и везде использовать просто port_t *

Код

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

Объясните мне пожалуйста, зачем эту структуру содержащую адреса регистров портов нужно
объявлять volatile ? Недопонимаю...

[ReAl]

Объясните мне пожалуйста, зачем эту структуру содержащую адреса регистров портов нужно объявлять volatile ? Недопонимаю...

В данном конкретном случае - может и не нужно. Но рано или поздно захочется написать
while( pport->pin & mask ) { }
и тут уже в полный рост встанет то, что оптимизатор компилятора не имеет права делать предположения типа "раз только что прочитано это значение, то ещё раз его читать не нужно".

Проблемма в том что я обращения к портам еще не увидел
А читать структру содержащую адреса портов можно и через volatile, тока зачем ?

Да не адреса портов в структуре!
Вся структура "мапится" на пространство ввода-вывода, у нас указатель на структуру, мы говорим, что она лежит по такому-то адресу, а там не память, а порты.

[singlskv]

Да не адреса портов в структуре!
Вся структура "мапится" на пространство ввода-вывода, у нас указатель на структуру, мы говорим, что она лежит по такому-то адресу, а там не память, а порты.

В таком варианте, согласен, тока я не понял как предполагается реализация того что спрашивал
автор поста, те по сути, нужно некоторое количество например входов, которые "произвольным"
образом размазаны по портам и их нужно опрашивать.