Skip to content

Conversation

@Craftzman7
Copy link
Contributor

@Craftzman7 Craftzman7 commented Nov 11, 2025

Description

Closes #2091

I know the pins and the FOV are correct, unsure about the PWM range or frequency. I haven't had a chance to test this but I was able to figure this out when attempting to turn on the LEDs on a blank install of Raspberry Pi OS back in 2023. I will have access to an LL3 tomorrow and hope to test then.

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why
  • This PR has been linted.
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with settings back to v2025.3.2
  • If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • If this PR addresses a bug, a regression test for it is added

@Craftzman7 Craftzman7 requested a review from a team as a code owner November 11, 2025 18:18
@github-actions github-actions bot added the documentation Anything relating to https://docs.photonvision.org label Nov 11, 2025
@thatcomputerguy0101
Copy link
Contributor

PWM range and frequency can probably be left unspecified to use default values.

@Craftzman7
Copy link
Contributor Author

The current values are just yanked from the LL2+ config.

@thatcomputerguy0101
Copy link
Contributor

ledPWMFrequency is not currently parsed by PV, and ledPWMRange defaults to [0, 100]. It seems like ledPWMFrequency was forgotten in the documentation and the LL2+ config after #127 removed the option.

@Craftzman7
Copy link
Contributor Author

Tested, working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Anything relating to https://docs.photonvision.org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hardware config for LL3

2 participants