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

[MaDiEQ]

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

[Реклама]

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

[Viktor26]

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

[Реклама]

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

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

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

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

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

[Alex]

Весь код нужно видеть. Вы выложили кусок, который абсолютно никак не относится к приёму/передаче по I2C.
 

[Реклама]

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

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

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

[MaDiEQ]

задержка имеет случайный характер изредка работает как надо

Изменено 24 ноября, 2016 пользователем MaDiEQ

[Alex]

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

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

[MaDiEQ]

я только пару недель на avr пишу чем можно заменить сдвиг(если надо установить 1 в определенный бит)

[Alex]

Зачем код удалили ?

Сдвиг у Вас не в установках бита, а в проверке.
 

[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);
}

 

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

Изменено 24 ноября, 2016 пользователем MaDiEQ

[Alex]

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

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

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

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

[MaDiEQ]

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

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

[Alex]

Тогда я вообще запутался в Вашем коде  Что делает эта функция ? 
 

[MaDiEQ]

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

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

[Alex]

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

[MaDiEQ]

проблему решил спасибо всем кто отписался =)

Изменено 24 ноября, 2016 пользователем MaDiEQ

[MaDiEQ]

PS: Хотелось бы получить совет от опытных программистов по поводу оптимизации кода для avr =)

 

[Alex]

Дак Вы код свой постоянно удаляете, как лиса хвостом
 

[MaDiEQ]

да просто там простыня некрасиво на странице смотрится )))

[Alex]

Тег spoler есть для этого
 

[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();
	}
}

 

 

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

[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;
}

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

[MaDiEQ]

спасибо , учту )))