I am designing a LED display using a 5X7 matrix of LEDs the columns are connected to a 4017 decade counter and the rows are driven directly by PORT1 of the micro, using P3.0 and P3.1 i reset and clock the 4017.
I have an array of hex digits ( in groups of five) each group of five digits when displayed on the display column by column makes a character or digit. when i do this digit by dgit i get whatever i want to display, but by using DPTR I am not able to do it. here is the code, what is wrong with it?


#include 8051.H

;EQUATES

clk = P3.0
rst = P3.1


.org 0000H

BEGIN: MOV P1,#255
ACALL RESET
CLR clk

MOV DPTR,#DATA
SHOW MOV R3,#255
L2 MOV R4,#25
L1 MOV R5,#5
pop DPL
pop DPH
L3 CLR A
MOVC A,@A+DPTR
MOV P1,A
INC DPTR
NOP
NOP
NOP
MOV P1,#255
ACALL CLOCK
DJNZ R5,L3
ACALL RESET
DJNZ R4,L1
DJNZ R3,L2
ACALL DELAY1
AJMP SHOW



;This routine resets the 4017 and puts the counter at zero
RESET:
SETB rst
NOP
NOP
CLR rst
NOP
RET

;Clocks the 4017

CLOCK:
CLR clk
NOP
NOP
NOP
SETB clk
NOP
NOP
RET

DELAY RET
DELAY1
MOV R0,#255
loop1 MOV R1,#255
LOOP2 MOV R2,#1
djnz R2,$
DJNZ R1,LOOP2
djnz R0,loop1
RET

DATA .DB 0C0H,0B7H,0B7H,0B7H,0C0H
80H,0B6H,0B6H,0B6H,0C9H
0C1H,0BEH,0BEH,0BEH,0DDH
80H,0BEH,0BEH,0BEH,0C1H
80H,0B6H,0B6H,0B6H,0BEH
80H,0B7H,0B7H,0B7H,0BFH
0C1H,0BEH,0B6H,0B6H,0D1H
80H,0F7H,0F7H,0F7H,80H
0BEH,0BEH,80H,0BEH,0BEH
0FFH,0FDH,0FEH,0FEH,81H
80H,0F7H,0E7H,0DBH,0BCH
80H,0FEH,0FEH,0FEH,0F1H
80H,0DFH,0E7H,0DFH,80H
80H,0DFH,0E3H,0FDH,80H
0C1H,0BEH,0BEH,0BEH,0C1H
0C0H,0B7H,0B7H,0B7H,0CFH
0C1H,0BEH,0BAH,0BCH,0C0H
80H,0B7H,0B3H,0B5H,0CEH
0CDH,0B6H,0B6H,0B6H,0D9H
0BFH,0BFH,80H,0BFH,0BFH
80H,0FEH,0FEH,0FEH,80H
87H,0F9H,0FEH,0F9H,87H
80H,0FDH,0F3H,0FDH,80H
0BEH,0DDH,0E3H,0DDH,0BEH
0BFH,0DFH,0E0H,0DFH,0BFH
0BCH,0BAH,0B6H,0AEH,9EH

0C1H,0BEH,0BEH,0BEH,0C1H
0FEH,0EEH,0DFH,80H,0FEH
0D8H,0B6H,0B6H,0B6H,0CEH
0DDH,0BEH,0B6H,0B6H,0C9H
0F3H,0EBH,0DBH,80H,0FBH
85H,0B6H,0B6H,0B6H,0B9H
0C1H,0B6H,0B6H,0B6H,0D9H
0BFH,0B8H,0B7H,0AFH,9FH
0C9H,0B6H,0B6H,0B6H,0C9H
0CDH,0B6H,0B6H,0B6H,0C1H

.end