A função `clearInterval` deve receber o retorno da função `setInterval`, porém como você encapsulou a `setInterval`, acabou passando a função `timer` como parâmetro, logo seu `setInterval` vai continuar a ser executado.


----------

Você pode criar uma variável para guardar o retorno do `setInterval`, com isso será possível passar a mesma como parâmetro para a `clearInterval`:

<!-- begin snippet: js hide: false console: true babel: false -->

<!-- language: lang-js -->

    let intervalo = null;

    var timer = function(){
        let count = 0;
        intervalo = setInterval(() => {
            console.log(count);
            count++
            }, 200)}

    var stop = function(){
        setTimeout(() => {
            clearInterval(intervalo)
            console.log('executei');        
                }, 2000)
        }

    timer();
    stop();

<!-- end snippet -->


----------

Também é possível alterar a função `timer` para retornar o intervalo e a função `stop` para receber o intervalo por parâmetro:

<!-- begin snippet: js hide: false console: true babel: false -->

<!-- language: lang-js -->

    var timer = function(){
        let count = 0;

        const intervalo = setInterval(() => {
            console.log(count);
            count++
            }, 200);

        return intervalo;
    }

    var stop = function(intervalo){
        setTimeout(() => {
            clearInterval(intervalo);
            console.log('executei');        
                }, 2000);
    }

    let intervalo = timer();
    stop(intervalo);

<!-- end snippet -->


----------


> Documentações:

> https://developer.mozilla.org/pt-BR/docs/Web/API/WindowOrWorkerGlobalScope/setInterval

> https://developer.mozilla.org/en-US/docs/Web/API/WindowOrWorkerGlobalScope/clearInterval