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

hdspmixer: Hardware output loopback toggle buttons #4

Closed
wants to merge 3 commits into from

Conversation

pfandl
Copy link
Contributor

@pfandl pfandl commented Feb 2, 2021

Hi, as promised in the sound/pci/rme9652 patch here are the user land changes which provide the user the ability to toggle the hardware output loopback functionality per output channel. I tried to hide the buttons for devices which do not expose the snd controls yet, but due to probably unnecessary struggle i cancelled that. Please let me know though should you want them removed for not yet enabled devices.

This adds "LPBK" buttons to the output strip
channels for toggling the hardware output
loopback functionality recently added for
HDSP9632.

The preset data is just appended to the
preset file so that it can still be loaded
in older versions.
This adds "LPBK" buttons to the output strip
channels for toggling the hardware output
loopback functionality recently added for
HDSP9632.

The preset data is just appended to the
preset file so that it can still be loaded
in older versions.
@pfandl pfandl changed the title Hardware output loopback toggle buttons hdspmixer: Hardware output loopback toggle buttons Feb 2, 2021
@pfandl pfandl closed this Feb 4, 2021
@perexg perexg reopened this Feb 4, 2021
@perexg
Copy link
Member

perexg commented Feb 4, 2021

What was the reason to close this PR? I see you sent the patch to ML, but we can merge the patches from github, too.

@pfandl
Copy link
Contributor Author

pfandl commented Feb 4, 2021

Hi, I was asked to send it to ML and since this is a little different than the ones I sent on the ML i closed this.
Patch no. 1 isn't here considered at all and the save/load routines are different. You can ignore this PR please.

@perexg
Copy link
Member

perexg commented Feb 4, 2021

I don't see the ML request in your conversation with @tiwai .

Link: https://lore.kernel.org/alsa-devel/2e9bb0b8-ee14-9dc9-1bd8-ecffca3982ac@gmail.com/

I'll close this when merged.

@pfandl
Copy link
Contributor Author

pfandl commented Feb 4, 2021

I asked him directly via email since I wasn't sure if this PR got noticed because the reaction time on the ML was so fast.

@pfandl
Copy link
Contributor Author

pfandl commented Feb 27, 2021

Hi @perexg the patch was merged, I'll close this if I may. Thanks

@pfandl pfandl closed this Feb 27, 2021
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.

None yet

2 participants