Skip to content

Implement USB Powershare feature#639

Open
Fede2782 wants to merge 1 commit into
BeardOverflow:mainfrom
Fede2782:powershare
Open

Implement USB Powershare feature#639
Fede2782 wants to merge 1 commit into
BeardOverflow:mainfrom
Fede2782:powershare

Conversation

@Fede2782
Copy link
Copy Markdown
Contributor

@Fede2782 Fede2782 commented Mar 8, 2026

Tested on C1MG

@glpnk
Copy link
Copy Markdown
Collaborator

glpnk commented Mar 8, 2026

why you closed previous PR?

@Fede2782
Copy link
Copy Markdown
Contributor Author

Fede2782 commented Mar 8, 2026

why you closed previous PR?

I changed branch named to something which has a actual meaning, also rebased the change to use to the correct values for the G2_0 Fw group which was wrongly placed in old PR.

Also want to make my local clone more organized since I'm currently working on other features (KBD backlight mode change) for this project to complete support for my EC Control app.

@Fede2782
Copy link
Copy Markdown
Contributor Author

it would be cool if other owners could add their EC adresses if they support this function.

@glpnk
Copy link
Copy Markdown
Collaborator

glpnk commented Mar 17, 2026

https://codeberg.org/Sparronator9999/YAMDCC/issues/89 there is some leaked doc about EC structure

@Fede2782
Copy link
Copy Markdown
Contributor Author

https://codeberg.org/Sparronator9999/YAMDCC/issues/89 there is some leaked doc about EC structure

Cool. Probably all gen2 have this set in 0xbf but I'd like to hear some gen1 user first.

@glpnk
Copy link
Copy Markdown
Collaborator

glpnk commented Mar 17, 2026

You can read "support bit" 0xBF bit 3 to detect if feature available to dynamically unlock it

But gen 1 seems to be unsupported completely

@Fede2782
Copy link
Copy Markdown
Contributor Author

Fede2782 commented Apr 4, 2026

You can read "support bit" 0xBF bit 3 to detect if feature available to dynamically unlock it

But gen 1 seems to be unsupported completely

Sorry for late reply. That's a great idea but I'm gonna keep it this way so that it matches the rest of the application. In case you believe the bit check is required for you to consider merging this, I'd be more than happy to accordingly overhaul this.

@glpnk
Copy link
Copy Markdown
Collaborator

glpnk commented Apr 4, 2026

Please migrate to bits

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

Successfully merging this pull request may close these issues.

2 participants