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

Please remove my name from the readme.md #14

Open
wern-apfel opened this issue Feb 11, 2023 · 3 comments
Open

Please remove my name from the readme.md #14

wern-apfel opened this issue Feb 11, 2023 · 3 comments

Comments

@wern-apfel
Copy link

Thanks. No more comments from me.

Qonfused added a commit to Qonfused/ASUS-ZenBook-Pro-Duo-15-UX582-Hackintosh that referenced this issue Feb 27, 2023
@wern-apfel
Copy link
Author

Actually, I did not want to say anything more on this subject.
For lack of time, I usually just skim the posts. Maybe I just misunderstood something. If that is really the case, I must apologize.

What do you mean by: (as he complained here) for some SSDTs for Qonfused's work ??
I have fixed the ScreenPad toggle and brightness adjustment, as well as the battery threshold and FN key lock. Even though the fix looks different, that doesn't mean it's from him.

@Qonfused
Copy link

Qonfused commented Apr 23, 2023

The screenpad fix in my repo is not from you; please take a look here: src/ACPI/SSDT-SPLC.dsl

This re-implements the DEVS methods based off work from @Plippo in the asus-wmi-screenpad repository. I made a note of this here when you first mentioned it for your EFI. I have a breakdown of references + notes here that explains how it works. I've given another explanation of how these WMI methods work in his repo for other screenpad devices here based on this work, as his implementation is not yet complete.

The FN key lock however was documented by @hieplpvip in his AsusSMC repo. Additionally, based on your implementation for the keyboard SKBV method, I believe this to be your original source. I'm also upstreaming changes to that repo here if you want a better look at how DSTS + DEVS methods are used by the screenpad driver (windows).

I have given you attribution in my own repo here and @shiecldk's repo (fork/PR) here for not just the FN+Lock and screenpad changes; your work encompassed the ATKD functionality of the repo. I've since moved away from using ACPI patches to control ATKD methods and the screenpad in #40. The reason why your ACPI patches aren't more commonplace is that they're incorporated in projects like AsusSMC that work between kernelspace + userspace.

@Qonfused
Copy link

Qonfused commented Apr 23, 2023

I've outlined the rationale for this in my #6 issue that I linked after you first mentioned/introduced your screenpad fix in #4. I discuss the reasoning for moving away from direct EC register writes here, and for why using DSTS + DEVS methods are preferrable across different devices + BIOS versions.

I have made no error in misrepresenting attribution or the source of own my work, and neither has @shiecldk.

If you're new to the open-source world, attribution is for giving credit based on people's work + time invested; rarely is there novelty in the kind of work done. There is no adversarial relationship here.

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

No branches or pull requests

2 participants