| |
|
  |
 Чтение состояния порта в массив, inline assembler gcc stm32f4xx |
|
|
|
|
 Jun 27 2016, 11:20
|
Частый гость
Группа: Участник
Сообщений: 182
Регистрация: 14-09-10
Из: Уфа
Пользователь №: 59 479
|
Здравствуйте, уважаемые товарищи! В си-коде, для проверки состояния порта используется вставка на ассемблере, приведенная ниже. Но где-то что-то идет не так задумано, и в буфере, что передается в функцию по ссылке, ничего кроме нулей не обнаруживается. Возможно, я что-то недопонимаю и делаю не так, как надо. Хотелось бы услышать идеи, с помощью которых можно было бы починить код... 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)
:
);
}
|
|
|
|
|
|
|
|
Jun 27 2016, 11:24
|
Гуру
Группа: Свой
Сообщений: 10 713
Регистрация: 11-12-04
Пользователь №: 1 448
|
Не уверен насчет "сообразительности" инлайн ассемблера gcc: Код "LDR r0, =(GPIO_BSRR_BS_5) \n \t" здесь peg может быть затерт. И почему не оформить код в виде нормального ассемблерного модуля?
|
|
|
|
|
|
|
|
Jun 27 2016, 11:29
|
Знающий
Группа: Участник
Сообщений: 756
Регистрация: 14-11-14
Пользователь №: 83 663
|
Цитата(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ом пошагово почему не посмотреть что в регистры грузится? И не нужны будут ничьи идеи.
--------------------
Пролетарий умственного труда.
|
|
|
|
|
|
|
|
Jun 27 2016, 11:30
|
Знающий
Группа: Участник
Сообщений: 756
Регистрация: 14-11-14
Пользователь №: 83 663
|
Цитата(aaarrr @ Jun 27 2016, 15:24) Не уверен насчет "сообразительности" инлайн ассемблера gcc: Код "LDR r0, =(GPIO_BSRR_BS_5) \n \t" здесь peg может быть затерт. И почему не оформить код в виде нормального ассемблерного модуля? R0 должен содержать 0x20
--------------------
Пролетарий умственного труда.
|
|
|
|
|
|
|
|
Jun 27 2016, 11:35
|
Гуру
Группа: Свой
Сообщений: 5 228
Регистрация: 3-07-08
Из: Омск
Пользователь №: 38 713
|
Цитата(artkam @ Jun 27 2016, 17:20) Возможно, я что-то недопонимаю и делаю не так, как надо. Где сохранение разрушаемых R4,R5? Цитата(artkam @ Jun 27 2016, 17:20) Хотелось бы услышать идеи, с помощью которых можно было бы починить код... Вынести функцию в асм-файл. И читать документацию компилятора насчёт соглашений вызова.
|
|
|
|
|
|
|
|
Jun 27 2016, 12:29
|
Гуру
Группа: Модераторы
Сообщений: 8 455
Регистрация: 15-05-06
Из: Рига, Латвия
Пользователь №: 17 095
|
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 использовать разные смещения в командах.
--------------------
На любой вопрос даю любой ответ"Write code that is guaranteed to work, not code that doesn’t seem to break" ( C++ FAQ)
|
|
|
|
|
|
|
|
Jun 28 2016, 00:37
|
Частый гость
Группа: Участник
Сообщений: 182
Регистрация: 14-09-10
Из: Уфа
Пользователь №: 59 479
|
Цитата(aaarrr @ Jun 27 2016, 16:24) И почему не оформить код в виде нормального ассемблерного модуля? Наверно, можно оформить и в виде отдельного модуля... Но я не понимаю пока, какие преимущества я из этого смогу извлечь?
|
|
|
|
|
|
|
|
Jun 28 2016, 05:36
|
Частый гость
Группа: Участник
Сообщений: 182
Регистрация: 14-09-10
Из: Уфа
Пользователь №: 59 479
|
Поэтому и обратился на форум, чтоб направили в правильном направлении, разъяснили основные моменты... В итоге, сделал вот так, в какой-то степени работает... 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"... Подскажите, возможно есть еще какая-то очевидная ошибка?
|
|
|
|
|
|
|
|
Jun 28 2016, 07:42
|
Знающий
Группа: Участник
Сообщений: 756
Регистрация: 14-11-14
Пользователь №: 83 663
|
Пусть тренируется человек, вон уже дошёл до факта, что смещение в виде константы [Rx, #(+-)N] имеет весьма ограниченный диапазон. ARM® and Thumb®-2 Instruction Set: 
QRC0001_UAL.pdf ( 226.46 килобайт )
Кол-во скачиваний: 69Если не надоест много занятного и полезного узнает.
--------------------
Пролетарий умственного труда.
|
|
|
|
|
|
|
|
1 чел. читают эту тему (гостей: 1, скрытых пользователей: 0)
Пользователей: 0
|
|
|
