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

[Feature Request] Implement PWM frequency option for newer ISSI drivers #14044

Closed
Thiird opened this issue Aug 17, 2021 · 1 comment
Closed

Comments

@Thiird
Copy link

Thiird commented Aug 17, 2021

Newer ISSI led drivers like IS31FL3733B give the user the possibility to set the PWM frequency by manipulating the PFS register.

immagine

This can be used to avoid hearing a buzzing noise from the driver's decoupling capacitors when the led's brightness is set above a certain threshold.

ISSI itself speaks about this issue in the IS31FL3733B's datasheet, page 25/30.

I think eliminating the buzzing in software would be a better solution than using tantalum capacitors, because it wouldn't add cost to the project and it would let us take advantage of the hardware that we already have on hand.

Sample of the noise from a single driver setup.

I will personally make a build with a dual IS31FL3733B setup in the near future (the older version IS31FL3733 is being phased out and is hard to find), so I would be happy to test this feature myself once it's implemented.

@zvecr zvecr changed the title [Feature Request] Implement PWN frequency option for newer ISSI drivers [Feature Request] Implement PWM frequency option for newer ISSI drivers Aug 17, 2021
@Thiird
Copy link
Author

Thiird commented Nov 6, 2021

Closing issue as it has been addressed by #14983.

@Thiird Thiird closed this as completed Nov 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant