Der Transistor ist falsch herum. Und das mit der Batterie ist auch falsch eingezeichnet, da der eine Anschluss Masse ist (auch wenn ich verstehe was du meinst).
Source kommt bei P-FETs an Vcc. Und Drain Richtung Masse. Anders herum als bei N-FETs. Die Steuerspannung ist aber immer noch Vgs (gate-source). Wenn am Gate 3,3V anliegt ist Vgs also 0V und der Transistor sperrt. Wenn 0V anliegt hat man 0V-3,3V = -3,3V und der Transistor leitet.
Das ist auch der Grund weshalb man u.U. den NPN braucht. 5V-12V = -7V. Damit sperrt der Transistor nie. Der NPN dient dann dazu die Gate-Spannung anzuheben
Der Widerstand kann kleiner sein. Das wird zwar auch so gehen da die Schaltgeschwindigkeit egal ist, aber ein paar mA werden hier schon gehen