That's the correct function yes. Each example of millis() I've seen uses variables based on the time between changes. I think my issue is storing 2 lots of data from the pots and having them act upon the LED simultaniously, which I find easiest with delays.