Is this code not initialising the LCD properly?

[Archangel]

RW bit ? are you going to read from the LCD memory to the PIC ? IF not
just tie it to ground, I do the same for the contrast bit, pin 3. I NEVER use a contrast pot and have yet to need one.
You might try adding FLAGS = 1 into your code, at least until you are sure everything is working. Check the Data Sheet and see if that PIC has comparators on portB and turn them off if so. And Finally TRISB=0
so your PIC knows the LCD requires outputs.

"PIC working ok LED lights up"
That's because the HIGH command will manipulate the TRIS register, I think probably the TRISB = 0 or TRISB = %00000000 will make it work.
Oh BTW I see you are new here, WELCOME !

[Platypus]

Thanks all for the suggestions, I'll give them a go when I get home from work.

Cheers

[Platypus]

Thanks all,

The solution was ANSELH
I had thought ANSEL covered this chip but I guess there is a high register that needs setting.

Back to the datasheet...........

[Archangel]

Thanks all,

The solution was ANSELH
I had thought ANSEL covered this chip but I guess there is a high register that needs setting.

Back to the datasheet...........

And Thank You for the feedback, It adds to my arsenal of experience.