Глюк с UART при засыпании

[Wiza_]

ISR(USART1_TX_vect)					// обработчик передачи на шину rs485
	{

		if (tx1_len == 0)
			PORTG &= ~(1 ‹‹ 2);							// switch tranceiver to receive
		if (tx1_len › 0)
			{
				TCNT0 = 0x100 - rs485_timeout;
				UDR1 = tx1_transmission[tx1_ptr];
				tx1_ptr++;
				tx1_len--;
			}
	}

В своё время делал так

[av0000]

_Артём_, TXCIE как флаг не годится - байт ещё не передан в этот момент. Всё равно ждать TXC. А вот сам TXC при пустом буфере - вполне . И да, сон - это всё, что кроме Idle (либо ADC, либо PWR_DOWN), а 485-й - закладка на универсальность, потому как бывает и так, и эдак.


Godzilla82, не, _delay_ms сама по себе - зло мало того, что "тормозит" программу без учёта скорости передачи, так ещё и от неё код распухает ибо она практически везде инлайнится. У кого-то подсмотрел и активно пользую обёртку с принудительным __noinline__ для экономии

Wiza_, Ну, как-то так и у меня сделано. Пока, правда, раздельно в UDRIE и TXCIE, но, по мере чистки, сольётся в TXCIE.

[Someone]

- отправили последний байт в обработчике TXC
- счетчик байт обнулили /TXC сбросился/
- О! вот тут ответ на вопрос - здесь же запретили прерывания, раз счётчик нулевой.
- в tx_flush ждём взвода TXC

Ну, можно и так, усложнить, и потом героически преодолеть.
Хотя проще в прерывании ТХС уменьшить счётчик, и если он стал равен 0, выход из прерывания (для 485 интерфейса отключаем передатчик, включаем приём). Иначе пихаем у UDR очередной символ из буфера, и выход из прерывания. И тогда флуш вообще не нужон, ведь в прерывание мы попадаем уже после завершения передачи символа, т.е. условие флуша выполнено, в отличии от прерывания UDRE.

[_Артём_]

TXCIE как флаг не годится - байт ещё не передан в этот момент. Всё равно ждать TXC. А вот сам TXC при пустом буфере - вполне .

Годится в случае использования для передачи прерывания TXC, а не DRE. Тогда пока передача активна прерывание TXC разрешено, после передачи последнего байта прврывание TXC запрещается (TXCIE=0). В основной программе проверяем TXCIE.

[Godzilla82]

Тогда пока передача активна прерывание TXC разрешено, после передачи последнего байта прврывание TXC запрещается (TXCIE=0). В основной программе проверяем TXCIE.

В сотый раз говорю - зачем запрещать - разрешать. Ни к чему эти телодвижения. Да ещё при отправке каждого байта.

У нас и так есть флаг _tx.cnt - количество байт, требующих отправки. Если ноль - значит всё. Ждём, пока передаётся последний байт и в сон или на дачу - кому как. А ждать можно и через delay(). Не намного она код распушает. Да и используем её только в 1 месте - перед сном, ну попозже уснули на миллисекунду - ничё страшного, подумаешь - бессоница мучала.

[Wiza_]

Поддерживаю. ТХС запрещать не имеет никакого смысла
delay использовать - тоже никакого

[av0000]

Народ!
Не ссорьтесь Никто и не говорит о запрете/разрешении TXCIE при передаче каждого байта! Только по окончании передачи буфера. Повторюсь, хоть какой, но признак окончания отправки таки нужен при использовании глубокого сна...

Кажется, дошло, что имелось в виду:

ISR(USART_TXC_vect) {
  if (_tx.cnt)  _tx.cnt--; 
  if (_tx.cnt) {
   _tx.out = (_tx.out + 1) & UART_TX_BUF_MASK;
   UDR = _tx.buf[_tx_out];
  }
}
// ...
void flush_tx() {
  while(_tx.cnt ) ;
} 

[Godzilla82]

что имелось в виду:

PHP код:

ISR(USART_TXC_vect) { if (_tx.cnt) _tx.cnt--; if (_tx.cnt) { _tx.out = (_tx.out + 1) & UART_TX_BUF_MASK; UDR = _tx.buf[_tx_out]; } } // ... void flush_tx() { while(_tx.cnt ) ; }

Так оно побыстрее работать будет, да и код поменьше:

 
ISR(USART_TXC_vect)
{
if (_tx.cnt)    {
                _tx.cnt--;
                _tx.out++;
                _tx.out &= UART_TX_BUF_MASK;
                UDR = _tx.buf[_tx_out];
                }
}

void flush_tx()
{
while(_tx.cnt );
_delay_ms(5);
} 

_delay_ms(5) необходима, так как _tx.cnt становится равным нулю ещё до окончания передачи байта.

Для скорости 115200 достаточно _delay_us(90);

[Someone]

Повторюсь, хоть какой, но признак окончания отправки таки нужен при использовании глубокого сна

И чем не устраивает равенство нулю счётчика байт?

ISR(USART1_TX_vect)     // прерывание завершения передачи символа
{
 ByteCountUART--;     // уменьшаем счётчик переданных байт
 if(!ByteCountUART)     // если передали все байты
 {
  TransiverOff();     // отключаем передатчик RS485
  return;       // завершаем прерывание
 }
 UDR1 = *PosBuferUART1++;   // если послали не всё, посылаем очередной байт
}

Очевидно же, что если ByteCountUART равен 0, передача была полностью завершена, и можно спать или заниматься чем угодно.

[av0000]

Someone, в итоге примерно так и вышло.
Оговорка: декремент счётчика без проверки может вызвать переполнение, потому, уменьшаем только если не ноль

А по поводу UDRE - я понял, в чём бонус - в отличие от TXC, прерывание возникает при пустом буфере - не требуется дублировать код отправки первого байта в uart_putch().