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

Случайная задержка перед вызовом фунции 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

Сравнительное тестирование аккумуляторов EVE Energy и Samsung типоразмера 18650

Инженеры КОМПЭЛ провели сравнительное тестирование аккумуляторов EVE и Samsung популярного для бытовых и индустриальных применений типоразмера 18650. 

Для теста были выбраны аккумуляторы литий-никельмарганцевой системы: по два образца одного наименования каждого производителя – и протестированы на двух значениях тока разряда: 0,5 А и 2,5 А. Испытания проводились в нормальных условиях на электронной нагрузке EBD-USB от ZKEtech, а зарядка осуществлялась от лабораторного источника питания в режиме CC+CV в соответствии с рекомендациями в даташите на определенную модель. Подробнее>>

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

Alex Наставник

Alex

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

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

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

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

Литиевые аккумуляторы EVE Energy и решения для управления перезаряжаемыми источниками тока (материалы вебинара)

Опубликованы материалы вебинара Компэл, посвященного литиевым аккумуляторам EVE Energy и решениям для управления перезаряжаемыми источниками тока.

На вебинаре мы представили информацию не только по линейкам аккумуляторной продукции EVE, но и по решениям для управления ею, что поможет рассмотреть эти ХИТ в качестве дополнительной альтернативы для уже выпускающихся изделий. Также рассмотрели нюансы работы с производителем и сервисы, предоставляемые Компэл по данной продукции. Подробнее>>

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

Литиевые батарейки и аккумуляторы от мирового лидера  EVE в Компэл

Компания Компэл, официальный дистрибьютор EVE Energy, бренда №1 по производству химических источников тока (ХИТ) в мире, предлагает продукцию EVE как со склада, так и под заказ. Компания EVE широко известна в странах Европы, Америки и Юго-Восточной Азии уже более 20 лет. Недавно EVE была объявлена поставщиком новых аккумуляторных элементов круглого формата для электрических моделей «нового класса» компании BMW.

Продукция EVE предназначена для самого широкого спектра применений – от бытового до промышленного. Подробнее>>

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

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 пользователей онлайн

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