Skip to content

Conversation

spacey-sooty
Copy link
Member

No description provided.

Signed-off-by: Jade Turner <[email protected]>
@spacey-sooty
Copy link
Member Author

Untested

@mcm001
Copy link
Contributor

mcm001 commented Mar 10, 2025

How is this different from the other limelight images exactly?

@Juniormunk
Copy link
Contributor

I believe the config.txt needs to change so that dtoverlay=ov9281,cam0

@spacey-sooty
Copy link
Member Author

How is this different from the other limelight images exactly?

It's the same as the 3g image cause I just needed something to test builds off and the hardware is very similar

@spacey-sooty spacey-sooty changed the title Add limelight-4 supportg Add limelight 4 support Mar 12, 2025
Signed-off-by: Jade Turner <[email protected]>
Signed-off-by: Jade Turner <[email protected]>
Signed-off-by: Jade Turner <[email protected]>
@spacey-sooty
Copy link
Member Author

This works when I modify a stable image but fresh images dont work due to libcamera abi break, working on fixing libcamera will then test this as a fresh image

@spacey-sooty
Copy link
Member Author

Works with PhotonVision/photonvision#1769

@spacey-sooty spacey-sooty marked this pull request as ready for review September 12, 2025 15:02
@@ -0,0 +1 @@
trigger ci
Copy link

Choose a reason for hiding this comment

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

Remove

# Install our new config.txt with OV9281 overlay
install -m 644 limelight4/config.txt /boot/

# Add the one extra file for the LL3
Copy link

Choose a reason for hiding this comment

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

LL3?

@@ -0,0 +1,45 @@
# Automatically load overlays for detected DSI displays/cameras
display_auto_detect=1
camera_auto_detect=1
Copy link

Choose a reason for hiding this comment

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

Auto detect, but we also set dtoverlay?

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.

4 participants