Skip to content

L4T UVC/HID manual patch for JP6 - #13055

Merged
Nir-Az merged 1 commit into
realsenseai:developmentfrom
dmipx:l4t-patch-jp6
Jun 20, 2024
Merged

Nir-Az merged 1 commit into
realsenseai:developmentfrom
dmipx:l4t-patch-jp6

Conversation

@dmipx

@dmipx dmipx commented Jun 19, 2024

Copy link
Copy Markdown

Jetson Orin™ with JetPack 6.0 manual patches for UVC compatibility with LRS

Tracked-by: LRS-1066

@dmipx
dmipx requested a review from Nir-Az June 19, 2024 13:28
Comment thread doc/installation_jetson.md Outdated
⮕ Use the V4L Native backend by applying the kernel patching

The method was verified with **Jetson AGX** boards with JetPack **4.4**[L4T 32.4.3], **4.6.1**[L4T 32.7.1] and **5.0.2**[L4T 35.1.0].
The method was verified with **Jetson Orin™** with JetPack 6.0, **Jetson AGX** boards with JetPack **4.2.3**[L4T 32.2.1,32.2.3], **4.3**[L4T 32.3.1], **4.4**[L4T 32.4.3], **4.6.1**[L4T 32.7.1] and **5.0.2**[L4T 35.1.0].

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can you please reduce the tested version of the JP?
**4.4**[L4T 32.4.3], **4.6.1**[L4T 32.7.1] and **5.0.2**[L4T 35.1.0].
I did the same on the public website

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Orin is also AGX
So I suggest:
The method was verified with **Jetson AGX Orin™** with JetPack 6.0, **Jetson Xavier AGX** boards

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

done


# verify that git is installed
if ! which git > /dev/null ; then
echo "ERROR: git is not installed. If your linux distro is 10.04 or later,"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We can remove the part about linux 10.04 right?
Too old

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Oh this file is taken from NVIDIA directly right?
If so ignore my last comment :)

Comment thread doc/installation_jetson.md Resolved
Comment thread scripts/patch-realsense-ubuntu-L4T.sh Resolved
Comment thread scripts/patch-realsense-ubuntu-L4T.sh Outdated
esac
echo -e "\e[32mL4T ${JETSON_L4T_VERSION} to use patches revision ${PATCHES_REV}\e[0m"

# TODO: JP6.0 fails on:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What is that?
Will it fail? is that comment for the users or it works now and this is leftovers?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Lately JP6.0 was updated to revision 0.2 so this package was updated and i seen no more issue with that. I will remove comment.

git init
git remote add origin git://nv-tegra.nvidia.com/linux-${KERNEL_RELEASE}
# Use NVIDIA script instead to synchronize source tree and peripherals
#git clone git://nv-tegra.nvidia.com/linux-${KERNEL_RELEASE}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We shouldn't clone?

@dmipx dmipx Jun 20, 2024 •

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

No, getting tags only

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

So lets remove that?

cp ./scripts/Tegra/$TEGRA_SOURCE_SYNC_SH ${sdk_dir}/Tegra
#Download NVidia source, disregard errors on module tag sync
#Download NVIDIA source, disregard errors on module tag sync
./Tegra/$TEGRA_SOURCE_SYNC_SH -k ${TEGRA_TAG} || true

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This will will require thorough testing. Please open another ticket for that.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

NP

@dmipx
dmipx force-pushed the l4t-patch-jp6 branch 2 times, most recently from 1b4eb92 to f67f1ec Compare June 20, 2024 09:10
**Jetson Orin™** with JetPack 6.0 manual patches for UVC compatibility with LRS

Tracked-by: LRS-1066

Signed-off-by: Dmitry Perchanov <dmitry.perchanov@intel.com>
@Nir-Az

Nir-Az commented Jun 20, 2024

Copy link
Copy Markdown
Collaborator

Please try to avoid force push on review changes, now I need to go over it all again..

@Nir-Az
Nir-Az merged commit 5af61fb into realsenseai:development Jun 20, 2024
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.

2 participants