Не работает защита от КЗ

[vavan1983]

unsigned short i=8,l=1;
void main(void)
{
OPTION_REG=0x7f;      //глобал разрешение подтяжкирезисторов
WPU=0x12;             //подтягиваем резисторы на ножки GP1 и GP4
TRISIO=0X12;          //ножки GP1 и GP4 как входы остальные выходы
OSCCON=0X75;          //внутреней тактовый ген 8 МГц
CMCON0=0X07;          //компораторы выключены
ANSEL=0X00;           //все порты ввода вывода цифровые
GPIO=0;               //установка лог. 0 на всех портах
PWM1_Init(40000);     //работа CCP в режиме ШИМ на частоте 40 КГц
PWM1_Set_Duty(i);     //начальная скважность 8.
PWM1_Start();         //старт CCP
Delay_ms(500);
GPIO.GP0=1;                     //включает реле для подачи плюса 280 вольт на MOSFET-ы
if(!GPIO.GP1 && !GPIO.GP4)      //проверка КЗ на выходах J1, J2, J3 (см. схему)
{
for(i=5; i‹=128; i++)           //КЗ между J1, J2, J3 нет. Увеличиваем скважность ШИМ до 128
{
Delay_ms(1);
PWM1_Set_Duty(i);
}}
while(1)
{
if((GPIO.GP1 || GPIO.GP4)&&l)  //произошло КЗ между J1, J2, J3
{
GPIO.GP0=0;                    //отключает плюс от полумоста на MOSFET-тах
PWM1_Set_Duty(8);              //сброс скважности до стартовой
Delay_ms(500);
GPIO.GP0=1                     //включает реле для подачи плюса 280 вольт на MOSFET-ы
l=0;
}
if(!GPIO.GP1 && !GPIO.GP4&&!l)   //проверка КЗ на выходах J1, J2, J3 (см. схему)
{
l=1;
Delay_ms(500);
for(i=8; i‹=128; i++)             //КЗ между J1, J2, J3 нет. Увеличиваем скважность ШИМ до 128
{
Delay_ms(1);
PWM1_Set_Duty(i);
}
}
}
}

В целом блок питание понижающий с 220 переменного на 48 вольт двух полярного по 24 вольта в плечё работает отлично. Мягкий старт есть.

Возникла проблемка защиты от КЗ каждого плеча в отдельности, а именно если накоротко замкнуть J1 и J2 то всё хорошо защита срабатывает как надо но когда замыкаю отдельно J1 и J3 или J2 и J3 то защита не срабатывает, то есть выполняются все условия if() по порядку в теле цикла while(1).

[akegor]

!GPIO.GP4&&!l

Что Вы ожидаете получить, сделав "!0х01"? Отсюда и ноги растут у непоняток.

[vavan1983]

Что Вы ожидаете получить, сделав "!0х01"? Отсюда и ноги растут у непоняток.

Предложите ваш вариант если не трудно.

[akegor]

!GPIO.GP4&&!l

Эта конструкция вообще лишняя. "0" есть "ложь", не "0" - "истина". Истина будет при любом ненулевом результате, поэтому "&&1" и подобное - излишество. Считаете необходимым подчеркнуть или просто явно указать для лучшего понимания (или других соображений, скажем, особенности компилятора) - используйте true и false. Или они уже определены в либах, или определите сами.

[vavan1983]

Вот и я столкнулся с ЭМИ влияющим на МК

[vavan1983]

Народ кто знает какое сопротивление имеют внутреннее резисторы в МК pic12f683?

[ForcePoint]

Народ кто знает какое сопротивление имеют внутреннее резисторы в МК pic12f683?

Точно - знает datasheet. А так - десятки килоом.

[johanh]

Опросил бит порта оптрона 1 - послал бит(или инвертированный) на релюху 1
Опросил бит порта оптрона 2 - послал бит(или ннвертированный) на релюху 2
"если" тут не нужно

[vavan1983]

Что Вы ожидаете получить, сделав "!0х01"? Отсюда и ноги растут у непоняток.

if(!GPIO.GP1 && !GPIO.GP4&&!l)
{
}

или вот так

if(GPIO.GP1==0 && PIO.GP4==0 && l==0)
{
}
можно вот

if(!(GPIO.GP1 && GPIO.GP4&&l))
{
}

или даже так

if((GPIO.GP1 && GPIO.GP4&&l)==0)
{
}

а результат во всех случаях будет одинаков.

[akegor]

Просю пардона, Поторопился и не все досмотрел.

ПС. Однако. Неплохо бы разделять логику и арифметику. На эту тему попадалась статья, что не во всех компиляторах микс будет работать как ожидаемо. Связано с представлением false и true, соответственно и восприятие компилятором.