[Atmega8] PD4 als Eingang trotz USART

[Hugo (Gast)]

Hallo zusammen,

ich möchte bei einem Atmega8 den Pin PD4 (XCK/T0) als Eingang benutzen. 
Leider ist PIND4 immer low obwohl ich mit dem Multimeter 5V am Pin 
messen kann.
Kann es sein, dass der Pin PD4 nicht also IO-port genutzt werden kann, 
wenn ich UART benutze? Eigentlich sollte es doch gehen, wenn ich UART im 
asynchronen Modus habe.

Folgende einstellungen mache ich in meinem Programm:

  DDRB=0b00111101;
  DDRC=0b00111111;
  DDRD=0b11000000;
  PORTB=0;
  PORTC=0;
  PORTD=0;
  TCCR1A=0;
  TCCR1B=0b00000001;
  TIMSK =0b00011100;
  OCR1A =0x7fff;
  OCR1B =0x7fff;
  MCUCR|=0b00001010;
  GICR|=0b11000000;

Die Einstellungen für UART sind:

/* Set baud rate */
void USART_Init( unsigned int ubrr)
{
/* Set baud rate */
UBRRH = (unsigned char)(ubrr>>8);
UBRRL = (unsigned char)ubrr;
/* Enable receiver and transmitter */
UCSRB = (1<<RXEN)|(0<<TXEN)|(1<<RXCIE);
/* Set frame format: 8data, 2stop bit */
UCSRC = (1<<URSEL)|(1<<USBS)|(3<<UCSZ0);
}

Gruß
Hugo

[holger (Gast)]

>Kann es sein, dass der Pin PD4 nicht also IO-port genutzt werden kann,
>wenn ich UART benutze?

Nein, wenn du auf asynchron einstellst kannst du den Pin nutzen.

Zeig mal das komplette Programm.

>/* Enable receiver and transmitter */
>UCSRB = (1<<RXEN)|(0<<TXEN)|(1<<RXCIE);

Senden möchtest du nicht?

[Hugo (Gast)]

Ich möchte sowohl Senden als auch Empfangen.
Es soll ein 2-KanalInkrementalgeber ausgelesen werden. Die Abfrage, die 
Probleme bereitet ist in INT0. In zweiter Interrupt hängt am Int1 und 
fuktioniertr tadellos.

Hier das komplette Programm bis jetzt.

#include <avr/io.h>
#include <avr/delay.h>
#include <avr/interrupt.h>
#include "USART.h"

#define F_CPU 8000000L
#define FOSC 8000000L// Clock Speed
#define BAUD 9600
#define MYUBRR FOSC/16/BAUD-1

int uart_state;
int16_t M1,M2;
int16_t pos1,pos2;

void USART_Init( unsigned int ubrr);
void USART_Transmit( unsigned char data )
{
  
 //Wait for empty transmit buffer 
while ( !( UCSRA & (1<<UDRE)) )
;
// Put data into buffer, sends the data 
UDR = data;
}
unsigned char USART_Receive( void )
{
/* Wait for data to be received */
while ( !(UCSRA & (1<<RXC)) )
;
/* Get and return received data from buffer */
return UDR; 
}
void USART_Init( unsigned int ubrr)
{
/* Set baud rate */
UBRRH = (unsigned char)(ubrr>>8);
UBRRL = (unsigned char)ubrr;
/* Enable receiver and transmitter */
UCSRB = (1<<RXEN)|(0<<TXEN)|(1<<RXCIE);
/* Set frame format: 8data, 2stop bit */
UCSRC = (1<<URSEL)|(1<<USBS)|(3<<UCSZ0);
}



void init(){
  USART_Init ( MYUBRR );
  DDRB=0b00111101;
  DDRC=0b00111111;
  DDRD=0b11000000;
  PORTB=0;
  PORTC=0;
  PORTD=0;
  TCCR1A=0;
  TCCR1B=0b00000001;
  TIMSK =0b00011100;
  OCR1A =0x7fff;
  OCR1B =0x7fff;
  MCUCR|=0b00001010;
  GICR|=0b11000000;
  
  uart_state=0;
  pos1=0;
  pos2=0;
}

ISR(INT0_vect){
  if((PIND&0b10000)>>4){
    pos2++;
    USART_Transmit('+');
  }else{
    pos2--;
    USART_Transmit('-');
  }
}
ISR(INT1_vect){
  if((PIND&0b100000)>>5){
    pos1++;
  }else{
    pos1--;
  }
}


ISR(TIMER1_OVF_vect){
  PORTB|=0b00000110;
}
ISR(TIMER1_COMPA_vect){
  PORTB&=0b11111101;
}
ISR(TIMER1_COMPB_vect){
  PORTB&=0b11111011;
}
ISR(USART_RXC_vect)
{
  char tmp;
  tmp=USART_Receive();
  switch (uart_state)
  {
  case 0:
    if(tmp=='s')uart_state=1;
    break;
  case 1:
    if(tmp=='e') {
      uart_state=2;
    }else{
      uart_state=0;
    }  
    break;
  case 2:
  if(tmp=='t') {
      uart_state=3;
    }else{
      uart_state=0;
    }
    break;
  case 3:
  
    if(tmp=='r'){
      M1=10;
      USART_Transmit('r');  
    }
    if(tmp=='l'){
      M1=-10;
      USART_Transmit('l');
    }
    if(tmp=='u'){
      M2=10;
      USART_Transmit('u');  
    }
    if(tmp=='d'){
      M2=-10;
      USART_Transmit('d');
    }
    uart_state=0;
  }
  USART_Transmit(uart_state);
  
}  


int main(void)
{
  init();
  sei();
    while(1)
    {
    
    }
}

[Hugo (Gast)]

holger schrieb:
>>/* Enable receiver and transmitter */
>>UCSRB = (1<<RXEN)|(0<<TXEN)|(1<<RXCIE);
>
> Senden möchtest du nicht?

Das Senden war nur als versuch ausgeschaltet. Es muss natürlich

UCSRB = (1<<RXEN)|(0<<TXEN)|(1<<RXCIE);

heißen.
Das ist aber nicht das problem, sondern der Pin PD4 als eingang.

[Karl H. (kbuchegg)]

Du hast, soweit ich das sehen kann, keine Pullups am PortD aktiviert.

Ansonsten, auch wenn deine Schreibweisen furchtbar unübersichtlich sind, 
sehe ich nichts was die Funktion verhindern würde.

Hast du schon mal versucht ein einfaches Testprogramm zu schreiben, mit 
dem du nur den Pin PD4 testest?

[H.Joachim S. (crazyhorse)]

Und das hier ist Absicht?
UCSRC = (1<<URSEL)|(1<<USBS)|(->3<- <<UCSZ0);
Hab jetzt aber nicht nachgeschaut, was daraus wird. Unüblich find ich es 
auf jeden Fall.

[holger (Gast)]

>holger schrieb:
>>>/* Enable receiver and transmitter */
>>>UCSRB = (1<<RXEN)|(0<<TXEN)|(1<<RXCIE);
>>
>> Senden möchtest du nicht?
>
>Das Senden war nur als versuch ausgeschaltet. Es muss natürlich
>
>UCSRB = (1<<RXEN)|(0<<TXEN)|(1<<RXCIE);
>
>heißen.

Senden ist damit immer noch nicht möglich.
Wozu dann die ganzen USART_Transmit('u'); ?

>Das ist aber nicht das problem, sondern der Pin PD4 als eingang.

Woher weisst du ob das da ein Problem ist? Wie kommst
du in den Int0?

Schreib dir mal ein Miniprogramm wo du PIND4 abfragst.

[Karl H. (kbuchegg)]

Einschub, weil ich jetzt schon 3 Minuten 0-en zähle

 if((PIND&0b10000)>>4){

das schreibt man nicht so. Programmierer sind faule Hunde. Das hier zu 
lesen und zu analysieren erfordert viel zu viel Aufwand und ist auch 
viel zu fehleranfällig zu schreiben.

Erstens:
In C ist ALLES logisch wahr, was ungleich 0 ist. Ob in deinem Ausdruck 
daher das Bitmuster 0b00010000 oder 0b00000001 rauskommt, spielt 
überhaupt keine Rolle. Ungleich 0 ist ungleich 0.
Daher: Du brauchst dir das eine ausmaskierte Bit nicht zurechtschieben.

Daher

 if( PIND & 0b10000 ){

macht exakt und genau das gleiche wie deine Version. Der einzige 
Unterschied: Du laufst in deiner Version Gefahr, dass die Anzahl der 
Verschiebungen nicht stimmt und dann kommt nicht 1 raus, sonder 0. Und 
das ist dann tatsächlich ein logisches FALSE.

Also: in C gilt
        ein Wert von 0          gilt als logisch FALSE
        ein Wert ungleich 0     gilt als logisch TRUE

und bei ungleich 0 spielt es keine Rolle, was der Wert genau ist. Er 
darf nur nicht 0 sein.

Zweitens:
Diese ewige 0-en Zählerei in den Binärkonstanten ist genauso 
fehleranfällig. Überhaupt dann, wenn du nicht alle 8 Bits eines Bytes 
anschreibst.

  if( PIND % (1 << PD4) )

jetzt braucht kein Mensch mehr 0-en zählen. Auch ein Blinder kann hier 
greifen, dass du den Pin PD4 abfrägst. Die Abfrage lautet in normalem 
Deutsch: Wenn Bit 4 am Port PIND auf 1 ist, dann mach was.
Und in dieser Schreibweise gibt es dann auch kaum mehr blödsinnige 
Fehler, die nur deshalb entstehen, weil man sich irgendwo mit den 0-en 
und 1-en verzählt hat.


Aber: Das ist nicht das Problem. So wie du das geschrieben hast, läuft 
es aufs gleiche raus. Es ist nur schwerer zu lesen und es gibt mehr 
Möglichkeiten für Schreibfehler.
Und es dauert 5 mal so lange, sich davon zu überzeugen, dass die Abfrage 
in Ordnung ist.

[holger (Gast)]

>  if( PIND % (1 << PD4) )
>
>jetzt braucht kein Mensch mehr 0-en zählen. Auch ein Blinder kann hier

Eine Taste daneben;)
  if( PIND & (1 << PD4) )

[Karl H. (kbuchegg)]

:-)
Danke holger.

[holger (Gast)]

Ich frag mich immer noch wie er darauf kommt das PIND4
nicht funktioniert.

Den UART Transmit hat er sich ja selber abgeknipst:

>UCSRB = (1<<RXEN)|(0<<TXEN)|(1<<RXCIE);

Da wird also kein
    USART_Transmit('+');
oder
    USART_Transmit('-');
aus dem Int0 kommen.

Kommt da überhaupt ein '+' oder '-' am Terminal?

Das könnte helfen:

UCSRB = (1<<RXEN)|(1<<TXEN)|(1<<RXCIE);

Transmitter einschalten.

Wie er in den Int0 kommt ist seine Sache.
Der Rest ist mir jetzt echt zu blöd.

[Hugo (Gast)]

Danke für eure Hilfe.
Es war jedoch ein komplett anderer Fehler. Die Gabellichtschranke war um 
ein paar Grad verdreht. Dadurch war der Phasenversatz von den beiden 
Signalen des Inkrementalgebers so gut wie Null und immer wenn INT0 
ausgelöst hast war PD4 auf 0V egal in welche richtung man den 
Inkrementalgeber gedreht hat. Der Code funtioniert einwandfrei (auch 
wenn man ihn schöner schreiben könnte).

Gruß Hugo