Простенький вопрос по SPI и две ATMEGA8

[ibro30]

Всем здравствуйте!! По-тихоньку изучаю программирование на Меге. Я новичок, поэтому прошу сильно не бить.
Задачу поставил простую себе: 2 меги, Мастер передаёт слэйву 1 бит, Слэйв включает светодиод. Всё просто, но у слэйва, видимо, я написал неправильно в условиях вывода (скорее всего условие IF). Правда при запуске в протеусе всё стопорится почему-то на SS... мистика))
master:

void SPI_Init(void)
{
DDRB |=(1‹‹PB2)|(1‹‹PB3)|(1‹‹PB5); //Настроить выводы MOSI,SS,SCK на выход
PORTB|=(1‹‹PB2); //Установить "1" на линии SS

SPCR = 0; //Обнулить регистр SPCR
SPSR = 0; //Обнулить регистр SPSR

SPCR |= (1‹‹MSTR)|(1‹‹SPR0); //Режим мастер, F=Fosc/16
SPSR |=(1‹‹SPI2X); //F=Fosc/8

SPCR |=(1‹‹SPE);//Включить SPI
}

void SPI_Send_byte(char data)
{
PORTB &=~(1‹‹PB2); //Установить "0" на линии SS
SPDR = data; //Отправить байт
while(!(SPSR&(1‹‹SPIF))) ; //Дождаться окончания передачи
PORTB |=(1‹‹PB2); //Установить "1" на линии SS
}

int main(void)
{
SPI_Init();
SPI_Send_byte('1');
while(1)
{

}
}


слэйв:

void SPI_Init_Slave(void)
{
DDRB&= ~(1‹‹PB2) | ~(1‹‹PB3) |~(1‹‹PB5);
DDRB|=(1‹‹PB4);

SPCR=0;
SPSR=0;

SPCR |= (1‹‹SPR0)|(1‹‹SPIE);
SPSR |=(1‹‹SPI2X);

SPCR |=(1‹‹SPE);
}


ISR(SPI_STC_vect)
{
PORTB = SPDR; // вывод данных
return ;
}

int main(void)
{
sei();
SPI_Init_Slave();

DDRB|=(1‹‹PC0); //Настраиваем ножку PB2 в режим выхода
while(1) //бесконечный цикл
{
if(SPDR&=~0)//Если в SPDR поступил байт
{
PORTB|=(1‹‹PC0);//Зажигаем светодиод
}
else// если не поступил, тогда
{
PORTB&=~(1‹‹PC0);//Гасим светодиод
}
}
}

[H4LF]

ibro30, тэг [код] можете поставить? Нечитаемо. А лучше [PHP] - тогда немного цветов появится, парсить полегче будет...

SPCR = 0; //Обнулить регистр SPCR

зачем писать такие бесполезные комментарии - и так понятно, что нуль записывается в SPCR. Написали бы уж "обнулить SPI Control Register", так хоть понятно будет его назначение.

Если так и написано:

if(SPDR&=~0) 

То у вас тут идёт запись в порт, компилятор не предупреждает? Советую обращать внимание на предупреждения.
Окончание передачи надо смотреть по флагу SPIF в регистре SPSR, потому он и "статус" называется[регистр]. Фух, чуть глаза не сломал, в следующий раз такое проигнорирую, и, думаю, не только я.

Будете больше читать - будут меньше бить на форумах. Синяки конечно добавляют мотивации, но не делают умнее... Вам, видимо, было лень читать DS на свою мегу, а ведь там даже примеры есть на асме и на Си:

void SPI_SlaveInit(void)
{
/* Set MISO output, all others input */
DDR_SPI = (1‹‹DD_MISO);
/* Enable SPI */
SPCR = (1‹‹SPE);
}
char SPI_SlaveReceive(void)
{
/* Wait for reception complete */
while(!(SPSR & (1‹‹SPIF)))
;
/* Return data register */
return SPDR;
} 

[ForcePoint]

Помимо все уже указанного - а в чём вопрос-то? В этом?

Правда при запуске в протеусе всё стопорится почему-то на SS... мистика))

Что значит "стопорится на SS"? У ведущего или ведомого?

Проштудируйте раздел "SS Pin Functionality" в документации.

[nml]

Напомню еще - у слейва SS должен быть притянут к земле.

[H4LF]

Да не теряются такие скобки нигде, ни в каких текстах... кроме, как когда вставляешь #include пропало в треугольных скобках avr/io.h . не веришь? попробуй сам написать такие скобки в любом месте и формате ответа.

Хорошо, попробую, но в эту тему, чтобы не засорять ту. Это интересно. У него там все сдвиги превратились в просто одинарный ‹ и не хватает точек с запятой и много чего ещё. Неужели ты думаешь, что он накопипастил с такими ошибками, да ещё как-то умудрился скомпилировать? И к тому же я уже видел такое на этом форуме, потому и не стал придираться.

void UARTInit(void) {
UBRRH = 0;
UBRRL = 51; //baud rate 9600
UCSRB = (1‹‹RXEN)|(1‹‹TXEN)|(1‹‹RXCIE);
UCSRC = (1‹‹URSEL)|(1‹‹UCSZ1)|(1‹‹UCSZ0); //8 bit, 1 stop bit
}

А вот тут - в режиме расширенного ответа - всё нормально выглядит, скопипащу как сюда:

Нажмите, чтобы открыть спойлер

void SPI_Init(void)
{
DDRB |=(1‹‹PB2)|(1‹‹PB3)|(1‹‹PB5); //Настроить выводы MOSI,SS,SCK на выход
PORTB|=(1‹‹PB2); //Установить "1" на линии SS

SPCR = 0; //Обнулить регистр SPCR
SPSR = 0; //Обнулить регистр SPSR

SPCR |= (1‹‹MSTR)|(1‹‹SPR0); //Режим мастер, F=Fosc/16
SPSR |=(1‹‹SPI2X); //F=Fosc/8

SPCR |=(1‹‹SPE);//Включить SPI
}

void SPI_Send_byte(char data)
{
PORTB &=~(1‹‹PB2); //Установить "0" на линии SS
SPDR = data; //Отправить байт
while(!(SPSR&(1‹‹SPIF))) ; //Дождаться окончания передачи
PORTB |=(1‹‹PB2); //Установить "1" на линии SS
}

int main(void)
{
SPI_Init();
SPI_Send_byte('1');
while(1)
{

}
}


слэйв:

void SPI_Init_Slave(void)
{
DDRB&= ~(1‹‹PB2) | ~(1‹‹PB3) |~(1‹‹PB5);
DDRB|=(1‹‹PB4);

SPCR=0;
SPSR=0;

SPCR |= (1‹‹SPR0)|(1‹‹SPIE);
SPSR |=(1‹‹SPI2X);

SPCR |=(1‹‹SPE);
}


ISR(SPI_STC_vect)
{
PORTB = SPDR; // вывод данных
return ;
}

int main(void)
{
sei();
SPI_Init_Slave();

DDRB|=(1‹‹PC0); //Настраиваем ножку PB2 в режим выхода
while(1) //бесконечный цикл
{
if(SPDR&=~0)//Если в SPDR поступил байт
{
PORTB|=(1‹‹PC0);//Зажигаем светодиод
}
else// если не поступил, тогда
{
PORTB&=~(1‹‹PC0);//Гасим светодиод
}
}
} 

[H4LF]

МВА, я точно вставлял нормальный код, но без тегов. что и требовалось доказать.

[МВА]

МВА, я точно вставлял нормальный код, но без тегов. что и требовалось доказать.

void SPI_Init(void)
{
DDRB |=(1‹‹PB2)|(1‹‹PB3)|(1‹‹PB5); //Настроить выводы MOSI,SS,SCK на выход
PORTB|=(1‹‹PB2); //Установить "1" на линии SS

SPCR = 0; //Обнулить регистр SPCR
SPSR = 0; //Обнулить регистр SPSR

SPCR |= (1‹‹MSTR)|(1‹‹SPR0); //Режим мастер, F=Fosc/16
SPSR |=(1‹‹SPI2X); //F=Fosc/8

Странно, я скопировал с Вашего сообщения кусок кода и вставил в сообщение. Не кажется ли что это мистика? В предварительном просмотре все отображено нормально.
Тогда я не прав, прошу прощенья. Удалю свое предыдущее сообщение.

[H4LF]

МВА, да, я тоже вот только и заметил, когда ответ тут стал писать. Это форумный движок не выкидывает подозрительное (как он может сделать, чтобы не пропускать какие-нибудь XSS и тому подобное), а просто не отображает в обычном режиме.
PS: плохо, что предварительный просмотр отображает не совсем так, как будет выглядеть...