Что не так с кодом - делаю по книге Белова Разраб. уст-в на мк AVR

Всем привет. Вот повторяю код из книги: задача - нажал кнопку- загорелся светодиод, нажал еще раз - погас. Вначале кода много воды до слоа while, т.к. делал собственно в визарде

Код (Text):

#include <io.h>

// Declare your global variables here

void main(void)
{
// Declare your local variables here

// Crystal Oscillator division factor: 1
#pragma optsize-
CLKPR=(1<<CLKPCE);
CLKPR=(0<<CLKPCE) | (0<<CLKPS3) | (0<<CLKPS2) | (0<<CLKPS1) | (0<<CLKPS0);
#ifdef _OPTIMIZE_SIZE_
#pragma optsize+
#endif

// Reset Source checking
if (MCUSR & (1<<PORF))
   {
   // Power-on Reset
   MCUSR=0;
   // Place your code here

   }
else if (MCUSR & (1<<EXTRF))
   {
   // External Reset
   MCUSR=0;
   // Place your code here

   }
else if (MCUSR & (1<<BORF))
   {
   // Brown-Out Reset
   MCUSR=0;
   // Place your code here

   }
else
   {
   // Watchdog Reset
   MCUSR=0;
   // Place your code here

   }

// Input/Output Ports initialization
// Port A initialization
// Function: Bit2=In Bit1=In Bit0=In
DDRA=(0<<DDA2) | (0<<DDA1) | (0<<DDA0);
// State: Bit2=T Bit1=T Bit0=T
PORTA=(0<<PORTA2) | (0<<PORTA1) | (0<<PORTA0);

// Port B initialization
// Function: Bit7=Out Bit6=Out Bit5=Out Bit4=Out Bit3=Out Bit2=Out Bit1=Out Bit0=Out
DDRB=(1<<DDB7) | (1<<DDB6) | (1<<DDB5) | (1<<DDB4) | (1<<DDB3) | (1<<DDB2) | (1<<DDB1) | (1<<DDB0);
// State: Bit7=1 Bit6=1 Bit5=1 Bit4=1 Bit3=1 Bit2=1 Bit1=1 Bit0=1
PORTB=(1<<PORTB7) | (1<<PORTB6) | (1<<PORTB5) | (1<<PORTB4) | (1<<PORTB3) | (1<<PORTB2) | (1<<PORTB1) | (1<<PORTB0);

// Port D initialization
// Function: Bit6=In Bit5=In Bit4=In Bit3=In Bit2=In Bit1=In Bit0=In
DDRD=(0<<DDD6) | (0<<DDD5) | (0<<DDD4) | (0<<DDD3) | (0<<DDD2) | (0<<DDD1) | (0<<DDD0);
// State: Bit6=P Bit5=P Bit4=P Bit3=P Bit2=P Bit1=P Bit0=P
PORTD=(1<<PORTD6) | (1<<PORTD5) | (1<<PORTD4) | (1<<PORTD3) | (1<<PORTD2) | (1<<PORTD1) | (1<<PORTD0);

// Timer/Counter 0 initialization
// Clock source: System Clock
// Clock value: Timer 0 Stopped
// Mode: Normal top=0xFF
// OC0A output: Disconnected
// OC0B output: Disconnected
TCCR0A=(0<<COM0A1) | (0<<COM0A0) | (0<<COM0B1) | (0<<COM0B0) | (0<<WGM01) | (0<<WGM00);
TCCR0B=(0<<WGM02) | (0<<CS02) | (0<<CS01) | (0<<CS00);
TCNT0=0x00;
OCR0A=0x00;
OCR0B=0x00;

// Timer/Counter 1 initialization
// Clock source: System Clock
// Clock value: Timer1 Stopped
// Mode: Normal top=0xFFFF
// OC1A output: Disconnected
// OC1B output: Disconnected
// Noise Canceler: Off
// Input Capture on Falling Edge
// Timer1 Overflow Interrupt: Off
// Input Capture Interrupt: Off
// Compare A Match Interrupt: Off
// Compare B Match Interrupt: Off
TCCR1A=(0<<COM1A1) | (0<<COM1A0) | (0<<COM1B1) | (0<<COM1B0) | (0<<WGM11) | (0<<WGM10);
TCCR1B=(0<<ICNC1) | (0<<ICES1) | (0<<WGM13) | (0<<WGM12) | (0<<CS12) | (0<<CS11) | (0<<CS10);
TCNT1H=0x00;
TCNT1L=0x00;
ICR1H=0x00;
ICR1L=0x00;
OCR1AH=0x00;
OCR1AL=0x00;
OCR1BH=0x00;
OCR1BL=0x00;

// Timer(s)/Counter(s) Interrupt(s) initialization
TIMSK=(0<<TOIE1) | (0<<OCIE1A) | (0<<OCIE1B) | (0<<ICIE1) | (0<<OCIE0B) | (0<<TOIE0) | (0<<OCIE0A);

// External Interrupt(s) initialization
// INT0: Off
// INT1: Off
// Interrupt on any change on pins PCINT0-7: Off
// Interrupt on any change on pins PCINT8-10: Off
// Interrupt on any change on pins PCINT11-17: Off
MCUCR=(0<<ISC11) | (0<<ISC10) | (0<<ISC01) | (0<<ISC00);
GIMSK=(0<<INT1) | (0<<INT0) | (0<<PCIE0) | (0<<PCIE2) | (0<<PCIE1);

// USI initialization
// Mode: Disabled
// Clock source: Register & Counter=no clk.
// USI Counter Overflow Interrupt: Off
USICR=(0<<USISIE) | (0<<USIOIE) | (0<<USIWM1) | (0<<USIWM0) | (0<<USICS1) | (0<<USICS0) | (0<<USICLK) | (0<<USITC);

// USART initialization
// USART disabled
UCSRB=(0<<RXCIE) | (0<<TXCIE) | (0<<UDRIE) | (0<<RXEN) | (0<<TXEN) | (0<<UCSZ2) | (0<<RXB8) | (0<<TXB8);

// Analog Comparator initialization
// Analog Comparator: Off
// The Analog Comparator's positive input is
// connected to the AIN0 pin
// The Analog Comparator's negative input is
// connected to the AIN1 pin
ACSR=(1<<ACD) | (0<<ACBG) | (0<<ACO) | (0<<ACI) | (0<<ACIE) | (0<<ACIC) | (0<<ACIS1) | (0<<ACIS0);
// Digital input buffer on AIN0: On
// Digital input buffer on AIN1: On
DIDR=(0<<AIN0D) | (0<<AIN1D);


while (1)
   {
   while (PIND.0==1) {}
    if (PINB.0==1)
     { PORTB.0=0; }
   else
     { PORTB.0=1; }
   while (PIND.0==0) {}
   };
}

 

Код компилируется, исп. внутренний кварц. Но код не работает - светодиод не зажигается ((( Что не так?

 

Закиньте код в тэг кода, так он трудночитаем.

А, вот уже вижу. На чем компилировать пытаетесь? на Arduino IDE? Он этих команд не понимает. Это код от CodeVision.

И еще, оно ждет 1 на ноге, а дождавшись выключает эту же ногу. Какой смысл в этом?

 

Я вроде в тег засунул. Эээ попробую поправить

 

на форуме теги в квадратных скобочках

 

Светодиод точно на ноге 0 порта B? А кнопка на ноге 0 порта D? Кнопка притянута к питанию через резистор, а при нажатии контачит с землей? И компилите на CodeVision? Если так, то программа должна работать.

Попробуйте еще так:

Код (C++):

while (1)
   {
   while (PIND.0) {}
   PORTB.0=!PINB.0;
   while (!PIND.0) {}
   };
}

Для Arduino IDE и AVRS будет так:

Код (C++):

void setup() {
  DDRD = 0x00;
  DDRB = 0x01;
}

void loop() {
  while (PIND & 1);
  PORTB = PINB ^ 1;
  while (!(PIND & 1));
}
 

 

Я собственно исходники взял от сюда: http://book.mirmk.ru/wdisk/pp.htm . Делал строго по книге, но у меня CodeVision другой версии и код генерит который я приложил выше. Из ссылки выше - открываем архив и выбираем Ppog2 - там код из книги который вы приложили, но он генерируется, но на деле не запускается, а вот зашил hex - и все заработало. Т.е. если я правильно Вас понимаю, когда читаем книги используем IDe собственно из книги, которая старая. Я делал все через Мастер, т.е. как в книге написано выбрать для порта IN/OUT и поставить 1/0 в нужных местах.

 

Arduino сейчас не интересует, т.к. хочу прошивать 2313 и др. мк. Но код пригодиться

 

Ардуино не на МК разве? Никто не мешает писать на чистом Си ни там ни сям. Порты, по вашему, только на Ардуино?

 

Если я праивльно понимаю: создаю проект в codevision - мигание светодиода на такой-то ноге, а потом захожу в uniprog - выбираю загрузить hex- затру ли я загрузчик таким действием??? (программатор 5 проводков)

 

Автора Ардуино не интересует. Непосредственно в МК проще всего программатором заливать.

 

Спасибо, это тоже опыт - о там не знал.

 

Если вы перед началом оптимизации не упираетесь ни в один предел, ни по производительности, ни по памяти, то считайте, что зря тратите время. Имхо лучше уделить внимание архитектуре приложения и чистоте кода.

 

А как же перфекционизм?

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

 

Перфекционизм и преждевременная необоснованная оптимизация это совершенно разные вещи, в некотором смысле противоположные.

 

Мне вот не столько за Тини13 обидно, которую можно безболезненно заменить на Тини85, сколько за Тини2313 - замечательный МК, у которого все портит мизерное количество памяти. Есть Тини4313, но это такая экзотика.

 

Есть еще tiny1634
Они все в одинаковом корпусе. Получается самый маленький МК в QFN20 с аппаратным UARTом на борту и уже вполне приличным для своего класса флэшем.