Skip to content

Update instruction for setting up TD2 on Compute Modules #4084

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

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

JamesH65
Copy link
Contributor

Supercedes #4072

@JamesH65
Copy link
Contributor Author

Something wrong here - will sort out tomorrow.

Update instruction for setting up TD2 on Compute Modules

Typo

Add information on setting up TD2 on Compute modules

Updated from review comments

Updated after review comments

URL fixup

Hopefully last fixup from review comments
@JamesH65 JamesH65 force-pushed the Update_TD2_setup_on_CMs branch from 225cee9 to 2d237cb Compare April 29, 2025 08:41
@JamesH65 JamesH65 requested a review from lurch April 29, 2025 08:43
@JamesH65
Copy link
Contributor Author

OK, sorted out all the merge issues. I think this is ready to go.

Copy link
Contributor

@lurch lurch left a comment

Choose a reason for hiding this comment

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

LGTM, but pinging @6by9 for one final check.

Copy link
Contributor

@6by9 6by9 left a comment

Choose a reason for hiding this comment

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

Mentioning J6 should be in there, but feel free to ignore the other two comments.

Unrelated, but rereading that page has also made me note that https://www.raspberrypi.com/documentation/computers/compute-module.html#connect-one-camera says dtoverlay=ov5647,cam1 etc.

There is no cam1 override, only cam0. Using cam1 is the default, and the firmware will be logging that cam1 is an unknown override.

@lurch lurch requested a review from 6by9 April 29, 2025 15:02

Touch Display 2 is a 720x1280 7" LCD display designed specifically for Raspberry Pi devices (see https://www.raspberrypi.com/products/touch-display-2/). It connects in the same way as the original touch display, but the software setup on Compute Modules is slightly different as it uses a different display driver. See xref:../accessories/touch-display-2.adoc[Touch Display 2] for connection details.

Edit the /boot/firmware/config.txt file and add the following to enable Touch Display 2 on DISP1/DSI1. You will also need to add jumpers to J6 as indicated on the silkscreen.
Copy link
Contributor

Choose a reason for hiding this comment

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

Looking at the earlier TD1 instructions, is this J6 jumper specific to CM4 IO board? 🤔

Copy link
Contributor

Choose a reason for hiding this comment

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

CM4IO J6 connects I2C to CAM0 and DISP0
CM5IO J6 connects I2C to CAM/DISP1.

No comment on consistency!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Its J6 on both.

Copy link
Contributor

Choose a reason for hiding this comment

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

(and we probably also want CM 1 / 3 / 4S hookup instructions for TD2?)

Copy link
Contributor

Choose a reason for hiding this comment

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

Are the jumper settings etc. identical for TD1 and TD2? In which case perhaps the "You will also need to add jumpers to J6 as indicated on the silkscreen." phrase here should be removed, and replaced with something like "See the earlier Touch Display instructions for details of required jumper settings." ?
Or (as a bigger change) perhaps it worth separating out the electrical-hookup instructions (jumpers etc.) which are the same for both TD1 and TD2, from the config.txt instructions which are different for TD1 and TD2? 🤔

Copy link
Contributor

Choose a reason for hiding this comment

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

Are the jumper settings etc. identical for TD1 and TD2

Yes. It's a board level thing, not what display is connected.

lurch added a commit that referenced this pull request Apr 29, 2025
Remove unnecessary `cam1` directives, as detailed in #4084 (review)
@lurch lurch mentioned this pull request Apr 29, 2025
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.

3 participants