Asus r510vx Skylake - battery status

[Deviad]

Note that the return you refer to is conditional, therefore the code following is not unreachable.
Your mistake was incorrect disassembly procedures.



https://www.tonymacx86.com/threads/guide-patching-dsdt-ssdt-for-laptop-backlight-control.152659/

Im considering the hypotesis of a missunderstanding since it just works. Contrarywise when I applied a wrong patch I got the "stop sign".

Our scenario (Pseudocode)
if (something) {
return 0; // bail, since condition is met
argA
argB
argC
} else {
return 1;
}

If it reaches return, it won't reach the rest of the code up to }, so I deleted argA, argB, argC (or whatever after return, I am at work now). Of course I did not delete the else block.
This is what I meant.

Does this apply to Assembly as well?

I am asking this stuff for my own knowledge and cause it might come in handy in the future knowing the right way to this thing.

 

[RehabMan]

Hi RehabMan,
I have used Clover's F4 procedure.
I did not know that about return, therefore here are the files you asked in your previous reply.

Thank you.

Disassemble with refs.txt as per guide. Result in DSDT.dsl: no errors.

https://www.tonymacx86.com/threads/guide-patching-laptop-dsdt-ssdts.152573/

 

[Deviad]

Disassemble with refs.txt as per guide. Result in DSDT.dsl: no errors.

https://www.tonymacx86.com/threads/guide-patching-laptop-dsdt-ssdts.152573/

Thank you RehabMan,
I will dive into it when I am back home.

 

[Deviad]

Thank you RehabMan, I got it working without errors with your latest suggestion. I applied all the patches and now all is fine.
I just have to add increasing and decreasing backlight by pressing some buttons on the keyboard, I will do it in the weekend if I do not have time now as I am starting different things:

http://www.github.com/Deviad

Thank you again.