Wenn R4 beim Einschalten falsch steht sind die Backlight-LEDs hinüber
(zusätzlicher Mindestwiderstand). Anstatt VSS würde ich lieber VCC
schreiben, VSS hat eine andere Bedeutung. Wenn Du noch nicht oft
programmiert hast, solltest Du jedem Pin des Controllers, der auch als
Ausgang geschaltet werden kann einen Schutzwiderstand spendieren, wenn
es sonst zu einer Überlastung kommen könnte (z.B. bei dem Encoder oder
D4-D7 des LCDs). Beim Reset sind die Pins des Controllers hochohmig. Der
Transistor für das Relais könnte dabei zerstört werden (pulldown
einbauen). Die Spannung am Ausgang des Spannungsreglers sollte nie höher
als am Eingang sein. Deshalb würde ich C4 weglassen. Nach X1-1 könnte
eine 1A-Diode eine Verpolung der Versorgungsspannung verhindern. Ob man
AVCC an VCC hängen darf, müsste ich im Datenblatt des Controllers
nachsehen. Es gab da schon bei Atmels Probleme, wenn man die interne
Referenz wählt. Dem Display würde ich zwischen VCC und GND auch noch
einen 100nF-Kondensator geben.
Ich kann Stromlaufpläne, bei denen die Powersymbole nach der Regel hohe
Spannung oben, niedrige Spannung unten und Powersymbole nicht
quergestellt besser lesen, aber das ist sicher eine Geschmacksfrage. So,
genug gemeckert.