Модифицируем народный линейный драйвер для тактовой кнопки

Corvax · 19.11.2015, 18:25

Цитата:

Сообщение от Tamagotchi :

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

Может выложите какой-нибудь из своих вариантов в исходных текстах? С минимумом наворотов.

Tamagotchi · 19.11.2015, 18:29

Corvax, Моих линейников с исходниками достаточно выложено.

Corvax · 19.11.2015, 19:03

Все с силовой кнопкой и только под 13А, или я пропустил. Конечно, Вам решать, но лично я был бы признателен за грамотно написанный костяк прошивки для тактовой кнопки под 85.

fnksb · 19.11.2015, 19:11

Цитата:

Сообщение от Tamagotchi :

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

А что с ним не так?

Похожей прошивкой под 13-ю от Alex63 пользуюсь уже полгода, пока что все нравится.

Tamagotchi · 19.11.2015, 20:19

Цитата:

Сообщение от fnksb :

А что с ним не так?

Я не вникал - просто промотал и что бросилось в глаза:

Это для наглядности

Нажмите на изображение для увеличения
Название: Untitled-1.jpg
Просмотров: 591
Размер: 50.7 Кб
ID: 153265

В старшем байте у нас 8 значимых бит и 2 в младшем, вот как они в этой прошивке обрабатываются:

Код:

unsigned char av0;      // Младший разряд измерения напряжения                                       
unsigned char av1;      // Старший разряд изменения напряжения                                                      
unsigned char adch_val; // Напряжение акка

av0=ADCL;
av1= ADCH;  

av1= av1<<1;            // Сдвинули влево на 1, последний 0
av0=av0>>7;			    // Сдвинули вправо на 7, остался мл разряд
adch_val = av1+av0; 	// По идее получили 8 разр число = код/2

берется старший байт и сдвигается влево... мы сразу теряем старший бит... т.е. мы теперь не можем отличить 255 от 127... все, что больше 127 мы не можем опознать: было записано 129 - сдвинули влево и у нас теперь 2. Дальше значение младшего байта сдвигается вправо 7 раз и мы теряем один бит из двух - младший т.е. вместо от 0 до 3 мы имеем только или 0 или 1. Затем эти два байта складываются. Зачем? Плюс почему автор складывает и байта и значение пишет в переменную размером 1 байт? Оно же вполне может просто не поместится (хотя тут эта ситуация невероятна т.к. мы уже 1 бит пропотеряли) и это потеря еще 1 старшего бита.

Дальше - в программе при переключении опоры нигде не пропускается ни одно значение, не выжидается необходимое значение тактов чтобы в АЦП все "устаканилось" (это требование есть в даташите), а она переключается при каждом замере.

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

Tamagotchi · 19.11.2015, 20:37

В общем решение простое. ADLAR обычно ставят, чтобы использовать только 8 бит АЦП из 10. Просто используем ADCH и не берем во внимание ADCL и все. У меня так в народном линейнике сделано, но это обусловлено жесткой нехваткой памяти. А тут памяти море - можно и все 10 бит юзять: uint16_t adch_val = ADCW; ВСЕ!!!

fnksb · 19.11.2015, 20:39

Цитата:

Сообщение от Tamagotchi :

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

Хмм, если могут улететь пороги по температуре и напряжению, то неприятно...

Tamagotchi · 19.11.2015, 20:42

Цитата:

Сообщение от fnksb :

если могут улететь пороги по температуре и напряжению,

Они правильно (более-менее) будут (без учета пропуска тактов - только обработка) только в узком диапазоне значений. Но это легко правится, а вот правильную работу с АЦП сюда сложнее вставить.

Alex63 · 22.11.2015, 07:48

Цитата:

Сообщение от Tamagotchi :

мы сразу теряем старший бит... т.е. мы теперь не можем отличить 255 от 127... все, что больше 127 мы не можем опознать: было записано 129 - сдвинули влево и у нас теперь 2. Дальше значение младшего байта сдвигается вправо 7 раз и мы теряем один бит из двух - младший т.е. вместо от 0 до 3 мы имеем только или 0 или 1. Затем эти два байта складываются. Зачем? Плюс почему автор складывает и байта и значение пишет в переменную размером 1 байт? Оно же вполне может просто не поместится (хотя тут эта ситуация невероятна т.к. мы уже 1 бит пропотеряли) и это потеря еще 1 старшего бита.

Если чуть внимательнее посмотреть, то мы убираем старший (принят за 0) и младший бит (шум) из 10 битов ( т е из 1024, а вовсе не из 255) и используем все остальные.
При этом мы "теряем" корректность измерения температуры выше 512 - 273 = 239 гр цельсия и напряжения меньше 1,1*2 = 2,2 в. Во всем остальном

Цитата:

Сообщение от Tamagotchi :

узком диапазоне значений

измерения корректны. Вам этого диапазона мало в фонаре?
Мне хватает, сорри за неиспользование всех возможностей.

Цитата:

Сообщение от Tamagotchi :

Дальше - в программе при переключении опоры нигде не пропускается ни одно значение, не выжидается необходимое значение тактов чтобы в АЦП все "устаканилось" (это требование есть в даташите), а она переключается при каждом замере.

Справедливо, но вот уж это сделать элементарно. Не понимаю, почему такому гуру

Цитата:

Сообщение от Tamagotchi :

правильную работу с АЦП сюда сложнее вставить

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

Alex63 · 22.11.2015, 09:05

Народ, кто имеет опыт работы с МК, глупый вопрос (даташит как то по разному можно перевести, например, при первом измерении оно длиннее на 13 тактов - может, это и надо для устаканивания переключения опорного), что лучше сделать для большей корректности измерений АЦП
- выкидывать результат первого измерения после смены опорного напряжения, и использовать только результат второго измерения
- вставить задержку на 14 тактов между сменой опорного и самим измерением

Tamagotchi · 22.11.2015, 10:08

Цитата:

Сообщение от Alex63 :

Если чуть внимательнее посмотреть, то мы убираем старший (принят за 0) и младший бит (шум) из 10 битов ( т е из 1024, а вовсе не из 255) и используем все остальные.

Я внимательно смотрел - не правильно там сделано. Читаем Ваш же комментарий: "По идее получили 8 разр число = код/2" - Да нифига не получаем. Теряем старший бит - это уже 50% потерь в старшем байте, потом еще зачем-то прибавляем остатки младшего вместо того, чтобы дописать их справа. Ради чего это сделано? Оъяснение этому есть? Ситуация напоминает анекдот: "Если отчет не сходится - в нем ошибка, а если сходится - две."

Цитата:

Сообщение от Alex63 :

Народ, кто имеет опыт работы с МК

Оставьте старший байт целиком - не надо его никуда двигать, а младший игнорируйте. IMHO это простейший способ получить правильную работу с достаточной точностью и минимальными переделками.

ЗЫЖ Сорри, если я слишком жестко написал.

Alex63 · 22.11.2015, 10:49

Цитата:

Сообщение от Tamagotchi :

Теряем старший бит - это уже 50% потерь в старшем байте, потом еще зачем-то прибавляем остатки младшего вместо того, чтобы дописать их справа. Ради чего это сделано? Оъяснение этому есть? Ситуация напоминает анекдот: "Если отчет не сходится - в нем ошибка, а если сходится - две."

Уже говорил - при измерении в 1024 старший бит что в температуре, что в напряжении 0 во всем диапазоне реальных значений. Дальше как раз и дописываем старший бит младшего байта на последний бит (0 после сдвига) значения.
Что то я не пойму, много шума из ничего, т е криков "все пропало, шеф!" а что пропало то? Где что потерялось, если без истерики?
Сдвинули влево старший байт, последний 0, сдвинули вправо младший байт, последний стал старший бит, сложили, получили 10 битовое число /2 округленное до 1 в 8 значном представлении. Или есть только ваш единственный правильный алгоритм, остальное - бред?

Цитата:

Сообщение от Tamagotchi :

Оставьте старший байт целиком - не надо его никуда двигать, а младший игнорируйте. IMHO это простейший способ получить правильную работу с достаточной точностью и минимальными переделками.

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

Tamagotchi · 22.11.2015, 10:54

Alex63, Я останусь при своем мнении. Зачем все это безобразие сделано Вы объяснить просто не можете.

Alex63 · 22.11.2015, 13:06

Цитата:

Сообщение от Tamagotchi :

Зачем все это безобразие сделано Вы объяснить просто не можете.

В предыдущем посте пояснял, зачем. Повысить точность с 8 до 9 разрядов, поскольку старший бит всегда 0 во всем диапазоне реальных значений измеряемых величин.
Если не хотите утруждаться и разобраться/понять что и зачем, то я точно не собираюсь тратить свое время чтобы как школьнику объяснять простые алгоритмы. Видите ошибку - покажите где конкретно. Имеете просто мнение без аргументов - оставайтесь при нем ради бога.

Про опыт с МК я спрашивал что лучше - пропустить измерение или несколько тактов добавить. Вот это конструктив бы был, а то что попало.

Tamagotchi · 22.11.2015, 13:10

Цитата:

Сообщение от Alex63 :

я точно не собираюсь тратить свое время чтобы как школьнику объяснять субъективность

Давай не будем забывать, на основе чьих исходников ты написал что-то свое. На основе, чьих исходников и Керамик написал прошивку, которую ты взял за основу и как ты буквально два дня назад пытался еще дополнительно от импульсника прошивку выпросить.

Если научился не сильно вникая дергать куски кода из чужих программ, то зазнаваться (как и предлагать платные услуги) еще мягко говоря рано.

Alex63 · 22.11.2015, 14:05

Цитата:

Сообщение от Tamagotchi :

Давай не будем забывать, на основе чьих исходников ты написал что-то свое. На основе, чьих исходников и Керамик написал прошивку, которую ты взял за основу и как ты буквально два дня назад пытался еще дополнительно от импульсника прошивку выпросить.

Справедливо, за основу взял прошивку Керамика, а он вашу. Насчет выпросить - спросил как у коллеги, а что, это криминал? Извините, был неправ, больше не повторится.

Цитата:

Сообщение от Tamagotchi :

Если научился не сильно вникая дергать куски кода из чужих программ, то зазнаваться (как и предлагать платные услуги) еще мягко говоря рано.

Дмитрий, а может не будем переходить на личности? Я скорее токарь, чем программист. Все детали для своих разработок точил/фрезеровал сам, да и программу для 2х диодов писал сам, поскольку вы же говорили про 2000 рулей / час за разработку программы для МК и дикого количества часов для простых адаптаций?

И в чем зазнаваться? Что хочу услышать аргументы?

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

Tamagotchi · 22.11.2015, 14:09

Цитата:

Сообщение от Alex63 :

Что не лизнул гуру когда он высказал неаргументированное мнение

Alex63, Не смеши меня, я все четко и побитно рассказал и написал, что в узком диапазоне значений это будет работать. А ты наивно считаешь свой код делением на 2

(деление на 2 это сдвиг вправо, а не влево) и даже не можешь объяснить зачем сделал именно так, а не не по нормальному, причем сделать не теряя половину данных даже проще.

Tamagotchi · 22.11.2015, 14:13

В общем больше никому никаких исходников я не дам и советов тоже не дам - пишите любую ересь и радуйтесь, что в программе 2 ошибки.

Цитата:

Сообщение от Alex63 :

как школьнику объяснять простые алгоритмы

напиши САМ что-то с 0, а потом поговорим. Давай - жду прошивку к импульснику, на СВОЕМ регуляторе, а не слизанном.

Alex63 · 22.11.2015, 16:19

Цитата:

Сообщение от Tamagotchi :

В общем больше никому никаких исходников я не дам и советов тоже не дам - пишите любую ересь и радуйтесь, что в программе 2 ошибки.

Цитата:
Сообщение от Alex63 :
как школьнику объяснять простые алгоритмы

напиши САМ что-то с 0, а потом поговорим. Давай - жду прошивку к импульснику, на СВОЕМ регуляторе, а не слизанном.

Спасибо на добром слове. И за неизвестные мне советы (не помню ни одного конструктивного).
Понимаю, время такое, всем трудно.
В общем, без подколов, здоровья и удачи, Дмитрий.

Tamagotchi · 22.11.2015, 16:22

Цитата:

Сообщение от Alex63 :

не помню ни одного конструктивного

19.11.2015, 18:29	362
Tamagotchi Регистрация: 07.09.2011 Последняя активность: 04.08.2023 18:59 Адрес: Волгоградская обл. Сообщений: 13684 Сказал(а) спасибо: 3333 Поблагодарили: 7887 раз(а) в 3523 сообщениях	Re: Модифицируем народный линейный драйвер для тактовой кнопки Corvax, Моих линейников с исходниками достаточно выложено. __________________ Всегда в наличии надежные драйверы для SKILHUNT H02, Н03 и H04. Группа в ВК, посвященная кастомизации фонарей.

19.11.2015, 19:03	363
Corvax Ветеран Фонарёвки Регистрация: 31.07.2013 Последняя активность: 04.04.2024 09:50 Адрес: Великий Новгород Сообщений: 2741 Сказал(а) спасибо: 1208 Поблагодарили: 597 раз(а) в 505 сообщениях	Re: Модифицируем народный линейный драйвер для тактовой кнопки Все с силовой кнопкой и только под 13А, или я пропустил. Конечно, Вам решать, но лично я был бы признателен за грамотно написанный костяк прошивки для тактовой кнопки под 85.

19.11.2015, 20:37	366
Tamagotchi Регистрация: 07.09.2011 Последняя активность: 04.08.2023 18:59 Адрес: Волгоградская обл. Сообщений: 13684 Сказал(а) спасибо: 3333 Поблагодарили: 7887 раз(а) в 3523 сообщениях	Re: Модифицируем народный линейный драйвер для тактовой кнопки В общем решение простое. ADLAR обычно ставят, чтобы использовать только 8 бит АЦП из 10. Просто используем ADCH и не берем во внимание ADCL и все. У меня так в народном линейнике сделано, но это обусловлено жесткой нехваткой памяти. А тут памяти море - можно и все 10 бит юзять: uint16_t adch_val = ADCW; ВСЕ!!! [Исправлено: Tamagotchi, 19.11.2015 в 20:38]

22.11.2015, 09:05	370
Alex63 Увлеченный Регистрация: 16.08.2012 Последняя активность: 19.01.2018 08:37 Адрес: Новосибирск Сообщений: 475 Сказал(а) спасибо: 44 Поблагодарили: 270 раз(а) в 117 сообщениях	Re: Модифицируем народный линейный драйвер для тактовой кнопки Народ, кто имеет опыт работы с МК, глупый вопрос (даташит как то по разному можно перевести, например, при первом измерении оно длиннее на 13 тактов - может, это и надо для устаканивания переключения опорного), что лучше сделать для большей корректности измерений АЦП - выкидывать результат первого измерения после смены опорного напряжения, и использовать только результат второго измерения - вставить задержку на 14 тактов между сменой опорного и самим измерением [Исправлено: Alex63, 22.11.2015 в 09:10]

22.11.2015, 10:54	373
Tamagotchi Регистрация: 07.09.2011 Последняя активность: 04.08.2023 18:59 Адрес: Волгоградская обл. Сообщений: 13684 Сказал(а) спасибо: 3333 Поблагодарили: 7887 раз(а) в 3523 сообщениях	Re: Модифицируем народный линейный драйвер для тактовой кнопки Alex63, Я останусь при своем мнении. Зачем все это безобразие сделано Вы объяснить просто не можете.