Attiny 861 - nach reset anderes verhalten

[Holger M. (holger78)]

Hallo liebes Forum,

ich habe einen Attiny 861 und programmiere diesen mit dem neusten MPLAP 
XC8.

Recht schnell konnte ich erfolge erzielen und der Code wuchs.
Nach einiger zeit musste ich jedoch feststellen das, wenn ich ein 
Reset(Power cycle) machte, meine PCB mit dem µC nicht das machte was sie 
machen sollte. Nach neu Programmieren mit dem ICD3 über UPDI war das 
Verhalten wieder wie gewünscht ohne Fehler.

Ausfindig konnte ich den PORT C machen der nicht mehr die Eingänge 
erkennt.

Mein Code ist folgender:

#define F_CPU 3333333
#include <xc.h>
#include <util/delay.h>

#define LED 0x04        //PA2 - Ausgang - LED 
#define TA  0x7         //PC3 - Eingang - Taster

#define LED_ON          PORTA.OUT &= ~LED       // Schematisch D1 = EIN
#define LED_OFF         PORTA.OUT |= LED        // Schematisch D1 = AUS
#define TASTER          !(PORTC.IN & ~TA)       // Schematisch SW1, Sensor meldet ob Roboter auf Position steht

int main(void)
{
    _delay_ms(1000);
    // PORT 
    PORTA.DIR = 0b10000100;
    PORTC.DIR = 0b0000;
    // PULL UP 
    PORTC.PIN3CTRL = PORT_PULLUPEN_bm;  // PC3 = Taster
    
    LED_ON;
    _delay_ms(1000);
    LED_OFF;
    
    while(1)
    {
     if(TASTER) LED_ON;
     else LED_OFF;
     _delay_ms(500);
    }
}

Mein Weg ist folgender:
1. power
2. make and program aus MPLAB
3. RESULTAT:
LED leuchtet 1 Sekunde, danach kann ich mit dem Taster die LED schalten
->Funktion OK
4. power cycle (ob mit oder ohne ICD3 spiel keine Rolle)
5. Resultat:
LED leuchtet 1 Sekunde. Danach kann der Taster nicht mehr ausgewertet 
werden LED leuchtet nicht mehr.

Was mache ich falsch? Wie kann ich das beheben?
Ich freue mich auf eure Antworten
Gruß Holgus

[Veit D. (devil-elec)]

Hallo,

das define kommt mir komisch vor zusammen mit der if Abfrage.

#define TASTER          !(PORTC.IN & ~TA)

Zum Pin schalten/toggeln würde ich auch die vordefinierten "Namen" aus 
dem Headerfile verwenden.
Bsp.

PORTF.OUTTGL = PIN2_bm;

Statt binär würde ich auch die vordefinierten Bitmasken verwenden.

PORTF.DIR |= PIN2_bm;

Du kannst dir auch wiederverwendbare Funktionen bauen statt defines. Das 
mit den defines würde ich mir erst gar nicht so angewöhnen. Man kommt 
schwer wieder davon los und muss dann für alles defines schreiben ob man 
will oder nicht.

static inline bool PORTF_get_pin_level(const uint8_t pin)
{
  return VPORTF.IN & (1 << pin);
}

das wird von ... verwendet

static inline bool SW0_get_level()
{
  return PORTF_get_pin_level(6);
}

dann rufste nur noch das auf - Bsp.

if ( !SW0_get_level() )
{
  LED0_set_level(false);
}

Die Pinkodierung musste ja nicht hart kodieren, kannste variabel 
gestalten.

Noch was vordefiniertes aus dem Headerfile - zum abschauen.  :-)

static inline void PORTF_set_pin_level(const uint8_t pin, const bool level)
{
  if (level == true) {
    VPORTF.OUT |= (1 << pin);
  } else {
    VPORTF.OUT &= ~(1 << pin);
  }
}

[Ben B. (Firma: Funkenflug Industries) (stromkraft)]

> PORTC.DIR = 0b0000;
Das ist miese Syntax, auch wenn es in dem Fall nicht stört weil sowieso 
'ne Null draus wird, aber entweder schreibe alle Bits hin oder einfach 
0.

[foobar (Gast)]

> Ausfindig konnte ich den PORT C machen der nicht mehr die Eingänge
> erkennt.

Wundert mich jetzt nicht, laut Datenblatt hat der Attiny861 ja gar 
keinen Port C.

Wie auch immer:

#define TA  0x7         //PC3 - Eingang - Taster
#define TASTER          !(PORTC.IN & ~TA)       // Schematisch SW1,

Das Makro TASTER liefert true, wenn Bits 3-7 von PORTC.IN alle 0 sind. 
Sicher, dass du das willst?!?  Oder meintest du nicht eher:

#define TA      0x08    //PC3 is SW1
#define TASTER  ((PORTC.IN & TA)==0)

[Holger M. (holger78)]

Vielen, vielen Dank!!!
Damit habt ihr mir sehr geholfen.
Die ganzen Tipps für die Syntax und vor allem der Hinweis, das der 
Attiny861 kein Port C hat. Oh man... das ist mir peinlich!
Nochmals Danke.
Gruß Holgus

[Peter D. (peda)]

Holger M. schrieb:
> #define F_CPU 3333333

Wo kommen denn die 3,3MHz her?
Der Attiny861 läuft ab Werk mit 1MHz.