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

Implement blue light filters #1065

Merged
merged 7 commits into from
Feb 21, 2019
Merged

Implement blue light filters #1065

merged 7 commits into from
Feb 21, 2019

Conversation

panicbit
Copy link
Contributor

@panicbit panicbit commented Apr 15, 2018

This change is Reviewable

@panicbit
Copy link
Contributor Author

I just noticed that the LUT gets reset when waking up from standby.

@@ -50,8 +50,12 @@ AccessControlInfo:
# The kernel extension removes svc perms checks, so below is just to avoid a makerom error
Backdoor: 123
KernelSetState: 124
BindInterrupt: 80
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't really used, see above comment

do {
u32 pos = idx & 0xFF;

GPU_FB_TOP_COL_LUT_INDEX = pos;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You only need to set the index once. It increases automatically on each write.

@ghost ghost deleted a comment from therealbungus Jul 11, 2018
@TuxSH
Copy link
Member

TuxSH commented Aug 1, 2018

Sorry for having not reviewed it yet, will do later

@lagseeing
Copy link

Did it get reviewed? I want to try this.

@916253
Copy link

916253 commented Feb 2, 2019

When rebased against the latest commit and built, rosalina's menu no longer opens after setting a filter. Dunno if that happened initially, but its something that needs to be resolved before this can be merged.

@panicbit
Copy link
Contributor Author

panicbit commented Feb 2, 2019

I'll have a look at it again (but it may take a few weeks).

@TuxSH
Copy link
Member

TuxSH commented Feb 21, 2019

Sorry for the very long delay, was busy with other stuff, the code seems totally fine.

Maybe the bug is due to the fact that the menu stack size was not actually bumped (in the array declaration), will check later.

EDIT: yep it's that & I have it working now

sizeof(menuThreadStack) not matching 0x3000
@TuxSH TuxSH merged commit f437d92 into LumaTeam:master Feb 21, 2019
Sahlo97 pushed a commit to Sahlo97/Luma3DS that referenced this pull request Dec 9, 2021
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.

7 participants