Guten Morgen,

also die Spannung der Frequenz würde ich mit einem Spannungsteiler reduzieren.
Von welcher Frequenz sprechen wir denn hier überhaupt? Weißt du das?

Ich würde zusätzlich für den Reset MCLR einen Taster einbauen (entprellt), dass du den µC auch gewollt zurück setzen kannst, ohne ihn von der Spannung zu trennen. Zumindest gehört noch eine Kapazität vor den MCLR um ein Delay zu erzeugen.
Dann ergeben zwei LEDs an Port RB1 für mich keinen Sinn(doppelte Anzeige?). Ich dachte du wolltest dir da eine richtige Anzeige bauen.
Also quasi so:
1. LED -> Schaltpunkt zu früh/Drehzahl zu niedrig
2. LED -> Schaltpunkt zu richtig/Drehzahl passt
3. LED -> Schaltpunkt zu spät/Drehzahl zu hoch

Also das man auch Zeit hat sich beispielsweise auf den Schaltpunkt vorzubereiten.
Oder soll das Gerät zu eine bestimmte Drehzahl anzeigen?!

Gruß