Contribute
Register

HP 430 G4 volume control problem

Status
Not open for further replies.
Sorry, I was wrong config.plist file
Using the "config.plist" file from probook.git, do not need to change, everything is normal.

So, no need for this diff:
Code:
NUC6i7KYK:probook.git rehabman$ git diff config_parts/config_CX8200.plist 
diff --git a/config_parts/config_CX8200.plist b/config_parts/config_CX8200.plist
index 2b92d90..69a9cff 100755
--- a/config_parts/config_CX8200.plist
+++ b/config_parts/config_CX8200.plist
@@ -20,7 +20,7 @@
                                <key>Comment</key>
                                <string>from AppleALC, purpose unknown</string>
                                <key>Disabled</key>
-                               <true/>
+                               <false/>
                                <key>Name</key>
                                <string>com.apple.driver.AppleHDA</string>
                                <key>Find</key>
@@ -32,7 +32,7 @@
                                <key>Comment</key>
                                <string>from AppleALC, purpose unknown</string>
                                <key>Disabled</key>
-                               <true/>
+                               <false/>
                                <key>Name</key>
                                <string>com.apple.driver.AppleHDA</string>
                                <key>Find</key>
 
Noted:
- changes/additions in pinconfigs
You can leave the old values, i expect no disadvantages.
Noted:
- VREF entry you mention (needs comments)
Node 0x19 Set Pin Widget Control "In Enable", "VRefEn Signal Level 80%"
Noted:
- second command { 0x01, 0xA7, 0x08, 0x85 } looks useless as all triggers are false... (I should probably remove it)
You can remove it, i have added it for additional tests. The Unsolicited Response Control works.
And then Platforms.plist diffs:
They are ok.
 
So, no need for this diff:
Code:
NUC6i7KYK:probook.git rehabman$ git diff config_parts/config_CX8200.plist
diff --git a/config_parts/config_CX8200.plist b/config_parts/config_CX8200.plist
index 2b92d90..69a9cff 100755
--- a/config_parts/config_CX8200.plist
+++ b/config_parts/config_CX8200.plist
@@ -20,7 +20,7 @@
                                <key>Comment</key>
                                <string>from AppleALC, purpose unknown</string>
                                <key>Disabled</key>
-                               <true/>
+                               <false/>
                                <key>Name</key>
                                <string>com.apple.driver.AppleHDA</string>
                                <key>Find</key>
@@ -32,7 +32,7 @@
                                <key>Comment</key>
                                <string>from AppleALC, purpose unknown</string>
                                <key>Disabled</key>
-                               <true/>
+                               <false/>
                                <key>Name</key>
                                <string>com.apple.driver.AppleHDA</string>
                                <key>Find</key>

Yes!
 
This assertion needs testing first.
In most cases, Apple has configured the line-in pin as Combination "71E8B", and the internal microphone pins as "71EA0" Unknown. So i expect that it works.
Now i see that i've made a mistake, i wanted to configure the internal microphone as Other Digital "71EA6" and not Other Analog.

@enjm115
If possible, try the internal microphone at full input volume with the last SSDT i uploaded, if you hear any distortion, and compare it with the newly updated SSDT. Because that's the case on my laptop.
 

Attachments

  • SSDT-4x0-G4-Kabylake.aml.zip
    4.9 KB · Views: 65
If possible, try the internal microphone at full input volume with the last SSDT i uploaded, if you hear any distortion, and compare it with the newly updated SSDT. Because that's the case on my laptop.

It looks good, and the test doesn't feel any change.
The microphone is working properly.
 
In most cases, Apple has configured the line-in pin as Combination "71E8B", and the internal microphone pins as "71EA0" Unknown. So i expect that it works.
Now i see that i've made a mistake, i wanted to configure the internal microphone as Other Digital "71EA6" and not Other Analog.

@enjm115
If possible, try the internal microphone at full input volume with the last SSDT i uploaded, if you hear any distortion, and compare it with the newly updated SSDT. Because that's the case on my laptop.
It looks good, and the test doesn't feel any change.
The microphone is working properly.

Current diffs for layout20_HDEF.asl:
Code:
NUC6i7KYK:probook.git rehabman$ git diff hotpatch/include/layout20_HDEF.asl
diff --git a/hotpatch/include/layout20_HDEF.asl b/hotpatch/include/layout20_HDEF.asl
index fa8ffe0..b5dabf5 100644
--- a/hotpatch/include/layout20_HDEF.asl
+++ b/hotpatch/include/layout20_HDEF.asl
@@ -35,8 +35,9 @@
                             Package(){},
                             0x16, 0x012b1040,
                             0x17, 0x90170010,
+                            0x18, 0x400000F0,
                             0x19, 0x018b1030,
-                            0x1a, 0x90a00020,
+                            0x1a, 0x90A60020,
                         },
                     },
                 },
@@ -49,12 +50,30 @@
                         "Command", Buffer()
                         {
                             0x01, 0x77, 0x0c, 0x02,
-                            0x01, 0x67, 0x0c, 0x02
                         },
                     },
                 },
             },
         },
+        "CodecCommander", Package()
+        {
+            "Version", 0x00020600,
+            "14f1_2008", Package()
+            {
+                "Custom Commands", Package()
+                {
+                    Package(){},
+                    Package()
+                    {
+                        //Node 0x19 Set Pin Widget Control "In Enable", "VRefEn Signal Level 80%"
+                        "Command", Buffer() { 0x01, 0x97, 0x07, 0x24 },
+                        "On Init", ">y",
+                        "On Sleep", ">n",
+                        "On Wake", ">y",
+                    },
+                },
+            },
+        },
     })

 //EOF

Final?
 
It looks good, and the test doesn't feel any change.
The microphone is working properly.
Thanks for testing
Current diffs for layout20_HDEF.asl:
Code:
NUC6i7KYK:probook.git rehabman$ git diff hotpatch/include/layout20_HDEF.asl
diff --git a/hotpatch/include/layout20_HDEF.asl b/hotpatch/include/layout20_HDEF.asl
index fa8ffe0..b5dabf5 100644
--- a/hotpatch/include/layout20_HDEF.asl
+++ b/hotpatch/include/layout20_HDEF.asl
@@ -35,8 +35,9 @@
                             Package(){},
                             0x16, 0x012b1040,
                             0x17, 0x90170010,
+                            0x18, 0x400000F0,
                             0x19, 0x018b1030,
-                            0x1a, 0x90a00020,
+                            0x1a, 0x90A60020,
                         },
                     },
                 },
@@ -49,12 +50,30 @@
                         "Command", Buffer()
                         {
                             0x01, 0x77, 0x0c, 0x02,
-                            0x01, 0x67, 0x0c, 0x02
                         },
                     },
                 },
             },
         },
+        "CodecCommander", Package()
+        {
+            "Version", 0x00020600,
+            "14f1_2008", Package()
+            {
+                "Custom Commands", Package()
+                {
+                    Package(){},
+                    Package()
+                    {
+                        //Node 0x19 Set Pin Widget Control "In Enable", "VRefEn Signal Level 80%"
+                        "Command", Buffer() { 0x01, 0x97, 0x07, 0x24 },
+                        "On Init", ">y",
+                        "On Sleep", ">n",
+                        "On Wake", ">y",
+                    },
+                },
+            },
+        },
     })

 //EOF

Final?
Yes, since enljm115 has confirmed that it works with microphone connection type Unknown, Other Digital, Other Analog, it should work.
Alternatively, you can leave Mirones layout and add this as a new one.
 
Alternatively, you can leave Mirones layout and add this as a new one.

I'd like to just switch the repo instead of having more options that people have to experiment with.
 
I'd like to just switch the repo instead of having more options that people have to experiment with.

Changes for Conexant CX8200 (as discussed in this thread so far) checked into the probook github repo.
 
Status
Not open for further replies.
Back
Top