|
at91, помогите разобраться |
|
|
|
Aug 15 2008, 19:08
|

Участник

Группа: Участник
Сообщений: 35
Регистрация: 8-07-07
Пользователь №: 28 963

|
Помогите разобраться, пожалуйста. в Header AT91SAM7S256.h файле написано: Код typedef struct _AT91S_PMC { AT91_REG PMC_SCER; // System Clock Enable Register AT91_REG PMC_SCDR; // System Clock Disable Register AT91_REG PMC_SCSR; // System Clock Status Register AT91_REG Reserved0[1]; // AT91_REG PMC_PCER; // Peripheral Clock Enable Register AT91_REG PMC_PCDR; // Peripheral Clock Disable Register AT91_REG PMC_PCSR; // Peripheral Clock Status Register AT91_REG Reserved1[1]; // AT91_REG PMC_MOR; // Main Oscillator Register AT91_REG PMC_MCFR; // Main Clock Frequency Register AT91_REG Reserved2[1]; // AT91_REG PMC_PLLR; // PLL Register AT91_REG PMC_MCKR; // Master Clock Register AT91_REG Reserved3[3]; // AT91_REG PMC_PCKR[3]; // Programmable Clock Register AT91_REG Reserved4[5]; // AT91_REG PMC_IER; // Interrupt Enable Register AT91_REG PMC_IDR; // Interrupt Disable Register AT91_REG PMC_SR; // Status Register AT91_REG PMC_IMR; // Interrupt Mask Register } AT91S_PMC, *AT91PS_PMC; далее ниже Код #define AT91C_BASE_PMC ((AT91PS_PMC) 0xFFFFFC00) // (PMC) Base Address в программе: Код AT91C_BASE_PMC->PMC_SCER = AT91C_PMC_UDP; правильно ли я понял, что AT91PS_PMC - указатель на структуру_AT91S_PMC. AT91C_BASE_PMC - ее начальный адресс, строчка в программе означает, в регистр PMC_SCER (по адресу AT91C_BASE_PMC + 5*AT91_REG) записать значение AT91C_PMC_UDP? ----------------- теперь файл: lib_AT91SAM7S256.h Код __inline unsigned int AT91F_AIC_ConfigureIt ( AT91PS_AIC pAic, // \arg pointer to the AIC registers unsigned int irq_id, // \arg interrupt number to initialize unsigned int priority, // \arg priority to give to the interrupt unsigned int src_type, // \arg activation and sense of activation void (*newHandler) () ) // \arg address of the interrupt handler { unsigned int oldHandler; unsigned int mask;
oldHandler = pAic->AIC_SVR[irq_id];
mask = 0x1 << irq_id; //* Disable the interrupt on the interrupt controller pAic->AIC_IDCR = mask; //* Save the interrupt handler routine pointer and the interrupt priority pAic->AIC_SVR[irq_id] = (unsigned int) newHandler; //* Store the Source Mode Register pAic->AIC_SMR[irq_id] = src_type | priority; //* Clear the interrupt on the interrupt controller pAic->AIC_ICCR = mask;
return oldHandler; } как компилятор узнает адрес регистра AIC_IDCR? Код pAic->AIC_IDCR = mask; ведь pAic это есть AT91PS_AIC, но на какой начальный адрес он указывает НЕТ в библиотеке Код typedef struct _AT91S_AIC { AT91_REG AIC_SMR[32]; // Source Mode Register AT91_REG AIC_SVR[32]; // Source Vector Register AT91_REG AIC_IVR; // IRQ Vector Register AT91_REG AIC_FVR; // FIQ Vector Register AT91_REG AIC_ISR; // Interrupt Status Register AT91_REG AIC_IPR; // Interrupt Pending Register AT91_REG AIC_IMR; // Interrupt Mask Register AT91_REG AIC_CISR; // Core Interrupt Status Register AT91_REG Reserved0[2]; // AT91_REG AIC_IECR; // Interrupt Enable Command Register AT91_REG AIC_IDCR; // Interrupt Disable Command Register AT91_REG AIC_ICCR; // Interrupt Clear Command Register AT91_REG AIC_ISCR; // Interrupt Set Command Register AT91_REG AIC_EOICR; // End of Interrupt Command Register AT91_REG AIC_SPU; // Spurious Vector Register AT91_REG AIC_DCR; // Debug Control Register (Protect) AT91_REG Reserved1[1]; // AT91_REG AIC_FFER; // Fast Forcing Enable Register AT91_REG AIC_FFDR; // Fast Forcing Disable Register AT91_REG AIC_FFSR; // Fast Forcing Status Register } AT91S_AIC, *AT91PS_AIC; спасибо
Сообщение отредактировал i.dmitry - Aug 15 2008, 19:10
|
|
|
|
|
 |
Ответов
(75 - 87)
|
Aug 19 2008, 08:52
|

Гуру
     
Группа: Свой
Сообщений: 13 372
Регистрация: 27-11-04
Из: Riga, Latvia
Пользователь №: 1 244

|
Цитата(aaarrr @ Aug 19 2008, 08:15)  Тем, кто имеет слабость лениться думать самостоятельно, наличие этой библиотеки может навредить. Для начинающих это еще сразу дает образчик дурнейшего стиля программирования с функциями в хидерах, многократно обернутых, зачастую просто небрежно написанных... При этом все запихивается а один файл....Чему это может научить? Ну разве только инлайнить  , правда бездумно  , даже одну строчку. Цитата(aaarrr @ Aug 19 2008, 10:36)  ...проблем с multiple definition быть не должно. Да линкер у него ругается, когда находит кучу продублированных функций - всегда безусловно инлайнить компиляторы не обязаны, как впрочем и выбрасывать неиспользованые функции, хоть они и с __inline.
--------------------
Feci, quod potui, faciant meliora potentes
|
|
|
|
|
Aug 19 2008, 10:34
|

кекс
     
Группа: Свой
Сообщений: 3 825
Регистрация: 17-12-05
Из: Киев
Пользователь №: 12 326

|
Цитата(zltigo @ Aug 19 2008, 11:52)  это еще сразу дает образчик дурнейшего стиля программирования с функциями в хидерах, многократно обернутых, зачастую просто небрежно написанных... Нормальный там стиль. inline функциям как раз и место только в хидерах, они по своей природе равносильны макросам. Качество кода я тоже демонстрировал - абсолютно ничего лишнего, одного взгляда достаточно для понимания сути. Коментарии тоже в порядке. Цитата При этом все запихивается а один файл....Чему это может научить? А почему бы и нет. Чем это плохо если речь идет о hal функциях. Неужели удобнее цеплять с десяток хидеров, по одному для каждой мелочи типа SPI? Цитата как впрочем и выбрасывать неиспользованые функции, хоть они и с __inline Хоть один компилятор не выбрасывающий неиспользуемые __inline функции можете назвать?
|
|
|
|
|
Aug 19 2008, 11:19
|
Гуру
     
Группа: Свой
Сообщений: 10 713
Регистрация: 11-12-04
Пользователь №: 1 448

|
Цитата(defunct @ Aug 19 2008, 14:34)  Качество кода я тоже демонстрировал - абсолютно ничего лишнего, одного взгляда достаточно для понимания сути. Коментарии тоже в порядке. По поводу качества кода: Вы AT91F_PITInit не нашли? Или мы обсуждаем только положительные стороны lib_? Copy->paste комментарии - это порядок? Цитата(defunct @ Aug 19 2008, 14:34)  А почему бы и нет. Чем это плохо если речь идет о hal функциях. Неужели удобнее цеплять с десяток хидеров, по одному для каждой мелочи типа SPI? А Вам удобно редактировать файл размером 150 кбайт? А если туда еще HAL для EMAC'а и USB запихать - еще удобней станет? Да, нашел я у себя в закромах Вашу версию lib, AT91F_PITInit там, естественно, есть. Код //* File Name : lib_AT91SAM7X256.h //* Object : AT91SAM7X256 inlined functions //* Generated : AT91 SW Application Group 09/12/2005 (15:39:14) ... //* CVS Reference : /lib_pitc_6079A.h/1.2/Thu Nov 4 14:01:11 2004//
...
/* ***************************************************************************** SOFTWARE API FOR PITC ***************************************************************************** */ //*---------------------------------------------------------------------------- //* \fn AT91F_PITInit //* \brief System timer init : period in µsecond, system clock freq in MHz //*---------------------------------------------------------------------------- __inline void AT91F_PITInit( AT91PS_PITC pPITC, unsigned int period, unsigned int pit_frequency) { pPITC->PITC_PIMR = period? (period * pit_frequency + 8) >> 4 : 0; // +8 to avoid %10 and /10 pPITC->PITC_PIMR |= AT91C_PITC_PITEN; }
|
|
|
|
|
Aug 19 2008, 12:18
|

кекс
     
Группа: Свой
Сообщений: 3 825
Регистрация: 17-12-05
Из: Киев
Пользователь №: 12 326

|
Цитата(aaarrr @ Aug 19 2008, 14:19)  По поводу качества кода: Вы AT91F_PITInit не нашли? Или мы обсуждаем только положительные стороны lib_? Нашел. Да с Вами можно согласиться - это баг. Правда ничего ужасающе страшного в нем на мой взгляд нет - вопрос округления. Для маленьких значений period'a (единицы us) точного значения не будет практически никогда, добавляй туда -1 или не добавляй, на 55Mhz их реализация будет даже более точно работать. Для больших значений периода доли us не играют роли. Плюс это не единственная функция для настойки PIT'a в этой библиотеке. Цитата Copy->paste комментарии - это порядок? Не коментарии, а шапки функций. Это простительно, на brief редко кто смотрит, а в коде с коментариями порядок. Цитата А Вам удобно редактировать файл размером 150 кбайт? А если туда еще HAL для EMAC'а и USB запихать - еще удобней станет? 1. Мне привычно работать с большими файлами. 2. Меня никто не заставляет его редактировать. (зачем его редактировать?)
|
|
|
|
|
Aug 19 2008, 12:34
|
Гуру
     
Группа: Свой
Сообщений: 10 713
Регистрация: 11-12-04
Пользователь №: 1 448

|
Цитата(defunct @ Aug 19 2008, 16:18)  Правда ничего ужасающе страшного в нем на мой взгляд нет - вопрос округления. Для маленьких значений period'a (единицы us) точного значения не будет практически никогда, добавляй туда -1 или не добавляй, на 55Mhz их реализация будет даже более точно работать. Для больших значений периода доли us не играют роли. Ну, тут можно поспорить. Если посмотреть документацию на SAM7, то в ней PITC позиционируется как таймер для точного отсчета интервалов времени: Цитата It is designed to offer maximum accuracy and efficient management, even for systems with long response time. А в библиотеке для установки этого таймера нам предлагают передать целое число МГц MCK, и вычисляют значение регистра с ошибкой. Цитата(defunct @ Aug 19 2008, 16:18)  Плюс это не единственная функция для настойки PIT'a в этой библиотеке. Да, есть еще AT91F_PITSetPIV. Если человек способен сам вычислить значение PIV, то вряд ли ему нужны костыли в виде библиотечной функции для установки поля регистра. Цитата(defunct @ Aug 19 2008, 16:18)  Не коментарии, а шапки функций. Это простительно, на brief редко кто смотрит, а в коде с коментариями порядок. Там, где они есть. Беглый просмотр показывает, что часть функций не откомментированна. Цитата(defunct @ Aug 19 2008, 16:18)  2. Меня никто не заставляет его редактировать. (зачем его редактировать?) За тем, что он содержит ошибки.
|
|
|
|
|
Aug 19 2008, 12:42
|

Гуру
     
Группа: Свой
Сообщений: 13 372
Регистрация: 27-11-04
Из: Riga, Latvia
Пользователь №: 1 244

|
Цитата(defunct @ Aug 19 2008, 12:34)  inline функциям как раз и место только в хидерах, они по своей природе равносильны макросам. Не совсем, компилятор имеет над ними некоторую власть по оптимизации, если конечно специально прагмами типа inline=forced не пользоваться. Цитата Качество кода я тоже демонстрировал - абсолютно ничего лишнего Да уж  что может быть лишнее, если в подавляющем большинстве там по одной строчке, а там, где уже 3-5 и лишнее, не криминальное, но если позиционировать сей код,как образец, то лишнее  Цитата Неужели удобнее цеплять с десяток хидеров, по одному для каждой мелочи типа SPI? А какая проблема? Тем более,как человек широко использующий тот-же SPI скажу, что SPI использую по максимуму его возможностей а не просто прочитать-записать словечко интерфейс к которому сводят его "библиотеки". Цитата Хоть один компилятор не выбрасывающий неиспользуемые __inline функции можете назвать? Не интересовался по причине нахренненужности  - тут лучше Вы скажите, где написано, что все компиляторы обязаны это делать?
--------------------
Feci, quod potui, faciant meliora potentes
|
|
|
|
|
Aug 20 2008, 00:09
|

кекс
     
Группа: Свой
Сообщений: 3 825
Регистрация: 17-12-05
Из: Киев
Пользователь №: 12 326

|
Цитата(zltigo @ Aug 19 2008, 15:42)  Не совсем, компилятор имеет над ними некоторую власть по оптимизации, если конечно специально прагмами типа inline=forced не пользоваться. имеет конечно. но для того чтобы он имел над ними власть их надо вызвать хотя бы раз. Цитата А какая проблема? Тем более,как человек широко использующий тот-же SPI скажу, что SPI использую по максимуму его возможностей а не просто прочитать-записать словечко интерфейс к которому сводят его "библиотеки". Вот и я думаю. Чем один файл плох. Какая разница 1 файл или 10 все одно речь о железе. И еще думаю, что же интересно может помешать создать свой собственный файл не отказываясь от библиотеки. Цитата Не интересовался по причине нахренненужности  - тут лучше Вы скажите, где написано, что все компиляторы обязаны это делать? Вопросом на вопрос - нипойдет. Хотя A function where all the declarations (including the definition) mention "inline" and never "extern". There must be a definition in the same translation unit. No stand-alone object code is emitted. You can (must?) have a separate (not inline) definition in another translation unit, and the compiler might choose either that or the inline definition. http://www.greenend.org.uk/rjk/2003/03/inline.htmlЦитата(aaarrr @ Aug 19 2008, 15:34)  За тем, что он содержит ошибки. Ну и что? ;> С какой радости мне править библиотечный код, который обновляется и поддерживается не мной. Законы Мерфи помните? Особо подходящие под этот случай: 3. Все программы содержат ошибки, просто о некоторых мы не догадываемся. 5. Если Вы находите ошибки, исправить которые дело долгое и нудное, ничего не делайте - просто внесите их в список особенностей. 25. Усилия, прилагаемые для исправления ошибки, увеличивают ее в геометрической прогрессии по отношению к затраченному времени. Поэтому или закрыть глаза на ошибки или вообще ничем не пользоваться. Править ошибку в библиотеке я не стану (себе дороже), проще найдя такую - обойти ее на месте. Цитата Там, где они есть. Беглый просмотр показывает, что часть функций не откомментированна. Справедливое замечание. Однако там где нет коментариев, коментировать просто нечего. Тупые коментарии вида: a = b; // в переменную a поместить значение переменной b никому не нужны.
|
|
|
|
|
Aug 20 2008, 09:26
|

Гуру
     
Группа: Свой
Сообщений: 13 372
Регистрация: 27-11-04
Из: Riga, Latvia
Пользователь №: 1 244

|
Цитата(defunct @ Aug 20 2008, 02:09)  Вопросом на вопрос - нипойдет. Хотя Это относится конкретно к GNU реализации, а отнюдь не ко всем. Посему ответа на вопрос Цитата что все компиляторы обязаны это делать? нет.
--------------------
Feci, quod potui, faciant meliora potentes
|
|
|
|
|
Aug 20 2008, 16:51
|

Частый гость
 
Группа: Свой
Сообщений: 91
Регистрация: 10-10-07
Из: Воронежа
Пользователь №: 31 250

|
Цитата(Сергей Борщ @ Aug 20 2008, 14:12)  Не могли бы вы более подробно описать, как вы это делаете? Ведь AT91C_BASE_XXX описаны через #define и Эклипс просто не видит этих структур. у меня все прекрасно видится (eclipse 3.4)
...пардон, не заметил слов "при отладке". Возможно автор имел ввиду не отладку, а написание программы...
Сообщение отредактировал injen-d - Aug 20 2008, 16:56
--------------------
- Бендер, ты же робот, зачем тебе пить пиво? - Незачем! Я могу бросить в любой момент!
|
|
|
|
|
Aug 21 2008, 07:05
|
Частый гость
 
Группа: Участник
Сообщений: 99
Регистрация: 22-03-07
Из: Novosibirsk
Пользователь №: 26 415

|
Цитата(Сергей Борщ @ Aug 20 2008, 17:12)  Не могли бы вы более подробно описать, как вы это делаете? Ведь AT91C_BASE_XXX описаны через #define и Эклипс просто не видит этих структур. Нужно объявить экземпляр указателя на структуру/тип, когда компилятор сгенерит отладочную информацию. Я пользуюсь at91lib (ленивый программист  ), там, в драйверах, они обычно уже объявлены. Для отладки можно глобальный указатель объявить, но я не часто смотрю регистры, мне хватает локальных
|
|
|
|
|
  |
2 чел. читают эту тему (гостей: 2, скрытых пользователей: 0)
Пользователей: 0
|
|
|