Se feito de forma correta nao vejo problemas.

nenhuma. claro que precisa ser bem pensada, mas nao vejo nenhuma restrição !

uma macro ou função é um conjunto de instruções
certo ?
então quando se chama outra macro ou função simplesmente se está executando outro conjunto
tanto em chamada quanto em se escrever as instruções se tem que pensar é na logica do que se quer fazer

macro1
   instrução1
   instrução2
         macro2  >>>  dá na mesma que colocar a instrução3 dentro da macro
   instrução4
fimmacro1

macro2
   instrução1
   instrução2
       macro3   >>>   dá na mesma que colocar a instrução3 dentro da macro
   instrução4
fimmacro2

macro3  >>>   " feita para ser usada em vaias outras macros"
  instrução3
fimmacro3

claro que tudo tem que ser bem estruturado da mesma forma que uma macro unica

Só lembrando que o VBA não permite macros "aninhadas". Ou seja, não é possível usar assim:

Sub Teste1() (ou Function1())
  blá, blá, blá...
  Sub Teste2() (ou Function2())
     blá, blá, blá...
  End Sub (End Function)
End Sub (End Function)

Já nas antigas versões do Basic/GBasic/Turbo Basic isso não era permitiado.

Por outro lado, o velho Turbo Pascal (mil novecentos e guaraná-com-rolha) permitia Procedures e Functions aninhadas.

Valeu pelo retorno pessoal.