Skip to content

feat(com): passed first communication - #25

Merged
husamettinarabaci merged 1 commit into
developfrom
feat/first-base-communication
Oct 17, 2025
Merged

husamettinarabaci merged 1 commit into
developfrom
feat/first-base-communication

Conversation

@husamettinarabaci

Copy link
Copy Markdown
Member

Communicated with mobile app first time


🧩 Affected Module(s)

Mark the modules impacted by this PR:

  • Firmware
  • Hardware
  • Mechanic
  • Documentation
  • CI / Infra

This pull request primarily focuses on simplifying and cleaning up the firmware codebase, especially around error handling, RGB LED control, and AT command dispatch. Several unused error variants were removed, code paths were streamlined, and some legacy or unnecessary methods were commented out or deleted. Additionally, a minor update was made to the project configuration files.

Error Handling Cleanup:

  • Removed unused firmware update-related error variants from the Error enum and their associated error codes in firmware/src/error/mod.rs. This reduces complexity and avoids confusion about error states that are no longer relevant. [1] [2]

AT Command Dispatch Simplification:

  • Refactored the AT command dispatcher by removing the usb_tx parameter from both the dispatcher method and its call site, indicating that USB transmission is now handled elsewhere or is unnecessary for dispatch errors. [1] [2]

Firmware Update Flow Adjustment:

  • Removed an unnecessary unsafe block from the firmware update handler, simplifying the transition to BOOTSEL mode for firmware updates.

RGB LED Task and API Cleanup:

  • Commented out the unused off method in RgbLed, clarifying the code and preventing accidental use.
  • Simplified the RGB task loop by handling only the relevant message type and removing the match statement, making the code easier to read and maintain.

Project Configuration Update:

  • Added pnpm-lock.yaml to the list of tracked files in REUSE.toml for better package management and license compliance.

Communicated with mobile app first time
@github-project-automation github-project-automation Bot moved this to 📥 Inbox / Ideas in hexaTune Project Oct 17, 2025
@github-actions github-actions Bot added default:unclassified No specific category matched this PR or file path. module:firmware Firmware type:infra Infrastructure, CI/CD, build system labels Oct 17, 2025
@husamettinarabaci
husamettinarabaci merged commit d5ddb2f into develop Oct 17, 2025
9 checks passed
@husamettinarabaci
husamettinarabaci deleted the feat/first-base-communication branch October 17, 2025 08:43
@github-project-automation github-project-automation Bot moved this from 📥 Inbox / Ideas to Done in hexaTune Project Oct 17, 2025
husamettinarabaci added a commit that referenced this pull request Oct 22, 2025
* feat(setup): added basic at command (#19)

Added basic at command implementation. It support usb-midi sysex protocol

* feat(setup): setted up pre commit hooks (#20)

Added reuse prettier commitlint hook

* feat(setup): setted up pre commit hooks (#21)

Added reuse prettier commitlint hook 3

* feat(async): setted up async tasks (#23)

Added rgb usb at async tasks and channel before firmware update

* feat(firmware): added basic firmware update (#24)

Added simple bootsel FWUPDATE command to At

* feat(com): passed first communication (#25)

Communicated with mobile app first time

* feat(freq): added basic freq handler (#27)

Added simple at freq handler not check and control

* feat(core): added multi core (#28)

System is configured for working multi-core. DDS task is working on the Core-1 and others on the Core-0

* feat(core): at command is made by id based (#29)

Changed all message types to at command base and id forced for all at command

* docs(basic): added basic docs Added simple documantation for version v1.0.0 (#30)
husamettinarabaci added a commit that referenced this pull request Oct 30, 2025
* feat(setup): added basic at command (#19)

Added basic at command implementation. It support usb-midi sysex protocol

* feat(setup): setted up pre commit hooks (#20)

Added reuse prettier commitlint hook

* feat(setup): setted up pre commit hooks (#21)

Added reuse prettier commitlint hook 3

* feat(async): setted up async tasks (#23)

Added rgb usb at async tasks and channel before firmware update

* feat(firmware): added basic firmware update (#24)

Added simple bootsel FWUPDATE command to At

* feat(com): passed first communication (#25)

Communicated with mobile app first time

* feat(freq): added basic freq handler (#27)

Added simple at freq handler not check and control

* feat(core): added multi core (#28)

System is configured for working multi-core. DDS task is working on the Core-1 and others on the Core-0

* feat(core): at command is made by id based (#29)

Changed all message types to at command base and id forced for all at command

* docs(basic): added basic docs Added simple documantation for version v1.0.0 (#30)

* feat(multicore): fixed multicore

Fixed channel for multicore tasks

* feat(multicore): fixed multicore

Fixed channel for multicore tasks
@husamettinarabaci husamettinarabaci mentioned this pull request Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

default:unclassified No specific category matched this PR or file path. module:firmware Firmware type:infra Infrastructure, CI/CD, build system

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant