LM2907 schematic critique
No, your "charge pump" circuit doesn't make any sense and won't work.

If you look at the equivalent circuit in the datasheet (figure 51 on page 21), and then read the functional description on page 8, you'll see that it's much more complicated than that.

The charge pump monitors the output of the input comparator, and on every transition, it charges or discharges the timing capacitor C1 between 0.25&times;Vcc and 0.75&times;Vcc using a constant-current source. While C1 is charging or discharging, a fixed current flows out of pin 3 and through R1. This converts each input transition into a fixed-width output pulse. The resulting duty cycle is then averaged by C2 in order to turn it into an analog voltage, for which the output opamp provides buffering.