caxapa.ru :: Перечитал ветку еще раз - мы говорим о разном и друг друга не слышим.

Cкpипaч, пророк (23.06.2024 07:39, просмотров: 106) ответил VladislavS. на А почему бы не экономить, особенно если это бесплатно делает компилятор? И не преждевременно, а всегда и без устали.

Перечитал ветку еще раз - мы говорим о разном и друг друга не слышим.

Вы написали тонкую оптимизацию битовых операций - ок. Пусть.

Но необходимости это делать нет.

То что говорю я - операции с регистрами должны быть вынесенны из прикладного кода.

Пресловутый STRANGE_LED_ON() это не библиотека работы с регистрами как таковыми, это функция именно этого, прикладного светика.

Хотите оптимизаций - внутри ее можете писать код с помощью своей волшебной библиотеки, но обычные сишные "и-или-не" над битами внутри нее вполне уместны.

Просто разбавить прикладной код битовой арифметикой, обернутой в ~~макросы~~ шаблоны - неправильный путь.

... но не любой ценой.

Ответить

- Вы противоречите своему примеру идеалного метода включения диода в который поместили работу с битами и регистрами конкретного контроллера. Вы этот метод будете переписывать раз за разом, а я нет. Битовая арифметика с регистрами на каком-то уровне всё равно появится, но этот уровень ниже чем у вас. Gpio, spi, i2c, usb - это всё должно быть прослойкой абстракциией, позволяющей писать прикладной уровень не думая для какого контроллера ты пишешь. Написана эта абстракция VladislavS.(97 знак., 23.06.2024 08:28 - 09:05)
  - Я - буду. Поменяю имя регистра или номер бита, если нужно. А как вы получили гарантию что не регистр, не не номер бита никогда не изменится в вашем случае? - Cкpипaч(23.06.2024 08:41)
    - Гарантия одна, в методе включения светодиода не использовать ни имя регистра, ни номер бита. Использовать абстракцию, я же приводил код метода LED::On(); VladislavS.(123 знак., 23.06.2024 09:03)
      - У вас есть неявное предположение что все LED гарантировано ведут себя одинаково. Я в таком случае введу прикладной номер LED, а то какой номер, к какому биту какого регистра относится - спрячу внутри модуля bsp. Cкpипaч(236 знак., 23.06.2024 09:12)
        
        Никак нет, проектирование начинается с создания кирпичиков из которых строится дом. Панельный дом собирается куда быстрее. Как нумерация или название связано с битами показано прямо в сообщении на которое вы отвечаете. И да все светодиоды ведут себя одинаково - погашены, горят (с разной яркостью) или мигают. Пишете их поведение один раз и больше туда не лазите. - VladislavS.(23.06.2024 09:23)
        
        У Nuvoton есть процы с глубокой спячкой и локом GPIO при проходе через ресет при выходе из спячки. Так что не всегда при переходе на вектор ресет имеем дефолтное состояние периферии. Деинит играет новыми красками. И даже сам вопрос необходимости (или нет) запускать инициализацию может повергнуть в уныние кишки самого кода инициализации. Особенно если ретеншн есть на совсем небольшую часть ОЗУ, или нужен более мелкожручий режим без ретеншна, т.е. даже сохранять стек нет смысла Vit(1 знак., 23.06.2024 09:59)
        
        Светодиод должен обо всём этом знать? - VladislavS.(23.06.2024 10:08)
        
        Если будете писать конструктор, то должны будете знать. Светодиод это рафинированный пример обернутого битового обращения от вас же. Представьте себе, что это выключатель питания GSM-модуля. Всё тот же GPIO, но с полной, а не с ограниченной отвественностью:) - Vit(23.06.2024 10:21)
        
        Ну если это выключатель GSM-модуля, то у него и логика будет не от светодиода. А метод включения будет всё равно GSM::On(). И дёргать ногой он будет точно так же через пин (gpio или расширителя - пофиг) как и светодиод не думая об его портах и битах, ибо это не его зона абстракции. - VladislavS.(23.06.2024 10:42)
        
        у него вводные условия появляются - прежде чем собственно поднять ногу нужно осмотреться, а доступна ли она, а не получится ли вдруг вступить в кучу:) - Vit(23.06.2024 10:48)
        
        Всегда есть космический крейсер, кореллийские лучи смерти, галактическая чума, угрожающая погубить всю планету... И чтобы обыватели не сошли с ума, они ничего не должны об этом знать! (c) MiB - VladislavS.(23.06.2024 11:05)
        
        toggle выполняет действие со скрытым от юзера предыдущим состоянием. иногда не пофигу результат. ЗЫ. если у класса GSM и ключ, и светодиод, то и ключу нужен класс GPIO, и классу LED. вывод: класс LED сам по себе не нужен:) - Vit(23.06.2024 11:38)
        
        Кто-то запрещает прочитать состояние, если это надо для работы модуля? Класс LED нужен, иначе вы описываете логику его работы в классе GSM, тем самым усложняя его и делая менее переносимым. - VladislavS.(23.06.2024 11:54)
        
        насчет запрещает - не совсем так, но почти, потому как глобальные разрешения (снятие лока GPIO) это вопрос не одного класса. а насчет класса LED - он по сути алиас для части GPIO. такое при построении сверху вниз чаще не нужно как класс, ибо лишний уровень абстракции. но в отдельной программе-тесте приходится затягивать из кода приложений, и по сути ручками делается тот же класс. это хоть и ручками, но не дольше, чем наоборот. - Vit(23.06.2024 13:02)
        
        Даже одна возможность того что светодиод может включаться 1 или 0 уже отличает его от простой ноги. Вот захардкодите вы дёргание ноги, а завтра схему поменяют... - VladislavS.(23.06.2024 13:17)
        
        Не стесняюсь писать Up и Down, а в On и Off указывать вызовы, соответствующие полярности. Бывало с рулением полярностью, но как показала практика - скорее правится плата и появляется BSP для неё в виде набора файлов, чем калечится предыдущий код путём вставок ветвления. В приложении нужны On и Off, при миграции код приложения остаётся. - Vit(23.06.2024 17:10)
        
        Никаких ветвлений в рантайме нет. Инверсия - константный пареметр. Он на этапе компиляции работает. - VladislavS.(23.06.2024 17:26)
        
        Щщазз! Инверсия получается после чтения входа этого же gpio. В качестве примера посмотрите strapping для любого ethernet phy как там подключаются индикаторные светодиоды. - 3m(25.06.2024 17:49)
        
        Внимательно следим за руками. Определяю два светодиода - один с прямым включением, второй с инверсным. Делаем LED::On() обоим и смотрим что получается. VladislavS.(119 знак., 26.06.2024 15:31, картинка, картинка)
        
        даже на Си, если не на макросах, то если полярность указывается константно, обычно мелкое инлайнится до отсутствия ветвления самим компилятором - Vit(23.06.2024 17:41)
        
        Да не даже на С, а это любой компилятор должен константные данные оптимизировать. Если ему руки всякими "отлаженными модулями" не выкручивать. - VladislavS.(23.06.2024 18:21)
        
        насчет "любой" и "должен" я бы не был столь категоричен. хотя да, это теперь скорее норма. только не говорите это мелкочипам - они даже GCC так кастрировали, что это обобщение не к ним:) ЗЫ я уверен, что вам не досталось счастья использовать "компилятор" MPLAB C16. оно могло один оператор в строке и никаких структур... - Vit(23.06.2024 19:21)
        
        Возможно мне повезло, но моё мнение по выбору чипов всегда учитывали. А я в свою очередь всегда учитывал наличие вменяемых средств разработки. - VladislavS.(23.06.2024 19:39)
        
        это прям тост. категорически поддерживаю. Vit(396 знак., 23.06.2024 20:04)
        
        За более чем тридцать лет в бизнесе, не припомню ни одного договора, в котором бы оговаривался размер кода в байтах :) - Cкpипaч(23.06.2024 18:24)
        
        Зато я помню проекты, где сотни каналов АФАР нужно за микросекунды перестроить. - VladislavS.(23.06.2024 18:37)
        
        такое делать без запаса по быстродействию с упором на одно программирование не комильфо. полагаю, что выбор кремния был не пальцем в небо. Vit(1032 знак., 23.06.2024 19:57)
        
        Естественно, программирование лишь часть проекта, без fpga такое не взлетает. А язык даёт выигрыш, опыт это подтверждает. Естественно, руками можно всё то же самое и на С написать. Но когда за тебя это делает компилятор работа быстрее идёт. - VladislavS.(23.06.2024 20:20)
        
        Ввиду полного отсутствия ограничений по ресурсам на стороне компилятора, какой-либо практической пользы в Си против Плюсов - не вижу. ~~Когда пишешь на Паскале, какой компилятор вообще похер.~~ - Cкpипaч(23.06.2024 20:15)
        
        Так вы, коллега, "шататель основ"?! 8) Что-ж, не вы один такой, вырывание из задачи кусков с последующим "к пуговицам претензии есть?" - очень характерно для плюсовиков, видимо язык накладывает отпечаток. - Cкpипaч(23.06.2024 09:52)
        
        Причём тут претензии? В области вашей работы всегда есть сущности, которые вы используете из проекта в проект. Вполне логично написать для них качественные классы, которые потом просто и быстро использовать. В итоге вы просто занимаетесь прикладухой, а не переписываете раз от раза метод включения светодиода. А ведь у него ещё и метод выключения есть :) - VladislavS.(23.06.2024 10:15)
        
        Вообще-то, именно так. Переписываю. Порядка трети тикетов - перенос уже отработанных прикладных алгоритмов на другой bsp (точнее, укладка нескольких алгоритмов в один bsp). Он состоит в переписывании двух десятков функций, большая часть из которых состоит из одной строки - поднять или опустить где-то какой-то бит. - Cкpипaч(23.06.2024 10:23 - 10:40)
        
        Итить колотить! Вам больше заняться нечем, как сидеть и переписывать функции? И эти люди запрещают мне ковырять в носу (с) :) - VladislavS.(23.06.2024 10:40)
        
        Лол. Это приносит деньги фирме :) Причем в человеко-часах это существенно эффективнее чем плодить классы, наследование и вот-это-вот-всё. - Cкpипaч(23.06.2024 10:41 - 10:43)
        
        Могло бы приносить больше, если бы вы занимались делом, а не переписыванием одного и того же. - VladislavS.(23.06.2024 10:43)
        
        Меньше. И не давите на меня, я - директор, я считал и сравнивал. - Cкpипaч(23.06.2024 10:45)
        
        та же фигня:) - Vit(23.06.2024 11:05)
      - В прикладном коде вам нужно будет найти инициализацию объекта LED и исправить там. А если окажется что еще и способ нужно поменять, то искать "способ" придется уже где-то в другом месте. Два места вместо одного, специально выделенного в отдельный модуль. Я это называю "объектный спагетти-код". - Cкpипaч(23.06.2024 09:04)
        
        Зачем вы свои проблемы на меня перекладываете? Беру схему, в заголовочном файле описываю (именно описываю) что куда подключено. Для светодиода это пин и наличие инверсии, для eeprom порт i2c, для синтезатора частот порт spi и пин gpio для контроля ФАПЧ и т.д. А дальше забота компилятора всё это связать. Нет никаких двух и более мест, по которым вы зачем-то рыскаете. - VladislavS.(23.06.2024 09:12)
        
        Эти идеальности не всех посещают. У меня вполне уживаются и светики на GPIO, и светики, подключенные через I2C расширители. И для этой группы устройств в базовом проекте туча таргетов для 6-и процов (один кетайский), 2 типа ядер, 3 типа периферии проца, под конкретные конфигурации плат (там есть несколько исполнений с тучей разных интерфейсов и т.п.) с выбранной индикацией (со своими разношерстными расширителями)... Сильно проще идти от прикладной задачи, имея подготовленные Vit(3 знак., 23.06.2024 09:30)
        
        У вас же расширитель имеет интерфейс обычного порта и предоставляет доступ к пинам так же как gpio контроллера? И повесить светодиод на пин расширителя можно так же как на пин контроллера? - VladislavS.(23.06.2024 09:49)
        
        у меня расширители PCF8574/A и PCA9534/A с жесткой адресацией на шине. естественно стоит автодедект. но в зависимости от варианта аппаратной сборки подключается разный набор светодиодов (да и некоторых ключиков питания). причем сами эти расширители имеют внутри разную физически конфигурацию выводов - в PCF8574/A неотключаемая подтяжка, а в PCA9534/A включаемая. для управления ключиками это важно. светодиоды всегда можно погасить и тут же зажечь как нужно (вспомная анекдот о Vit(114 знак., 23.06.2024 10:16)
        
        Сам бог велел спрятать всё это непотребство в класс расширителя и выдать светодиоду интерфейс пина к которому он подключен. - VladislavS.(23.06.2024 10:26)
        
        Это впорос реализации, но не применения. У меня приложение вызывает функцию управления функциональным узлом - светодиод обмена данными включить. Под капотом оно и проверит, а инициализирован ли драйвер, если нет, то запустит инициализацию, и другие проверки, ну или просто дёрнет GPIO. Класс там у драйвера или модуль (как идентифицированная единица компиляции) с набором функций - это не суть важно. - Vit(23.06.2024 10:57)
        
        Всегда есть особенности реалищации, но не надо делать из этого винегрет. Светодиод не должен заниматься проверкой инициализации порта. Это дело самого порта. Если у вас такой хитрый порт, то и наворачивайте его логику. Оставьте светодиоду светодидово. Так код будет проше и понятней. - VladislavS.(23.06.2024 11:14)
        
        При чём тут винегрет? Да, есть драйвер I2C_GPIO, он сам себя инициализирует, если до того не вызывали. Да, оно под капотом и кишки наружу не торчат. Но функция включения светодиода в приложении вызывает ручками вписанную (назначенную) функцию от этого драйвера или от CPU_GPIO (драйвер, как ни странно, тоже бывает нужен). И она действительно чихать хотела как оно реализовано. В CPU_GPIO может быть bit banding, работа с Set/Reset регистрами, может быть чтение-модификация запись Vit(890 знак., 23.06.2024 12:44)
        
        Вы всё красиво и правильно расписываете. Я лишь за то чтобы код gsm-модуля ничего об этом не знал. - VladislavS.(23.06.2024 13:10)
        
        :) я ж вроде как за то же. - Vit(23.06.2024 17:13)
        
        Бугага! (только пожалуйста не обижайтесь, это не переход на личности) А как насчет необходимости по-быстренькому подменить пару входов заглушками, потому что в целевом bsp нет свободных? 8) Тоже броситесь прикладной алгоритм переписывать? У вас это дешевле? :) Cкpипaч(176 знак., 23.06.2024 11:25 - 11:38)
        
        Когда вы принимаете решение использовать чужую библиотеку на развитие которой не можете влиять, то приниманте на себя все риски. Какая разница как она написана? Даже если вляпались в какашку, кто-то держит пистолет у виска с требованием обновить её? - VladislavS.(23.06.2024 12:26)
        
        Мы живем не в стране розовых пони, вы забыли? :) - Cкpипaч(23.06.2024 12:29)
        
        В голове должно что-то срабатывать задолго до момента когда вы решаетесь править чужие библиотеки. - VladislavS.(23.06.2024 12:42)
        
        Вы меня с кем-то перепутали. - Cкpипaч(23.06.2024 12:45)
        
        Не я предлагал gsm-библиотеку править. - VladislavS.(23.06.2024 12:51)
        
        И не я. Cкpипaч(86 знак., 23.06.2024 12:53)
        
        Значит показалось. А как вы вообще библиотеки делаете, если там постоянно в функциях что-то переписывать приходится? Это то мне хоть не показалось? - VladislavS.(23.06.2024 12:54)
        
        Нужно не путать библиотеки и модули. Есть собственно программа, состоящая из дерева модулей и есть используемые библиотеки. Если заказчику понадобилась хоть немного иная программа - дерево модулей копируется в отдельный, новый проект. А библиотеки - нет. Cкpипaч(644 знак., 23.06.2024 13:29)
        
        Можно поподробней о разнице между библиотекой и модулем? И то и другое переиспользуется в разных проектах, в чём разница то? - VladislavS.(23.06.2024 14:00)
        
        Django это библиотека, а алгоритм регулирования давления двумя насосами на одном частотнике - модуль :) Cкpипaч(529 знак., 23.06.2024 14:15 - 14:34)
        
        Библиотека это цивилизованное переиспользование кода, а модуль тупая копипаста из проекта в проект? Чёт сомнительно. - VladislavS.(23.06.2024 14:23)
        
        Несовсем копипаста, CVS-репозитарий никто не отменял, но в принципе - да. Самый короткий путь - прямой. - Cкpипaч(23.06.2024 14:29)
        
        Я не знаю что такое "в bsp нет свободных". У заглушки в любом случае должен быть штатный интерфейс и вы его напишете. Какая разница в модуле или классе? У меня, например, есть готовый порт-пустышка. Светик будет думать что всё работает, а реально он никуда не подключен. - VladislavS.(23.06.2024 11:40)
        
        Разница огромная. Если у вас есть функция, где происходит переход от прикладного уровня к аппаратно-зависимому, то можно нужные две строчки поставить там. Без (1) введения новых классов и прочесывания прикладного кода, на предмет где подправить инициализацию, и без (2) засорения библиотек, ради частных случаев применения. - Cкpипaч(23.06.2024 11:41)
        
        Это потому что у вас спагетти-код и неправильное разделение на абстракции. - VladislavS.(23.06.2024 12:03)
        
        Вам виднее :) *Именно на этом мы и зарабатываем деньги. - Cкpипaч(23.06.2024 12:14 - 12:55)
        
        . - VladislavS.(26.06.2024 23:06)

Средства и методы разработки