From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7C8E6335BB6 for ; Thu, 16 Oct 2025 13:01:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760619683; cv=none; b=lygBEnleIyKvtT9C/mbTa42bPGsI74TPrnY6BzdHZGlpTzt2TcVRac9ENWaGmPmG9ZtBROizPP8tIXflmmZXotSnBeYxLZivT8ZHncVNz2ODOwS7xzrpUdEEB7LBtz2FIfXZV17su879vODXnY0LxnC+yB694D47JtyeQTeJ6Ug= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760619683; c=relaxed/simple; bh=qXb1XoO8Rcab2MwMwV+iHpjwImn+WrVqCpt7sZr6S+I=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=Ugrn9TpO06uh1o8oPn+1p0AFsW6j9vKSzQHWmujgjbhRnkPsq60wUV1D535xkZSQ9if2VnwboWuo/TY/H3A9S3/IFK4CybkZwPwSu4ViDs4Y4uAEXpyIQkOnny4604k6gEfLIs5L29qrowaC1gUtQ3rE4AizDrjzj2QKeYeNjtw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=H4Pu6sI6; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="H4Pu6sI6" Received: from pps.filterd (m0279873.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 59G7TfHA016695 for ; Thu, 16 Oct 2025 13:01:20 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= lOoCmU3FK8p4YHctcwVkp7NyEzPdsNMWVTf99V0SNrE=; b=H4Pu6sI6SfYCZoNv DPzGruOLqKRTVLTOaFOMP/wSawUuQWp5R4UxceBhvYN1+FUAApxbbc94dzXkZTsJ NU4iy4adLXNlts40YBdM7eVJtrtpK9I9ki8qDC9vDxiqEbhfeAZPsUbTWCUK8WW1 Bx0Li/FZK63IiZv0UvPveSiB1MFgchv7bJJMZNytdUnnTZG7wh60z56eTjZiU8hi f9UchSdO+pMJW4hOTRq3TFniAZfqiDZvfNKIhlxfDAP/P0R+sCyY8KCZrtyxo6ot GaoGch/EbtJKoOyozGWzn/gC1Y65M5FkPDuvu/NJR8p07rY20ehN+35/8snnjyEB vYb7TA== Received: from mail-qk1-f197.google.com (mail-qk1-f197.google.com [209.85.222.197]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 49rtrtda5h-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Thu, 16 Oct 2025 13:01:20 +0000 (GMT) Received: by mail-qk1-f197.google.com with SMTP id af79cd13be357-88e133107adso274734585a.1 for ; Thu, 16 Oct 2025 06:01:20 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1760619679; x=1761224479; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=lOoCmU3FK8p4YHctcwVkp7NyEzPdsNMWVTf99V0SNrE=; b=FQ3BWDTX9LSdvRhswlL/Gh5WvzBuKYqswtvWl35eBYKabvQG0zRSfVVnVh6mYTT/Bw +H9gcOlmMEvxm4C4KQOIxAOvc7ar6Akt/tM6IWvDhnHGsTNVpAiAB1j4LSc6clsx2bZQ rE4lMRvKfILZ6Lak+7UsiEcYaIvuGBnbMcWTSR+nt4VxVByXh3mCj4rPzwu8srgyJWuf xoymA3JET4lBSY/38O4FA22zPs9mHJiu8MMMDzZZE/90IGuFs5hgN3FCG2I929O0bgxy PNnF7tssk1gk8/nBGMM6czZHO1uveix1aK5QWx98T7D2xMvINF9xxvY5oLwSYCP4nyfG ZIhA== X-Forwarded-Encrypted: i=1; AJvYcCXz955+GSZx4d5mPtGWQPS6SM9mvLqsnd0nASzTADZK9gh2wLP1g/GhfD5xwOcnCNCEpA8ob0sc9g9DKOuJ@vger.kernel.org X-Gm-Message-State: AOJu0YxD7qCdRWrJHzuO5BMVBa8XXxxSPuCCAsQCqfoUshq8itbUgp3R Ey2RcjuBfPfZWWe+J8Geg8wZNV4TmP+xMYpefbcfZbRuyGfGcTRg7TBbVPeNxl5dWwuCjoqpW8s iGwhhZ8CPKjslMbFq/0M4A5BGsf+jqzN0WyiDbqz2KNfM8RLxup6/KQzut6c2CRRlfwrc X-Gm-Gg: ASbGncu1CLsovkfUNk4fwyP9NDZ1rxYFwek8MIpx/KGnvNu3xvBWe3VaIR49i9y3WuI KfKzEevaBCXLcESAxzeXreSBNl7hSA/Sf6r7T/ChhCIjCOdLRnZwuOfKpQIC9xbYxY0qarAbKRY LVQIsq00mUA+ltU0WZvbAU9XNXKg3r9VB3FMURbn2YiUVMM81p2Td3j9P/5KsiHXdtetaaML3Rv t/ZgBTSNX28DCzzqpXI5vgZhIWIU5Kk636Br+32pg9Ih4xiKbdIuE2jFhJGuxP8sN3u8DZv/iv5 ZhEJxneefsVkjeDZTcRl30VXH2zrPAzH2c2F6ftAGZrpsuWxxXYsEF8sezCBu5jmz717b5wH2/K HLxxNVIQjPrLjF0XyBSLUor9p1wVJT/1qi2D8VX4IEINi57Y4los/QGleVgnO0c8= X-Received: by 2002:a05:620a:45a6:b0:852:b230:2202 with SMTP id af79cd13be357-88354ea76afmr4310616385a.49.1760619673873; Thu, 16 Oct 2025 06:01:13 -0700 (PDT) X-Google-Smtp-Source: AGHT+IE1cYR672WhfNwRUL4xIcz1CTBk3GVUXvgO9v8Ao4Y5P8X2lqJHJSvMIx09uSEnclFHZLcREw== X-Received: by 2002:a05:620a:45a6:b0:852:b230:2202 with SMTP id af79cd13be357-88354ea76afmr4310491085a.49.1760619663149; Thu, 16 Oct 2025 06:01:03 -0700 (PDT) Received: from [10.111.161.126] (Global_NAT1_IAD_FW.qualcomm.com. [129.46.232.65]) by smtp.gmail.com with ESMTPSA id af79cd13be357-88f38bf1f80sm190854185a.55.2025.10.16.06.00.58 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 16 Oct 2025 06:01:02 -0700 (PDT) Message-ID: Date: Thu, 16 Oct 2025 21:00:54 +0800 Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/3] media: dt-bindings: Add qcom,sm6150-camss To: Bryan O'Donoghue , Robert Foss , Todor Tomov , Vladimir Zapolskiy , Mauro Carvalho Chehab , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bjorn Andersson , Konrad Dybcio Cc: linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20251016-sm6150-camss-v1-0-e7f64ac32370@oss.qualcomm.com> <20251016-sm6150-camss-v1-1-e7f64ac32370@oss.qualcomm.com> Content-Language: en-US From: Wenmeng Liu In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Proofpoint-ORIG-GUID: DyOPGEPoG0E0h-pRBTvtVEcCMwkKcrus X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMDEzMDAyMiBTYWx0ZWRfX4sh+mUfblSk7 ePGgwvmSL6v0dSasmhOEHiXXaB/cf3ZXNRw0UITq9Ka/GdRvOXezLA5dMuDsPLMf1WjrprcSrFs aK8y33dOPlfQQxiBv1LVzMDU9xNE+QBALqtriR8BZfhFZXPpX+WxwYUMdQBBQvxz+HI/klVRTvb //2K9qO1L9hR0d+0NX5/TSUnlvJ0TVl3ariOAOunqkKRDAEKyiQ+O4YkWndSeSvI1GOxjoxabp3 5+DnBGkNXZSbmgZXIw+BKj9tMwcZGzJmzIOnFvLVm3UlEdBdPRCf+uGVAFnjVyCNgtYyJZHy0SV JI1hJXtDnzLbJJ4LIhtXzZkVHL2W8bzRrks9btXjcJ/BzB0BGhwOpG7OTgAwKm3ee1M3jPzKA1k A5bUlOOqoysH9FpYPDwXGxDxiry77w== X-Authority-Analysis: v=2.4 cv=SfD6t/Ru c=1 sm=1 tr=0 ts=68f0eca0 cx=c_pps a=50t2pK5VMbmlHzFWWp8p/g==:117 a=C3Dk8TwHQYyIj7nOf9RCJw==:17 a=IkcTkHD0fZMA:10 a=x6icFKpwvdMA:10 a=VkNPw1HP01LnGYTKEx00:22 a=gEfo2CItAAAA:8 a=EUspDBNiAAAA:8 a=tlDJjdemDny6dp0K7XAA:9 a=8-qNIi-7vQFwNDMI:21 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=IoWCM6iH3mJn3m4BftBB:22 a=sptkURWiP4Gy88Gu7hUp:22 X-Proofpoint-GUID: DyOPGEPoG0E0h-pRBTvtVEcCMwkKcrus X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.80.40 definitions=2025-10-16_02,2025-10-13_01,2025-03-28_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 malwarescore=0 clxscore=1015 adultscore=0 phishscore=0 lowpriorityscore=0 bulkscore=0 impostorscore=0 priorityscore=1501 spamscore=0 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2510020000 definitions=main-2510130022 On 10/16/2025 6:43 PM, Bryan O'Donoghue wrote: > On 16/10/2025 11:22, Wenmeng Liu wrote: >> Add bindings for qcom,sm6150-camss in order to support the camera >> subsystem found in Qualcomm Talos EVK board. > > Understood you are doing this to support the Talos EVK but the yaml > should describe the specific SoC. > ACK>> >> Signed-off-by: Wenmeng Liu >> --- >>   .../bindings/media/qcom,sm6150-camss.yaml          | 283 +++++++++++ >> ++++++++++ >>   1 file changed, 283 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/media/qcom,sm6150- >> camss.yaml b/Documentation/devicetree/bindings/media/qcom,sm6150- >> camss.yaml >> new file mode 100644 >> index >> 0000000000000000000000000000000000000000..758bed0970f2ceee7df30b579a0f60d583a9230c >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/media/qcom,sm6150-camss.yaml >> @@ -0,0 +1,283 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/media/qcom,sm6150-camss.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Qualcomm SM6150 Camera Subsystem (CAMSS) >> + >> +maintainers: >> +  - Wenmeng Liu >> + >> +description: >> +  The CAMSS IP is a CSI decoder and ISP present on Qualcomm platforms. >> + >> +properties: >> +  compatible: >> +    const: qcom,sm6150-camss >> + >> +  reg: >> +    maxItems: 9 >> + >> +  reg-names: >> +    items: >> +      - const: csid0 >> +      - const: csid1 >> +      - const: csid_lite >> +      - const: csiphy0 >> +      - const: csiphy1 >> +      - const: csiphy2 >> +      - const: vfe0 >> +      - const: vfe1 >> +      - const: vfe_lite >> + >> +  clocks: >> +    maxItems: 21 >> + >> +  clock-names: >> +    items: >> +      - const: camnoc_axi >> +      - const: cpas_ahb >> +      - const: csiphy0 >> +      - const: csiphy0_timer >> +      - const: csiphy1 >> +      - const: csiphy1_timer >> +      - const: csiphy2 >> +      - const: csiphy2_timer >> +      - const: gcc_axi_hf >> +      - const: soc_ahb >> +      - const: vfe0 >> +      - const: vfe0_axi >> +      - const: vfe0_cphy_rx >> +      - const: vfe0_csid >> +      - const: vfe1 >> +      - const: vfe1_axi >> +      - const: vfe1_cphy_rx >> +      - const: vfe1_csid >> +      - const: vfe_lite >> +      - const: vfe_lite_cphy_rx >> +      - const: vfe_lite_csid >> + >> +  interrupts: >> +    maxItems: 9 >> + >> +  interrupt-names: >> +    items: >> +      - const: csid0 >> +      - const: csid1 >> +      - const: csid_lite >> +      - const: csiphy0 >> +      - const: csiphy1 >> +      - const: csiphy2 >> +      - const: vfe0 >> +      - const: vfe1 >> +      - const: vfe_lite >> + >> +  interconnects: >> +    maxItems: 2 >> + >> +  interconnect-names: >> +    items: >> +      - const: ahb >> +      - const: hf_mnoc >> + >> +  iommus: >> +    maxItems: 1 >> + >> +  power-domains: >> +    items: >> +      - description: IFE0 GDSC - Image Front End, Global Distributed >> Switch Controller. >> +      - description: IFE1 GDSC - Image Front End, Global Distributed >> Switch Controller. >> +      - description: Titan GDSC - Titan ISP Block, Global Distributed >> Switch Controller. >> + >> +  power-domain-names: >> +    items: >> +      - const: ife0 >> +      - const: ife1 >> +      - const: top >> + >> +  vdd-csiphy-1p2-supply: >> +    description: >> +      Phandle to a 1.2V regulator supply to CSI PHYs. >> + >> +  vdd-csiphy-1p8-supply: >> +    description: >> +      Phandle to 1.8V regulator supply to CSI PHYs pll block. >> + >> +  ports: >> +    $ref: /schemas/graph.yaml#/properties/ports >> + >> +    description: >> +      CSI input ports. >> + >> +    patternProperties: >> +      "^port@[0-2]+$": >> +        $ref: /schemas/graph.yaml#/$defs/port-base >> +        unevaluatedProperties: false >> + >> +        description: >> +          Input port for receiving CSI data from a CSIPHY. >> + >> +        properties: >> +          endpoint: >> +            $ref: video-interfaces.yaml# >> +            unevaluatedProperties: false >> + >> +            properties: >> +              data-lanes: >> +                minItems: 1 >> +                maxItems: 4 >> + >> +            required: >> +              - data-lanes >> + >> +required: >> +  - compatible >> +  - reg >> +  - reg-names >> +  - clocks >> +  - clock-names >> +  - interrupts >> +  - interrupt-names >> +  - interconnects >> +  - interconnect-names >> +  - iommus >> +  - power-domains >> +  - power-domain-names >> +  - vdd-csiphy-1p2-supply >> +  - vdd-csiphy-1p8-supply >> + >> +additionalProperties: false >> + >> +examples: >> +  - | >> +    #include >> +    #include >> +    #include > > rpmh should come after qcs615. > ACK>> +    #include >> +    #include >> +    #include >> +    #include >> + >> +    soc { >> +        #address-cells = <2>; >> +        #size-cells = <2>; >> + >> +        camss: isp@acb3000 { >> +            compatible = "qcom,sm6150-camss"; >> + >> +            reg = <0x0 0x0acb3000 0x0 0x1000>, >> +                  <0x0 0x0acba000 0x0 0x1000>, >> +                  <0x0 0x0acc8000 0x0 0x1000>, >> +                  <0x0 0x0ac65000 0x0 0x1000>, >> +                  <0x0 0x0ac66000 0x0 0x1000>, >> +                  <0x0 0x0ac67000 0x0 0x1000>, >> +                  <0x0 0x0acaf000 0x0 0x4000>, >> +                  <0x0 0x0acb6000 0x0 0x4000>, >> +                  <0x0 0x0acc4000 0x0 0x4000>; >> +            reg-names = "csid0", >> +                        "csid1", >> +                        "csid_lite", >> +                        "csiphy0", >> +                        "csiphy1", >> +                        "csiphy2", >> +                        "vfe0", >> +                        "vfe1", >> +                        "vfe_lite"; >> + >> +            clocks = <&camcc CAM_CC_CAMNOC_AXI_CLK>, >> +                     <&camcc CAM_CC_CPAS_AHB_CLK>, >> +                     <&camcc CAM_CC_CSIPHY0_CLK>, >> +                     <&camcc CAM_CC_CSI0PHYTIMER_CLK>, >> +                     <&camcc CAM_CC_CSIPHY1_CLK>, >> +                     <&camcc CAM_CC_CSI1PHYTIMER_CLK>, >> +                     <&camcc CAM_CC_CSIPHY2_CLK>, >> +                     <&camcc CAM_CC_CSI2PHYTIMER_CLK>, >> +                     <&gcc GCC_CAMERA_HF_AXI_CLK>, >> +                     <&camcc CAM_CC_SOC_AHB_CLK>, >> +                     <&camcc CAM_CC_IFE_0_CLK>, >> +                     <&camcc CAM_CC_IFE_0_AXI_CLK>, >> +                     <&camcc CAM_CC_IFE_0_CPHY_RX_CLK>, >> +                     <&camcc CAM_CC_IFE_0_CSID_CLK>, >> +                     <&camcc CAM_CC_IFE_1_CLK>, >> +                     <&camcc CAM_CC_IFE_1_AXI_CLK>, >> +                     <&camcc CAM_CC_IFE_1_CPHY_RX_CLK>, >> +                     <&camcc CAM_CC_IFE_1_CSID_CLK>, >> +                     <&camcc CAM_CC_IFE_LITE_CLK>, >> +                     <&camcc CAM_CC_IFE_LITE_CPHY_RX_CLK>, >> +                     <&camcc CAM_CC_IFE_LITE_CSID_CLK>; >> + >> +            clock-names = "camnoc_axi", >> +                          "cpas_ahb", >> +                          "csiphy0", >> +                          "csiphy0_timer", >> +                          "csiphy1", >> +                          "csiphy1_timer", >> +                          "csiphy2", >> +                          "csiphy2_timer", >> +                          "gcc_axi_hf", >> +                          "soc_ahb", >> +                          "vfe0", >> +                          "vfe0_axi", >> +                          "vfe0_cphy_rx", >> +                          "vfe0_csid", >> +                          "vfe1", >> +                          "vfe1_axi", >> +                          "vfe1_cphy_rx", >> +                          "vfe1_csid", >> +                          "vfe_lite", >> +                          "vfe_lite_cphy_rx", >> +                          "vfe_lite_csid"; >> + >> +            interconnects = <&gem_noc MASTER_APPSS_PROC >> QCOM_ICC_TAG_ACTIVE_ONLY >> +                             &config_noc SLAVE_CAMERA_CFG >> QCOM_ICC_TAG_ACTIVE_ONLY>, >> +                            <&mmss_noc MASTER_CAMNOC_HF0 >> QCOM_ICC_TAG_ALWAYS >> +                             &mc_virt SLAVE_EBI1 QCOM_ICC_TAG_ALWAYS>; >> +            interconnect-names = "ahb", >> +                                 "hf_mnoc"; >> + >> +            interrupts = , >> +                         , >> +                         , >> +                         , >> +                         , >> +                         , >> +                         , >> +                         , >> +                         ; >> +            interrupt-names = "csid0", >> +                              "csid1", >> +                              "csid_lite", >> +                              "csiphy0", >> +                              "csiphy1", >> +                              "csiphy2", >> +                              "vfe0", >> +                              "vfe1", >> +                              "vfe_lite"; >> + >> +            iommus = <&apps_smmu 0x820 0x40>; >> + >> +            power-domains = <&camcc IFE_0_GDSC>, >> +                            <&camcc IFE_1_GDSC>, >> +                            <&camcc TITAN_TOP_GDSC>; >> +            power-domain-names = "ife0", >> +                                 "ife1", >> +                                 "top"; >> + >> +            vdd-csiphy-1p2-supply = <&vreg_l11a_1p2>; >> +            vdd-csiphy-1p8-supply = <&vreg_l12a_1p8>; >> + >> +            ports { >> +                #address-cells = <1>; >> +                #size-cells = <0>; >> + >> +                port@0 { >> +                    reg = <0>; >> +                    csiphy_ep0: endpoint { >> +                        data-lanes = <0 1>; >> +                        remote-endpoint = <&sensor_ep>; >> +                    }; >> +                }; >> +            }; >> +        }; >> +    }; >> > will update in next version. Thanks, Wenmeng