Ошибка в коде MDR32F9Qx_mil_std_1553.c функция MIL_STD_1553_Cmd

[nw53harry]

Функция должна сбрасывать и устанавливать бит MR но устанавливает еще несколько старших бит,
и сбивает раннее сделаные настройки (ошибка из за неправильгого значения MR_EN_Reset),
предлагается такой вариант:

void MIL_STD_1553_Cmd(MDR_MIL_STD_1553_TypeDef * MIL_STD_1553x, FunctionalState NewState)
{
/* Check the parameters */
assert_param(IS_MIL_STD_1553_ALL_PERIPH(MIL_STD_1553x));
assert_param(IS_FUNCTIONAL_STATE(NewState));

if(NewState == ENABLE){
	/* Enable the selected MIL_STD_1553 by clearingsetting the MR bit in the CONTROL register */
	MIL_STD_1553x->CONTROL &= ~MR_EN_Set;
}
else{
	/* Disable the selected MIL_STD_1553 by setting the MR bit in the CONTROL register */
	MIL_STD_1553x->CONTROL |= MR_EN_Set; // setting only the MR bit in the CONTROL register
	//MIL_STD_1553x->CONTROL |= ~MR_EN_Reset; // it is wrong, unnecessary high bits are set! (оставил ошибку для сравнения)
}

}

[Eldar Khayrullin]

Уже не поддерживаю данный репозиторий, но могу слить ваши изменения (под вашу ответственность), если откроете Pull request

[nw53harry]

Здравствуйте! Отвечать конечно не хочется ;) но запрос открыл. Код у меня проверен на железе (ошибка кстати может наверно и спалить интерфейсные микросхемы  на миландровской отладке они сильно грелись благо  успел выключить.)  

Воскресенье, 12 марта 2023, 16:00 +03:00 от Eldar Khayrullin ***@***.***>:     Уже не поддерживаю данный репозиторий, но могу слить ваши изменения (под вашу ответственность), если откроете Pull request — Reply to this email directly, view it on GitHub , or unsubscribe . You are receiving this because you authored the thread. Message ID: <eldarkg/emdr1986x-std-per-lib/issues/59/1465192480 @ github . com>

    С уважением, Гарий Горбач ***@***.***  

[Eldar Khayrullin]

Fixed #60