MicroBlaze GPIO and timer interrupt

Question

Hi
I want to use GPIO and timer interrupt. I use interrupt controller IP.
my target to get 1ms interrupt using timer and GPIO interrupt.
the problem is initialize timer interrupt second after initialize GPIO interrupt >>>> it works , but GPIO int doesn't work
then I put  
XIntc_Enable(&Intc, 0);     // Enable GPIO interrupt high priority
timer interrupt GPIO interrupt work . but sitll there is a problem which is after GPIO interrupt the program hang and never return to main
this is the first post for me , I hope finding helpful replys
thanks
this is my code:
//************************************ for main **********************************************//
int main()
/*----------------------------------initializations--------------------------------------*/
        init_platform();
        print("Hello World\n\r");
        // initialize GPIO
                Gpio_initialization(&LED_GPIO,GPIO_DEVICE_ID);
        // initialize GPIO interrupt
                GpioIntr_initialization(&Intc,&LED_GPIO,GPIO_DEVICE_ID,
                                       XPAR_INTC_0_GPIO_0_VEC_ID, BUTTON_INTERRUPT);
                                       //INTC_DEVICE_ID
        // initialize Timer
        Status = TmrCtrIntr_start(&InterruptController,
                          &TimerCounterInst,
                          TMRCTR_DEVICE_ID,
                          TMRCTR_INTERRUPT_ID,
                          TIMER_CNTR_0);
while(1){
//************************************ for timer **********************************************//
int TmrCtrIntr_start(INTC* IntcInstancePtr,
            XTmrCtr* TmrCtrInstancePtr,
            u16 DeviceId,
            u16 IntrId,
            u8 TmrCtrNumber)
    int Status;
     * Initialize the timer counter so that it's ready to use,
     * specify the device ID that is generated in xparameters.h
    Status = XTmrCtr_Initialize(TmrCtrInstancePtr, DeviceId);
    if (Status != XST_SUCCESS) {
        return XST_FAILURE;
    print("Timer initialized \n\r");
     * Perform a self-test to ensure that the hardware was built
     * correctly, use the 1st timer in the device (0)
    Status = XTmrCtr_SelfTest(TmrCtrInstancePtr, TmrCtrNumber);
    if (Status != XST_SUCCESS) {
        return XST_FAILURE;
     * Connect the timer counter to the interrupt subsystem such that
     * interrupts can occur.  This function is application specific.
    Status = TmrCtrSetupIntrSystem(IntcInstancePtr,
                    TmrCtrInstancePtr,
                    DeviceId,
                    IntrId,
                    TmrCtrNumber);
    if (Status != XST_SUCCESS) {
        return XST_FAILURE;
     * Setup the handler for the timer counter that will be called from the
     * interrupt context when the timer expires, specify a pointer to the
     * timer counter driver instance as the callback reference so the handler
     * is able to access the instance data
    XTmrCtr_SetHandler(TmrCtrInstancePtr, TimerCounterHandler,
                       TmrCtrInstancePtr);
     * Enable the interrupt of the timer counter so interrupts will occur
     * and use auto reload mode such that the timer counter will reload
     * itself automatically and continue repeatedly, without this option
     * it would expire once only
    XTmrCtr_SetOptions(TmrCtrInstancePtr, TmrCtrNumber,
                XTC_INT_MODE_OPTION | XTC_AUTO_RELOAD_OPTION);
     * Set a reset value for the timer counter such that it will expire
     * eariler than letting it roll over from 0, the reset value is loaded
     * into the timer counter when it is started
    XTmrCtr_SetResetValue(TmrCtrInstancePtr, TmrCtrNumber, RESET_VALUE);
     * Start the timer counter such that it's incrementing by default,
     * then wait for it to timeout a number of times
    XTmrCtr_Start(TmrCtrInstancePtr, TmrCtrNumber);
    return XST_SUCCESS;
//********************************************** for GPIO ************************************************************//
int GpioSetupIntrSystem(XIntc *IntcInstancePtr, XGpio *InstancePtr,
            u16 DeviceId, u32 IntrId, u32 IntrMask)
    int Result;
    GlobalIntrMask = IntrMask;
     * Initialize the interrupt controller driver so that it's ready to use.
     * specify the device ID that was generated in xparameters.h
    Result = XIntc_Initialize(IntcInstancePtr, INTC_DEVICE_ID);
    if (Result != XST_SUCCESS) {
        return Result;
    /* Hook up interrupt service routine */
    XIntc_Connect(IntcInstancePtr, IntrId,
              (Xil_ExceptionHandler)GpioHandler, InstancePtr);
    /* Enable the interrupt vector at the interrupt controller */
    XIntc_Enable(IntcInstancePtr, IntrId);
     * Start the interrupt controller such that interrupts are recognized
     * and handled by the processor
    Result = XIntc_Start(IntcInstancePtr, XIN_REAL_MODE);
    if (Result != XST_SUCCESS) {
        return Result;
     * Enable the GPIO channel interrupts so that push button can be
     * detected and enable interrupts for the GPIO device
    XGpio_InterruptEnable(InstancePtr, IntrMask);
    XGpio_InterruptGlobalEnable(InstancePtr);
     * Initialize the exception table and register the interrupt
     * controller handler with the exception table
    Xil_ExceptionInit();
    Xil_ExceptionRegisterHandler(XIL_EXCEPTION_ID_INT,
             (Xil_ExceptionHandler)INTC_HANDLER, IntcInstancePtr);
    /* Enable non-critical exceptions */
    Xil_ExceptionEnable();
    return XST_SUCCESS;
}

Answer 1

Well Well
Thank you so much stephenm . yes it works now
The error is initialize INTC two times. It should be initialize once and use XIntc_Connect two times. one for GPIO and other for timer.
Thank you for your help.