toleda Posted August 20, 2012 Share Posted August 20, 2012 Get the following warnings: 11048 Warning Not all control paths return a value (_HID) 11048 Warning Reserved method must return a value (Integer/String required for _HID) Using this patch: into_all all code_regex (If\s\(TCMF)\s*\) replace_matched begin If (TCMF) \n { \n Return (Zero) \n } \n end  The result is: How do I get the return in the right place and avoid the extra set of brackets? Method (_HID, 0, NotSerialized) { If (TCMF) { Return (Zero) } {} Else { Return (0x0201D824) } }  DSDT: Original Device (_SB.PCI0.LPCB.TPM) { Method (_HID, 0, NotSerialized) { If (TCMF) {} Else { Return (0x310CD041) } } Fixed Device (_SB.PCI0.LPCB.TPM) { Method (_HID, 0, NotSerialized) { If (TCMF) {} Else { Return (0x310CD041) } Return (Zero) } test.aml.zip Quote Link to comment Share on other sites More sharing options...
Cassio Posted August 20, 2012 Share Posted August 20, 2012 You are talking about 2 different fixes. First one is mentioned here http://Olarila.com/forum/viewtopic.php?f=19&t=1752#p14711 into_all all code_regex If\s\(TCMF\)\s\{\} replace_matched begin If (TCMF)\n {\n Return (Zero)\n } end The second is simpler into method label _HID parent_label _SB.PCI0.LPCB.TPM insert begin Return (Zero) end Quote -Donations- PayPal HERE - Stripe HERE - BuyMeaCoffee HERE - Mercado Livre HERE Skrill danielnmaldonado@gmail.com - BTC 33HeGCuCSh4tUBqdYkQqKpSDa1E7WeAJQ3 PicPay @danielnmaldonado - PiX @danielnmaldonado@gmail.com Premium Users HERE - Problems with Paypal HERE Link to comment Share on other sites More sharing options...
toleda Posted August 20, 2012 Author Share Posted August 20, 2012 You are talking about 2 different fixes. Didn't mean to trouble you twice on this. Works perfectly. Thanks again. Quote Link to comment Share on other sites More sharing options...