Schieberegister mit Pic ansteuern

hallo miteinander

ich wollte mit hilfe von MPLABX und dem XC8 compiler meinen Pic16f877a 
dazu bringen, dass das Schieberegister ein einfaches muster herausgibt. 
an den Q's des Schieberegister sind  LED's angeschlossen.
den uC programmiere ich über den Brenner 8P von Sprut via ICSP.
das programm ist ziemlich einfach in C realisiert. Wenn der uC auf run 
ist scheint das SR wie tot. wenn der uC auf stop ist ( um ein Programm 
rüberzuladen) flackern alle led's unontrolliert. das wäre mir noch egal 
aber sobald er auf run geht sollte sich das SR füllen und mein gewähltes 
muster ausgeben. Ihr könnt mir sicher helfen. im anhang befindet sich 
ein photo und das schema.

das programm sieht so aus.

/** I N C L U D E S *******************************************/
#include <htc.h>
#include <stdio.h>
#include <stdlib.h>
#include <pic.h>

// PIC16F877A Configuration Bit Settings

// CONFIG
#pragma config FOSC = XT        // Oscillator Selection bits (XT 
oscillator)
#pragma config WDTE = OFF       // Watchdog Timer Enable bit (WDT 
disabled)
#pragma config PWRTE = OFF      // Power-up Timer Enable bit (PWRT 
disabled)
#pragma config BOREN = ON       // Brown-out Reset Enable bit (BOR 
enabled)
#pragma config LVP = ON         // Low-Voltage (Single-Supply) 
In-Circuit Serial Programming Enable bit (RB3/PGM pin has PGM function; 
low-voltage programming enabled)
#pragma config CPD = OFF        // Data EEPROM Memory Code Protection 
bit (Data EEPROM code protection off)
#pragma config WRT = OFF        // Flash Program Memory Write Enable 
bits (Write protection off; all program memory may be written to by 
EECON control)
#pragma config CP = OFF         // Flash Program Memory Code Prot


#ifndef _XTAL_FREQ
#define _XTAL_F REQ 4000000L // (4mhz)
#endif

#define clk RA1
#define data RA0

void clock(void);

/** H A U P T P R O G R A M M *********************************/
void main(void)
{
/** D E F I N E S *********************************************/
    TRISD = 0x00;
    TRISA = 0x00;



  while(1) {          //Hauptschleife
            data =1;
            clock();
            data =0;
            clock();
             data =1;
            clock();
            data =0;
            clock();
             data =1;
            clock();
            data =0;
            clock();
             data =1;
            clock();
            data =0;
            clock();
  }
}


void clock(void)
{
    clk=0;
    NOP();
    clk=1;
   NOP();
    clk=0;
}