Слой Абстракции Для Spi

[COKPOWEHEU]

Написал слой абстракции для мастера SPI, как программного так и аппаратного. Кому не лень посмотрите что можно исправить или доработать.

Единственное, что мне не нравится, что avr-gcc ругается warning'ом на неинициализированную переменную. Я-то знаю, что после 9 сдвигов в ней не останется ни одного родного бита, но как это объяснить компилятору?

char SPI_send(char data){
 char res,i=7;

spi.zip

[Реклама]

Реклама: ООО ТД Промэлектроника, ИНН: 6659197470, Тел: 8 (800) 1000-321

[Vascom]

Так может инициализировать переменную?

[Реклама]

20% скидка на весь каталог электронных компонентов в ТМ Электроникс!

Акция "Лето ближе - цены ниже", успей сделать выгодные покупки!

Плюс весь апрель действует скидка 10% по промокоду APREL24 + 15% кэшбэк и бесплатная доставка!

Перейти на страницу акции

Реклама: ООО ТМ ЭЛЕКТРОНИКС, ИНН: 7806548420, info@tmelectronics.ru, +7(812)4094849

[Alex]

А почему бы её не инициализировать ?

ЗЫ: Опоздал

Изменено 25 июля, 2016 пользователем Alex

[Реклама]

Организация питания на основе надежных литиевых аккумуляторов EVE и микросхем азиатского производства

Качественное и безопасное устройство, работающее от аккумулятора, должно учитывать его физические и химические свойства, профили заряда и разряда, их изменение во времени и под влиянием различных условий, таких как температура и ток нагрузки. Мы расскажем о литий-ионных аккумуляторных батареях EVE и нескольких решениях от различных китайских компаний, рекомендуемых для разработок приложений с использованием этих АКБ. Представленные в статье китайские аналоги помогут заменить продукцию западных брендов с оптимизацией цены без потери качества. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[COKPOWEHEU]

А зачем, если это значение все равно сотрется? Если не получится вразумить компилятор, так и сделаю, но неужели нет другого способа?

[Реклама]

Сравнительное тестирование аккумуляторов EVE Energy и Samsung типоразмера 18650

Инженеры КОМПЭЛ провели сравнительное тестирование аккумуляторов EVE и Samsung популярного для бытовых и индустриальных применений типоразмера 18650. 

Для теста были выбраны аккумуляторы литий-никельмарганцевой системы: по два образца одного наименования каждого производителя – и протестированы на двух значениях тока разряда: 0,5 А и 2,5 А. Испытания проводились в нормальных условиях на электронной нагрузке EBD-USB от ZKEtech, а зарядка осуществлялась от лабораторного источника питания в режиме CC+CV в соответствии с рекомендациями в даташите на определенную модель. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[Vascom]

Затем, что это вырабатывает правильные навыки программирования. Компилятор лишь предупреждает, что так делать нехорошо, именно поэтому всего лишь Warning, а не Error.

Если тебе не важно начальное значение - инициализируй нулём. Предупреждение пропадёт и всё будет работать как надо.

Именно таков верный способ вразумить компилятор!

[Реклама]

Литиевые аккумуляторы EVE Energy и решения для управления перезаряжаемыми источниками тока (материалы вебинара)

Опубликованы материалы вебинара Компэл, посвященного литиевым аккумуляторам EVE Energy и решениям для управления перезаряжаемыми источниками тока.

На вебинаре мы представили информацию не только по линейкам аккумуляторной продукции EVE, но и по решениям для управления ею, что поможет рассмотреть эти ХИТ в качестве дополнительной альтернативы для уже выпускающихся изделий. Также рассмотрели нюансы работы с производителем и сервисы, предоставляемые Компэл по данной продукции. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[Alex]

Вообще, он правильно ругается. Есть вероятность, что из функции вернётся неинициализированное значение.

В SPI_IN(x) , в условии проверки линии, поставь else. Установка бита в "x" стоит, а сброса нет. Это даже ошибка

Или инициализируй переменную нулём.

[Реклама]

Литиевые батарейки и аккумуляторы от мирового лидера  EVE в Компэл

Компания Компэл, официальный дистрибьютор EVE Energy, бренда №1 по производству химических источников тока (ХИТ) в мире, предлагает продукцию EVE как со склада, так и под заказ. Компания EVE широко известна в странах Европы, Америки и Юго-Восточной Азии уже более 20 лет. Недавно EVE была объявлена поставщиком новых аккумуляторных элементов круглого формата для электрических моделей «нового класса» компании BMW.

Продукция EVE предназначена для самого широкого спектра применений – от бытового до промышленного. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[Стальной]

В *.h (заголовочных) файлах реализации функций не пишут! Там только прототипы, а в отдельном *.c файле реализации.

[Alex]

Ерунда. С таким успехом, можно сказать, что .c файлы не подключаются includ'ом.

Если .h файл подключается всего в одном месте, то почему бы там не реализовывать код и не объявлять переменные ?

[Стальной]

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

Как говорилось выше, хороший стиль программирования

[Alex]

В разных местах - да, будет. И то, не всегда. Возможны просто создания одинаковых экземпляров.

Но, как я уже выше сказал :

Если .h файл подключается всего в одном месте, то почему бы там не реализовывать код и не объявлять переменные ?

[Vascom]

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

Ты сюда за помощью пришел или спорить? Тебе говорят как улучшить код, избавиться от ворнинга. Зачем делать неправильно?

[Alex]

Ты сюда за помощью пришел или спорить? Тебе говорят как улучшить код, избавиться от ворнинга. Зачем делать неправильно?

Это кому адресовано ?

[Vascom]

А, не посмотрел на имена, с телефона писал, сорри.

Тогда можете считать это размышлениями вслух.

[COKPOWEHEU]

Установка бита в "x" стоит, а сброса нет. Это даже ошибка

Вроде как Си гарантирует заполнение сдвинутого бита нулем, зачем его еще раз присваивать?

В *.h (заголовочных) файлах реализации функций не пишут! Там только прототипы, а в отдельном *.c файле реализации.

Специально для этого предусмотрены макроопределения MULTISOURCE_PROJECT и HEADER_IMPLEMENTATION, позволяющие при надобности отключить реализацию. Собственно, единственное почему не рекомендуется в *.h файл включать реализацию - дублирование функций с одинаковыми именами в разных *.o файлах, а линкер потом с ума сходит не зная какой использовать. С другой стороны, для проверки какой-нибудь мелочи подключать к проекту лишний *.c файл, переконфигурировать его и т.п. не оправдано. Поэтому и пошел на такой компромисс - если этот модуль используется только в одном файле, реализация подключена, если в нескольких - в настройках указывается MULTISOURCE_PROJECT, а в одном любом *.c файле (проще всего, наверное, main.c) HEADER_IMPLEMENTATION. На тестовых примерах это работает, будет ли удобно в реальности - пока не знаю.

Затем, что это вырабатывает правильные навыки программирования. Компилятор лишь предупреждает, что так делать нехорошо

Нет, компилятор не знает, что начальное значение в данном случае ни на что не повлияет. Так что, нет никакого атрибута или еще какого указания компилятору "я знаю про эту особенность, это не баг а фича"?

[Alex]

Вроде как Си гарантирует заполнение сдвинутого бита нулем, зачем его еще раз присваивать?

Это писателю известно, а компилятор считает, что переменная может не измениться.

Всё-таки, ради интереса, попробуй else добавить.

[COKPOWEHEU]

Ничего не изменилось. На счет неинициализированной переменной наверное и правда не стоит извращаться и инициализировать, пусть компилятор спит спокойно.