Програмирование pic на СИ.

[uldemir]

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

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

[Vano79]

XC8 ver. 1.45 ругается

Код: Выделить всё

implicit signed to unsigned conversion

на такую строку

Код: Выделить всё

lcd_putch(r[0] + 48);

Аргументы функции и переменная r[0] объявлены как u8. В ver. 1.38 такого нет, у обоих

Код: Выделить всё

 --warn=-3

.
Забить?

[КРАМ]

[uquote="uldemir",url="/forum/viewtopic.php?p=3331001#p3331001"]это про XC8 - платный он дюже дорог. А триальными 60-ю днями я уже в прошлом году воспользовался и это счастье больше мне не светит.[/uquote]
Пользуйтесь помесячной лицензией. 30 дней - 1800 рублей.
А вообще, я как то обхожусь для 8-разрядных АСМом. Если уж на них делается объемный проект со стеками протоколов, то тогда имеет смысл минимизировать затраты помесячной лицензией.
http://www.microchip.com/developmenttoo ... 006021-sub
https://www.microchipdirect.com/product ... 006021-sub
ЗЫ. К слову, лицензий с бесконечным сроком нет. Есть либо месячная, либо годовая.

[oleg110592]

[uquote="Vano79",url="/forum/viewtopic.php?p=3331248#p3331248"]XC8 ver. 1.45 ругается[/uquote]
тож ругается на старую проверенную чужую библиотеку, но в железке работает - где то на форуме Микрочипа вроде обещали, что поправят

[driver_gv]

[uquote="Vano79",url="/forum/viewtopic.php?p=3331248#p3331248"]

Код: Выделить всё

lcd_putch(r[0] + 48);

Аргументы функции и переменная r[0] объявлены как u8.

Код: Выделить всё

 --warn=-3

.[/uquote]

По умолчанию число 48 - signed
Микрочип рекомендует:
The warning is caused by the "i - 1"; this results in a signed value which is being used as an array index. Make it "i - 1U" instead.

Ваш код

Код: Выделить всё

lcd_putch(r[0] + 48U);

А от разработчика компилятора

I don't know if this qualifies as a "good" answer, but it's the only one I've got:

XC8 generates that message in a number of situations where it arguably should not. We don't yet know precisely why it does (and thus we don't know how we can change the compiler to stop it), which is why it was designated a level "-3" warning, when the compiler's default warning level is "0". But MPLAB X recently changed to explicitly set the compiler's warning level to "-3" so you get the warning now.

You can eliminate many often-bothersome warning messages by changing your MPLAB X project settings to specify a warning level of "-2" (which is the level I have historically recommended as being most helpful while still least distracting). Selecting this level means you will miss out on some useful warnings, but you will also avoid a lot of noise. It's up to you if this tradeoff is acceptable.

Что соответствует выбору - "забить"

[Vano79]

По умолчанию число 48 - signed

Глупый вопрос - это стандарт?

Ваш код

Код: Выделить всё

lcd_putch(r[0] + 48U);

Тем не менее приводится в норму без предупреждений:

Код: Выделить всё

r[0] += 48;
lcd_putch(r[0]);

Код: Выделить всё

Что соответствует выбору - "забить"

Мне кажется не тот случай, путаница с типами не такая уже редкая ошибка. И предупреждения здесь уместны.
Только сейчас заметил, если аргументом функции lcd_putch() является signed переменная, то XC8 1.38 не предупреждает при

Код: Выделить всё

--warn=-3

Вывод: один и тот же уровень "WarningLevel" разных версий ХС8 отвечает за разный набор кодов предупреждений.

[driver_gv]

[uquote="Vano79",url="/forum/viewtopic.php?p=3331326#p3331326"]

По умолчанию число 48 - signed

Глупый вопрос - это стандарт?
.[/uquote]

ХЗ. я обычно не применяю не именованные константы, делаю так

Код: Выделить всё

const uint8_t divider = 48;

Потом настройки программы можно делать из одного места а не лазить по всему коду.

[oleg110592]

48 - ascii код нуля, интересно прокатит ли lcd_putch(r[0] + '0');
з.ы. в itoa.c в папке xc8 есть

Код: Выделить всё

char      c;
....
c += '0';
....

[Аlex]

Код: Выделить всё

lcd_putch(r[0]+'0');

не ругается.
Версия 1.45

[DarkWolf]

Доброго времени суток! Решил в этой ветке задать вопрос: пробую использовать таймер 0 в контроллере PIC18f47J53. Но почему то результата 0

Код: Выделить всё

void init_Timer0()
{
   T5CON.T08BIT  = 0;   // 16 бит
   TMR0L = 0x47;       
   TMR0H = 0x71;     
   T0CON.PSA = 0;      
   T0CON.T0CS = 0;
   T0CON.T0SE = 0;
   T0CON.T0PS0 = 0;   //*****************
   T0CON.T0PS1 = 1;   // прескаллер 1:128 *
   T0CON.T0PS2 = 1;   //*****************
   T0CON.TMR0ON = 1; // старт таймера
}

void interrupt() {
     //USB Interrupt
     USB_Interrupt_Proc();
     
     if (INTCON.TMR0IF&&INTCON.TMR0IF)
     {
       PORTD.f5 = ~(PORTD.f5); // моргаю светодиодом
       INTCON.TMR0IF = 0;
       TMR0L = 0x47;
       TMR0H = 0x71;
     }

}

Светодиод зажигается один раз и горит постоянно. Частота МК 24МГц

[driver_gv]

Для 18 серии надо писать в LAT.
Пример макроса

Код: Выделить всё

#define LED_1_Toggle()             do { LATBbits.LATB4 = ~LATBbits.LATB4; } while(0)

[DarkWolf]

[uquote="driver_gv",url="/forum/viewtopic.php?p=3351369#p3351369"]Для 18 серии надо писать в LAT.[/uquote]


Попробовал писать в LATD светодиод быстро моргает 3 раз и затем постоянно горит.

[driver_gv]

Еще возникает вопрос по этой строке

Код: Выделить всё

T5CON.T08BIT  = 0;   // 16 бит

Если в ней ошибка и надо TOCON то еще поменяйте местами строки

Код: Выделить всё

       TMR0L = 0x47;
       TMR0H = 0x71;

Так как запись в таймер происходит по записи в L регистр а в H из защелки
-----
Similarly, a write to the high byte of Timer0 must also
take place through the TMR0H Buffer register. The high
byte is updated with the contents of TMR0H when a
write occurs to TMR0L. This allows all 16 bits of Timer0
to be updated at once.
-----

[DarkWolf]

Подозрение, что не заходит в обработчик прерывания. Сейчас все лишнее из программы удалил оставил только обработчик таймера. В итоге светодиод не горит.

Добавлено after 2 hours 21 minute 28 seconds:
Проблемы с английским вытекают в глупые ошибки. Не правильно перевел даташит.. Небходимо было разрешить прерывание от таймера.

INTCON.T0IE=1;

[electronik_tver]

Коллеги мне нужна помощь, есть готовое устройство, схема обсуждению не подлежит ее менять нельзя, нужно написать на него программу. При старте программа читает положение джамперов для настройки один раз и далее динамическая индикация.
С чтением проблем вроде нет, по Х поочередно даем 0, Y - входы с подтяжкой.
X1=0;X2=1;X3=1;Set[1-1]=pinY2;Set[4-1]=pinY1; //читаем джамперы J1,J4
X1=1;X2=0;X3=1;Set[2-1]=pinY2;Set[5-1]=pinY1; //читаем джамперы J2,J5
X1=1;X2=1;X3=0;Set[3-1]=pinY2;Set[6-1]=pinY1; //читаем джамперы J3,J6
и заполняем массив Set[6]
а вот с динамической индикацией голову сломал, ну понятно, делаем прерывание с нужной частотой.
Далее рассуждения такие: X3=0 и больше его не трогаем. Поочередно по выходам X1 или X2 даем 1.
Если допустим X2=1 и светодиод D1 нужно зажечь то Y2=0 а если потушить то Y2 нужно перевести как вход без подтяжки (потому что если сделать просто Y2=1 то он может встретится с X=0 через D16 или D18 и возможно замкнутые джамперы )
и в свою очередь если светодиод должен быть потушен и на X2=1 то он получит на Y2 низкий уровень через джампер и диод с X1 и X3 и все равно будет светиться. Получается и X если он не 1 нужно переводить на вход без подтяжки, как-то все коряво и сложно получается на элементарной вроде схеме.
Подскажите пож как красиво обслужить такого типа схемку на динамической индикации и может опрос входов тоже проще можно сделать??

[vk696]

о обучении.
в компиляторах есть библиотеки готовых функций и описаны они с хорошим вкусом и их можно и нужно использовать в качестве обучающего материала.
к тому же описание на сам компилятор является главным учебным пособием по программированию PIC.

[pokk]

Подскажите как сделать ассемблерную вставку, что-то типа такого

Код: Выделить всё

#define SetBit(adr, bit) asm("BSF adr, bit")

[Linuxoid91]

Добрый день. Нужно вывести из ножки PIC24 8-битное число, начиная со старшего бита (для сдвигового регистра). Написал такую функцию.

Код: Выделить всё

HC164(int x) //record the byte to 74HC164
{ unsigned char b;
   char j;
   for(j=7;j>-1;j--)
   {b=x; //записываем входное число от 0 до 255 в 8-битную переменную без знаков.
     b»j; //сдвигаем на j, чтоб записываемый бит оказался младшим (начинаем запись с самого старшего)
     PORTCbits.RC0=b; //serial data пишем бит
     PORTBbits.RB3=1; //clock дрыгаем clock
     PORTBbits.RB3=0;}} //clock

В результате, на RC0 результат совершенно отличается от ожидаемого (то стабильно в единице, то стабильно в нуле). Сам порт и регистр работает (проверял записью конкретно заданных бит).

Я правильно понимаю, что PORTbits записывает в порт только младший бит? Проверил записью чисел больше 1. При записи нечетных светодиод горит, при четных - не горит. Осциллограф отобразил появление только младшего бита. Если код в корне не правильный, то как сделать правильно?

Добавлено after 7 minutes 22 seconds:
Также попробовал PORTAbits.RC0<<b; результат тот же.

[uldemir]

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

ой, как всё запущено.

Код: Выделить всё

for(j=7;j>-1;j--)

на эту строчку варнингов нет? j у вас по умолчанию без знаковое. так что сравнение всегда истинно.

Код: Выделить всё

PORTCbits.RC0=b; //serial data пишем бит

тут я не уверен.... что писаться будет именно младший бит. Для надёжности я бы замаскировал остальные биты типа

Код: Выделить всё

PORTCbits.RC0 = b & 0x01u; //serial data пишем бит

Может было бы правильнее просто двигать маску:

Код: Выделить всё

unsigned char mask = 0x80;
for (j=0 ; j < 8; j++) {
  PORTCbits.RC0 = (x & mask) ? 1 : 0 ;
  mask >> 1;
}

[Linuxoid91]

Код: Выделить всё

? 1 : 0

Можно подробней про эту команду? Что она делает?

Добавлено after 28 minutes 39 seconds:
Кстати, таки заработало. Благодарствую.

Добавлено after 1 hour 46 minutes 41 second:
А, понял. Сишная запись "if-else"