media: qcom: add X1E80100 C-PHY camera receive support - #86
Draft
ooaklee wants to merge 4 commits into
Draft
Conversation
Document the Sony IMX681 image sensor and its currently validated 3840x2640 RAW10 profile. The profile uses a single CSI-2 C-PHY trio at 2406 Msymbol/s and an external 19.2 MHz clock. List turbinebmw as maintainer, following the original public camera review series from which the sensor work was developed. Link: https://github.com/turbineBMW/surface-pro-11-linux/blob/05e5335bc72476d44390336701cf03efa5fd0165/kernel/sp11-camera-review.patch Link: d915d67 Signed-off-by: Leon Silcott <leon@boasi.io>
Add a standalone V4L2 driver for the IMX681 front-camera mode validated on the Surface Pro 11. The driver programs 3840x2640 RAW10 at about 30 frames per second and supports frame length, 24-bit exposure, analog gain, and digital gain controls. Reject firmware descriptions other than the validated one-trio C-PHY topology, 1.203 GHz V4L2 link frequency, and 19.2 MHz input clock before powering the sensor. This keeps the observed register sequence scoped to the hardware profile on which it was tested. The register tables and initial driver were developed from turbinebmw's public camera review series, then corrected and runtime-validated in the SP11 integration branch. Link: https://github.com/turbineBMW/surface-pro-11-linux/blob/05e5335bc72476d44390336701cf03efa5fd0165/kernel/sp11-camera-review.patch Link: d915d67 Signed-off-by: Leon Silcott <leon@boasi.io>
Add an explicit MIPI C-PHY mode and teach the X1E80100 combo PHY to accept only the validated one-trio, 2.406-Gsymbol/s receiver profile. Program the bounded X1E register sequence, clear its interrupt state, and apply its shutdown sequence while the PHY clocks remain available. Keep the existing D-PHY configuration and shutdown paths unchanged. The C-PHY clock vote, interrupt, programming, and shutdown paths are selected only after the consumer requests C-PHY and the X1E capability data accepts trio 0 and the exact symbol rate. The register observations were independently transcribed from runtime MMIO activity on the Surface Pro 11. This port builds on turbineBMW's public CAMSS review series and geocausa's Windows/QTI parity analysis. Link: https://github.com/turbineBMW/surface-pro-11-linux/blob/05e5335bc72476d44390336701cf03efa5fd0165/kernel/sp11-camera-review.patch Link: d915d67 Signed-off-by: Leon Silcott <leon@boasi.io>
Accept C-PHY endpoints only for X1E80100 CAMSS and only for the validated one-trio topology at the 1.203-GHz V4L2 link frequency. Propagate the bus type through CSIPHY and CSID, select the X1E receive clocks, and program the CSID C-PHY receiver fields. Leave the existing D-PHY clock, lane, and receiver programming paths unchanged. This re-lift excludes the source integration's VFE write-mode, CSID crop and drop-engine changes, status logging, and sensor-specific format handling. Non-SP11 D-PHY hardware testing remains required. The receiver routing builds on turbineBMW's public CAMSS review series and geocausa's Windows/QTI parity analysis. Link: https://github.com/turbineBMW/surface-pro-11-linux/blob/05e5335bc72476d44390336701cf03efa5fd0165/kernel/sp11-camera-review.patch Link: d915d67 Signed-off-by: Leon Silcott <leon@boasi.io>
This was referenced Aug 31, 2026
ooaklee
added a commit
to ooaklee/linux_ms_dev_kit-sp11
that referenced
this pull request
Sep 1, 2026
Integrate jglathe#86 for combined SP11 testing.
jglathe
force-pushed
the
jg/ubuntu-qcom-x1e-7.2.y
branch
from
September 3, 2026 10:32
87bcf07 to
4943ee7
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Dependency and status
This PR is intended to target
jg/ubuntu-qcom-x1e-7.2.ydirectly. Review it after #82, #83, and the preceding IMX681 sensor topic:The head is stacked on
x/sp11-camera-imx681, so its GitHub diff is cumulative until the sensor topic is merged into the release branch. The C-PHY-specific portion is the two commits afterddc5bc6c2804. It does not merge the broad-sp11staging branch or replay the correction already merged through #81.This PR is a draft. The Denali OLED camera graph follows separately, and both the exact cumulative SP11 stack and an existing non-SP11 X1E D-PHY camera need hardware testing before this shared-driver topic is ready.
Containment
qcom,x1e80100-camsscompatiblePHY_MODE_MIPI_CPHY, trio 0, one data lane/trio, and the exact 2.406 Gsymbol/s ratePHY_MODE_MIPI_CPHYenum addition is inert until a consumer explicitly requests that modeProvenance
The two C-PHY commits are authored by Leon Silcott, carry the authorized
Signed-off-by: Leon Silcott <leon@boasi.io>, and preserve source links to the final integration state in d915d679423e.The receiver work builds on @turbineBMW's public camera review patch at immutable revision
05e5335bc724and credits @geocausa's Windows/QTI parity analysis in the commit messages. These are source and analysis credits; no contributorSigned-off-byorCo-authored-bytrailer has been invented for the rewritten commits.Validation
git diff --check ddc5bc6c2804..HEADfor the incremental C-PHY topicQUALCOMM MIPI CSI2 PHY DRIVERentryubuntu_x1e_defconfigW=1builds of the two Qualcomm MIPI CSI-2 PHY objects and the four affected CAMSS/CSIPHY/CSID objectsHardware checks still required