Перейти к содержанию

Случайная задержка перед вызовом фунции Atmega

MaDiEQ

Автор MaDiEQ,
в AVR

 Поделиться

Рекомендуемые сообщения

MaDiEQ Новобранец

MaDiEQ

Опубликовано
Опубликовано

Прошу помочь разобраться в причине странного поведения Atmega8515L суть заключается в следующем: МК принимает по i2c свой slave адрес в следующем такте отвечает ACK и далее должен передать 8 битный код в данном примере (0b10001100), но функция передачи кода иногда запаздывает на случайное число тактов . Кто-нибудь сталкивался с подобным явлением, в чем может быть причина?

111.png

Без-имени-4.png

0
Ссылка на комментарий
Поделиться на другие сайты

Реклама: ООО ТД Промэлектроника, ИНН: 6659197470, Тел: 8 (800) 1000-321

Viktor26 Профи

Viktor26

Опубликовано
Опубликовано

Возможно прирывания от таймеров или какие нить ещё прирывания. Попробуйте во время процедуры общения и2с отключать прерывания глобально

1

Не знаеш как? Спроси у Google'а !!!

Ссылка на комментарий
Поделиться на другие сайты

20% скидка на весь каталог электронных компонентов в ТМ Электроникс!

Акция "Лето ближе - цены ниже", успей сделать выгодные покупки!

Плюс весь апрель действует скидка 10% по промокоду APREL24 + 15% кэшбэк и бесплатная доставка!

Перейти на страницу акции

Реклама: ООО ТМ ЭЛЕКТРОНИКС, ИНН: 7806548420, info@tmelectronics.ru, +7(812)4094849

Выбираем схему BMS для заряда литий-железофосфатных (LiFePO4) аккумуляторов

Обязательным условием долгой и стабильной работы Li-FePO4-аккумуляторов, в том числе и производства EVE Energy, является применение специализированных BMS-микросхем. Литий-железофосфатные АКБ отличаются такими характеристиками, как высокая многократность циклов заряда-разряда, безопасность, возможность быстрой зарядки, устойчивость к буферному режиму работы и приемлемая стоимость. Но для этих АКБ очень важен контроль процесса заряда и разряда для избегания воздействия внешнего зарядного напряжения после достижения 100% заряда. Инженеры КОМПЭЛ подготовили список таких решений от разных производителей. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

Alex Наставник

Alex

Опубликовано
Опубликовано

У Вас задержка между передачей байтов. Что в коде между вызовами i2c_transact ? Скорее всего, там собака порылась.

И ещё. Уж очень "тяжёлая" арифметика у Вас в функции. Int'ы, сдвиги на переменную, ... С'оптимизируйте код.
 

1
Ссылка на комментарий
Поделиться на другие сайты
MaDiEQ Новобранец

MaDiEQ

Опубликовано
  • Автор
Опубликовано (изменено)

ну да он проверяет j -й бит - переменную величину неизвестную заранее, как можно это упростить для МК? 

int chek_kb(){
	press_release=0;
	keycode=0;
	
	for (int i=0;i<=3;i++){
		DDRA|=0b00001111;
		PORTA=0b00001111 & out_line[i];
		_delay_us(10);
		if (PINB!=key_mask[i]){
			
			for (int j=0;j<=7;j++){
				///
				if( ((PINB^key_mask[i])&(1<<j))!=0) {
					
					if((PINB&(1<<j))==(1<<j)){
						press_release=0b00000000;
					}else{
						press_release=0b10000000;
					}
					keycode=table_codes[i*8+j];
					
					key_mask[i]=PINB;
					break;
				}
			}
			break;
		}
	}
	
	return (keycode|press_release);
}

 

в чем причина хаотичной задержки до начала передачи непонятно

Изменено пользователем MaDiEQ
0
Ссылка на комментарий
Поделиться на другие сайты
Alex Наставник

Alex

Опубликовано
Опубликовано

Какой кошмар... предыдущий код был куда оптимальнее :)

					if((PINB&(1<<j))==(1<<j)){
						press_release=0b00000000;
					}else{
						press_release=0b10000000;
					}

А что потом делается с press_release ? Так все 8 раз и пишется в 7-ой бит ?

Погуглите примеры I2C. Думаю, на просторах этого г..на навалом.

1
Ссылка на комментарий
Поделиться на другие сайты
MaDiEQ Новобранец

MaDiEQ

Опубликовано
  • Автор
Опубликовано

вроде бы не 8 раз ))) он 1 раз как только напишет должен выходить из цикла по break; 

а вот PINB&(1<<j))==(1<<j) наверно стоит заменить на PINB&(1<<j))!=0 или компилятор сам такое правит?

0
Ссылка на комментарий
Поделиться на другие сайты
MaDiEQ Новобранец

MaDiEQ

Опубликовано
  • Автор
Опубликовано

к первым 4м битам порта а подключены 4 строки клавиатуры (китайская мембранная типа замыкает строку-столбец), 8 столбцов подключены к порту B эта функция передает поочереди 0 на каждый пин из 4 пинов в порте А и ищет изменение состояния на порте Б относительно запомненного ранее для каждой строки

потом передает код клавиши в функцию передающую по i2c

0
Ссылка на комментарий
Поделиться на другие сайты
Alex Наставник

Alex

Опубликовано
Опубликовано

Такс... Возвращаемся к нашим баранам...
Где функция передачи байта в I2C и код между передачами ? Функция скана клавиатуры нас не интересует.
 

1
Ссылка на комментарий
Поделиться на другие сайты
MaDiEQ Новобранец

MaDiEQ

Опубликовано
  • Автор
Опубликовано
Скрытый текст 

#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
////
int key_mask[4]={255,255,255,255};
int out_line[4]={0b11111110,0b11111101,0b11111011,0b11110111};//PORTA&=0b11110111;
int table_codes[32]={12, 13, 14, 21, 41, 39, 37, 30, 15, 16, 17, 22, 25, 26, 38, 27,18, 19, 20, 23, 40, 33, 42, 28,31, 11, 32, 24, 35, 34, 36, 29};
//
int l_numb=0;//0-3
int col_numb=0;//0-7
int press_release=0b00000000;
int keycode=0b00000000;
//////
int slave_addr=0b10110110;
///////
int state_i2c_machine=0;
////
int next=0;


void chek_start(){
	DDRA=0b00001111;
	PORTA=0;
	int point=0;
	while(true){
		/////////////////////
		if( ((PINA&0b10000000)!=0)&&((PINA&0b01000000)!=0) && point==0){
			point=1;
		}else if ( ((PINA&0b10000000)!=0)&&((PINA&0b01000000)==0) && point==1){
			break;
		}else{
			point=0;
		}
	}
}

int chek_kb(){
	press_release=0;
	keycode=0;
	
	for (int i=0;i<=3;i++){
		DDRA|=0b00001111;
		PORTA=0b00001111 & out_line[i];
		_delay_us(10);
		if (PINB!=key_mask[i]){
			
			for (int j=0;j<=7;j++){
				///
				if( ((PINB^key_mask[i])&(1<<j))!=0) {
					
					if((PINB&(1<<j))==(1<<j)){
						press_release=0b00000000;
					}else{
						press_release=0b10000000;
					}
					keycode=table_codes[i*8+j];
					
					key_mask[i]=PINB;
					break;
				}
			}
			break;
		}
	}
	
	return (keycode|press_release);
}

void i2c_transact(int sendvar){
	int i=7;
	int jmpnext=0;
	int last=0;
	while (true){
		
		if ((PINA&0b10000000)==0) {
			if (jmpnext==0){
				if ((sendvar&(1<<i))!=0){
					//Z
					DDRA=0b00001111;
					PORTA=0;
					//PORTC=255;//////////tst
				}else{
					//0
					DDRA=0b01001111;
					PORTA=0;
					//PORTC=0;/////////////tst
				}
				jmpnext=1;
				if (last==1){
					break;
				}
			}
		}else{
			if (jmpnext==1){
				if (i>0){
					i--;
				}else{
					last=1;
				}
				jmpnext=0;
			}
		}
	}
}

void reset(){
DDRA=0b00001111;
DDRB=0;
DDRC=255;
DDRD=255;

PORTA=0b00001111;
PORTB=255;
PORTC=0;
PORTD=0;
}

int i2c_reciever(){
	int jmpnext=0;
	int result=0;
	int i=7;
	while (true){
		if (((PINA&0b10000000)!=0)){
			if (jmpnext==0){
				jmpnext=1;
				if ((PINA&0b01000000)!=0){
					result|=(1<<i);
				}
			}
		}else{
			if (jmpnext==1){
				if (i>0){
					i--;
				}else{
					return result;
				}
			}
			jmpnext=0;
		}
	}
}

int chek_ack(){//vhod v func pri scl=0
	DDRA=0b00001111;
	PORTA=0;
	while (true){
		if ((PINA&0b10000000)!=0){
			if ((PINA&0b01000000)==0){
				return 1;//ack=true
			}else{
				return 0;
			}
		}
	}
}



void i2c_state_machine(){
	//int jmplow=0;//state_i2c_machine
	if(state_i2c_machine==0){
		chek_start();
		
		state_i2c_machine=1;
		//PORTC=250;//////////////////////////////tst
	}else if ((state_i2c_machine==1)&&((PINA&0b10000000)==0)){
		int tsti2crec=i2c_reciever();
		//PORTC=tsti2crec;//////////////////////////////tst
		if (tsti2crec==slave_addr){
			state_i2c_machine=2;
			
		}else{
			state_i2c_machine=0;
			//PORTC=241;//////////////////////////////tst
		}
	}else if( ((state_i2c_machine==2)&&((PINA&0b10000000)==0)) || ((state_i2c_machine==2)&& (next==1))){
		//PORTC=~PORTC;//////////tst
		next=1;
		if ((PINA&0b10000000)!=0)
		{
			//PORTC=~PORTC;//////////tst
			state_i2c_machine=3;
			next=0;
		}else{
			DDRA=0b01001111;//send ACK
			PORTA=0;
		}
	}else if((state_i2c_machine==3)&&((PINA&0b10000000)==0)){
		//PORTC=~PORTC;//////////tst
		int code_s=chek_kb();
		//PORTC=~PORTC;//////////tst
		i2c_transact(code_s);
		//PORTC=~PORTC;//////////tst
		DDRA=0b00001111;
		PORTA=0b00000000;
		state_i2c_machine=0;
	}
}

int main(void)
{
	cli();
	reset();
	
	while (true){
		i2c_state_machine();
	}
}

 

 

кстати там была ошибка при которой он ВООБЩЕ не должен был работать так что эти случайные задержки выглядят еще более странно =)

0
Ссылка на комментарий
Поделиться на другие сайты
Alex Наставник

Alex

Опубликовано
Опубликовано

Во первых - все int'ы, где не нужна большая разрядность, заменить на unsigned char.
 

Во вторых.
Очень часто видишь код типа : 

for(i=0;i<8;i++){
    if(var & (1<<i))    ....
    else                ....
}

Его лучше заменить на : 

for(i=0;i<8;i++){
    if(var & 0x01)      ....
    else                ....
    var>>=1;
}

Выполняться будет в разы быстрее.
 

1
Ссылка на комментарий
Поделиться на другие сайты

Присоединяйтесь к обсуждению

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

Гость
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Ответить в этой теме...

×   Вставлено с форматированием.   Восстановить форматирование

  Разрешено использовать не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отображать как обычную ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставлять изображения напрямую. Загружайте или вставляйте изображения по ссылке.

Загрузка...
×
 Поделиться
Перейти к списку тем

  • Последние посетители   0 пользователей онлайн

    • Ни одного зарегистрированного пользователя не просматривает данную страницу
×
×
  • Создать...