Message-ID: <ef82f126-af16-8eba-4da9-eb08106a03db@quicinc.com>
Date:   Thu, 23 Feb 2023 15:49:30 +0530
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.1
Subject: Re: [RESEND v8 4/5] clk: qcom: lpassaudiocc-sc7280: Merge AHB clocks
 into lpass_aon
Content-Language: en-US
To:     Stephen Boyd <swboyd@chromium.org>, <agross@kernel.org>,
        <andersson@kernel.org>, <broonie@kernel.org>,
        <konrad.dybcio@somainline.org>,
        <krzysztof.kozlowski+dt@linaro.org>,
        <linux-arm-msm@vger.kernel.org>, <linux-clk@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <mturquette@baylibre.com>,
        <quic_plai@quicinc.com>, <quic_rohkumar@quicinc.com>,
        <quic_visr@quicinc.com>, <robh+dt@kernel.org>
CC:     Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com>
References: <20230215151330.539885-1-quic_mohs@quicinc.com>
 <20230215151330.539885-5-quic_mohs@quicinc.com>
 <CAE-0n53mDzqua47jEqrJwQBhcQcyBjJAtNvH2J-tCPhkvV9JtA@mail.gmail.com>
From:   Mohammad Rafi Shaik <quic_mohs@quicinc.com>
In-Reply-To: <CAE-0n53mDzqua47jEqrJwQBhcQcyBjJAtNvH2J-tCPhkvV9JtA@mail.gmail.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk


On 2/16/2023 11:01 AM, Stephen Boyd wrote:
> Quoting Mohammad Rafi Shaik (2023-02-15 07:13:29)
>> @@ -828,8 +830,9 @@ static int lpass_aon_cc_sc7280_probe(struct platform_device *pdev)
>>          if (of_property_read_bool(pdev->dev.of_node, "qcom,adsp-pil-mode")) {
>>                  lpass_audio_cc_sc7280_regmap_config.name = "cc";
>>                  desc = &lpass_cc_sc7280_desc;
>> -               ret = qcom_cc_probe(pdev, desc);
>> -               goto exit;
>> +               ret = qcom_cc_probe_by_index(pdev, 1, desc);
> Where is the patch to the binding yaml file?
Thanks for comment.

The dtbinding  yaml file for this is already present in existing yaml.

Below is the snippet for the same:

compatible = "qcom,sc7280-lpassaoncc";
reg = <0x3380000 0x30000>;

Actually what ever the  CC register region <required range is  0x3389000 
0x24> which we are accessing is already present in the above reg map 
range of dtsi, so we have not updated the 
/devicetree/bindings/clock/qcom,sc7280-lpasscorecc.yaml file.