IntelRealSense / realsense_mipi_platform_driver

Intel® RealSense™ camera driver for GMSL* interface
GNU General Public License v2.0
15 stars 12 forks source link

Jp5 2d457 1 des #160

Closed dmipx closed 1 year ago

dmipx commented 1 year ago

update for compatibility with development platform

Nir-Az commented 1 year ago

Was this tested with JP5 and JP4 as working?

dmipx commented 1 year ago

This was tested on JP5 single and dual camera and on JP4

dmipx commented 1 year ago

@jnzw the JP4 d4xx driver is not aligned to new cool serdes management of JP5. can we merge this feature to jp4?

jnzw commented 1 year ago

@jnzw the JP4 d4xx driver is not aligned to new cool serdes management of JP5. can we merge this feature to jp4?

I suppose you mean the dual D457 feature (#144) for JP4? I think it's doable, but it will require non-trivial effort to implement and validate. The d4xx and serdes drivers diverged quite a bit between JP4 and JP5.

dmipx commented 1 year ago

@jnzw the JP4 d4xx driver is not aligned to new cool serdes management of JP5. can we merge this feature to jp4?

I suppose you mean the dual D457 feature (#144) for JP4? I think it's doable, but it will require non-trivial effort to implement and validate. The d4xx and serdes drivers diverged quite a bit between JP4 and JP5.

Not the "Dual" for JP4 but d4xx/serdes development. i can merge this feature but all commits with history will be lost. I will use same driver for Alderlake, we want to have only one copy of the driver to maintain.

We also have issue with y12i on viewer on JP4 and it resolved on dual version.

jnzw commented 1 year ago

We also have issue with y12i on viewer on JP4 and it resolved on dual version.

That's strange. There was an issue with Y12I indeed but it was with JP5 and it should have been fixed by #145. I don't think I had issue with y12i with JP4 before doing porting.