Skip to content

media: ipu7: set CSI2 port count by hardware version - #105

Open
khaiwenng wants to merge 1 commit into
intel:mainfrom
khaiwenng:csi_port_fix
Open

khaiwenng wants to merge 1 commit into
intel:mainfrom
khaiwenng:csi_port_fix

Conversation

@khaiwenng

Copy link
Copy Markdown

IPU7 provides four CSI2 ports, while IPU7P5 and IPU8 provide three. The common platform data initialization currently uses the four-entry offset table for every hardware version, causing an invalid fourth CSI2 entity to be registered on IPU7P5 and IPU8.

Pass the PCI-selected hardware version to the platform data initializer and set the number of ports accordingly.

IPU7 provides four CSI2 ports, while IPU7P5 and IPU8 provide three.
The common platform data initialization currently uses the four-entry
offset table for every hardware version, causing an invalid fourth CSI2
entity to be registered on IPU7P5 and IPU8.

Pass the PCI-selected hardware version to the platform data initializer
and set the number of ports accordingly.

Signed-off-by: Khai Wen, Ng <khai.wen.ng@intel.com>
@khaiwenng

Copy link
Copy Markdown
Author

Before
media-ctl -p | grep -ie entity | grep -ie CSI

- entity 129: Intel IPU7 CSI2 0 (9 pads, 9 links, 4 routes)
- entity 139: Intel IPU7 CSI2 1 (9 pads, 8 links, 1 route)
- entity 149: Intel IPU7 CSI2 2 (9 pads, 8 links, 1 route)
- entity 159: Intel IPU7 CSI2 3 (9 pads, 8 links, 1 route)

After
media-ctl -p | grep -ie entity | grep -ie CSI

- entity 97: Intel IPU7 CSI2 0 (9 pads, 9 links, 1 route)
- entity 107: Intel IPU7 CSI2 1 (9 pads, 8 links, 1 route)
- entity 117: Intel IPU7 CSI2 2 (9 pads, 8 links, 1 route)

khaiwenng added a commit to intel/Intel-MIPI-CSI-Camera-Reference-Driver that referenced this pull request Sep 24, 2026
IPU7 supports 4 mipi ports while IPU7P5 (IPU75XA)
and IPU8 supports 3 mipi ports. Check hw_ver for nports.

Similar fix is submitted to ipu7-drivers for review.
intel/ipu7-drivers#105

Signed-off-by: Khai Wen, Ng <khai.wen.ng@intel.com>

This branch has not been deployed

No deployments
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.

1 participant