Помощь - Поиск - Пользователи - Календарь
Полная версия этой страницы: Чтение состояния порта в массив, inline assembler gcc stm32f4xx
Форум разработчиков электроники ELECTRONIX.ru > Микроконтроллеры (MCs) > ARM
artkam
Jun 27 2016, 11:20
Здравствуйте, уважаемые товарищи!
В си-коде, для проверки состояния порта используется вставка на ассемблере, приведенная ниже.
Но где-то что-то идет не так задумано, и в буфере, что передается в функцию по ссылке, ничего кроме нулей не обнаруживается.
Возможно, я что-то недопонимаю и делаю не так, как надо.
Хотелось бы услышать идеи, с помощью которых можно было бы починить код...

CODE

void f(unsigned int* peg)
{
__ASM volatile
(

".equ PERIPH_BASE, 0x40000000 \n \t"
".equ AHB1PERIPH_BASE, 0x00020000 \n \t"
".equ GPIOD_BASE, 0x0C00 \n \t"
".equ GPIOB_BASE, 0x0400 \n \t"
".equ GPIO_IDR, 0x10 \n \t"
".equ GPIO_BSRRL, 0x18 \n \t"
".equ GPIO_BSRRH, 0x1A \n \t"

".equ GPIO_BSRR_BS_5, 0x20 \n \t"


"LDR r0, =(GPIO_BSRR_BS_5) \n \t"
"LDR r1, =(PERIPH_BASE + AHB1PERIPH_BASE + GPIOB_BASE + GPIO_BSRRH) \n\t"
"LDR r2, =(PERIPH_BASE + AHB1PERIPH_BASE + GPIOB_BASE + GPIO_BSRRL) \n\t"
"LDR r3, =(PERIPH_BASE + AHB1PERIPH_BASE + GPIOD_BASE + GPIO_IDR) \n\t"

"LDR r4, [%[data_ptr], 0x00] \n \t"

//0
"STRH r0, [r2, 0x00] \n \t" // set clock
"LDR r5, [r3, 0x00] \n \t" // считываем GPIOD->IDR
"STRH r0, [r1, 0x00] \n \t" // reset clock
"STR r5, [r4, 0] \n \t" // записываем то, что было в GPIOD->IDR, в память со смещением

//1
"STRH r0, [r2, 0x00] \n \t"
"LDR r5, [r3, 0x00] \n \t"
"STRH r0, [r1, 0x00] \n \t"
"STR r5, [r4, 4] \n \t"

//2
"STRH r0, [r2, 0x00] \n \t"
"LDR r5, [r3, 0x00] \n \t"
"STRH r0, [r1, 0x00] \n \t"
"STR r5, [r4, 8] \n \t"

/* Множество раз повторяется. */

//681
"STRH r0, [r2, 0x00] \n \t"
"LDR r5, [r3, 0x00] \n \t"
"STRH r0, [r1, 0x00] \n \t"
"STR r5, [r4, #2724] \n \t"

:
: [data_ptr] "r" (peg)
:
);
}
aaarrr
Jun 27 2016, 11:24
Не уверен насчет "сообразительности" инлайн ассемблера gcc:
Код
"LDR r0, =(GPIO_BSRR_BS_5) \n \t"

здесь peg может быть затерт.

И почему не оформить код в виде нормального ассемблерного модуля?
Obam
Jun 27 2016, 11:29
Цитата(artkam @ Jun 27 2016, 15:20) *
Здравствуйте, уважаемые товарищи!
В си-коде, для проверки состояния порта используется вставка на ассемблере, приведенная ниже.
Но где-то что-то идет не так задумано, и в буфере, что передается в функцию по ссылке, ничего кроме нулей не обнаруживается.
Возможно, я что-то недопонимаю и делаю не так, как надо.
Хотелось бы услышать идеи, с помощью которых можно было бы починить код...

CODE

void f(unsigned int* peg)
{
__ASM volatile
(

".equ PERIPH_BASE, 0x40000000 \n \t"
".equ AHB1PERIPH_BASE, 0x00020000 \n \t"
".equ GPIOD_BASE, 0x0C00 \n \t"
".equ GPIOB_BASE, 0x0400 \n \t"
".equ GPIO_IDR, 0x10 \n \t"
".equ GPIO_BSRRL, 0x18 \n \t"
".equ GPIO_BSRRH, 0x1A \n \t"

".equ GPIO_BSRR_BS_5, 0x20 \n \t"


"LDR r0, =(GPIO_BSRR_BS_5) \n \t"
"LDR r1, =(PERIPH_BASE + AHB1PERIPH_BASE + GPIOB_BASE + GPIO_BSRRH) \n\t"
"LDR r2, =(PERIPH_BASE + AHB1PERIPH_BASE + GPIOB_BASE + GPIO_BSRRL) \n\t"
"LDR r3, =(PERIPH_BASE + AHB1PERIPH_BASE + GPIOD_BASE + GPIO_IDR) \n\t"

"LDR r4, [%[data_ptr], 0x00] \n \t"

//0
"STRH r0, [r2, 0x00] \n \t" // set clock
"LDR r5, [r3, 0x00] \n \t" // считываем GPIOD->IDR
"STRH r0, [r1, 0x00] \n \t" // reset clock
"STR r5, [r4, 0] \n \t" // записываем то, что было в GPIOD->IDR, в память со смещением

//1
"STRH r0, [r2, 0x00] \n \t"
"LDR r5, [r3, 0x00] \n \t"
"STRH r0, [r1, 0x00] \n \t"
"STR r5, [r4, 4] \n \t"

//2
"STRH r0, [r2, 0x00] \n \t"
"LDR r5, [r3, 0x00] \n \t"
"STRH r0, [r1, 0x00] \n \t"
"STR r5, [r4, 8] \n \t"

/* Множество раз повторяется. */

//681
"STRH r0, [r2, 0x00] \n \t"
"LDR r5, [r3, 0x00] \n \t"
"STRH r0, [r1, 0x00] \n \t"
"STR r5, [r4, #2724] \n \t"

:
: [data_ptr] "r" (peg)
:
);
}


Ну а под JTAGом пошагово почему не посмотреть что в регистры грузится? И не нужны будут ничьи идеи.
scifi
Jun 27 2016, 11:30
Цитата(artkam @ Jun 27 2016, 14:20) *
Хотелось бы услышать идеи, с помощью которых можно было бы починить код...

Вот такая идея: подключить внутрисхемный отладчик, пройтись по шагам в окне дизассемблера и разобраться, что именно идёт не так.
Obam
Jun 27 2016, 11:30
Цитата(aaarrr @ Jun 27 2016, 15:24) *
Не уверен насчет "сообразительности" инлайн ассемблера gcc:
Код
"LDR r0, =(GPIO_BSRR_BS_5) \n \t"

здесь peg может быть затерт.

И почему не оформить код в виде нормального ассемблерного модуля?


R0 должен содержать 0x20
jcxz
Jun 27 2016, 11:35
Цитата(artkam @ Jun 27 2016, 17:20) *
Возможно, я что-то недопонимаю и делаю не так, как надо.

Где сохранение разрушаемых R4,R5?

Цитата(artkam @ Jun 27 2016, 17:20) *
Хотелось бы услышать идеи, с помощью которых можно было бы починить код...

Вынести функцию в асм-файл. И читать документацию компилятора насчёт соглашений вызова.
Сергей Борщ
Jun 27 2016, 12:29
QUOTE (artkam @ Jun 27 2016, 14:20) *
CODE
        :
        : [data_ptr] "r" (peg)
        :
1) Тут должны быть перечислены все используемые во вставке регистры и память ("=m", если правильно помню)
2) лучше дать возможность компилятору самому выделить наиболее подходящие регистры, т.е. задать там [имя] "+r" и во вставке использовать %[имя] вместо имени конкретного регистра - это развяжет ему руки.
3) аналогично peg можно заставить компилятор передавать во вставку адреса GPIOB->BSRRH, GPIOB->BSRRL, GPIOD->IDR чтобы брать их из заголовочного файла, а не прибивать гвоздями внутри вставки.
4) можно сэкономить регистр, занеся в регистр адрес GPIOB, а для обращений к BSRRH, BSRRL использовать разные смещения в командах.
artkam
Jun 28 2016, 00:37
Цитата(aaarrr @ Jun 27 2016, 16:24) *
И почему не оформить код в виде нормального ассемблерного модуля?


Наверно, можно оформить и в виде отдельного модуля...
Но я не понимаю пока, какие преимущества я из этого смогу извлечь?
jcxz
Jun 28 2016, 03:39
Цитата(artkam @ Jun 28 2016, 06:37) *
Наверно, можно оформить и в виде отдельного модуля...
Но я не понимаю пока, какие преимущества я из этого смогу извлечь?

А я думаю многие тут не понимают: какие преимущества Вы извлекаете нагородив этот огород с инлайн-ассемблером вместо простой си-функции?
При этом даже не понимаете как взаимодействует Ваш асм-код с телом си-функции в которую его впихнули.
artkam
Jun 28 2016, 05:36
Поэтому и обратился на форум, чтоб направили в правильном направлении, разъяснили основные моменты...

В итоге, сделал вот так, в какой-то степени работает...

CODE

void ClockAsm(unsigned int *peg)
{
__ASM volatile
(

//"LDR r3, 0x4002041A \n \t"
//"LDR r1, 0x40020418 \n \t"

//"LDR r2, 0x40020C10 \n \t"

//"MOV r3, 0x00000020 \n \t"
".equ PERIPH_BASE, 0x40000000 \n \t"
".equ AHB1PERIPH_BASE, 0x00020000 \n \t"
".equ GPIOD_BASE, 0x0C00 \n \t"
".equ GPIOB_BASE, 0x0400 \n \t"
".equ GPIO_IDR, 0x10 \n \t"
".equ GPIO_BSRRL, 0x18 \n \t"
".equ GPIO_BSRRH, 0x1A \n \t"

".equ GPIO_BSRR_BS_5, 0x20 \n \t"

"LDR r3, [%[data_ptr], 0x00] \n \t"

"LDR v1, =(GPIO_BSRR_BS_5) \n \t"
"LDR v2, =(PERIPH_BASE + AHB1PERIPH_BASE + GPIOB_BASE + GPIO_BSRRH) \n \t"
"LDR v3, =(PERIPH_BASE + AHB1PERIPH_BASE + GPIOB_BASE + GPIO_BSRRL) \n\t"
"LDR v6, =(PERIPH_BASE + AHB1PERIPH_BASE + GPIOD_BASE + GPIO_IDR) \n\t"

//0
"STRH v1, [v3, 0x00] \n \t"
"LDR v5, [v6, 0x00] \n \t"
"STRH v1, [v2, 0x00] \n \t"
"STR v5, [r3, 0] \n \t"

//1
"STRH v1, [v3, 0x00] \n \t"
"LDR v5, [v6, 0x00] \n \t"
"STRH v1, [v2, 0x00] \n \t"
"STR v5, [r3, 4] \n \t"

...

//338
"STRH v1, [v3, 0x00] \n \t"
"LDR v5, [v6, 0x00] \n \t"
"STRH v1, [v2, 0x00] \n \t"
"STR v5, [r3, #1352] \n \t"
:
: [data_ptr] "r" (&peg)
: "r3", "v1", "v2", "v3", "v5", "v6", "memory"
);
}


Теперь, следующая проблема: мне необходима выборка длиной 600 семплов, а, при увеличении шагов до 339, например, компилятор ругается: "Error: offset out of range"...
Подскажите, возможно есть еще какая-то очевидная ошибка?
jcxz
Jun 28 2016, 06:02
Цитата(artkam @ Jun 28 2016, 11:36) *
Поэтому и обратился на форум, чтоб направили в правильном направлении, разъяснили основные моменты...

Правильное направление: пишите на си.
Асм уместен только там, где реально нужен (очень редко!). И то - очень не советую использовать инлайн-асм. Лучше отдельный асм-файл.
Obam
Jun 28 2016, 07:42
Пусть тренируется человек, вон уже дошёл до факта, что смещение в виде константы [Rx, #(+-)N] имеет весьма ограниченный диапазон.

ARM® and Thumb®-2 Instruction Set:
Нажмите для просмотра прикрепленного файла

Если не надоест много занятного и полезного узнает.
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.
Invision Power Board © 2001-2025 Invision Power Services, Inc.