Помощь - Поиск - Пользователи - Календарь
Полная версия этой страницы: STM32F4 ETHERNET + CAN
Форум разработчиков электроники ELECTRONIX.ru > Микроконтроллеры (MCs) > ARM
DmitryG
Здравствуйте!

Проект: Управление отправкой и приёмом CAN сообщений через Ethernet с помощью браузера.

Проблема: Не работает CAN, как на приём, так и на передачу сообщений.

Судя по флагам, микроконтроллер проходит по всем строчкам инициализации CAN, заходит в функцию отправки сообщений, однако, ничего не отправляет.
Коммутация рабочая.

Подскажите, пожалуйста, как чинить?

Инициализация:

CODE
void CAN1_Config(void)
{
GPIO_InitTypeDef GPIO_InitStructure;

/* CAN GPIOs configuration **************************************************/

/* Enable GPIO clock */
RCC_AHB1PeriphClockCmd(RCC_AHB1Periph_GPIOB, ENABLE);

/* Connect CAN pins */
GPIO_PinAFConfig(GPIOB, GPIO_PinSource8, GPIO_AF_CAN1);
GPIO_PinAFConfig(GPIOB, GPIO_PinSource9, GPIO_AF_CAN1);

/* Configure CAN RX and TX pins */
GPIO_InitStructure.GPIO_Pin = GPIO_Pin_8 | GPIO_Pin_9;
GPIO_InitStructure.GPIO_Mode = GPIO_Mode_AF;
GPIO_InitStructure.GPIO_Speed = GPIO_Speed_50MHz;
GPIO_InitStructure.GPIO_OType = GPIO_OType_PP;
GPIO_InitStructure.GPIO_PuPd = GPIO_PuPd_UP;
GPIO_Init(GPIOB, &GPIO_InitStructure);

/* CAN configuration ********************************************************/
/* Enable CAN clock */
RCC_APB1PeriphClockCmd(RCC_APB1Periph_CAN1, ENABLE);

/* CAN register init */
CAN_DeInit(CAN1);
CAN_StructInit(&CAN_InitStructure);

/* CAN cell init */
CAN_InitStructure.CAN_TTCM = DISABLE;
CAN_InitStructure.CAN_ABOM = DISABLE;
CAN_InitStructure.CAN_AWUM = DISABLE;
CAN_InitStructure.CAN_NART = DISABLE;
CAN_InitStructure.CAN_RFLM = DISABLE;
CAN_InitStructure.CAN_TXFP = DISABLE;
CAN_InitStructure.CAN_Mode = CAN_Mode_Normal;
CAN_InitStructure.CAN_SJW = CAN_SJW_1tq;

/* CAN Baudrate = 1MBps (CAN clocked at 30 MHz) */
CAN_InitStructure.CAN_BS1 = CAN_BS1_6tq;
CAN_InitStructure.CAN_BS2 = CAN_BS2_8tq;
CAN_InitStructure.CAN_Prescaler = 2;
CAN_Init(CAN1, &CAN_InitStructure);

/* CAN filter init */
CAN_FilterInitStructure.CAN_FilterNumber = 0;
CAN_FilterInitStructure.CAN_FilterFIFOAssignment = CAN_Filter_FIFO0;
CAN_FilterInitStructure.CAN_FilterMode = CAN_FilterMode_IdMask;
CAN_FilterInitStructure.CAN_FilterScale = CAN_FilterScale_32bit;
CAN_FilterInitStructure.CAN_FilterActivation = ENABLE;

CAN_FilterInitStructure.CAN_FilterIdHigh = 0;
CAN_FilterInitStructure.CAN_FilterIdLow = 0;
CAN_FilterInitStructure.CAN_FilterMaskIdHigh = 0;
CAN_FilterInitStructure.CAN_FilterMaskIdLow = 0;

CAN_ITConfig(CAN1, CAN_IT_FMP0, ENABLE);
CAN_ITConfig(CAN1, CAN_IT_TME, ENABLE);

CAN_FilterInit(&CAN_FilterInitStructure);
}



void NVIC_Config(void)
{
NVIC_InitTypeDef NVIC_InitStructure;

NVIC_InitStructure.NVIC_IRQChannel = CAN1_RX0_IRQn;
NVIC_InitStructure.NVIC_IRQChannelPreemptionPriority = 0x0;
NVIC_InitStructure.NVIC_IRQChannelSubPriority = 0x1;
NVIC_InitStructure.NVIC_IRQChannelCmd = ENABLE;
NVIC_Init(&NVIC_InitStructure);

NVIC_InitStructure.NVIC_IRQChannel = CAN1_TX_IRQn;
NVIC_InitStructure.NVIC_IRQChannelPreemptionPriority = 0x0;
NVIC_InitStructure.NVIC_IRQChannelSubPriority = 0x1;
NVIC_InitStructure.NVIC_IRQChannelCmd = ENABLE;
NVIC_Init(&NVIC_InitStructure);
}

Функция отправки:

Код
void ASK_1(void)
{
  CanTxMsg TxMessage;

//     if (CAN_GetITStatus (CAN1, CAN_IT_TME))
//     {
//        CAN_ClearITPendingBit(CAN1,CAN_IT_TME);

        ASK1_DATA0='1';  

        TxMessage.StdId = 0x101;
        TxMessage.RTR = CAN_RTR_DATA;
        TxMessage.IDE = CAN_ID_STD;
        TxMessage.DLC = 1;
        TxMessage.Data[0] = ASK1_DATA0;  
        
        //CAN1->TSR&CAN_TSR_TXOK0;
        CAN_Transmit(CAN1,&TxMessage);
        
//     }
}

Функция приёма:

Код
void CAN1_RX0_IRQHandler(void)
{
    CanRxMsg RxMessage;
    
    if (CAN_GetITStatus (CAN1, CAN_IT_FMP0))
    {
        CAN_ClearITPendingBit(CAN1,CAN_IT_FMP0);

        CAN_Receive(CAN1,CAN_FIFO0, &RxMessage);
        CAN_FIFORelease (CAN1,CAN_FIFO0);
        
        READ_DATA0=RxMessage.Data[0];
        READ_DATA1=RxMessage.Data[1];
        READ_DATA2=RxMessage.Data[2];
        READ_DATA3=RxMessage.Data[3];
        
        Can1Flag = ENABLE;
    }
}
pitt
Не использую НИКОГДА CUBE, SPL или HAL и Вам не советую.
Попробуйте сначала программную заглушку потом аппаратную и пусть поговорит сам с собой.
DmitryG
Цитата(pitt @ Feb 18 2017, 18:58) *
Не использую НИКОГДА CUBE, SPL или HAL и Вам не советую.
Попробуйте сначала программную заглушку потом аппаратную и пусть поговорит сам с собой.


Если вас не затруднит, не могли бы вы разъяснить начинающему, что такое CUBE, SPL или HAL?

Я раньше опирался на примеры уже готовых программ, а они были написаны в подобной форме. Что я делаю не так, и где нужно почитать, чтобы делать правильно?

Спасибо
Forger
Не слушаете коллегу, все правильно вы делаете - начинаете работу с чужих примеров и решений.
Переход на прямое обращение к регистрам, это уж скорее от безделья или от скуки sm.gif
Но справедливости ради стоить отметить, что в в критичных к скорости работы участках кода ситуациях это имеет смысл (прерывания, программные интерфейсы через ногодрыгание).

Цитата
что такое CUBE, SPL или HAL?

CUBE, SPL или HAL - это готовые библиотеки от ST.
Ваш код выше написан на SPL - Std Periph Lib.
HAL пришел на смену SPL, новый проекты лучше начинать на HAL.

Цитата
Я раньше опирался на примеры уже готовых программ, а они были написаны в подобной форме.

Все верно вы делаете - нет ничего постыдного или зазорного в использовании чужого открытого кода
Делать наиболее простым и привычным способом - это разумно.

Цитата
Что я делаю не так, и где нужно почитать, чтобы делать правильно?

Скачивайте свежие библиотеки с сайта ST, внутри есть примеры для работы с разной периферией.
HAL есть внутри CUBE, поэтому ищите свежий CUBE.
Возможно, у вас аппаратно снаружи МК что-то не так подключено. Поищите в том направлении.
В гуглях полно информации по этой теме.
DmitryG
Цитата(Forger @ Feb 19 2017, 11:17) *
.



Понял Вас.
Большое Спасибо за помощь.
pitt
Примеры - это хорошо. Открытый код - это замечательно. Использовать CUBE, HAL, SPL можно только чтобы понимать что происходит в регистрах. Использовать ЭТО вслепую очень легко и ... самоубийственно. Конечно, что-то получится и даже заработает, но потом, почему-то перестанет, а знаний нет. Но это уже дело вкуса и требований к качеству. Если для инженера-электрика или хоббииста это простительно, но для профессионального программера - приговор. Как пелось в старой популярной песне:"думайте сами, решайте сами..."
Forger
Цитата(pitt @ Feb 20 2017, 07:48) *
Использовать CUBE, HAL, SPL можно только чтобы понимать что происходит в регистрах.

Вот уж точно заняться нечем, как изучать, что там и где пишется в регистры, особенно новичку sm.gif

Цитата
Использовать ЭТО вслепую очень легко и ... самоубийственно.

Если у вас не получается ЭТО использовать успешно и эффективно, то это вовсе не означает, что у других это тоже не получается ))

Цитата
Конечно, что-то получится и даже заработает, но потом, почему-то перестанет, а знаний нет.

Неужели такая большая проблема для опытного инженера, посмотреть и разобраться, что именно там неправильно настраивается в регистрах в этих библиотеках с открытыми исходниками?

Цитата
Но это уже дело вкуса и требований к качеству.

Дело вкуса - это в ресторане. Тут же все как правило упирается в некий компромисс: надежность кода/время.
Если через HAL/CUBE задача решается быстрее и эффективнее, то именно так ее и следует решать.
Но коли страх перед чужим пусть и бесплатным кодом сильнее, чем следует, то, разумеется, использовать чужой код категорически запрещается! sm.gif

Цитата
Если для инженера-электрика или хоббииста это простительно, но для профессионального программера - приговор.

Значит, подавляющее большинство сидящих тут - "инженеры-электрики или хоббиисты", и лишь один - "профессиональный программер" ... sm.gif
Neborak
Вроде еще нужно инициализировать тактирование альтернативных функций портов.
pitt
Цитата(Forger @ Feb 20 2017, 01:39) *

Желаю всяческих успехов: об'яснять слепому от рождения, что такое белый цвет я не буду.
Forger
Цитата(pitt @ Feb 20 2017, 16:10) *
об'яснять слепому от рождения, что такое белый цвет я не буду.

Нет ничего постыдного признаться в своем неумении что-то делать - нет людей, которые умеют все!
Стыдно другое - слепо убеждать обывателей в том, во что сам когда-то фанатично уверовал, не понимая истинной сути сего "учения" ... sm.gif

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

Я сам лично не пользуюсь кубом и пока не планирую, но и отговаривать от этого новичков не стану.
Пользуюсь HAL/SPL вперемешку с прямым обращением. Пока что этого мне достаточно.
Нисколько не навязываю, просто, делюсь своим опытом wink.gif
AHTOXA
Цитата(Forger @ Feb 20 2017, 11:39) *
Значит, подавляющее большинство сидящих тут - "инженеры-электрики или хоббиисты", и лишь один - "профессиональный программер" ... sm.gif

Откуда у вас такая статистика? У меня обратное впечатление. По крайней мере, с кубом здесь пока вижу только сплошные страдания.

Цитата(Forger @ Feb 20 2017, 19:16) *
Я сам лично не пользуюсь кубом и пока не планирую, но и отговаривать от этого новичков не стану.
Пользуюсь HAL/SPL вперемешку с прямым обращением. Пока что этого мне достаточно.

Вот в этом-то и дело. Вы знаете, как оно там всё устроено внутри, поэтому вы можете и куб осилить. А новички, которые начинают сразу с куба, совершенно беспомощны при возникновении любой нештатной ситуации. Потому и вопросы тут про куб такие - "я нагенерил, а оно не работает, что мне делать?". Наверное, куб допилят в конце-то концов. Но думать он отучает, это его очень большой минус.
Forger
Цитата(AHTOXA @ Feb 20 2017, 18:42) *
По крайней мере, с кубом здесь пока вижу только сплошные страдания.

А вы полагаете, что здесь будут писать не о проблемах, а о успехах освоения куба или чего-бы то ни было?? smile3046.gif
Сюда пишут в случае возникновения трудностей, это нормально. Было бы очень странно видеть туть обратное.
А в данном случае, код вообще написан на архаичном SPL и скопирован лишь частично из существующих примеров.

Цитата
Вот в этом-то и дело. Вы знаете, как оно там всё устроено внутри,

Ничего подобного, просто, мне действительно до-лампочки какие регистры пишутся при инициализации.
Мне важен результат и читаемость кода.
Трудно спорить, что HAL/SPL читается легче, чем прямое обращение к регистрам.
К регистром я обращаюсь напрямую лишь в критичных к скорости работы кусках кода, это редко нужно.

Цитата
поэтому вы можете и куб осилить.

Нечего там осиливать - даже школьник сможет получить результат, если не будет выпендриваться и копаться в даташитах раньше времени!

Цитата
А новички, которые начинают сразу с куба, совершенно беспомощны при возникновении любой нештатной ситуации.

Т.е. по-вашему логично копать яму под фундамент лопатой, вместо того, чтобы воспользоваться экскаватором?
CUBE - это инструмент, и им тоже нужно учиться пользоваться.
С него хорошо начинать, но на нем нельзя зацикливаться. Равно как и нельзя зацикливаться на прямом обращении к регистрам.
КОМПРОМИСС, его нужно искать. Самостоятельно!

Цитата
Потому и вопросы тут про куб такие - "я нагенерил, а оно не работает, что мне делать?".

Бесит, пролистывай тему )))
Да, ленивых имбицилов нынче еще больше, чем раньше. Это, увы, неизбежно ((
Но это не означает, что имбицил поумнеет пытаясь писать насильно код зарывшись в даташит! Он с равным успехом будет тупить в кубе )))
Некоторым горе-"программерам" лучше воообще не влезать в эту область, а идти работать в гипермаркет на кассу или еще куда-нить по-проще
Нормальный адекватный программер, пройдя через КУБ или что-то подобное, волей неволей сам разберется, интернеты/ютубы нынче очень дешевые )))

Такие программисты пишут на форумах мало, редко (в основном читают), проблемы свои решают самостоятельно, и чаще с большей готовностью помогают другим.
Вполне возможно, что среди них есть те, кто пользует КУБ. Если мы о них не знаем, вовсе не означет, что их нет ))

Цитата
Но думать он отучает, это его очень большой минус.

КУБ приучает получать результат в первую очередь, а это, порой, важнее некой гипотетической идеальности кода -
нахимичить с прямым доступом в регистры еще проще, чем взять чужой готовый пример на хале/кубе и его чуток допилить (по необходимости).

В будущем процы станут еще толще и сложнее по периферии и "чистых" писателей на прямом обращении к регистрам останутся единицы, в итоге, они вымрут также как вымерли в свое время чистые ассемблерщики sm.gif
Может, останется один-два, которые до пены у рта будут доказывать свою уникальность, но их будут воспринимать как чудаков-вассерманов, и постепенно забудут про их существование ...
jcxz
Цитата(Forger @ Feb 20 2017, 12:39) *
Значит, подавляющее большинство сидящих тут - "инженеры-электрики или хоббиисты", и лишь один - "профессиональный программер" ... :)

Да, профессионалов единицы. Это в любом деле так. Подавляющее большинство - любители, в массе своей способные только копировать чужое, не понимая.
Для них и придуманы все эти кало-кубы.

PS: блин - ещё один "эффективный" "программист".... :crying:
Forger
Цитата(jcxz @ Feb 20 2017, 20:48) *
Да, профессионалов единицы. Это в любом деле так. Подавляющее большинство - любители, в массе своей способные только копировать чужое, не понимая.

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

А действительно настоящие профессионалы вырастают из тех самых новичков, которые в свое время прошли весь этот путь самостоятельно, и вовсе не застряли на каком-то этапе.
Чтобы новичок вырос разносторонне развитым и действительно профессионалом, ему нужно давать инструменты, разные инструменты, но учиться пользоваться ими он должен по максимуму самостоятельно.

Если просит новичок помощи в проблеме - покажи ему примерное направление поиска, но не давай ему готовое решение, иначе никакого проку не будет ))
jcxz
Цитата(Forger @ Feb 21 2017, 00:00) *
Если просит новичок помощи в проблеме - покажи ему примерное направление поиска, но не давай ему готовое решение, иначе никакого проку не будет ))

Это направление ему дали ещё в первом посте - прочитать даташит. Если он последует этому совету - это и будет первый шаг по пути в профессионалы из чайников.
Forger
Цитата(jcxz @ Feb 20 2017, 21:26) *
Это направление ему дали ещё в первом посте - прочитать даташит.
Если он последует этому совету - это и будет первый шаг по пути в профессионалы из чайников.

...И бросили Боги с небес новичку-программеру божественную подачку - "Кури даташит, о, смертный!"
И перестали смертные верить в богов, ибо нет никаких богов/гуру, а есть ли те, кто неумело выдает себя за них ... wink.gif
jcxz
Цитата(Forger @ Feb 20 2017, 23:34) *
Может, останется один-два, которые до пены у рта будут доказывать свою уникальность, но их будут воспринимать как чудаков-вассерманов, и постепенно забудут про их существование ...

Интересно: а кто тогда в те благословенные времена писать будет все эти кубо-библиотеки, которыми будет пользоваться остальное стадо кубо-кодеров??? Или думаете в процы уже куб будут встраивать в виде карты соединения вентилей??? :biggrin:
Кто будет за всё остальное стадо читать и изучать даташиты? Да молиться будете на этих "один-два" чудаков! И платить им огромные гонорары за умения.
Forger
Цитата(jcxz @ Feb 20 2017, 21:49) *
Интересно: а кто тогда в те благословенные времена писать будет все эти кубо-библиотеки, которыми будет пользоваться остальное стадо кубо-кодеров??? Или думаете в процы уже куб будут встраивать в виде карты соединения вентилей??? biggrin.gif
Вот уж действительно нечем заняться, как думать кто и на чем там пишет библиотеки ...
"Ваш шашечки или ехать?"

Бессмысленно других приучать пользоваться везде лопатой лишь на том основании, что сам не фига умеешь пользоваться экскаватором! sm.gif

Цитата
Кто будет за всё остальное стадо читать и изучать даташиты?
Ну, да, в будущем же даташитов для простых смертных уже не будет и выдаваться они будут лишь избранным "гуру" после прохождения смертельных испытаний (скажем, по программировании регистров NVIC контроллера вслепую, без CMSIS)!
Интернеты будут доступны только для истинных Гуру, а те в свою очередь будут выдавать смертным Божественные Даташиты только после признаниями смертными истинной Божественности этих Гуру! ....
Ничего не напоминает? sm.gif


Цитата
Да молиться будете на этих "один-два" чудаков! И платить им огромные гонорары за умения.

"И остался он в старости один в полной нищете и забвении, ибо свято верил он в универсальность лопаты, как самого лучшего и единственного инструмента на свете ..."
jcxz
Цитата(Forger @ Feb 21 2017, 00:59) *
Бессмысленно других приучать пользоваться везде лопатой лишь на том основании, что сам не фига умеешь пользоваться экскаватором! sm.gif

Так может слезете со своего экскаватора и поможете наконец-то единоверцу-кубокодеру? Вместо того чтобы лить флейм экскаваторными ковшами.
Может сделаете что-то реально полезное вместо того, чтобы поносить других, кто пытался помочь, а сами ни слова по делу не сказали - один пустой трёп?
Пока от Вас во всех этих килобайтах пустого флейма не видно ни слова пользы, только пустое "обновите библиотеки". Видно "обновлять библиотеки" это всё что может кубо-кодер когда возникает проблема. laughing.gif
Forger
Цитата(jcxz @ Feb 21 2017, 09:06) *
Пока от Вас во всех этих килобайтах пустого флейма не видно ни слова пользы, только пустое "обновите библиотеки".

Вы увидели только то, что хотели увидеть или то, что вам удобно и выгодно видеть. Но это не повод сердиться!
Это вполне нормальное явление, молодежь всегда будет уделывать "стариков" по всем фронтам (кроме разве что сдержанности и жизненного опыта, но это уже к делу не относится) sm.gif
Объяснение тут простое - молодые думают шире и быстрее, у них еще нет зашоренности на привычном и ограниченном инструментарии (везде голый С, а то и АSМ, категоричное неприятие HAL/CUBE и т.п.).
Вот и топикстартер, судя по всему, понял суть дела и довольно быстро. А ведь он новичок ...

По делу: самый вероятный косяк в коде был указан в самой теме (пост #8), кратко и по делу.
А все остальное тут - самцовы игры "стариков", разве что от скуки wink.gif

ps. Я уже писал, что кубом не пользуюсь и не планирую далее. Хотя пробовал его разок, что греха таить )))
Уже давно проектирую проекты иначе, мне куб просто неудобен. А вот HAL/SPL пользовать не стесняюсь, и регистры напрямую юзаю, и SMSIS не грех потискасть wink.gif
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.
Invision Power Board © 2001-2025 Invision Power Services, Inc.