los errores que cometiste son varios, la declaración de la matriz no tiene una sintaxis correcta, en los fors de la variable j haces un incremento a la variable i y estas usando dos variables que ya están declarada en otro modulo (fpos, cpos) Mas 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;
}