Чтение DS18B20

[YURALL72]

Всем доброго время суток.Написал код общения с DS18B20.Возникла проблема,датчик шлёт данные младшим битом вперёд(((.После приёма байта написал цикл разворота этого байта.Этот код работает,Протеус симулирует и показывает температуру.
//чтение байта из датчика
unsigned char read_byte(void)
{
unsigned char read_byte=0,i,temp;
for (i=0;i‹8;i++)
{
DDRB.4=1;//Порд в выход
PORTB.4=0; //сбрасываем в 0
delay_us(10);//формируем синхроимпульс
DDRB.4=0; //порт к +5в
//PORTB.4=0;
delay_us(20);//ждём ответа датчика
if(PINB.4==1) read_byte++; //если на входе +5в, байт +1
if(i !=7)read_byte‹‹=1; //ну а если 0,и не последний проход цикла сдвигаем влево на 1 позицию,и готовим следующий бит
delay_us(60); //заканчуем слот
}
temp=read_byte;
read_byte=0;
for (i=8;i›0;i--)
{

if (temp & 1) read_byte++;
if(i!=1)read_byte‹‹=1;
temp››=1;
}
return read_byte;
}

Я захотел уменьшить код и и вышло такое:

//чтение байта из датчика
unsigned char read_byte(void)
{
unsigned char read_byte=0,i,temp;
for (i=8;i›0;i--)
{
DDRB.4=1;//Порд в выход
PORTB.4=0; //сбрасываем в 0
delay_us(10);//формируем синхроимпульс
DDRB.4=0; //порт к +5в
PORTB.4=0;
delay_us(20);//ждём ответа датчика
if(PINB.4==1) read_byte |= (1‹‹(i-1)); //если на входе +5в, устанавливаем 1 в нужном разряде
delay_us(60); //заканчуем слот
}

return read_byte;
}

но этот код не работает,хотя прогон в авр студио показывает корректную
работу кода.Т.е. при наличии на пине.4 лог. 1.Она записуется в соответствующий разряд.Если нетрудно,помогите разобраться .

[uk8amk]

Вот этот код в железе работает.

[YURALL72]

Большое спасибо!Но у меня есть рабочий код на си и на ассемблере (чтение байта из датчика).Я не могу понять почему второй вариант не работает(((

[akegor]

YURALL72, недостаточно информации. Какой контроллер? Подтяжка внешняя есть или только внутренняя?
Пишете в CV? Там есть функции специально для этого термометра, рожать не надо.

[YURALL72]

Прошу прощения,пишу в кодевижне. Подтяжка внешняя 4.7к. Для атмеги8. Встроенные функции не люблю,хочу понять где я наплужил во втором варианте.

[electroTrash]

хочу понять где я наплужил во втором варианте.

надо изменить порядок хранения битов

//чтение байта из датчика
unsigned char read_byte(void)
{
unsigned char read_byte=0,i,temp;
for (i=0;i‹8;i++)
{
DDRB.4=1;//Порд в выход
PORTB.4=0; //сбрасываем в 0
delay_us(10);//формируем синхроимпульс
DDRB.4=0; //порт к +5в
PORTB.4=0;
delay_us(20);//ждём ответа датчика
if(PINB.4==1) read_byte |= (1‹‹i); //если на входе +5в, устанавливаем 1 в нужном разряде
delay_us(60); //заканчуем слот
}
return read_byte;
}

[alvadep]

if(PINB.4==1) read_byte |= (1‹‹(i-1)); //если на входе +5в, устанавливаем 1 в нужном разряде

А i случаем не портится?

[YURALL72]

Нет,i не портится,проверял.electroTrash спасибо!Я чот не додумался до такого простого решения,нагородил кучу всего).Буду Ваш код использовать.Спасибо ВСЕМ.Удачи.
P/S Дошло!!!Младшим битом ВПЕРЁД!!!Я знал а пхал младший в старший!Ну как так можно перепутать?А ведь перепутал,и 2 вечера парился(((.Большое спасибо Ребята что вывели из ступора!)))

[_Артём_]

Я захотел уменьшить код и и вышло такое:

//чтение байта из датчика
но этот код не работает,хотя прогон в авр студио показывает корректную
работу кода.Т.е. при наличии на пине.4 лог. 1.Она записуется в соответствующий разряд.Если нетрудно,помогите разобраться .

Как-то сложно у вас. Зачем вычислять маску (1‹‹(i-1))?
Можно сдвигать вправо и устанавливать старший бит, если нужно.

unsigned char read_byte(void)
{
unsigned char read_byte,i;
PORTB.4=0;
for (i=0;i‹8;i++)
{
DDRB.4=1;//Порт в выход
//PORTB.4=0; //сбрасываем в 0 - не нужно
delay_us(10);//формируем синхроимпульс
DDRB.4=0;  //порт к +5в

delay_us(20);//ждём ответа датчика
read_byte››=1;
if(PINB.4) read_byte |= 0x80; 
delay_us(60);  
}

P.S. B PORTB.4 менять не обязательно, если есть подтяжка - достаточно DDRB.4 переключать в нужное состояние.

[oleg110592]

Как-то сложно у вас. Зачем вычислять маску (1‹‹(i-1))?
Ну у производителя микросхемы (APPLICATION NOTE 162) так:

if(read_bit()) value|=0x01‹‹i; // reads byte in, one byte
at a time and then shifts it left

http://www.maximintegrated.com/app-n...dex.mvp/id/162

Ваш код, наверное покомпактнее будет.