Problem with gen~ FM synth

Reilly Shellito's icon

EDIT: got it sorted out, needed a history operator because there was a feedback loop.

I'm still pretty new to Max and gen~, but I'm trying to make an FM synth and have ran into a problem. My goal was to make a 4 op FM synth with a matrix so every operator can modulate any other/itself.

So far I have 2 operators, and everything's working except for when I want to modulate op 2 with op 1. As I increase the amplitude, it sounds like it's just adding the value of the dial to the phasor's output instead of adding the operator signal. But when I do it the other way around, modulating op 1 with op 2, it works perfectly. The two operators are identical in the gen patcher, so I'm at a loss for what the problem is.

Basically, I'm unable to increase op 1's amplitude in order to modulate op 2.

In the gen patcher below, I disconnected the cycle from the multiplier because the DSP won't work upon startup if it's connected. After DSP is turned on, when I connect them, DSP will work but the result is the same. It just adds the dial's value to the phasor of op 2, which is what it does without the patch cable connected.

It's probably something simple, but again I'm new to Max.

fmclean2.maxpat
Max Patch

Reilly Shellito's icon

Whoops, I forgot to disconnect the multiplier like I noted in the gen patcher. It should be like this