Странный баг управления драйвером RS-485

[nml]

Но контроль последнего отправленного бита выполняется в строке
while(!( UCSR0A & (1 ‹‹ TXC0)));//

Если верить даташиту, то TXC сбрасывается либо при записи туда 1, либо при обработке соотв. прерывания. В вашем примере я не увидел ни того, ни того. Так что он взводится после окончания передачи ПЕРВОГО байта и не очищается никогда.

А буферизация у меги есть. Так что опережение на два байта вполне закономерно. Один в регистре сдвига - передается, второй в буфере.

[bdn62]

Еще раз спасибо за внимание!
Но:
while(!( UCSR0A & (1 ‹‹ TXC0)));//

UCSR0A |= (1 ‹‹ TXC0);
т е сначала жду установки TXC0, затем его обнуляю
А вот про буферизацию это интересно. Хотя как обрабатывать этот редчайший, но после "заскока" ежесекундно повторяющийся "глюк" пока не понимаю.

[j-Roger]

Странно; функция RS485_U0_send в посте #1 написана один-в-один по даташиту с учетом буферизации и ожиданием окончания передачи последнего бита. Все как бы должно работать. Эррата молчит.
А прерывания запрещать на время передачи ( вдруг кто-то портит порт в прерываниях ) тогда уж так:

void RS485_U0_send(unsigned char *buff, unsigned char len)
{
 int cnt;
 unsigned char tmp;

	tmp = SREG;
	__asm__ ("cli");
	PORTB |= (1 ‹‹ PB4);//and set high level

	cnt = 0;
	while(cnt ‹ len)
	{
		while(!( UCSR0A & (1 ‹‹ UDRE0)));
		UDR0 = buff[cnt ++];
	}
	while(!( UCSR0A & (1 ‹‹ TXC0)));//

	UCSR0A |= (1 ‹‹ TXC0); 
	PORTB &= ~(1 ‹‹ PB4);//set R/W low level
	SREG = tmp;
	inc_tranzaction_MB1();
}

Можно (на всякий случай ) еще бит прерывания этого передатчика почистить.

[akegor]

прерывания запрещать на время передачи ( вдруг кто-то портит порт в прерываниях )

Интересное предложение. Вообще-то, прерывания служат для того, чтобы выполнить что-то "именно здесь и сейчас". А если выполнение этого можно отложить на неопределенное время (миллисекунды!), как в данном предложении, то нахрена тогда использовать прерывания?!

опережение на два байта вполне закономерно. Один в регистре сдвига - передается, второй в буфере.

Очень уж должны сойтись звезды... Но тут поможет запрет переключения на прием при непустом регистре данных передачи.

Проверил поисковиком редактора все файлы проекта - нигде больше нет обращения к биту PORTB.PB4

Нужно проверить ВСЕ обращения к порту. Бит может быть изменен не только обращением к одному биту, групповуха тоже возможна.
Еще возможно формирование буфера на два байта короче (не записана контрольная сумма), потому как в самой функции передачи буфера, вроде бы, все в порядке.
А еще возможно пересечение стека и области переменных. Уже говорил. В этом случае возможны зависания, нарушения в алгоритме и вообще ХЗЧ.

[dgrishin]

Код программы написан в дурном стиле. Если в посылке несколько (десяток) байт - то тогда код сойдёт. Такие вещи "по взрослому" надо делать на DMA.
А от дурного программирования до дурного поведения программы один шаг.

[AR_Favorit]

Такие вещи "по взрослому" надо делать на DMA.

Ну вот и самые внимательные подтянулись))) ДМА в меге...

[mimuh64]

Но именно иногда происходит зависание, иллюстрируемое осциллограммой 1, которая и вызывает у меня оторопь. Причем в этом зависании устройство сидит "вечно", до аппаратной инициализации МК.

Про зависание... это тоже не понятно, самое главное что видно по осциллограмме - это потеря 2х байтов...

Допустим что ТС описал нам все так как происходит и с регистром UCSR0A, в момент работы макроса RS485_U0_send, никто ни чего не делает, то такой баг (пропуск контроля выдачи 2х байт) может возникать только тогда, когда бит TXC0 равен единице на момент его проверки процедурой while(!( UCSR0A & (1 ‹‹ TXC0)));. Почему? пока не понятно... Сам, многократно контролировал выдачу выдачу подобным образом, никогда проблем не было...

Согласно ДШ такого не должно быть - бит TXC0 устанавливается в 1 только тогда, когда из сдвигового регистра (TRANSMIT SHIFT REGISTER) все выдано (вплоть до стопового бита) и в буферном регистре UDR0 (UDR (Transmit)) тоже ничего нет. Что в общем-то противоречит тому что мы видим. Но, оно есть...

Если бы в макросе RS485_U0_send была ошибка, то он бы стабильно работал неправильно, смущает то, что периодически выдача идет правильно (если верить ТС).

Есть одно замечание по макросу RS485_U0_send. Насколько я знаю, - сброс ФЛАГОВ програмным путем регистров АВР должен делаться не через UCSR0A |= (1 ‹‹ TXC0); (чтение-модификация-запись), а через UCSR0A = (1 ‹‹ TXC0); - просто запись в регистр (именно так изменится один бит)...Но, настаивать не буду - это ТС легко проверит в железе... Достаточно установить в UCSR0A|=(1‹‹U2X0); двойную скорость... если выполнение команды UCSR0A = (1 ‹‹ TXC0); очистит регистр - восстановит скорость до нормальной (смотреть осциллографом), то сброс бита нужно делать через UCSR0A |= (1 ‹‹ TXC0);

Далее...ТС попробуй следующее...
Для такой реализации макроса RS485_U0_send, я бы его переписал следующим образом

void RS485_U0_send(unsigned char *buff, unsigned char len)
{
    int cnt;
    PORTB |= (1 ‹‹ PB4);//and set high level

    cnt = 0;
    while(cnt ‹ len)
    {
        UDR0 = buff[cnt ++]; // при первом запуске макроса регистр пустой...
                while(!( UCSR0A & (1 ‹‹ UDRE0))); // переставили контроль буфера...!!!!!!!!!

    }
    UCSR0A = (1 ‹‹ TXC0);  // сбросим перед проверкой, мы точно знаем что один байт еще не выдан....
    while(!( UCSR0A & (1 ‹‹ TXC0)));//

     UCSR0A = (1 ‹‹ TXC0); 
    PORTB &= ~(1 ‹‹ PB4);//set R/W low level
    inc_tranzaction_MB1();
} 

Такая реализаци даст возможность контролировать только один - последний байт. И если будет потеря, то только одного байта, тогда будем думать еще...
Сброс UCSR0A = (1 ‹‹ TXC0); перед while - на всякий случай. Нужно проверить с ним и без....
согласно ДШ сам он сбрасывается только через прерывание (прерываний нет, значит сброс программно...)

bdn62 , попробуй... и отпишись...

[pt200]

Если в момент передачи произойдет долгое прерывание( дольше передачи 1 или 2 байта), то TXC0 будет взведен( и сам уже не сброситься)!

Для начала можно попробовать такой велосипед:

while(cnt ‹ len)
{
	while(!( UCSR0A & (1 ‹‹ UDRE0)));
	__asm__ ("cli");
	UDR0 = buff[cnt ++];
	UCSR0A |= (1 ‹‹ TXC0); 
	__asm__ ("sei");
}
while(!( UCSR0A & (1 ‹‹ TXC0)));//

[bdn62]

Велосипед
__asm__ ("cli");
UDR0 = buff[cnt ++];
UCSR0A |= (1 ‹‹ TXC0);
__asm__ ("sei");
пока работает, буду ждать...

[akegor]

Если в момент передачи произойдет долгое прерывание( дольше передачи 1 или 2 байта), то

такого программера надо лупить розгами до просветления и осознания.

И вообще - организация передачи сообщения так, как сделано у ТС (циклом), заслуживает общественного осуждения и порицания. Нормально - с использованием прерываний, раз уж ПДП нету.