[PATCH] dt-bindings: regulator: qcom,rpmh-regulator: allow i and j as RPMh resource name suffix

Neil Armstrong posted 1 patch 2 years, 3 months ago
Documentation/devicetree/bindings/regulator/qcom,rpmh-regulator.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] dt-bindings: regulator: qcom,rpmh-regulator: allow i and j as RPMh resource name suffix
Posted by Neil Armstrong 2 years, 3 months ago
Add "i" and "j" to the allowed subffix list as they can be used as RPMh
resource name suffixes on new platforms.

Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
 Documentation/devicetree/bindings/regulator/qcom,rpmh-regulator.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/regulator/qcom,rpmh-regulator.yaml b/Documentation/devicetree/bindings/regulator/qcom,rpmh-regulator.yaml
index e758093365bc..7afffdb91581 100644
--- a/Documentation/devicetree/bindings/regulator/qcom,rpmh-regulator.yaml
+++ b/Documentation/devicetree/bindings/regulator/qcom,rpmh-regulator.yaml
@@ -92,7 +92,7 @@ properties:
         RPMh resource name suffix used for the regulators found
         on this PMIC.
     $ref: /schemas/types.yaml#/definitions/string
-    enum: [a, b, c, d, e, f, g, h, k]
+    enum: [a, b, c, d, e, f, g, h, i, j, k]
 
   qcom,always-wait-for-ack:
     description: |

---
base-commit: 28c736b0e92e11bfe2b9997688213dc43cb22182
change-id: 20230822-topic-sm8x50-upstream-rpmh-regulator-suffix-29f2922f5b5c

Best regards,
-- 
Neil Armstrong <neil.armstrong@linaro.org>
Re: [PATCH] dt-bindings: regulator: qcom,rpmh-regulator: allow i and j as RPMh resource name suffix
Posted by Konrad Dybcio 2 years, 3 months ago
On 22.08.2023 11:27, Neil Armstrong wrote:
> Add "i" and "j" to the allowed subffix list as they can be used as RPMh
> resource name suffixes on new platforms.
> 
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
Also 'n' and 'm' for 8550

Konrad
Re: [PATCH] dt-bindings: regulator: qcom,rpmh-regulator: allow i and j as RPMh resource name suffix
Posted by Konrad Dybcio 2 years, 3 months ago
On 22.08.2023 11:28, Konrad Dybcio wrote:
> On 22.08.2023 11:27, Neil Armstrong wrote:
>> Add "i" and "j" to the allowed subffix list as they can be used as RPMh
>> resource name suffixes on new platforms.
>>
>> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
>> ---
> Also 'n' and 'm' for 8550
Correction, the PMICs are indexed 'n' and 'm' but looks
like there are no RPMh-managed regulators on there

Konrad
Re: [PATCH] dt-bindings: regulator: qcom,rpmh-regulator: allow i and j as RPMh resource name suffix
Posted by Konrad Dybcio 2 years, 3 months ago
On 22.08.2023 11:29, Konrad Dybcio wrote:
> On 22.08.2023 11:28, Konrad Dybcio wrote:
>> On 22.08.2023 11:27, Neil Armstrong wrote:
>>> Add "i" and "j" to the allowed subffix list as they can be used as RPMh
>>> resource name suffixes on new platforms.
>>>
>>> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
>>> ---
>> Also 'n' and 'm' for 8550
> Correction, the PMICs are indexed 'n' and 'm' but looks
> like there are no RPMh-managed regulators on there
Correction of the correction ;) there are some

Konrad
Re: [PATCH] dt-bindings: regulator: qcom,rpmh-regulator: allow i and j as RPMh resource name suffix
Posted by Neil Armstrong 2 years, 3 months ago
Hi,

On 22/08/2023 11:42, Konrad Dybcio wrote:
> On 22.08.2023 11:29, Konrad Dybcio wrote:
>> On 22.08.2023 11:28, Konrad Dybcio wrote:
>>> On 22.08.2023 11:27, Neil Armstrong wrote:
>>>> Add "i" and "j" to the allowed subffix list as they can be used as RPMh
>>>> resource name suffixes on new platforms.
>>>>
>>>> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
>>>> ---
>>> Also 'n' and 'm' for 8550
>> Correction, the PMICs are indexed 'n' and 'm' but looks
>> like there are no RPMh-managed regulators on there
> Correction of the correction ;) there are some

Indeed, you're right, let's for for i, j, n & m then !

Neil

> 
> Konrad