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

Add imx335 sensor driver with higher FPS modes for FPV #21

Merged
merged 4 commits into from
Feb 16, 2024

Conversation

tipoman9
Copy link
Contributor

The driver is added in a separate folder, compiles, but does not interfere with others.
Needs to be copied by BuildRoot together with its ini to the camera in:

/usr/lib/sensors/libsns_imx335_fpv.so
/etc/sensors/imx335_fpv.ini

Then it can be used if requested by majestic.yaml

  sensorConfig: /etc/sensors/imx335_fpv.ini

The driver needs conditional compilation to fine tune some modes depending on Goke or Hisilicon platform, since it seems ISP does not perform the same.

Sensor modes added:

2592x1520 50fps , slightly cropped vertically to 16:9

for on hi356ev300 : Isp_FrameRate=50 , majestic set to 1920x1080 50fps => 50fps
for gk7205v300 : Isp_FrameRate=48 , majestic set to 1920x1080 48fps => 47fps

2592x1944 fullscale mode boosted to 40fps

for on hi356ev300 set Isp_FrameRate=45, majestic to 1920x1080 fps: 45 to get 39fps
for on gk7205v300 set Isp_FrameRate=36, majestic to 1920x1080 fps: 36 to get 33fps

1920x1080 cropped, max 90fps , zoom 1.5x

Isp_FrameRate=90 # can work at 90fps only when majestic is set to 1280x720 !!!
Max Isp_FrameRate=55 for 1920x1080

1296x972 binning max 65fps,

Isp_FrameRate=68 # # max supported value on hi356ev300 at 1280x720

Copy link
Member

@flyrouter flyrouter left a comment

Choose a reason for hiding this comment

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

Thank you, sir

@flyrouter flyrouter merged commit e81d290 into OpenIPC:main Feb 16, 2024
0 of 2 checks passed
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