c++ funktioner, skicka värde?

Det första och mest uppenbara felet du gör är att du hanterar lokala variabler fel.
Variabler som deklareras inuti en funktion syns bara inifrån den funktionen. Så den variabel min du deklarerar inuti funktion1() är inte samma variabel som den min som finns i main().
De min och max värden du räknat ut och sparat inuti funktion1() kommer därför att bara försvinna i tomma intet så fort funktionsanropet avslutas.

En sak till som kan orsaka svårt spårbara problem är att du igenom "using namespace std;" har alla funktioner och variabler i din namespace som syns ifrån iostream, vilket i din konkreta fall kan (men inte nödvändigtvis behöver, beroende på vilken compiler och tillhörande implementation av STL du använder) innehålla en definition av bl.a. std::min och std::max (se t.ex. [1]) som du försöker redefinera som int min i din kod. Så jag skulle rekommendera att du inte använder `using namespace std;` för nuvarande och skriver istället std::cin, std::cout, std::endl osv. - eller så kan du skriva "using std::cout, std::cin, std::endl;" i början istället för "using namespace std;" så behöver du inte ändra alla cin till std::cin osv.

using namespace std är lite overkill i många fall om man inte än vet vad man gör Lite som "from x import *" i Python, för de som kommer från det hållet.
Se också [2].

[1] https://stackoverflow.com/questions/19897722/why-can-stdmax-a...
[2] https://www.quora.com/Why-do-many-people-use-std-cout-instead...

PS: jag hade inte möjlighet just nu att testa eller reproducera just felet du kom på, i princip behöver man oftast se lite mer detaljer av felmeddelandet; men tipps ovan lär ändå hjälpa dig att undvika en hel del problem som inte har med det du försöker lösa att göra, som namespaces, scope etc.

#include <iostream>

using namespace std;



 float funktion1 (float inp)
{
    int max = INT_MIN;
    int nst = INT_MIN;
    int min = INT_MAX;
    int sum = 0;
    int antal = 0;
    float inp = 0; 


     while (true)
    {

        if (inp == 0) 
        {
            break;
        }

        if (inp > max) 
        {
            nst=max;
            max = inp;
        }
        if (inp > nst && inp < max) 
        {
            nst = inp;
        }

        if (inp < min)
        {
            min = inp;
        }
        sum += inp;
        antal++;
    }
    return inp;
}

int main ()
{
    int max = INT_MIN;
    int nst = INT_MIN;
    int min = INT_MAX;
    int sum = 0;
    int antal = 0;
    float inp = 0;
     
    cout << "Skriv in ett antal tal, avsluta med 0:" << endl;
    
    cin >> inp;

    inp = funktion1(inp);

    cout << "Summa är: " << sum << endl;
    cout << "Medelvärde är: " << (double)sum/(double)antal << endl;
    cout << "Störst tal är: " << max << endl;
    cout << "Näst störst tal är: " << nst << endl;
    cout << "Minst tal är: " << min << endl;
    
    
    
    return 0;
}

Nu tog jag en till titt på din kod och insåg vad som orsakade just felet med "previous definition" eller kanske hellre "previously declared" - du har deklarerat "inp" två gånger, nämligen först i funktionsdeklarationen som parameter: "float funktion1(float inp)" och sen igen några rad under det i själva funktionen, "float inp=0". Ta bort "float" om du vill bara sätta variablen till 0, men då behöver du inte heller ha det som argument till din funktion då vad än du överlämnar till funktionen blir ändå 0 i det rådet då. Så ta helt bort det andra. Men som andra redan sa måste du skriva om funktionen helt ändå för att åstadkomma det du vill göra.

Men fel gör man för att lära sig mer du är på rätt väg skulle jag säga. Lycka till!