| Message ID | 20250108143733.2761200-1-quic_depengs@quicinc.com |
|---|---|
| Headers | show |
| Series | media: qcom: camss: Add sm8550 support | expand |
On 08/01/2025 14:37, Depeng Shao wrote: > v8: > - Add correct version number for each patch - Krzysztof, Hans, Bryan > - Correct the copyright in patches [15/16] and [16/16] > - Link to v7: https://lore.kernel.org/all/20241225133523.4034820-1-quic_depengs@quicinc.com/ Patch #9 doesn't apply to media.git/next git remote add media git://linuxtv.org/media.git git fetch media git checkout -b media-next-25-10-01-camss-8550 media/next b4 shazam 20250108143733.2761200-1-quic_depengs@quicinc.com Grabbing thread from lore.kernel.org/all/20250108143733.2761200-1-quic_depengs@quicinc.com/t.mbox.gz Checking for newer revisions Grabbing search results from lore.kernel.org Analyzing 17 messages in the thread Analyzing 260 code-review messages Checking attestation on all messages, may take a moment... --- ✓ [PATCH v8 1/16] media: qcom: camss: csiphy-3ph: Fix trivial indentation fault in defines ✓ [PATCH v8 2/16] media: qcom: camss: csiphy-3ph: Remove redundant PHY init sequence control loop ✓ [PATCH v8 3/16] media: qcom: camss: csiphy-3ph: Rename struct ✓ [PATCH v8 4/16] media: qcom: camss: csiphy: Add an init callback to CSI PHY devices ✓ [PATCH v8 5/16] media: qcom: camss: csiphy-3ph: Move CSIPHY variables to data field inside csiphy struct ✓ [PATCH v8 6/16] media: qcom: camss: csiphy-3ph: Use an offset variable to find common control regs ✓ [PATCH v8 7/16] media: qcom: camss: csid: Move common code into csid core ✓ [PATCH v8 8/16] media: qcom: camss: vfe: Move common code into vfe core ✓ [PATCH v8 9/16] media: qcom: camss: Add callback API for RUP update and buf done ✓ [PATCH v8 10/16] media: qcom: camss: Add default case in vfe_src_pad_code ✓ [PATCH v8 11/16] media: qcom: camss: csid: Only add TPG v4l2 ctrl if TPG hardware is available ✓ [PATCH v8 12/16] dt-bindings: media: camss: Add qcom,sm8550-camss binding ✓ [PATCH v8 13/16] media: qcom: camss: Add sm8550 compatible ✓ [PATCH v8 14/16] media: qcom: camss: csiphy-3ph: Add Gen2 v2.1.2 two-phase MIPI CSI-2 DPHY support ✓ [PATCH v8 15/16] media: qcom: camss: Add CSID 780 support ✓ [PATCH v8 16/16] media: qcom: camss: Add support for VFE 780 --- ✓ Signed: DKIM/quicinc.com --- Total patches: 16 --- Base: using specified base-commit 8155b4ef3466f0e289e8fcc9e6e62f3f4dceeac2 Applying: media: qcom: camss: csiphy-3ph: Fix trivial indentation fault in defines Applying: media: qcom: camss: csiphy-3ph: Remove redundant PHY init sequence control loop Applying: media: qcom: camss: csiphy-3ph: Rename struct Applying: media: qcom: camss: csiphy: Add an init callback to CSI PHY devices Applying: media: qcom: camss: csiphy-3ph: Move CSIPHY variables to data field inside csiphy struct Applying: media: qcom: camss: csiphy-3ph: Use an offset variable to find common control regs Applying: media: qcom: camss: csid: Move common code into csid core Applying: media: qcom: camss: vfe: Move common code into vfe core Applying: media: qcom: camss: Add callback API for RUP update and buf done Patch failed at 0009 media: qcom: camss: Add callback API for RUP update and buf done error: patch failed: drivers/media/platform/qcom/camss/camss.c:2454
Hi Bryan, On 1/10/2025 9:50 PM, Bryan O'Donoghue wrote: > On 08/01/2025 14:37, Depeng Shao wrote: >> v8: >> - Add correct version number for each patch - Krzysztof, Hans, Bryan >> - Correct the copyright in patches [15/16] and [16/16] >> - Link to v7: https://lore.kernel.org/all/20241225133523.4034820-1- >> quic_depengs@quicinc.com/ > Patch #9 doesn't apply to media.git/next > Yes, below patch[1] is merged these days, so my series get conflict. I will rebase my series and verify it next Monday. [1] Revert "media: qcom: camss: Restructure camss_link_entities" > git remote add media git://linuxtv.org/media.git > git fetch media > > git checkout -b media-next-25-10-01-camss-8550 media/next > > b4 shazam 20250108143733.2761200-1-quic_depengs@quicinc.com > Grabbing thread from lore.kernel.org/all/20250108143733.2761200-1- > quic_depengs@quicinc.com/t.mbox.gz > Checking for newer revisions > Grabbing search results from lore.kernel.org > Analyzing 17 messages in the thread > Analyzing 260 code-review messages > Checking attestation on all messages, may take a moment... > --- > ✓ [PATCH v8 1/16] media: qcom: camss: csiphy-3ph: Fix trivial > indentation fault in defines > ✓ [PATCH v8 2/16] media: qcom: camss: csiphy-3ph: Remove redundant > PHY init sequence control loop > ✓ [PATCH v8 3/16] media: qcom: camss: csiphy-3ph: Rename struct > ✓ [PATCH v8 4/16] media: qcom: camss: csiphy: Add an init callback to > CSI PHY devices > ✓ [PATCH v8 5/16] media: qcom: camss: csiphy-3ph: Move CSIPHY > variables to data field inside csiphy struct > ✓ [PATCH v8 6/16] media: qcom: camss: csiphy-3ph: Use an offset > variable to find common control regs > ✓ [PATCH v8 7/16] media: qcom: camss: csid: Move common code into > csid core > ✓ [PATCH v8 8/16] media: qcom: camss: vfe: Move common code into vfe > core > ✓ [PATCH v8 9/16] media: qcom: camss: Add callback API for RUP update > and buf done > ✓ [PATCH v8 10/16] media: qcom: camss: Add default case in > vfe_src_pad_code > ✓ [PATCH v8 11/16] media: qcom: camss: csid: Only add TPG v4l2 ctrl > if TPG hardware is available > ✓ [PATCH v8 12/16] dt-bindings: media: camss: Add qcom,sm8550-camss > binding > ✓ [PATCH v8 13/16] media: qcom: camss: Add sm8550 compatible > ✓ [PATCH v8 14/16] media: qcom: camss: csiphy-3ph: Add Gen2 v2.1.2 > two-phase MIPI CSI-2 DPHY support > ✓ [PATCH v8 15/16] media: qcom: camss: Add CSID 780 support > ✓ [PATCH v8 16/16] media: qcom: camss: Add support for VFE 780 > --- > ✓ Signed: DKIM/quicinc.com > --- > Total patches: 16 > --- > Base: using specified base-commit > 8155b4ef3466f0e289e8fcc9e6e62f3f4dceeac2 > Applying: media: qcom: camss: csiphy-3ph: Fix trivial indentation fault > in defines > Applying: media: qcom: camss: csiphy-3ph: Remove redundant PHY init > sequence control loop > Applying: media: qcom: camss: csiphy-3ph: Rename struct > Applying: media: qcom: camss: csiphy: Add an init callback to CSI PHY > devices > Applying: media: qcom: camss: csiphy-3ph: Move CSIPHY variables to data > field inside csiphy struct > Applying: media: qcom: camss: csiphy-3ph: Use an offset variable to find > common control regs > Applying: media: qcom: camss: csid: Move common code into csid core > Applying: media: qcom: camss: vfe: Move common code into vfe core > Applying: media: qcom: camss: Add callback API for RUP update and buf done > Patch failed at 0009 media: qcom: camss: Add callback API for RUP update > and buf done > error: patch failed: drivers/media/platform/qcom/camss/camss.c:2454 > Thanks, Depeng
v8: - Add correct version number for each patch - Krzysztof, Hans, Bryan - Correct the copyright in patches [15/16] and [16/16] - Link to v7: https://lore.kernel.org/all/20241225133523.4034820-1-quic_depengs@quicinc.com/ v7: - Due to the patches in https://lore.kernel.org/all/20241217140656.965235-1-quic_vikramsa@quicinc.com/ are merged, so rebased below patches to fix the conflict. media: qcom: camss: csiphy-3ph: Remove redundant PHY init sequence control loop media: qcom: camss: csiphy-3ph: Move CSIPHY variables to data field inside csiphy struct media: qcom: camss: Add sm8550 compatible media: qcom: camss: csiphy-3ph: Move CSIPHY variables to data field inside csiphy struct media: qcom: camss: csiphy-3ph: Add Gen2 v2.1.2 two-phase MIPI CSI-2 DPHY support media: qcom: camss: Add support for VFE 780 - Add RB for "media: qcom: camss: Add CSID 780 support" - Bryan - Use TAG name for ICC and remove offline HW ICC - Bryan - Remove the logic that moving enable_irq();/disable_irq(); to wm_start() and wm_stop() to make sure no logical change in VFE refactor change. - Update the commit message and title for the TPG change. - Bryan - Link to v6: https://lore.kernel.org/all/20241211140738.3835588-1-quic_depengs@quicinc.com/ v6: - Add bus type property in dt-binding which will be limited by a latest change https://lore.kernel.org/all/20241209-camss-dphy-v1-0-5f1b6f25ed92@fairphone.com/ - Add RB for "media: qcom: camss: Add sm8550 compatible" and "media: qcom: camss: Add support for VFE 780" - Uppercase the hex in csiphy register list - Bryan - Add empty function for csid tpg - Vladimir - Set testgen mode to CSID_PAYLOAD_MODE_DISABLED in subdev init interface - encapsulate the guard __thus__ for new header - Bryan - Add a standalone patch for the platform which doesn't support CSID TPG to make sure new platform driver can set CSID_PAYLOAD_MODE_DISABLED to disable TPG - Update the csid for csid and vfe driver - Bryan - Link to v5: https://lore.kernel.org/all/20241205155538.250743-1-quic_depengs@quicinc.com/ v5: - Update dt-bindings required items order - Krzysztof - Sort the reg order based on the comments in sc7280 dt-binding - Vladimir - Change the irq type to IRQ_TYPE_EDGE_RISING - Vladimir - Remove the Krzysztof's RB tag from dt-binding patch due to above updates in dt-binding patch - Move regulator from csid resource to csiphy resource - Bryan, Vladimir - Move the change to add default case in vfe_src_pad_code to a standalone patch. - Bryan - Rename csid-gen3 as csid-780 - Bryan - use macros() to bury bit shifts - Bryan - Sort the macros by register offset order - Vladimir - Redefine a macro for rup_aup in csid driver - Vladimir - Remove the unused macros in vfe 780 driver - Vladimir - Add dummy function for unsupported hw_ops in vfe 780 driver - Vladimir, Bryan - Use a standalone patch for the callback API of RUP and buf done update - Use a standalone patch to make CSID TPG optional - Vladimir - Link to v4: https://lore.kernel.org/all/20240812144131.369378-1-quic_depengs@quicinc.com/ v4: - Update dt-bindings based on comments - Krzysztof, bod, Vladimir - Move common code into csid core and vfe core driver - bod - Remove *_relaxed in the csid and vfe drivers - Krzysztof - Reorganize patches in logical junks, make sure that new added structures have users in current patch - Krzysztof - Remove notify function and add new functions in camss for buf done and reg update - bod - Remove custom code to get csid base - bod - Remove ISR function in vfe780 driver since it is never fired - bod - Move csid_top_base to camss structure since we only have one csid top block, and just need to get base once for csid top - Add Vladimir's RB - Remove prerequisite-patch-id in the cover letter since the changes have been merged - Add dtsi patch link for reference - Krzysztof https://lore.kernel.org/all/20240807123333.2056518-1-quic_depengs@quicinc.com/ - Link to v3: https://lore.kernel.org/all/20240709160656.31146-1-quic_depengs@quicinc.com/ v3: - Rebased the change based on below change which will be merged firstly. "Move camss version related defs in to resources" Link: https://lore.kernel.org/all/20240522154659.510-1-quic_grosikop@quicinc.com/ - Rebased the change based on Bryan's csiphy optimization change and add these changes into this series, so that the new csiphy-3ph driver don't need to add duplicate code. This has got Bryan's permission to add his patches into this series. - Refactor some changes based on the comments to move the random code to patches where they are used. - Remove the vfe780 irq function since it isn't doing the actual work. - Add dt-binding for sm8550 camss driver. Link to V2: https://lore.kernel.org/all/20240320141136.26827-1-quic_depengs@quicinc.com/ v2: - Update some commit messages Link to V1: https://lore.kernel.org/all/20240320134227.16587-1-quic_depengs@quicinc.com/ v1: SM8550 is a Qualcomm flagship SoC. This series adds support to bring up the CSIPHY, CSID, VFE/RDI interfaces in SM8550. SM8550 provides - 3 x VFE, 3 RDI per VFE - 2 x VFE Lite, 4 RDI per VFE - 3 x CSID - 2 x CSID Lite - 8 x CSI PHY Bryan O'Donoghue (6): media: qcom: camss: csiphy-3ph: Fix trivial indentation fault in defines media: qcom: camss: csiphy-3ph: Remove redundant PHY init sequence control loop media: qcom: camss: csiphy-3ph: Rename struct media: qcom: camss: csiphy: Add an init callback to CSI PHY devices media: qcom: camss: csiphy-3ph: Move CSIPHY variables to data field inside csiphy struct media: qcom: camss: csiphy-3ph: Use an offset variable to find common control regs Depeng Shao (10): media: qcom: camss: csid: Move common code into csid core media: qcom: camss: vfe: Move common code into vfe core media: qcom: camss: Add callback API for RUP update and buf done media: qcom: camss: Add default case in vfe_src_pad_code media: qcom: camss: csid: Only add TPG v4l2 ctrl if TPG hardware is available dt-bindings: media: camss: Add qcom,sm8550-camss binding media: qcom: camss: Add sm8550 compatible media: qcom: camss: csiphy-3ph: Add Gen2 v2.1.2 two-phase MIPI CSI-2 DPHY support media: qcom: camss: Add CSID 780 support media: qcom: camss: Add support for VFE 780 .../bindings/media/qcom,sm8550-camss.yaml | 597 +++++++++++++ drivers/media/platform/qcom/camss/Makefile | 2 + .../platform/qcom/camss/camss-csid-4-1.c | 19 - .../platform/qcom/camss/camss-csid-4-7.c | 42 - .../platform/qcom/camss/camss-csid-780.c | 337 ++++++++ .../platform/qcom/camss/camss-csid-780.h | 25 + .../platform/qcom/camss/camss-csid-gen2.c | 60 -- .../media/platform/qcom/camss/camss-csid.c | 137 ++- .../media/platform/qcom/camss/camss-csid.h | 31 + .../qcom/camss/camss-csiphy-2ph-1-0.c | 6 + .../qcom/camss/camss-csiphy-3ph-1-0.c | 794 ++++++++++-------- .../media/platform/qcom/camss/camss-csiphy.c | 4 + .../media/platform/qcom/camss/camss-csiphy.h | 8 + .../media/platform/qcom/camss/camss-vfe-17x.c | 112 +-- .../media/platform/qcom/camss/camss-vfe-4-1.c | 9 - .../media/platform/qcom/camss/camss-vfe-4-7.c | 11 - .../media/platform/qcom/camss/camss-vfe-4-8.c | 11 - .../media/platform/qcom/camss/camss-vfe-480.c | 274 +----- .../media/platform/qcom/camss/camss-vfe-780.c | 159 ++++ drivers/media/platform/qcom/camss/camss-vfe.c | 274 ++++++ drivers/media/platform/qcom/camss/camss-vfe.h | 59 +- drivers/media/platform/qcom/camss/camss.c | 359 ++++++++ drivers/media/platform/qcom/camss/camss.h | 4 + 23 files changed, 2464 insertions(+), 870 deletions(-) create mode 100644 Documentation/devicetree/bindings/media/qcom,sm8550-camss.yaml create mode 100644 drivers/media/platform/qcom/camss/camss-csid-780.c create mode 100644 drivers/media/platform/qcom/camss/camss-csid-780.h create mode 100644 drivers/media/platform/qcom/camss/camss-vfe-780.c base-commit: 8155b4ef3466f0e289e8fcc9e6e62f3f4dceeac2