-
Notifications
You must be signed in to change notification settings - Fork 77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Добавить управление несколькими GPIO и чтение с входов АЦП #54
Comments
Здравствуйте, у проекта вполне конкретное предназначение, а именно -- конвертер USB2Serial. Абсолютно все что реализовано так или иначе относится непосредственно к данной задаче. Я не против добавлять что-то новое, но в основном только в тех случаях, когда эта функциональность имеет отношение к теме проекта. В исключительных случаях может быть добавлена какая-то не относящаяся непосредственно к проекту функциональность (например, как опция сборки), но для этого у меня должно быть четкое понимание зачем эта функциональность нужна и будет ли она полезна хоть сколько-то заметному количеству пользователей. Кроме того, хотелось бы иметь чуть более полную спецификацию задачи, чем вы изложили. К сожалению, пока я не вижу необходимости добавлять в проект требуемую вам функциональность. Да и до конца не понимаю что вам нужно, если честно. |
Где-то в форумах читал сообщение от Вас, что вы хотели добавить аналогичный функционал. |
У меня в планах есть добавление USB-Audio, как опция сборки. То есть, либо чистый USB2Serial (3UART), либо 2UART + USB Audio. Скорее всего вы читали именно это. Этот функционал мне нужен самому (да и многие знакомые радиолюбители просили добавить). Про добавление generic GPIO+ generic ADC я вряд ли писал. Опять же, с USB-Audio все понятно: будет композитное устройство 2 CDC + USB Audio Class, в вашем feature request непонятно в каком формате вы хотите это иметь. То есть, со стороны bluepill GPIO/ADC, а со стороны USB что? CDC или что-то другое? Как это увязать с имеющимися портами? Делать меньше? Или добавлять еще CDC (или что-то другое для этого). Непонятно. Был человек, который хотел сделать похожий функционал (GPIO+Virtual USB CDC+имеющийся шелл для управления GPIO), но дальше PoC дело не пошло. Какое-то время повисел недоделанный Pull Request, но потом пришлось его закрыть, поскольку человек бросил, а мне было неинтересно/ненужно (см. #45). У нас была договоренность, что если PR не ведет к деградации параметров USB2UART и соответсвует coding style проекта, то я могу рассмотреть его внедрение в основную ветку. Но до этого не дошло. В-целом получается вот что:
Резюмируя: в настоящий момент "GPIO+ADC" добавлять не планирую. |
Спасибо за развернутый ответ. |
Спасибо за понимание, в любом случае я буду иметь в виду что такая функциональность нужна некоторой части пользователей (и их теперь как минимум двое) и, возможно, в будущем пересмотрю планы. Но в любом случае, мне надо доделать сначала звук. Основная проблема в том, что у меня нет в голове концепции того как это лучше сделать и почему именно так. Ну и, конечно, ограниченный ресурс -- это некоторая объективная проблема. |
Если есть возможность просьба добавить возможность управление несколькими GPIO в режиме In/Out и чтение с нескольких входов АЦП.
The text was updated successfully, but these errors were encountered: