-
Notifications
You must be signed in to change notification settings - Fork 22
Fix readme #309
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
base: dev
Are you sure you want to change the base?
Fix readme #309
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the documentation to split the generic JetPack 6.x guide into separate guides for JetPack 6.0 and 6.2, improving specificity and clarity for users working with different versions.
Key Changes:
- Split README_JP6.md into version-specific documentation (README_JP6.0.md and README_JP6.2.md)
- Updated README_JP6.2.md with improved deployment instructions and correct version references
- Updated main README.md links to point to the new version-specific guides
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.
| File | Description |
|---|---|
| README_JP6.2.md | Updated deployment instructions, reorganized steps for clarity, and corrected version references from 6.0 to 6.2 |
| README_JP6.0.md | New file containing JetPack 6.0-specific setup guide with correct version references throughout |
| README.md | Updated navigation links to reference new version-specific documentation files |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| Assuming user 'nvidia' on Jetson with ip: `10.0.0.116` (if building natively on Jetson use $USER@localhost): | ||
|
|
||
| 1. Copy build artifacts: | ||
| If you build locally use those commands: |
Copilot
AI
Nov 13, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected grammar: 'use those commands' should be 'use these commands'.
| If you build locally use those commands: | |
| If you build locally use these commands: |
| sudo cp ./images/6.2/rootfs/boot/Image /boot/ | ||
| ``` | ||
| On Jetson target (user home folder) assuming backup step was followed: | ||
| In case of scp copy from host use this commands: |
Copilot
AI
Nov 13, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected grammar: 'use this commands' should be 'use these commands'.
| In case of scp copy from host use this commands: | |
| In case of scp copy from host use these commands: |
| 4. | ||
| Verify bootloader configuration |
Copilot
AI
Nov 13, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Step 4 heading is incomplete. The line '4.' should include the full description on the same line as 'Verify bootloader configuration' for consistency with other numbered steps.
| 4. | |
| Verify bootloader configuration | |
| 4. Verify bootloader configuration |
|
|
||
| ``` | ||
| # Configuration files | ||
| tar czf rootfs.tar.gz -C images/6.2/rootfs boot lib |
Copilot
AI
Nov 13, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Incorrect path reference: This file is for JetPack 6.0, but the tar command references 'images/6.2/rootfs'. It should reference 'images/6.0/rootfs' instead.
| tar czf rootfs.tar.gz -C images/6.2/rootfs boot lib | |
| tar czf rootfs.tar.gz -C images/6.0/rootfs boot lib |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-Pilot is right
| Following steps required: | ||
|
|
||
| 1. Copy build artifacts: | ||
| If you build locally use those commands: |
Copilot
AI
Nov 13, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected grammar: 'use those commands' should be 'use these commands'.
| If you build locally use those commands: | |
| If you build locally use these commands: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right again
| sudo cp ./images/6.0/rootfs/boot/dtb/tegra234-p3737-0000+p3701-0000-nv.dtb /boot/dtb/. | ||
| sudo cp ./images/6.0/rootfs/boot/Image /boot/ | ||
| ``` | ||
| In case of scp copy from host use this commands: |
Copilot
AI
Nov 13, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected grammar: 'use this commands' should be 'use these commands'.
| In case of scp copy from host use this commands: | |
| In case of scp copy from host use these commands: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
| 4. | ||
| Verify bootloader configuration |
Copilot
AI
Nov 13, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Step 4 heading is incomplete. The line '4.' should include the full description on the same line as 'Verify bootloader configuration' for consistency with other numbered steps.
| 4. | |
| Verify bootloader configuration | |
| 4. Verify bootloader configuration |
|
|
||
| ## JetPack manual build - cross compile x86-64 (CI deploy) | ||
|
|
||
| [NVIDIA® JetPack 6.2: Jetson Linux 36.4.3](https://developer.nvidia.com/embedded/jetson-linux-r3643) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is 6.0 doc
| ## JetPack manual build - cross compile x86-64 (CI deploy) | ||
|
|
||
| [NVIDIA® JetPack 6.2: Jetson Linux 36.4.3](https://developer.nvidia.com/embedded/jetson-linux-r3643) | ||
| 1. Download Jetson Linux Driver Package - [JetPack 6.2 BSP sources](https://developer.nvidia.com/downloads/embedded/l4t/r36_release_v4.3/release/jetson_linux_r36.4.3_aarch64.tbz2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
| sudo cp ./boot/dtb/tegra234-p3737-0000+p3701-0000-nv.dtb /boot/dtb/. | ||
| sudo cp ./boot/Image /boot/ | ||
| ``` | ||
| 2. Run $ `sudo /opt/nvidia/jetson-io/jetson-io.py`, to exit choose save & reboot: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to make sure why it didn't work when I ran this readme on a clean machine
No description provided.