linux-kernel - Re: [PATCH 1/3] interconnect: qcom: constify qcom_icc

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 27 Apr 2022 14:59:39 +0200
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Georgi Djakov <djakov@...nel.org>, Andy Gross <agross@...nel.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        linux-arm-msm@...r.kernel.org, linux-pm@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] interconnect: qcom: constify qcom_icc_desc

On 27/04/2022 14:17, Georgi Djakov wrote:
> On 22.04.22 18:25, Krzysztof Kozlowski wrote:
>> On 12/04/2022 12:26, Krzysztof Kozlowski wrote:
>>> struct qcom_icc_desc is not modified so it can be made const for safety.
>>>
>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>>> ---
>>>   drivers/interconnect/qcom/msm8916.c |  6 +++---
>>>   drivers/interconnect/qcom/msm8939.c |  8 ++++----
>>>   drivers/interconnect/qcom/msm8974.c | 12 ++++++------
>>>   drivers/interconnect/qcom/qcm2290.c | 12 ++++++------
>>>   drivers/interconnect/qcom/qcs404.c  |  6 +++---
>>>   drivers/interconnect/qcom/sc7180.c  | 26 +++++++++++++-------------
>>>   drivers/interconnect/qcom/sc7280.c  | 24 ++++++++++++------------
>>>   drivers/interconnect/qcom/sdm660.c  | 12 ++++++------
>>>   drivers/interconnect/qcom/sm8150.c  | 22 +++++++++++-----------
>>>   drivers/interconnect/qcom/sm8250.c  | 22 +++++++++++-----------
>>>   drivers/interconnect/qcom/sm8350.c  | 20 ++++++++++----------
>>>   drivers/interconnect/qcom/sm8450.c  | 22 +++++++++++-----------
>>
>> Hi folks,
>>
>> Any comments here?
> 
> Thanks for the patches! I had actually some of these changes in my local tree
> for a long time.

Indeed I see them now in linux-next. Sorry for pinging, I just did not
get any notification they got applied.

> Do you want to send a follow-up patch also for sc8280xp?

Yes, sir! I'll constify that one as well.


Best regards,
Krzysztof