Skip to content
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

Orin Camera support #546

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

emrahbillur
Copy link
Contributor

@emrahbillur emrahbillur commented Apr 8, 2024

Description of changes

Checklist for things done

  • Summary of the proposed changes in the PR description
  • More detailed description in the commit message(s)
  • Commits are squashed into relevant entities - avoid a lot of minimal dev time commits in the PR
  • Contribution guidelines followed
  • Ghaf documentation updated with the commit - https://tiiuae.github.io/ghaf/
  • PR linked to architecture documentation and requirement(s) (ticket id)
  • Test procedure described (or includes tests). Select one or more:
    • Tested on Lenovo X1 x86_64
    • Tested on Jetson Orin NX or AGX aarch64
    • Tested on Polarfire riscv64
  • Author has run nix flake check --accept-flake-config and it passes
  • The above item never completes.
  • All automatic Github Action checks pass - see actions
  • Author has added reviewers and removed PR draft status

Testing

The tests will be performed with the camera hardware by another team so these are not available in Ghaf side. The software side tests include installation of v4l-tools to check the video devices of corresponding cameras.

  1. Execute lsmod |grep avt_csi2 to see
    avt_csi2 106496 0
    and lsmod |grep tc358743 to see
    tc358743 49152 1
  2. Check if /dev/video0 (Toshiba) and /dev/video1 (Alvium) and /dev/media0 video4linux interface devices exist with
    ls -la /dev/video0 /dev/video1 /dev/media0
  3. install v4l-tools with
    nix-shell -p v4l-utils
  4. Execute `v4l-ctl --list-devices' in the nix-shell to see the below result.
NVIDIA Tegra Video Input Device (platformtegra-camrtc-ca):
               /dev/media0
ALVIUM 1800 C-158m 9-3c (platformtegra-capture-vi:0):
               /dev/video1
tc358743 10-000f (platformtegra-capture-vi:2):
               /dev/video0
  1. Execute below commands to get more detailed information about resolution colorspace and formats:
v4l2-ctl -d /dev/video0 --list-formats-ext
v4l2-ctl -d /dev/video0 --list-ctrls-menus
v4l2-ctl --device /dev/video0 --stream-mmap
v4l2-ctl -d /dev/video1 --list-formats-ext
v4l2-ctl -d /dev/video1 --list-ctrls-menus
v4l2-ctl --device /dev/video1 --stream-mmap

And the --stream-mmap will show the data input from the capture device.

  1. install vlc with
    nix-shell -p vlc

@emrahbillur emrahbillur temporarily deployed to internal-build-workflow April 8, 2024 08:28 — with GitHub Actions Inactive
@emrahbillur emrahbillur had a problem deploying to external-build-workflow April 8, 2024 08:28 — with GitHub Actions Failure
@emrahbillur emrahbillur changed the title Camera support Camera support - DRAFT Apr 8, 2024
@emrahbillur emrahbillur temporarily deployed to internal-build-workflow April 8, 2024 11:31 — with GitHub Actions Inactive
@emrahbillur emrahbillur temporarily deployed to external-build-workflow April 8, 2024 11:32 — with GitHub Actions Inactive
@brianmcgillion brianmcgillion marked this pull request as draft April 8, 2024 13:08
@emrahbillur emrahbillur changed the title Camera support - DRAFT Orin Camera support - DRAFT Apr 8, 2024
@emrahbillur emrahbillur temporarily deployed to internal-build-workflow April 10, 2024 07:00 — with GitHub Actions Inactive
@emrahbillur emrahbillur temporarily deployed to internal-build-workflow April 10, 2024 07:02 — with GitHub Actions Inactive
@emrahbillur emrahbillur temporarily deployed to internal-build-workflow April 10, 2024 08:28 — with GitHub Actions Inactive
@emrahbillur
Copy link
Contributor Author

A new issue is detected as my camera modules are built with 6.1.28 build your own kernel version of nvidia instead of the 5.10 of jetpack-nixos. I'll try to figure out where it overrides my settings.

@emrahbillur emrahbillur temporarily deployed to internal-build-workflow April 24, 2024 12:47 — with GitHub Actions Inactive
@emrahbillur emrahbillur temporarily deployed to internal-build-workflow April 24, 2024 13:49 — with GitHub Actions Inactive
@emrahbillur emrahbillur temporarily deployed to internal-build-workflow April 24, 2024 14:32 — with GitHub Actions Inactive
@emrahbillur emrahbillur self-assigned this Apr 26, 2024
@emrahbillur emrahbillur changed the title Orin Camera support - DRAFT Orin Camera support Apr 26, 2024
@emrahbillur
Copy link
Contributor Author

Funny things happen in checks side. I've built this PR with AGX and NX both aarch64 and cross compile X86_64 and had no error while the checks report error.

@emrahbillur emrahbillur marked this pull request as ready for review April 26, 2024 09:53
Signed-off-by: Emrah Billur <emrah.billur@unikie.com>

Doc fix

Signed-off-by: Emrah Billur <emrah.billur@unikie.com>
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.

None yet

1 participant