Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <1c6ce9e4-b9a4-9252-566a-f48c8e675d6b@quicinc.com>
Date:   Tue, 10 Jan 2023 11:57:10 +0530
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
Subject: Re: [PATCH v2] dt-bindings: remoteproc: qcom: sc7280-adsp-pil: Add
 reg-names and power-domain-names
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        <linux-remoteproc@vger.kernel.org>, <agross@kernel.org>,
        <andersson@kernel.org>, <lgirdwood@gmail.com>,
        <broonie@kernel.org>, <robh+dt@kernel.org>,
        <quic_plai@quicinc.com>, <bgoswami@quicinc.com>, <perex@perex.cz>,
        <tiwai@suse.com>, <srinivas.kandagatla@linaro.org>,
        <quic_rohkumar@quicinc.com>, <linux-arm-msm@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <swboyd@chromium.org>,
        <judyhsiao@chromium.org>, <devicetree@vger.kernel.org>,
        <mathieu.poirier@linaro.org>, <corbet@lwn.net>
References: <1672991425-898-1-git-send-email-quic_srivasam@quicinc.com>
 <c6ea3f37-fe5a-9a61-3a02-99bc036fd23e@linaro.org>
From:   Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com>
Organization: Qualcomm
In-Reply-To: <c6ea3f37-fe5a-9a61-3a02-99bc036fd23e@linaro.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 1/8/2023 9:06 PM, Krzysztof Kozlowski wrote:
Thanks for Your Time Krzyszto!!!
> On 06/01/2023 08:50, Srinivasa Rao Mandadapu wrote:
>> Add reg-names and power-domain-names for remoteproc ADSP pheripheral
>> loader. This is to make compatible with remoteproc ADSP PIL driver.
> I don't understand the reasoning. Before binding was "incompatible"?

Yes. Previous bindings needs fixes. Will add fixes tag for the same. So 
far corresponding device tree not

mainlined and no user till now, hence these missing errors not found.

>
>> Also change power domain from LCX to CX.
> Why? Hardware changed?

No, basically it's a fix. The power domain name wrongly mentioned in 
previous bindings.

The fixes tag mentioned above will be applicable here also.

>
>> Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com>
>> ---
>> Changes Since v1:
>>      -- Modify subject line to SoC specific.
>>
>>   .../devicetree/bindings/remoteproc/qcom,sc7280-adsp-pil.yaml  | 11 ++++++++++-
>>   1 file changed, 10 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-adsp-pil.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-adsp-pil.yaml
>> index 94ca7a0..b5bee2c 100644
>> --- a/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-adsp-pil.yaml
>> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-adsp-pil.yaml
>> @@ -23,6 +23,11 @@ properties:
>>         - description: qdsp6ss register
>>         - description: efuse q6ss register
>>   
>> +  reg-names:
>> +    items:
>> +      - const: qdsp6ss_base
>> +      - const: lpass_efuse
>> +
>>     iommus:
>>       items:
>>         - description: Phandle to apps_smmu node with sid mask
>> @@ -57,7 +62,11 @@ properties:
>>   
>>     power-domains:
>>       items:
>> -      - description: LCX power domain
>> +      - description: CX power domain
>> +
>> +  power-domain-names:
>> +    items:
>> +      - const: cx
>>   
>>     resets:
>>       items:
> Best regards,
> Krzysztof
>