Skip to content
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

Add hardware configuration for genXnoise devices #768

Merged
merged 4 commits into from
Dec 23, 2024
Merged

Conversation

probonopd
Copy link
Owner

@probonopd probonopd commented Nov 28, 2024

@probonopd probonopd marked this pull request as draft November 28, 2024 10:00
@probonopd probonopd changed the title Add hardware configuration for dXeus machina Add hardware configuration for genXnoise devices Nov 28, 2024
Copy link

github-actions bot commented Nov 28, 2024

Build for testing:
MiniDexed_2024-11-28-0c520b5
Use at your own risk.

Need to replace the default minidexed.ini in the root of the microSD card with the appropriate one from inside the hardware folder.

@Lonolo
Copy link

Lonolo commented Dec 8, 2024

'USBGadget=1' is not appearing in the desktop_module.ini of this build.

@probonopd
Copy link
Owner Author

probonopd commented Dec 8, 2024

According to https://github.com/probonopd/MiniDexed/wiki/Hardware#usb-gadget-mode

In USB Gadget mode the Pi will obtain its power from the USB host it is plugged into and it MUST NOT be powered independently.

On the desktop module as it is shipped, is there no way to inadvertently power it on another way than the USB host? (E.g., does the housing prevent from this?) Otherwise I'd be hesitant to enable this by default, in order not to have people who don't read the wiki brick their devices.

@Lonolo
Copy link

Lonolo commented Dec 8, 2024

The desktop module comes with a USB cable and no other PSU. Instructions explicitly say:

Configured as a USB gadget. Use the included USB cable and connect the micro side to the port labeled USB. Do not connect a power supply to the PWR port.

I've considered plugging the PWR port with these, to solve that posiibility of user-error I think I will:
Screen Shot 2024-12-08 at 5 43 46 PM

But that is the simplicity of the desktop module, powered and MIDI controlled by the USB port. That and a stereo audio cable are the only two connections:
IMG_3530

@probonopd
Copy link
Owner Author

So far this option is not in our default minidexed.conf at all (not even disabled by default) because it can be dangerous depending on the hardware setup, this is why the configuration doesn't get applied (yet). I am thinking about a solution. Thanks for bringing this to my attention.

@Lonolo
Copy link

Lonolo commented Dec 9, 2024

Thanks again for your time spent on this, I understand the option to enable networking could also complicate including USBGadget in the default configuration. Perhaps adding a clear warning in the #comments above the USBGadget parameter would help caution users about potential risks. Users of both the desktop and dXeus modules already have flexibility with multiple connectivity options, including USB-MIDI, USB-Gadget, and soon, networking access, allowing them to tailor their setup to their needs.

Copy link

Build for testing:
MiniDexed_2024-12-21-aa8bfb4
Use at your own risk.

@probonopd
Copy link
Owner Author

probonopd commented Dec 21, 2024

The ini file for the desktop module now contains:

# CAUTION: To prevent hardware damage, DO NOT use the port labeled 'PWR'
# (the microUSB port near the edge of the device) when USBGadget is set to 1!
# You need to disable USBGadget if you would like to use that port!
# See https://github.com/probonopd/MiniDexed/wiki/Hardware#usb-gadget-mode for more information
USBGadget=1

That doesn't guarantee that everyone will ever read it there though...

@Lonolo
Copy link

Lonolo commented Dec 23, 2024

Thanks - works great!

These will now ship with a plug in the PWR port:

desktopPlug

@probonopd probonopd marked this pull request as ready for review December 23, 2024 20:12
@probonopd probonopd merged commit ffb6449 into main Dec 23, 2024
1 check passed
@probonopd probonopd deleted the dxeus_machina branch December 23, 2024 20:12
@soyersoyer
Copy link
Contributor

MIDIThru=ttyS1,ttyS1 looks wrong. Is it?

@Lonolo
Copy link

Lonolo commented Dec 23, 2024

MIDIThru=ttyS1,ttyS1 looks wrong. Is it?

What looks wrong?

Using the RPI's Tx/Rx pins to a midi DIN circuit. MIDI chaining them all and works perfectly:

dXeus-racked

@soyersoyer
Copy link
Contributor

soyersoyer commented Dec 23, 2024

Shouldn't this forward MIDI IN to the THRU port?

I assume that ttyS0 is the IN port and ttyS1 is the THRU port.
And with this configuration, it forwards ttyS1 to ttyS1.

Edit:
Okay, I understand how it works :)
MIDI IN is the ttyS1 in, and MIDI Thru is the ttyS1 out.

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.

3 participants