zheka 1 12 августа, 2018 Опубликовано 12 августа, 2018 · Жалоба Так в приведенной вами выше ссылке так прямо сразу и написано - 100В. Покупал я у другого продавца, с более бедным описанием. Приводя ссылку вчера, я в нее не вчитывался. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
jcxz 172 12 августа, 2018 Опубликовано 12 августа, 2018 · Жалоба Или вы в целом про стиль, который усматривается из текста (вы же упомянули "программку") Если второе, то, былбы благодарен услышать вашу критику моего уменья в более конкретных выражениях) Стиль? Конкретные ошибки? ИЛи еще что? Я уже писал, что для того чтобы вывести напряжение/ток/температуру на LCD float в принципе не нужен. Это как вскапывать экскаватором грядки. Ваш экскаватор просто в огород не влезет. не удивлюсь что библиотека эмуляции float сожрёт половину из доступных 8 кБ флеша. Ну и в остальном - это не стиль, а какое-то недоразумение. Зачем хотя-бы дублировать одни и те же операции многократно? Ещё раз - флеша в этом МК мало, от слова "совсем". И на такой быдлокодинг его размер явно не рассчитан. Зачем использовать 32-битные переменные на 8-битном МК там где это совсем не нужно? ну и остальное - не лучше... Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
zheka 1 12 августа, 2018 Опубликовано 12 августа, 2018 · Жалоба Зачем хотя-бы дублировать одни и те же операции многократно? Это где это у меня дублирование? Зачем использовать 32-битные Вообще не мое, в шаблоне было. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
jcxz 172 12 августа, 2018 Опубликовано 12 августа, 2018 · Жалоба Это где это у меня дублирование? Посмотрите на свои LCD_Show_Voltage()/LCD_Show_Temperature()/LCD_Show_Current() - не видите схожести? Они почти идентичные за исключением нескольких мест. Зачем тогда столько раз продублировали одно и то же? Хотя сами говорите что не лезет во флешь.... Зачем BATTERY_LEVELS и DIGITS продублированы в ОЗУ и флешь, если используются как константы? Зачем эти полотенца из send_command(0xC7);send_data(0x01); send_command(0x97); во многих местах? Что такое циклы - знаете? PS: Удивительно - у меня в 8 кБ STM8 влезло управление PMSM-мотором с довольно сложным пользовательским интерфейсом, работой с EEPROM и UART и пр.. А у Вас простой вывод на семисегментник не лезет.... Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
zheka 1 12 августа, 2018 Опубликовано 12 августа, 2018 (изменено) · Жалоба Посмотрите на свои LCD_Show_Voltage()/LCD_Show_Temperature()/LCD_Show_Current() - не видите схожести? Ну если вы об этом... Они шлют разные команды, по разным адресам. Более того, значок процента для цифр в левой части экрана дисплей считает точкой (запятой) старшего разряда цифр, находящихся в правой части экрана. Управляя цифрами правой части экрана, нужно корректировать одну из цифр левой части, более того, нужно знать, что за параметр в данный момент в ней отображается (параметров 4, одновременно отображаемых - 2) Можно конечно выделить общее и написать универсальную функцию, но для этого должно до такой степени не хватать места, что трата времени на подобные хитрости себя оправдывает. Я же зная сколько места в контроллере, и имея опыт написания более объемных прошивок, полагал, что места хватит при самом небрежном отношении к коду. ДА его собственно и хватило, я поставил галку, как указал выше - все нормально. Эти 4 функции были написаны раздельно с целью упрощения рашифровки того, какой сегмент за что отвечает (даташита на дисплей нет). Зачем эти полотенца из send_command(0xC7);send_data(0x01); send_command(0x97); во многих местах? Что такое циклы - знаете? Есть функция LCD_Clear - там цикл уместен и я его использовал. А в остальных функциях.... А ну-ка, хорошо, вот Вам кусок кода, покажите, как бы Вы сделали из него цикл? Просто интересно. send_command(0x00); send_command(0x44); send_command(0xC7); if (int_num<10) send_data(0x00+percent_sign);else send_data(DIGITS[int_num/10]+percent_sign); send_command(0xCB); send_data(DIGITS[int_num%10]); send_command(0xCE); send_data(DIGITS[decimal_num]+1); send_command(0xCA); send_data(0x4); send_command(0x97); Но за критику в любом случае спасибо. Изменено 12 августа, 2018 пользователем zheka Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
jcxz 172 12 августа, 2018 Опубликовано 12 августа, 2018 · Жалоба А ну-ка, хорошо, вот Вам кусок кода, покажите, как бы Вы сделали из него цикл? Просто интересно. Самое простое и очевидное что можно сделать: написать функцию отправляющую не побайтно, а строку байт. Ну а в первую очередь - убрать float-ы. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
zheka 1 12 августа, 2018 Опубликовано 12 августа, 2018 · Жалоба Самое простое и очевидное что можно сделать: написать функцию отправляющую не побайтно, а строку байт. Не спорю. И это можно сделать. Но повторюсь, задача экономии передо мной не стояла, да и не стоит. Моя прошивка будет существенно меньше той, что была в оригинала (там было еще меню настроек). Мне нужно было побыстрее, пусть и топорно, сделать небольшой реверс-инжиниринг. Для себя, лишь бы работало. Что-то не получается у меня с внешними прерываниями: ..... INTERRUPT_HANDLER(EXTI_PORTD_IRQHandler, 6) { GPIO_WriteHigh(BACKLIGHT_PORT, BACKLIGHT); // если прерывание сработает, то должна загореться подсветка (это не конечная цель, а просто способ отладки) } .... main() { GPIO_Init(GPIOD, GPIO_PIN_4, GPIO_MODE_IN_PU_IT); // кнопка, нога контроллера подтянута к питанию ITC_SetSoftwarePriority(ITC_IRQ_PORTD, ITC_PRIORITYLEVEL_1); EXTI_SetExtIntSensitivity(EXTI_PORT_GPIOD, EXTI_SENSITIVITY_FALL); // кнопка замыкает на минус enableInterrupts(); // while(1); // для чистоты эксперимента, чтобы программа не шла дальше. Из комментариев все ясно. Брейкпоинты почему-то в STVD не ставятся... Прерывание не срабатывает. Вместе с тем есть еще одна странность - даже если отключить режим прерывания у пина, если вообще убрать текст обработчика (и сам обработчик), то как только вызвать enableInterrupts(); экран перестает работать. Как будто какое-то прерывание постоянно вызывается, что нарушает логику обмена информацией контроллером дисплея. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
jcxz 172 13 августа, 2018 Опубликовано 13 августа, 2018 · Жалоба Из комментариев все ясно. Брейкпоинты почему-то в STVD не ставятся... Может не стоит мучаться и взять IAR? Там всё работает. Он бесплатный для Вашего МК. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
Harbinger 10 13 августа, 2018 Опубликовано 13 августа, 2018 · Жалоба Не видя всего проекта, сложно что-то сказать. Работал только со связкой STVD+Raisonance, проблем вроде не было... более того, отладчик STVD позволяет читать-писать переменные "на лету" ("read/write on the fly"). Но в свете последних веяний от резонанса придётся отказаться. Может не стоит мучаться и взять IAR? Там всё работает. Он бесплатный для Вашего МК.А других альтернатив (с полноценной отладкой) в общем-то и нет. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
zheka 1 13 августа, 2018 Опубликовано 13 августа, 2018 (изменено) · Жалоба Бесплатнее некуда The evaluation license is completely free of charge and allows you to try the integrated development environment and evaluate its efficiency and ease of use. When you start the product for the first time, you will be asked to register to get your evaluation license. After installation, you have the following evaluation options to choose from: •a 30-day time-limited but fully functional license •a size-limited Kickstart license without any time limit Restrictions to the 30-day time-limited evaluation •A 30-day time limitation. •Source code for runtime libraries is not included. •No support for MISRA C. •Limited technical support. •Must not be used for product development or any other kind of commercial use. Restrictions to the Kickstart, size-limited evaluation •A 8 Kbyte code size limitation. •Source code for runtime libraries is not included. •No support for MISRA C. •Limited technical support. Информация с сайта iar Но это не беда. Мне предложили на выбор 30 дневное ограничение либо 8кб ограничение по коду. Я выбрал второе. Изменено 13 августа, 2018 пользователем zheka Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
Harbinger 10 13 августа, 2018 Опубликовано 13 августа, 2018 · Жалоба Второе и предлагалось. Cosmic на новом компе так и не получил лицензию - запрос проигнорировали, возможно, из-за того, что повторно на ту же почту. Raisonance за 32-килобайтную лицензию, бывшую когда-то бесплатной (аж на год!), дерёт 700 с чем-то евро, бесплатных вариантов не осталось вообще. Прикручивать sdcc к эклипсу или кодеблоксу - накладно по времени и результат под большим сомнением... Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
zheka 1 13 августа, 2018 Опубликовано 13 августа, 2018 (изменено) · Жалоба Н-да.. вы были правы. В IAR все работает. Продолжаю мучить контроллер. По нажатию одной из кнопок вхожу в режим halt. Другой кнопкой хочу выводить из этого режима. Настроено прерывание по спаду на эту кнопку. В принципе все работает - кнопку нажал, контроллер заснул. Нажал другую - контроллер пронулся и пошел дальше работать. Добавил я в цикл основную задачу - измерение напряжения и вывод его на экран. И вот тут-то пошли проблемы - контроллер по нажатию на вторую кнопку просыпается, но зависает в функции работы с АЦП в виделенной жирным строке: // ------------ ADC1 ------------------- uint16_t GetADCvalue(uint8_t ChanelNumb) { uint16_t tmphvalue; uint16_t tmplvalue; ADC1->CSR = ChanelNumb; // channel delay(10); ADC1->CR1 |= 0x61; while(!(ADC1->CSR & ADC1_CSR_EOC)){;} ADC1_ClearITPendingBit(ADC1_IT_EOC); tmplvalue = ADC1->DRL; tmphvalue = (uint16_t)ADC1->DRH << 8; tmphvalue = tmphvalue + tmplvalue; return (tmphvalue); } INTERRUPT_HANDLER(EXTI_PORTD_IRQHandler, 6) { /* In order to detect unexpected events during development, it is recommended to set a breakpoint on the following instruction. */ w=0; } void main(void) { CLK_SYSCLKConfig(CLK_PRESCALER_CPUDIV1); CLK_SYSCLKConfig(CLK_PRESCALER_HSIDIV1); // set 16 MHz for CPU GPIO_DeInit(BACKLIGHT_PORT); GPIO_Init(BACKLIGHT_PORT, BACKLIGHT, GPIO_MODE_OUT_PP_LOW_FAST); GPIO_WriteHigh(BACKLIGHT_PORT, BACKLIGHT); GPIO_Init(PWR_BTN_PORT, PWR_BTN, GPIO_MODE_IN_PU_NO_IT); //EXTI_SetExtIntSensitivity(EXTI_PORT_GPIOA, EXTI_SENSITIVITY_FALL_ONLY); GPIO_Init(MENU_BTN_PORT, MENU_BTN, GPIO_MODE_IN_PU_IT); EXTI_SetExtIntSensitivity(EXTI_PORT_GPIOD, EXTI_SENSITIVITY_FALL_ONLY); GPIO_DeInit(IO_PORT); GPIO_Init(IO_PORT, STB|SCLK|DIO|TM1722_POWER, GPIO_MODE_OUT_PP_LOW_FAST); GPIO_WriteHigh(BACKLIGHT_PORT, BACKLIGHT); GPIO_WriteHigh(IO_PORT, TM1722_POWER); ADC1_Config(); enableInterrupts(); LCD_Clear(0); while(1) { adc=GetADCvalue(4); ADC_Voltage=adc*(5.0/1024)*20; LCD_Show_Voltage(ADC_Voltage); if (GPIO_ReadInputPin(GPIOA,GPIO_PIN_2)) { LCD_Show_Spark(0); } else { LCD_Show_Spark(1); halt(); } } Заметьте, я вхожу в halt аккуратно - не по прерыванию, а по чтению кнопки после того, как АЦП завершил работу. Сам АЦП инициализирован так / ------------ ADC1 ------------------- void ADC1_Config(void) { GPIO_Init(GPIOD,GPIO_PIN_3,GPIO_MODE_IN_FL_NO_IT); ADC1->CR1 = 0x61; // 0b01100001; // enable ADC ADC1->CR2 = 0x08; // 0b00001000; // right otygnment ADC1->CR3 = 0x00; // 0; // data bufer disable ADC1->TDRL = 0xC0; // 0b11000000; // disable Schmitt triggers for AIN7 & AIN6 } Я решил эту проблему просто - перед halt() выключаю АЦП, после просыпания тут же включаю. Но что-то мне это кажется полумерой. Почему присходит сбой, так и должно быть? Изменено 13 августа, 2018 пользователем zheka Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
jcxz 172 13 августа, 2018 Опубликовано 13 августа, 2018 · Жалоба Почему присходит сбой, так и должно быть? Вы бы, батенька, даташит хотя-б открыли. Да почитали как входить в halt, как выходить, что должно быть выключено во время halt-а, как включать после..... Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
zheka 1 14 августа, 2018 Опубликовано 14 августа, 2018 · Жалоба Скачал полное руководство - RM0016 Описание halt занимает полстранички пункт 10.2.2, стр.103. Ценной информации там я не нашел, может покажете другой мануал? Формально заход в halt и выход из него по нажатию одной и той же кнопки я осилил. Теперь борюсь за потребление. Итак. Работающее устройство потребляет 55 мА (контроллер+подсветка, плюс драйвер LCD) Последовал совету, который гласил, что нужно переводить все болтающиеся ноги на вывод. Получил 200 мА )))) Почему? Питание драйвера LCD китайская схема осуществляет через PC4. Нет, эта ножка не включает какой-нибудь ключ, она просто питает микросхему. И когда я микросхему отключил - потребление парадоксально возросо до 200 мА. Причиной тому была утечка через DIO и SCK в микросхему скорее всего. Потому как перевод пинов, подающих команды, в режим входа с подтяжкой снизил общее потребление до 40 мА. Но, к сожалению, оно таким и остается. Я попробовал поставить halt сразу после void main(void) {.... получил 35 мА. Руки конечно потянулись, согласно советам в различных статьях, поиграться с тактовым генератором, но ведь у меня режим, в котором останавливается все.... Да... чтобы понять, к чему стремиться, прежде чем вытереть старую китайскую прошивку, я измерил, сколько потребляет устройство в спящем режиме - 12 мкА. Китайцы что-то знают, чего не знаю я. Скажите мне, если я вызываю halt() сразу же, в начале void main(void) - каков должен быть ток потребления? И нужно ли что-то отключать, если по умолчанию все уже выключено, а если и включено, то при остановленном тактовом генераторе, по идее не должно жрать. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
zheka 1 18 августа, 2018 Опубликовано 18 августа, 2018 (изменено) · Жалоба В общем я взял обычную китайскую отладочную плату, запитал ее не через имеющийся LDO, а непоредственно от разъема программирования. Сам LDO отпаял, потому как в этм случае ток куда-то в него уходит, потребляется то бишь. Простым вызовом halt() сразу после void main (void) { у меня получилось 35 мкА. манипуляции переводом в нужное состояние портов, выключений всего что нужно, удалось снизить потребление до 11 мкА. Иными словами, все пляски с бубнами, описанные даташиты, нужны там, где речь идет о борьбе за единицы микроампер. Возвращаемся к моей китайской плате - она в режиме Halt жрет 21 мА. Если же отключить ее от программатора - 50 мА. То есть проблема в схемотехнике. Но на плате ошибок нет, потому как, повторюсь, с исходной прошивкой она жрала те же 11 мкА. То есть китайская прошивка прежде чем перевести в halt, как то мотивирует обвязку контроллера на голодание. Господа не обессудьте, выкладываю то что набросал на листе бумаги. Понятно, что шлак, я не требую подсказать решение по этой схеме. Если дойдет до того, что в схеме дейтвительно кто-то "ворует электричество", а из-за ее качества непонятно, тогда нарисую в КАДе и выложу. А сейчас выкладываю этот шлак на случай, если косяк лежит на поверхности и его не оставит труда заметить даже на такой схеме Трехногая микруха слева - преоразователь питания на 3.3 вольт. Транзисторный ключ справа - управляет подсветкой. Неизвестный эемент вверху. левая нога которого идет к двум коненсаторам и земле - похожа на чип индуктивность. Микруха справа - драйвер дисплея. Питается с ноги PC4 14 пина (он отключен, если что). Нижняя кнопка - она-то и включает и выключает устройство. Может мне кто объяснит назначение МОСФЕТа внизу? Если вы обратите внимание, то с его истока проводник идет к LDO, питающему контроллер. То есть МОСФЕТ должен быть открыт. И открывается он внешним питанием, через резистор. Но видите ли, в этом случае эту функцию транзистора с успехом заменил бы обычный проводник. Но там еще два диода, развязывающие его от остальных частей схемы. Кстати, я дергал PD3 и в ноль и в плюс - управляя его затвором - эффекта не возымело. Будут мысли у кого-нибудь? Изменено 18 августа, 2018 пользователем zheka Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться