Skip to content

fix(boards): SPI and Led Pins#113

Open
3dprintpt wants to merge 4 commits intoRat-OS:v2.1.xfrom
3dprintpt:v2.1.x
Open

fix(boards): SPI and Led Pins#113
3dprintpt wants to merge 4 commits intoRat-OS:v2.1.xfrom
3dprintpt:v2.1.x

Conversation

@3dprintpt
Copy link
Contributor

@3dprintpt 3dprintpt commented Feb 6, 2026

Corrections on Manta M8P SPI pins, Octopus Pro V1.1 (H723) Led pin and Y1 enable pin.

Summary by CodeRabbit

  • Chores
    • Remapped motor SPI pins and stepper interface on the BTT Manta M8P board.
    • Updated Y1 enable pin alias on the BTT Octopus Pro H723 board.
    • Changed filament runout sensor pin mapping on the BTT EBB42 Gen2 board.

Corrections on Manta M8P SPI pins and Octopus Pro V1.1 (H723) Led pin
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6b331786-1f02-40a3-961c-e439260c9a01

📥 Commits

Reviewing files that changed from the base of the PR and between 22bbb0d and 8b8a44e.

📒 Files selected for processing (1)
  • configuration/boards/btt-octopus-pro-h723-11/config.cfg
🚧 Files skipped from review as they are similar to previous changes (1)
  • configuration/boards/btt-octopus-pro-h723-11/config.cfg

📝 Walkthrough

Walkthrough

SPI motor and stepper pin mappings moved to PG6/PG7/PG8 for the Manta M8P board; a Y1 enable alias changed from PA0 to PA2 on the Octopus Pro H723 board; filament runout pin moved from PA3 to PA10 on the BTT EBB42 Gen2 toolboard config.

Changes

Cohort / File(s) Summary
Manta M8P SPI Pin Migration
configuration/boards/btt-manta-m8p-20/board-definition.json
All MOTOR1–MOTOR8 SPI pins (mosi/miso/sclk) and stepperSPI pins updated from PC12/PC11/PC10PG6/PG7/PG8.
Octopus Pro H723 Pin Alias
configuration/boards/btt-octopus-pro-h723-11/config.cfg
Alias y1_enable_pin changed from PA0PA2 in the board pins/aliases section.
BTT EBB42 Gen2 Filament Sensor
configuration/boards/btt-ebb42-gen2-10/toolboard-config.cfg
filament_sensor_runout_pin mapping changed from PA3PA10 in the board pins section.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

I hopped through pins with nimble feet,
Swapped PC lanes for PG's neat suite;
A Y1 winked from PA to two,
A filament's new home, PA ten — who knew?
Hoppity cheers, the configs align 🐇🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions 'SPI and Led Pins' but the changeset includes three distinct updates: SPI pin migrations on Manta M8P, a Y1 enable pin change on Octopus Pro, and a filament sensor pin change on EBB42, with LED pin changes only partially covered. Consider a more comprehensive title like 'fix(boards): SPI, LED, and enable pins' or split into multiple PRs if these are unrelated fixes, as the current title omits the filament sensor and Y1 enable pin changes.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Updated wiring diagrams with all the accessories for stock hardware and most used boards.
Al octopus nut Max EZ, O2S, EBB42 (all versions) and EBB36 (all versions).
EBB 42 Gen 2 Corrected
Including Orbiter smart sensor using I2C Pins on EBB 42 Gen 2. Diagram also corrected.
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@configuration/boards/btt-ebb42-gen2-10/toolboard-config.cfg`:
- Line 41: The filament sensor pin is set incorrectly: change the configuration
key filament_sensor_runout_pin from PA10 back to the dedicated FIL input PA3 to
restore proper filament detection and avoid clobbering I²C (PA9/PA10) lines;
locate the filament_sensor_runout_pin entry in the toolboard-config.cfg and
replace the PA10 value with PA3 so the line reads
filament_sensor_runout_pin=PA3, ensuring trailing comma matches surrounding
entries.


# filament sensors
filament_sensor_runout_pin=PA3,
filament_sensor_runout_pin=PA10,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Search for PA3 and PA10 references in configuration files

echo "=== Files in the btt-ebb42-gen2-10 directory ==="
find configuration/boards/btt-ebb42-gen2-10 -type f 2>/dev/null | head -20

echo -e "\n=== Searching for PA3 references ==="
rg 'PA3' configuration/ --type cfg --type txt -C 1

echo -e "\n=== Searching for PA10 references ==="
rg 'PA10' configuration/ --type cfg --type txt -C 1

echo -e "\n=== Checking the full toolboard-config.cfg file ==="
cat -n configuration/boards/btt-ebb42-gen2-10/toolboard-config.cfg

Repository: Rat-OS/RatOS-configurator

Length of output: 2495


🌐 Web query:

BTT EBB42 Gen2 1.0 filament sensor pin PA10 pinout schematic

💡 Result:

On the BTT EBB42 GEN2 V1.0, MCU pin PA10 is the I²C SDA line, brought out on the I2C header/connector (with PA9 as SCL). (global.bttwiki.com)

If you’re looking for the dedicated filament/runout (“FIL”) switch input, that’s PA3 (separate from the I²C pins). (global.bttwiki.com)

Citations:


Revert filament_sensor_runout_pin from PA10 back to PA3.

According to the official BTT EBB42 GEN2 V1.0 pinout documentation, PA10 is the I²C SDA (data) line and PA9 is the I²C SCL (clock) line. The dedicated filament runout ("FIL") switch input is on PA3. The current mapping at line 41 (filament_sensor_runout_pin=PA10) is incorrect and would break both I²C communication and filament detection. Change it back to filament_sensor_runout_pin=PA3,.

🤖 Prompt for AI Agents
In `@configuration/boards/btt-ebb42-gen2-10/toolboard-config.cfg` at line 41, The
filament sensor pin is set incorrectly: change the configuration key
filament_sensor_runout_pin from PA10 back to the dedicated FIL input PA3 to
restore proper filament detection and avoid clobbering I²C (PA9/PA10) lines;
locate the filament_sensor_runout_pin entry in the toolboard-config.cfg and
replace the PA10 value with PA3 so the line reads
filament_sensor_runout_pin=PA3, ensuring trailing comma matches surrounding
entries.

Correcting VAOC led pin on Octopus Pro V1.1 (h273).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant