Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Sender: tdas=codeaurora.org@mg.codeaurora.org
DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org C5272C433D3
Subject: Re: [PATCH v2 3/6] dt-bindings: clock: Add SC7280 GPUCC clock binding
To:     Stephen Boyd <sboyd@kernel.org>,
        Michael Turquette <mturquette@baylibre.com>
Cc:     Rajendra Nayak <rnayak@codeaurora.org>,
        linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org,
        linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org, robh@kernel.org, robh+dt@kernel.org
References: <1619519590-3019-1-git-send-email-tdas@codeaurora.org>
 <1619519590-3019-4-git-send-email-tdas@codeaurora.org>
 <162261721239.4130789.8314129952052939804@swboyd.mtv.corp.google.com>
From:   Taniya Das <tdas@codeaurora.org>
Message-ID: <208ac6e4-749c-7ee7-5a05-1fef12e134d8@codeaurora.org>
Date:   Tue, 13 Jul 2021 08:27:10 +0530
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.11.0
MIME-Version: 1.0
In-Reply-To: <162261721239.4130789.8314129952052939804@swboyd.mtv.corp.google.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hello Stephen,

Thanks for your review.

On 6/2/2021 12:30 PM, Stephen Boyd wrote:
> Quoting Taniya Das (2021-04-27 03:33:07)
>> diff --git a/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml b/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml
>> index df943c4..7e3f9e7 100644
>> --- a/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml
>> +++ b/Documentation/devicetree/bindings/clock/qcom,gpucc.yaml
>> @@ -11,11 +11,12 @@ maintainers:
>>   
>>   description: |
>>     Qualcomm graphics clock control module which supports the clocks, resets and
>> -  power domains on SDM845/SC7180/SM8150/SM8250.
>> +  power domains on SDM845/SC7180/SC7280/SM8150/SM8250.
> 
> Can we stop updating this line? Just say "power domains on Qualcomm
> SoCs"?
> 

This will be updated in the next series.

>>   
>>     See also:
>>       dt-bindings/clock/qcom,gpucc-sdm845.h
>>       dt-bindings/clock/qcom,gpucc-sc7180.h
>> +    dt-bindings/clock/qcom,gpucc-sc7280.h
>>       dt-bindings/clock/qcom,gpucc-sm8150.h
>>       dt-bindings/clock/qcom,gpucc-sm8250.h
>>   
>> @@ -24,6 +25,7 @@ properties:
>>       enum:
>>         - qcom,sdm845-gpucc
>>         - qcom,sc7180-gpucc
>> +      - qcom,sc7280-gpucc
>>         - qcom,sm8150-gpucc
>>         - qcom,sm8250-gpucc
>>

-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation.

--