Работа с прерываниями в at91sam7s, не вызывается повторное прерывание Опции

[Bulat]

Конфигурация прерываний происходит следующим образом:

Код

void RT()
{
AT91F_AIC_ConfigureIt( AT91C_BASE_AIC, AT91C_ID_PIOA, 0x06, AT91C_AIC_SRCTYPE_INT_POSITIVE_EDGE, irq_RT);
AT91F_PIO_InterruptEnable(AT91C_BASE_PIOA,SW2_MASK);
AT91F_PIO_InterruptEnable(AT91C_BASE_PIOA,SW3_MASK); //Маска прерываний на PA23, PA24
//* set the interrupt by software
AT91F_AIC_EnableIt(AT91C_BASE_AIC, AT91C_ID_PIOA);
  while(1); //Ожидание прерывания
}

Обработчик прерываний:

Код

__ramfunc void irq_RT()
{  
  //Определение источника прерывания и запрет прерываний
  int i_status = AT91F_PIO_GetInterruptStatus(AT91C_BASE_PIOA);
  if(i_status&0x800000) AT91F_PIO_InterruptDisable(AT91C_BASE_PIOA,SW3_MASK);   //Прерывание пришло от PA23
  if(i_status&0x1000000) AT91F_PIO_InterruptDisable(AT91C_BASE_PIOA,SW2_MASK);   //Прерывание пришло от PA24
  AT91F_AIC_DisableIt(AT91C_BASE_AIC, AT91C_ID_PIOA);
  ...
  //программа обработки прерывания
  ...
  //Разрешение прерываний
  if(i_status&0x800000) AT91F_PIO_InterruptEnable(AT91C_BASE_PIOA,SW3_MASK);
  if(i_status&0x1000000) AT91F_PIO_InterruptEnable(AT91C_BASE_PIOA,SW2_MASK);
  AT91F_AIC_EnableIt(AT91C_BASE_AIC, AT91C_ID_PIOA);

Первое прерывание нормально обрабатывается, но второе прерывание не обрабатывается, хотя источником является тот же вывод (PA24). Чтобы обработать следующее прерывание, приходится презагружать устройство и заново вызывать прерывание. То есть больше одного прерывания подряд не обрабатывает. Почемутак происходит, я ведь правильно конфигурирую прерывания?
Заранее благодарен!

[SergeiCh]

Тему не в том разделе создали, надо в помощь начинающим.

Излишеств много. Запрещать прерывания не надо. Вы пока это не обработаете, вторично в него не попадете - приоритет один. Убедитесь, что возвращаетесь в RT(). И не меняется ли у Вас i_status.

Код

void RT(void)
{
    AT91F_AIC_ConfigureIt( AT91C_BASE_AIC, AT91C_ID_PIOA, 0x06, AT91C_AIC_SRCTYPE_INT_POSITIVE_EDGE, irq_RT);
    AT91F_PIO_InterruptEnable(AT91C_BASE_PIOA, SW2_MASK | SW3_MASK); //Маска прерываний на PA23, PA24
    //* set the interrupt by software
    AT91F_AIC_EnableIt(AT91C_BASE_AIC, AT91C_ID_PIOA);
    while(1); //Ожидание прерывания
}

__ramfunc void irq_RT(void)
{  
  //Определение источника прерывания
  int i_status = AT91F_PIO_GetInterruptStatus(AT91C_BASE_PIOA);
   ...
  //программа обработки прерывания
  ...
}

[dimka76]

Первое прерывание нормально обрабатывается, но второе прерывание не обрабатывается, хотя источником является тот же вывод (PA24).

Перед выходом из обработчика прерывания надо произвести запись в регистр AIC_EOICR любого значения.

The End of Interrupt Command Register is used by the interrupt routine to indicate treatment is complete.

[Troll]

Перед выходом из обработчика прерывания надо произвести запись в регистр AIC_EOICR любого значения.

Ага, и еще надо восстанавливать PC = LR - 4 и CPSR. Обычно и то и другое делается в общем irq-обработчике из startup.S.

Убедитесь, что возвращаетесь в RT().

И также, проверьте, что точно второй раз не вызывается функция irq_RT().

[aaarrr]

Ну, добавим еще традиционную и очевидную ошибку - AT91C_AIC_SRCTYPE_INT_POSITIVE_EDGE.
Вот почему всем неймется настроить прерывания от внутренних некомбинированных источников на работу по фронту?

Если писать без маразмов, то получится как-то так:

Код

void RT(void)
{
    AT91C_BASE_AIC->AIC_SVR[AT91C_ID_PIOA] = (unsigned int)irq_RT;
    AT91C_BASE_AIC->AIC_SMR[AT91C_ID_PIOA] = AT91C_AIC_SRCTYPE_INT_HIGH_LEVEL | 6;
    AT91C_BASE_AIC->AIC_IECR = (1UL << AT91C_ID_PIOA);
    AT91C_BASE_SYS->PIOA_IER = SW2_MASK | SW3_MASK;
}

__irq void irq_RT(void)
{
    unsigned int status = AT91C_BASE_SYS->PIOA_ISR;
    if(status & SW2_MASK)
    {
        ...
    }
    if(status & SW3_MASK)
    {
        ...
    }
    AT91C_BASE_AIC->AIC_EOICR = status;
}

[SergeiCh]

Если писать без маразмов,

Код

    AT91C_BASE_AIC->AIC_EOICR = status;

Как заметил Troll, в стандартном startup.s это уже есть.
INT_POSITIVE_EDGE недосмотрел

[aaarrr]

Как заметил Troll, в стандартном startup.s это уже есть.

Ну я же написал "без маразмов". "Стандартный" (и кто же его стандартизировал-то?) стартап к таковым относится.

[SergeiCh]

(и кто же его стандартизировал-то?)

Хорошо, данный нам в softpack Atmel и растиражированный IAR.

BTW, если его не подправить, Ваш пример с __irq работать не будет. В тамошнем startup, насколько помню, lr корректируется (sub lr, lr, #4), примерно то же делает директива __irq, т.е. от lr будет дважды отниматься 4. С другой стороны, если в таблице векторов уже ldr pc, [pc,#-0xF20], как в startup "по умолчанию" у Keil, без __irq оно могло залетать в какой-нибудь abort. Надо смотреть, что в startup у топикстартера.

[aaarrr]

Надо смотреть, что в startup у топикстартера.

Да, надо. Причем топикстартеру, а не нам.

Хорошо бы как объявление закрепить:

Внимание!

При создании темы типа "прерывание на ARM не работает" прикладывайте стартап или область векторов, не заставляйте окружающих прибегать к телепатии.

[Bulat]

Да, надо. Причем топикстартеру, а не нам.

Хорошо бы как объявление закрепить:

Спасибо за ответы.
Прикрепляю стартап, изменил расширение, так как s79 не дает загрузить.

Прикрепленные файлы

 Cstartup.txt ( 10.21 килобайт ) Кол-во скачиваний: 122

 

[Troll]

Не сильно хорошо разбираюсь в Ассемблере. Но на первый взгляд ничего "противозаконного" не заметил.
Проверьте после выхода из прерывания, программа возвращается в основной цикл или нет?

Хотя, на счет "все нормально" я поспешил.
Команда "В" изменяет регистр LR (адрес текущей команды) и после выполнения команд:

Код

            sub         lr, lr, #4
            stmfd       sp!, {lr}

будет сохранен отнюдь не адрес команды на которой произошло прерывание.
Вместо команды "В" используйте команду "LDR".

Сообщение отредактировал Troll - Jul 8 2009, 09:54

[Bulat]

Не сильно хорошо разбираюсь в Ассемблере. Но на первый взгляд ничего "противозаконного" не заметил.
Проверьте после выхода из прерывания, программа возвращается в основной цикл или нет?

В основном цикле включается светодиод, подключенный к РА0

Код

void RT(void)
{
    AT91C_BASE_AIC->AIC_SVR[AT91C_ID_PIOA] = (unsigned int)irq_RT;
    AT91C_BASE_AIC->AIC_SMR[AT91C_ID_PIOA] = AT91C_AIC_SRCTYPE_INT_HIGH_LEVEL | 6;
    AT91C_BASE_AIC->AIC_IECR = (1UL << AT91C_ID_PIOA);
    AT91C_BASE_SYS->PIOA_IER = SW2_MASK | SW3_MASK;
    
    while(1) { regs->PIOA_ODSR = 0x0080001; }
}

Так вот, после обработки прерывания светодиод загорается, если нет дерективы __irq перед обработчиком. Если стоит деректива, то светодиод не загорается, стало быть из прерывания не возвращается.
Но, даже когда загорается диод, то есть программа возвращается в основной цикл, повторно перейти к обработчику она не может, хотя прерывание вызывается также, как и в первом случае.

[Troll]

Так вот, после обработки прерывания светодиод загорается, если нет дерективы __irq перед обработчиком. Если стоит деректива, то светодиод не загорается, стало быть из прерывания не возвращается.

не знаю каким компилятором Вы пользуетесь, но в общем случае директива __irq вставляет в код функции строки:

Код

            sub         lr, lr, #4
            stmfd       sp!, {lr} //Список может быть и больше
... //тело фукнции
            ldmia       sp!, {pc}^

Поэтому с Вашим обработчиком и директивой IRQ из прерывания программа будет улетать в ближний космос. Об этом чуть выше уже писал SergeiCh.


Вопрос, а с чего Вы взяли, что светодиод загорается после выхода из обработчика?

Код

while(1) { regs->PIOA_ODSR = 0x0080001; }

этот код вообще не показатель.

[aaarrr]

Команда "В" изменяет регистр LR (адрес текущей команды)

Команда B не изменяет LR, изменяет BL.

Так вот, после обработки прерывания светодиод загорается, если нет дерективы __irq перед обработчиком.

С таким стартапом __irq не нужен.

Но я бы рекомендовал выкинуть IRQ_Handler_Entry, а вместо

Код

                B           IRQ_Handler_Entry  ; 0x18 IRQ

поставить

Код

    ldr        pc, [pc, #-0xf20]    ; IRQ

И вернуть __irq обработчикам.

Но, даже когда загорается диод, то есть программа возвращается в основной цикл, повторно перейти к обработчику она не может, хотя прерывание вызывается также, как и в первом случае.

Выложите полностью основную программу и обработчик в текущем состоянии. А то совершенно непонятно, как получаются такие выводы.

[Troll]

Но я бы рекомендовал выкинуть IRQ_Handler_Entry, а вместо

Код

                B           IRQ_Handler_Entry ; 0x18 IRQ

поставить

Код

    ldr        pc, [pc, #-0xf20]; IRQ

И вернуть __irq обработчикам.

aaarrr а с чем связана такая рекомендация? Как это работает я понимаю, но мне интересна Ваша точка зрения, чем один вариант лучше другого.

[aaarrr]

aaarrr а с чем связана такая рекомендация? Как это работает я понимаю, но мне интересна Ваша точка зрения, чем один вариант лучше другого.

Связана с явной глупостью "стандартного" стартапа - все прерывания обрабатываются как вложенные с соответствующим оверхедом.
При прямом переходе по вектору из AIC мы получаем возможность использовать обычные обработчики прерываний с модификатором __irq, а в случае использования вложенных прерываний - не сохранять контекст для прерываний с наивысшим приоритетом и сэкономить ручное чтение IVR (при некотором оверхеде по объему, но лишняя сотня байт на обработчик по современным понятиям не считается).

[Troll]

а в случае использования вложенных прерываний - не сохранять контекст для прерываний с наивысшим приоритетом и сэкономить ручное чтение IVR.

Что-то я не понял, что Вы сказали.
Вложенное прерывание, по моему представлению, это когда выполняется обработка одного прерывания и произошло прерывание с более высоким приоритетом и происходит вызов обработчика этого нового прерывания. Контекст - LR и другие РОН.

Что в этом случае значит "не сохранять контекст для прерываний с наивысшим приоритетом и сэкономить ручное чтение IVR"?

[aaarrr]

Что в этом случае значит "не сохранять контекст для прерываний с наивысшим приоритетом и сэкономить ручное чтение IVR"?

Я не совсем понятно выразился, похоже. Не сохранять контекст - значит не пихать LR_irq и SPSR_irq на стек и не переключать режим процессора для прерываний с высшим приоритетом.
Ручное чтение IVR исключается для всех прерываний.

[Bulat]

Команда B не изменяет LR, изменяет BL.


С таким стартапом __irq не нужен.

Но я бы рекомендовал выкинуть IRQ_Handler_Entry, а вместо

Код

                B           IRQ_Handler_Entry ; 0x18 IRQ

поставить

Код

    ldr        pc, [pc, #-0xf20]; IRQ

И вернуть __irq обработчикам.


Выложите полностью основную программу и обработчик в текущем состоянии. А то совершенно непонятно, как получаются такие выводы.

Спасибо! Прерывания нормально работают. Тут у меня новая неполадка выявилась)

[singlskv]

Связана с явной глупостью "стандартного" стартапа - все прерывания обрабатываются как вложенные с соответствующим оверхедом.
При прямом переходе по вектору из AIC мы получаем возможность использовать обычные обработчики прерываний с модификатором __irq, а в случае использования вложенных прерываний - не сохранять контекст для прерываний с наивысшим приоритетом и сэкономить ручное чтение IVR (при некотором оверхеде по объему, но лишняя сотня байт на обработчик по современным понятиям не считается).

+1000
Я в паре топиков говорил про вредность "общих" обработчиков допускающих вложенные прерывания,
но понимания не достиг..., наверное у Вас лучше получиться объяснить.

[Bulat]

Прерывание вызывается по внешнему сигналу от порта. В качестве сигнала вызова подпрограммы обработки прерывания у меня моргает светодиод 2 раза. Точнее, должен моргать. На самом деле, светодиод моргает 4 раза, то есть обработчик вызывается 2 раза. При этом осциллограф показал всего один импульс на входе внешнего прерывания. Я ввел в обработчик функции, запрещающие прерывания, но это не помогло.

Код

//Инициализация прерывания
__ramfunc void RT()
{
    //Настройка портов ввода-ввывода
    regs->PIOA_OER = 0x22490000;      
    regs->PIOA_ODR = ~0x22490000;
    
    regPIO->PIO_PPUDR = 0x63DDFFFF;  
    regs->PIOA_OWER = 0x2249ffff;
    regs->PIOA_OWDR = ~0x2249ffff;          
    regs->PIOA_ODSR = 0x0080000; //R/W=0, REG=0, STRB=1
      
    // Load System pAic Base address
        
    AT91F_AIC_ConfigureIt_MY( AT91C_BASE_AIC, AT91C_ID_PIOA, 0x06, AT91C_AIC_SRCTYPE_INT_HIGH_LEVEL, irq_RT);
    AT91F_PIO_InterruptEnable_MY(AT91C_BASE_PIOA,SW2_MASK);
    AT91F_PIO_InterruptEnable_MY(AT91C_BASE_PIOA,SW3_MASK);
    // set the interrupt by software
    AT91F_AIC_EnableIt_MY(AT91C_BASE_AIC, AT91C_ID_PIOA);  
                                                    
    while(1){regs->PIOA_ODSR = 0x0080000;}                                    //Ожидание прерывания
    
    
}//RT()

//Обработчик прерываний
__ramfunc void irq_RT()
{    
  int i_status = 0x0;
  i_status = AT91F_PIO_GetInterruptStatus(AT91C_BASE_PIOA);
  if(i_status&0x800000) AT91F_PIO_InterruptDisable(AT91C_BASE_PIOA,SW3_MASK);   //Прерывание пришло от PA23
  if(i_status&0x1000000) AT91F_PIO_InterruptDisable(AT91C_BASE_PIOA,SW2_MASK);   //Прерывание пришло от PA24
  AT91F_AIC_DisableIt_MY(AT91C_BASE_AIC, AT91C_ID_PIOA);
  
  //Прием командного слова
  if(i_status&0x800000) regs->PIOA_ODSR = 0x400000;                             //REG=0, R/W=1, STRB=0, CHA/CHB=0 ïðè CHA
  if(i_status&0x1000000) regs->PIOA_ODSR = 0x410000;                            //REG=0, R/W=1, STRB=0, CHA/CHB=1 ïðè CHB    
  PIO_copy = regs->PIOA_PDSR;                                                   //считываем данные с порта контроллера
  regs->PIOA_ODSR = 0x0080000;
  if(PIO_copy != 0x0) i_PIO=1;
  
   if(i_PIO == 1)
  {
    i = 300000;
      while(i!=0)
      {
        if(i>=200000 & i<=300000) regs->PIOA_ODSR = 0x0080001; //Лампочка горит
        if(i>=100000 & i<200000) regs->PIOA_ODSR = 0x0080000;  //не горит
        if(i<100000) regs->PIOA_ODSR = 0x0080001;                     //горит
        i--;
      }
    i_PIO=0;  
  }

AT91C_BASE_AIC->AIC_EOICR = status;                                          
  if(i_status&0x800000) AT91F_PIO_InterruptEnable(AT91C_BASE_PIOA,SW3_MASK);
  if(i_status&0x1000000) AT91F_PIO_InterruptEnable(AT91C_BASE_PIOA,SW2_MASK);
  AT91F_AIC_EnableIt(AT91C_BASE_AIC, AT91C_ID_PIOA);    
}

Почему прерывание 2 раза вызывается, если внешний сигнал только один?
Заранее благодарен. файл стартапа прилагается

Прикрепленные файлы

 Cstartup.txt ( 10.21 килобайт ) Кол-во скачиваний: 19

 

[sergeeff]

if(i>=200000 & i<=300000) regs->PIOA_ODSR = 0x0080001; //Лампочка горит
if(i>=100000 & i<200000) regs->PIOA_ODSR = 0x0080000; //не горит

Должно быть:

Код

        if(i>=200000 && i<=300000) regs->PIOA_ODSR = 0x0080001; //Лампочка горит
        if(i>=100000 && i<200000) regs->PIOA_ODSR = 0x0080000;  //не горит

[Troll]

Отключать прерывания не нужно.

Код

AT91F_AIC_DisableIt_MY(AT91C_BASE_AIC, AT91C_ID_PIOA);

После того как вновь включаете, должна быть команда очистки прерываний (не помню точно как регистр называется).

Должно быть:

Код

        if(i>=200000 && i<=300000) regs->PIOA_ODSR = 0x0080001; //Лампочка горит
        if(i>=100000 && i<200000) regs->PIOA_ODSR = 0x0080000;  //не горит

вот так, помоему даже красивше:

Код

        if((200000<=i) && (i<=300000)) regs->PIOA_ODSR = 0x0080001; //Лампочка горит

Код

AT91C_BASE_AIC->AIC_EOICR = status;

А вот этого уже не нужно. В стартапе уже есть

Код

;- Mark the End of Interrupt on the AIC
            ldr         r14, =AT91C_BASE_AIC
            str         r14, [r14, #AIC_EOICR]

Код

if(i_status&[b]0x800000[/b]) AT91F_PIO_InterruptDisable(AT91C_BASE_PIOA,[b]SW3_MASK[/b]);   //Прерывание пришло от PA23

Почему бы просто не написать i_status&SW3_MASK

И код

Код

  PIO_copy = regs->PIOA_PDSR;                                                   //считываем данные с порта контроллера
  if(PIO_copy != 0x0) i_PIO=1;
  
   if(i_PIO == 1)

Можно заменить просто одним условным оператором.

Сообщение отредактировал Troll - Aug 5 2009, 06:21

[Bulat]

спасибо за ответы, но это не решает основной проблемы. Лампочка мигает 4 раза вместо 2-х, то есть прерывание дважды вызывается, хотя я вызываю его один раз.

[sergeeff]

В конце обработчика прерывания сбросьте флаг перывания

Код

volatile int dummy = AT91C_BASE_PIOA->PIO_ISR;

[Bulat]

В конце обработчика прерывания сбросьте флаг перывания

Код

volatile int dummy = AT91C_BASE_PIOA->PIO_ISR;

Спасибо, заработало)

[Troll]

В конце обработчика прерывания сбросьте флаг перывания

Код

volatile int dummy = AT91C_BASE_PIOA->PIO_ISR;

А я решил, что PIO_ISR читается здесь

Код

i_status = AT91F_PIO_GetInterruptStatus(AT91C_BASE_PIOA);

[aaarrr]

А я решил, что PIO_ISR читается здесь

Привильно решили. Прерывание генерируется по любому изменению уровня PIO, поэтому один импульс вызовет два прерывания.

[zltigo]

Код

volatile int dummy = AT91C_BASE_PIOA->PIO_ISR;

"Труды" Редькина плодоносят, блин
Зачем здесь

Код

volatile int dummy =

[sergeeff]

Уважаемый Гуру!

Я прекрасно помню про вашу нелюбовь к подобным

Код

volatile int dummy

. Ваши идеи на сей счет прекрасно работают в С, но не везде в С++ (именно в нем я работаю). Посему мой совет - это copy/past из живого кода.

[zltigo]

Ваши идеи...

Это не идеи и тем более не мои. Это совершенно естественное поведение компилятора не имеющего никакого права игнорировать обращение к volatile переменной.

на сей счет прекрасно работают в С, но не везде в С++ (именно в нем я работаю).

Вам только осталось найти в стандарте на C++ отличия от C в поведении volatile переменых, позволяющие распоряжаться C++ компиляторам volatile по своему усмотрению.

[sergeeff]

Вы же сами признавались в топике (http://electronix.ru/forum/index.php?showtopic=54275&st=15), что

Ну что я могу сказать - много значит cтранностей в IARовском недоплюсовом компиляторе. Подчеркиваю еще раз ПЛЮСОВОМ, а тема начинслась как C/C99.

[singlskv]

В конце обработчика прерывания сбросьте флаг перывания

Код

volatile int dummy = AT91C_BASE_PIOA->PIO_ISR;

Не смотря на различия С и плюсов,
у Вас здесь явно лишним будет volatile перед int dummy,
PIO_ISR уже объявлен как volatile для чтения,
поэтому гарантировать запись этого(не нужного) значения куда-то
нет никакого смысла...