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 Jack Sense and EAPD support to ALC230 layout 13 & 20 and add WakeConfigData to layout 13 #618

Merged
merged 2 commits into from
Nov 2, 2020

Conversation

Ab2774
Copy link
Contributor

@Ab2774 Ab2774 commented Oct 28, 2020

All thank goes to @fewtarius

@ghost
Copy link

ghost commented Oct 29, 2020

Thanks @Ab2774, but I haven't been involved with this. Since I'm looking at it though, why did you change two layouts and how did you test them? Why did you also change the comments? Did you talk with the original layout author(s) yet? You should seek their approval, and also add more info about what you're changing and why you're changing it.

@Ab2774
Copy link
Contributor Author

Ab2774 commented Oct 29, 2020

Hi @fewtarius, I did test them on both my Lenovo laptops with ALC230 (Kaby Lake and Ice Lake), I changed the comments because not just Lenovo IdeaPad 320 laptops come with ALC230, I don't mind a review from @andreszerocross and @maiconjs, usually, in order to gain function of headphones after sleep ALCPlugFix should be installed, however, with this fix it no longer requires, but unfortunately, there are still some issues that I haven't been able to fix as using mic after sleep make audio disappears for few mins, Chrome doesn't support mic until one of the channels is disabled through Audio MIDI Setup, and finally mic input level in System Preferences.
I'd really appreciate it if you have any idea how to fix these.
Cheers

@Ab2774 Ab2774 changed the title Add Jack Sense and EAPD support to layout 13 & 20 and add WakeConfigData to layout 13 Add Jack Sense and EAPD support to ALC230 layout 13 & 20 and add WakeConfigData to layout 13 Oct 29, 2020
@ghost
Copy link

ghost commented Oct 29, 2020

unfortunately, there are still some issues that I haven't been able to fix as using mic after sleep make audio disappears for few mins, Chrome doesn't support mic until one of the channels is disabled through Audio MIDI Setup, and finally mic input level in System Preferences.
I'd really appreciate it if you have any idea how to fix these.
Cheers

It sounds like you need to create a new layout and fix the issues specific to your codec implementation. PRs aren't usually used for help, use the forums for that.

@Andrey1970AppleLife
Copy link
Contributor

You shan't makes changes without coordination with the author of an original layout.
You need to create a new layout.

@Ab2774
Copy link
Contributor Author

Ab2774 commented Oct 29, 2020

You shan't makes changes without coordination with the author of an original layout.
You need to create a new layout.

Hi, I do not mind making a new layout(s) but should all the existing users move to the new layout(s) and never get the issues fixed in the original layout in the first place?
Thanks

@vit9696
Copy link
Collaborator

vit9696 commented Oct 29, 2020

@Andrey1970AppleLife I checked the changes and they look adequate, we did add similar codes for various other codecs in the past when WakeConfigData was only starting to be a thing. I am not sure these layouts are currently maintained by anyone, and I think it will be better to take the risks and merge the changes and revert them if they cause issues.

@ghost
Copy link

ghost commented Oct 30, 2020

Since this was reopened I thought I'd take a closer look at it, hope you don't mind.

Codec 10EC 0230, layout 13
01271EA0 Mic in, Unknown
01471E17 Speaker, Other analog
01971E81 Line in, 1/8” jack
02171E2B HP out, Combo Jack

01271C10 01271D00 01271EA0 01271F90 01471C20 01471D00 01471E17 01471F90 01470C02 01971C30 01971D10 01971E81 01971F04 01970C02 02171C40 02171D10 02171E2B 02171F04 01371C50 01371D00 01371E00 01371F40 01D71C60 01D71DB0 01D71E66 01D71F40

01470C02 02170C02 01970725

This one may need to be adjusted, as you're enabling EAPD on the jack mic (node 19), but not on the headphone output (node 21). Might have better results if you remove 01970C02, and add 02170C02 at the end of node 21's configuration data.

Codec 10EC 0230, layout 20
01271EA0 Mic in, Unknown
01471E17 Speaker, Other analog
01971E81 Line in, 1/8” jack
02171E21 HP out, 1/8” jack

01471C10 01471D01 01471E17 01471F90 01470C02 02171C20 02171D10 02171E21 02171F13 02170C02 01271C30 01271D01 01271EA0 01271F90 01971C40 01971D11 01971E81 01971F93

01470C02 02170C02 01970725

This one looks ok.

@Ab2774
Copy link
Contributor Author

Ab2774 commented Oct 30, 2020

Hi @fewtarius, of course, I don't mind any improvements, if you can help with my other issues id really appreciate it if you're willing to, please continue on InsanelyMac as this not usually a please for discussion and help.
Thanks a lot

@vandroiy2013 vandroiy2013 merged commit 9938380 into acidanthera:master Nov 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants