STM32F030 ADC Handler CMSIS

VladislavS

стандартным вызовом. Разве это запрещено?

С дуру и хрен сломать можно.

Добавлено after 1 minute 50 seconds:

Вообще то я пользуюсь практикой SPL, не прибегая к ней самой. Там все выглядит именно так - проверка разрешения && проверка флага - сброс флага - обработчик.

Куда полезней своей головой пользоваться.

Реклама

jcxz

КРАМ писал(а):

Что не так?

Имха - неопределённый порядок чтения volatile-регистров.
Хотя возможно тут зависит от компилятора.... :dont_know:

Реклама

VladislavS

jcxz, вы абсолютно правы. Правая часть && не будет выполнена, если левая false. Это значит чтения из ADC1->ISR не произойдёт. А такое чтение, для некоторых периферийных устройств может менять состояние флагов.

Код:

//void ADC_IRQHandler (void) 
//{
//  if ((ADC1->IER & ADC_IER_ADRDYIE) && (ADC1->ISR & ADC_ISR_ADRDY)) 
        MOV      R0,#+1342177280
        LDR      R1,[R0, #+4]
        LSLS     R2,R1,#+31
        BPL.N    ??ADC_IRQHandler_0
        LDR      R1,[R0, #+0]
        LSLS     R2,R1,#+31
        ITT      MI 
//    ADC1->ISR = ADC_ISR_ADRDY;
        MOVMI    R1,#+1
        STRMI    R1,[R0, #+0]
//}
??ADC_IRQHandler_0:
        BX       LR               ;; return

jcxz

VladislavS писал(а):

Правая часть && не будет выполнена, если левая false. Это значит чтения из ADC1->ISR не произойдёт. А такое чтение, для некоторых периферийных устройств может менять состояние флагов.

Я о чём и говорю. Поэтому думаю - или у ТС в свойствах проекта выставлено подавление варнингов, или он просто не обращает внимания на них при компиляции или регистры определены неверно (без volatile) или ...?

BlackKilkennyCat

А разве на это выдаются предупреждение? Возможное невыполнение части условия - стандартное поведение, о котором просто надо помнить.

Реклама

jcxz

BlackKilkennyCat писал(а):

А разве на это выдаются предупреждение? Возможное невыполнение части условия - стандартное поведение, о котором просто надо помнить.

Лучше в начале ISR прочитать регистр флагов прерываний в локальную переменную и работать уже ней.
Да и не нужно многократно читать эти флаги (если обрабатывается несколько разными ветками алгоритма).

Реклама

VladislavS

jcxz писал(а):

Поэтому думаю - или у ТС в свойствах проекта выставлено подавление варнингов, или он просто не обращает внимания на них при компиляции или регистры определены неверно (без volatile) или ...?

Не, всё у него верно определено. Описания регистров из CMSIS взяты. Компиляторы действительно молчат.

STM32F030 ADC Handler CMSIS

Кто сейчас на форуме