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: <5d648cbe-de43-39a8-760e-1c9b9b2ec064@quicinc.com>
Date:   Wed, 12 Jul 2023 13:50:25 +0530
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.6.0
Subject: Re: [PATCH v3 1/3] dt-bindings: power: rpmhpd: Add Generic RPMh PD
 indexes
Content-Language: en-US
To:     Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        <agross@kernel.org>, <andersson@kernel.org>, <robh+dt@kernel.org>,
        <krzysztof.kozlowski+dt@linaro.org>, <conor+dt@kernel.org>
CC:     <linux-arm-msm@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
References: <1689054169-10800-1-git-send-email-quic_rohiagar@quicinc.com>
 <1689054169-10800-2-git-send-email-quic_rohiagar@quicinc.com>
 <2040226e-9b45-b409-3edd-a5b86d86daa8@linaro.org>
 <8a3124ce-a11d-2491-eaee-1695cec70b17@quicinc.com>
 <e885383b-a231-75a1-32ce-bf850f0cb022@linaro.org>
 <8c3468f0-3069-95e3-e2f1-279fca78a578@linaro.org>
 <4330df82-77a9-27ce-3f24-144718cc082e@linaro.org>
From:   Rohit Agarwal <quic_rohiagar@quicinc.com>
In-Reply-To: <4330df82-77a9-27ce-3f24-144718cc082e@linaro.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk


On 7/11/2023 8:29 PM, Dmitry Baryshkov wrote:
> On 11/07/2023 14:44, Konrad Dybcio wrote:
>> On 11.07.2023 08:22, Krzysztof Kozlowski wrote:
>>> On 11/07/2023 08:17, Rohit Agarwal wrote:
>>>>
>>>> On 7/11/2023 11:22 AM, Krzysztof Kozlowski wrote:
>>>>> On 11/07/2023 07:42, Rohit Agarwal wrote:
>>>>>> Add Generic RPMh Power Domain indexes that can be used
>>>>>> for all the Qualcomm SoC henceforth.
>>>>>>
>>>>>> Signed-off-by: Rohit Agarwal <quic_rohiagar@quicinc.com>
>>>>>> Suggested-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>>>>>> ---
>>>>>>    include/dt-bindings/power/qcom-rpmhpd.h | 30 
>>>>>> ++++++++++++++++++++++++++++++
>>>>>>    1 file changed, 30 insertions(+)
>>>>>>    create mode 100644 include/dt-bindings/power/qcom-rpmhpd.h
>>>>>>
>>>>>> diff --git a/include/dt-bindings/power/qcom-rpmhpd.h 
>>>>>> b/include/dt-bindings/power/qcom-rpmhpd.h
>>>>>> new file mode 100644
>>>>>> index 0000000..4da2e04
>>>>>> --- /dev/null
>>>>>> +++ b/include/dt-bindings/power/qcom-rpmhpd.h
>>>>> Filename based on compatible.
>>>> This is not specific for SDX75. These are generic ones that should be
>>>> used for all other targets.
>>>> Konrad suggested in v1 to avoid target specific prefixes everytime and
>>>> to create a new generic
>>>> dt-bindings that can be reused.
>>>>>> @@ -0,0 +1,30 @@
>>>>>> +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
>>>>>> +/*
>>>>>> + * Copyright (c) 2023, Qualcomm Innovation Center, Inc. All 
>>>>>> rights reserved.
>>>>>> + */
>>>>>> +
>>>>>> +#ifndef _DT_BINDINGS_POWER_QCOM_RPMHPD_H
>>>>>> +#define _DT_BINDINGS_POWER_QCOM_RPMHPD_H
>>>>>> +
>>>>>> +/* Generic RPMH Power Domain Indexes */
>>>>>> +#define CX               0
>>>>> These are very generic names, usually not used in global headers. 
>>>>> Please
>>>>> use some reasonable prefix.
>>>> This was based on the suggestion from Konrad in v2 to drop the RPMHPD
>>>> prefix and we can go only with names like CX, etc.
>>>
>>> I don't think having so generic name in tree-wide header is good idea.
>> Conversely, I think that it would be very clear in files including 
>> this header.
>
> It is expected to be included into dtsi and into rpmhpd driver (only). 
> However I'd also suggest adding some generic prefix (e.g. RPMHPD_CX). 
> Just `CX' is too short and has a sensible possibility of getting  a 
> symbol conflict.
>
Ok, Will add them back as in v2.

Thanks,
Rohit.
>>
>> Konrad
>>> CX can mean anything.
>>>
>>> Best regards,
>>> Krzysztof
>>>
>