Frage zum Portexpander 74HCT165

[Markus P. (sebastianwurst)]

Hallo,
ich habe folgende Schaltung (Anhang) verdrahtet und den Code (unten) 
benutzt, doch leider wird auf der Seriellen immer nur "11111111" 
ausgegeben auch wenn ich den Schalter drücke. Hat jemand eine Idee was 
ich falsch gemacht haben könnte?

Kleiner Nachtrag: Nachdem ersten übertragen ist das stopbit ja false und 
er schreibt "00000100" auf die serielle schnittstelle. Wenn ich dann 
nochmal schreiben lasse indem ich das stopbit zurücksetze wird immer nur 
"11111111" geschrieben, egal wie der Schalter steht.

DDRA    = 0x00000000;
PORTA   = 0x11111111; // 1 Pull up aktiv = Eingang 

DDRC    = 0x11111111:

  while(1)
  {

if (stopbit == false)
  {
  
  // Clock im Ruhezustand auf 1 
  PORTC |= (1<<PC4);

  // An PL mal stroben...

  // PL auf High
  PORTC |= (1<<PC5);
  // PL auf Low
  PORTC &= ~(1<<PC5);
  // PL auf High
  PORTC |= (1<<PC5);

  unsigned int m;

  for(m=1;m<=8;++m)
  {
  if (PINA & (1<<PINA0)) { // Logisch 1 wenn Schalter nicht gedrückt da Pullup an
    usart_write_str("0");
  } 
  else 
  {
    usart_write_str("1");
  }


  // An Clock mal stroben...

  // Clock auf High
  PORTC |= (1<<PC4);
  // Clock auf Low
  PORTC &= ~(1<<PC4);
  // Clock auf High
  PORTC |= (1<<PC4);
  }

 stopbit = true;
  }

Das stopbit setze ich dann über die serielle zurück, damit er mir nicht 
zyklisch was auf die Schnittstelle schreibt, sondern nur 1mal.

[Markus P. (sebastianwurst)]

Hat keiner eine Idee ?

[... (Gast)]

Markus P. schrieb:
> DDRA    = 0x00000000;
> PORTA   = 0x11111111; // 1 Pull up aktiv = Eingang
>
> DDRC    = 0x11111111:

Das ist schonmal ziemlich daneben, das kommt so nie durch den Compiler.
1. Binärformat fängt mit 0b an, 0x ist hexadezimal.
2. : statt ;
Deine Portkonfiguration stimmt also nicht. Zumindest PC5/PL ist bei Dir 
ein Eingang statt Ausgang.
Das setzen von DDRA kannst Du Dir eigentllich auch sparen, das ist nach 
einem Reset sowieso 0.

Am Taster/Pin13 fehlt ein PullUp. Die restlichen Eingänge floaten auch, 
ebenfalls keine so gute Idee. Leg die auf definierte Pegel.

[Markus P. (sebastianwurst)]

> Das ist schonmal ziemlich daneben, das kommt so nie durch den Compiler.
> 1. Binärformat fängt mit 0b an, 0x ist hexadezimal.

Stimmt, das war ein Schreibfehler kahm so durch den Compiler 
(Programmers Notepad)

> 2. : statt ;

Nur nen Kopierfehler hier ins Forum, sorry

> Deine Portkonfiguration stimmt also nicht. Zumindest PC5/PL ist bei Dir
> ein Eingang statt Ausgang.

Jetzt sieht es ja so aus:

  DDRC = 0b11111111;

Somit ist  der ganze PortC, also auch PL, ein Ausgang.


> Das setzen von DDRA kannst Du Dir eigentllich auch sparen, das ist nach
> einem Reset sowieso 0.
>
> Am Taster/Pin13 fehlt ein PullUp. Die restlichen Eingänge floaten auch,
> ebenfalls keine so gute Idee. Leg die auf definierte Pegel.


Ich benutze doch die internen PullUps, aktiviert durch
PORTA   = 0b11111111; // 1 Pull up aktiv = Eingang

und halte mich an das Beispiel(s.Anhang) ist das trotzdem falsch ?

[Seltener Gast (Gast)]

Markus P. schrieb:

> Ich benutze doch die internen PullUps

Es geht um die Eingänge des 74HCT165, und der hat keine internen 
Pullup-Widerstände. Daher gehören dort externe hin (bzw. sind unbenutzte 
Eingänge fix an VCC oder GND zu legen).

[Muprhy (Gast)]

Hi,

Pullup daher:

Du hast generell den Eingang als PullUp gezeichnet und ohne 
Tasterbetätigung = High im AVR hinterlegt. Möchtest du aber den Zustand 
wechseln (Taster gedrückt!) so mußt du einen "sauberen" Low-Pegel mit 
10k gegen Masse legen.

[Markus P. (sebastianwurst)]

Seltener Gast schrieb:
> Es geht um die Eingänge des 74HCT165, und der hat keine internen
> Pullup-Widerstände. Daher gehören dort externe hin (bzw. sind unbenutzte
> Eingänge fix an VCC oder GND zu legen)

Da ist der Kanckpunkt, denkfehler von mir, vielen Dank ich glaube das 
war es.....
Danke an Murphy auch !!!! Super....