Editing PCD8544.h (Solved)

Other Hardware Displays
1

I wanted to simplify the wiring for a 5110. I have been using the 5110 for some years and I modified the library at the outset but I'm not sure what I did. The new wiring is

8 9 10 11 12

CLK DIN DC CE RST

and I have lines 28 to 32:-

class PCD8544: public Print {
    public:
 // All the pins can be changed from the default values...
        PCD8544(unsigned char sclk  = 8,    /* clock        (display pin 5) */
                unsigned char sdin  = 9,    /* data-in      (display pin 4) */
                unsigned char dc    = 10,   /* data select  (display pin 3) */
                unsigned char sce   = 11,   /* enable       (display pin 2) */
                unsigned char reset = 12);  /* reset        (display pin 1) */

I get nothing on the display. I don't know if I have missed something or just have a mental block. I first assumed the wiring wasn't kosher but I have just installed a little clock programme that does not use a library, and it works OK so I now know the shield is fine.

2

OK, I would delete this if I knew how. The problem, for those interested, was the order of the pin assigment. The last two lines of the above code should be swapped to

                unsigned char reset = 12);  /* reset        (display pin 1) */
                unsigned char sce   = 11,   /* enable       (display pin 2) */