los errores que cometiste son varios, la **declaración de la matriz** no tiene una **sintaxis correcta**, en los **FOR** de la variable **j** haces un incremento a la variable **i**.

estás usando **dos variables** que ya están declaradas en otro módulo <kbd>fpos, cpos<>/kbd.

más abajo te dejo el código modificado con comentarios por si no entiendes muy bien los errores que digo. Las variables globales serian mas eficientes pero también la puedes hacer con el método que pusiste.

    #include <iostream>
    using namespace std;
    int ffpos=0, ccpos=0; //Declaramos las variables de forma global para poder usarlas en todas las funciones/eventos y cambiamos de fpos a ffpos y de cpos a ccpos un modulo ya contiene estas variables en uso
    
    void mostrarmat(int m[][4],int dfil){
    	const int dcol=4;
    	for(int i=0;i<dfil;i++){
    		for (int j=0;j<dcol;j++){ //El incremento se le aria a la variable j
    			cout<<m[i][j];
    			(j!=dcol-1)?cout<<"\t":cout<<"\n"; //Solo agrego esta linea para que la salida sea mucho mas intuitiva
    		}
    	}
    }
    void menor(int m[][4], int dfil){
    	/*mfil=0; 
    	mcol=0;*/ //Usaremos las variables globales fpos y ccpos
    	const int dcol=4;
    
    	for(int i=0;i<dfil;i++){
    		for(int j=0;j<dcol;j++){ //El incremento se le aria a la variable j.
    			if ((m[i][j])<(m[ffpos][ccpos])){
    				ffpos=i;
    				ccpos=j;
    			}
    		}
    	}
    }
    int main(){
    	const int nfil=5;
    	const int ncol=4;
    	//La sitaxis de una matriz seria la siguiente.
    	int mat[nfil][ncol] = {
    	{25,8 ,12,7},
    	{17,11,9 ,33},
    	{21,3 ,41,5},
    	{4 ,34,92,18},
    	{37,6 ,43,54}
    	};
    	cout<<"Matriz:"<<endl<<endl;
    	mostrarmat(mat, nfil); //Eliminamos los parametros que no usamos
    	/*int ffpos,ccpos;*/ //Esta linea ya no es necesaria.
    	menor(mat,nfil);
    	cout<<endl<<"El menor valor es "<<mat[ffpos][ccpos]; //Faltaban unos espacios para que la salida estubiera bien
    	cout<<" y se encuentra en la fila "<<ffpos<<" y la columna "<<ccpos<<endl; //Faltaban unos espacios para que la salida estubiera bien
    	return 0;
    }