lkml.org 
[lkml]   [2023]   [Mar]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH v3 1/4] dt-bindings: mfd: Add TI TPS6594 PMIC
    From


    On 3/22/23 09:16, Krzysztof Kozlowski wrote:
    > On 22/03/2023 09:01, Julien Panis wrote:
    >>>> + ti,multi-phase-id:
    >>>> + description: |
    >>>> + Describes buck multi-phase configuration, if any. For instance, XY id means
    >>>> + that outputs of buck converters X and Y are combined in multi-phase mode.
    >>>> + $ref: /schemas/types.yaml#/definitions/uint32-array
    >>> No improvements here. As Rob pointed out, this looks like coupled
    >>> regulators.
    >> I used 'oneOf' logic to handle mutual exclusion. But it seems that I did not
    >> understand what you and Rob expected.
    >> Does some generic property already exist for 'coupled regulators' ?
    > Yes, see regulator.yaml binding.

    Krzysztof, I talked with the regulator API/yaml maintainer.
    Actually, our multiphase concept is different than coupled regulators:
    https://lore.kernel.org/all/ZBr+7X3lcFdI8p%2Fo@sirena.org.uk/

    We must not use the generic 'coupled regulator' property here.
    So, 'ti,multi-phase-id' can be kept.

    >
    > Best regards,
    > Krzysztof
    >

    \
     
     \ /
      Last update: 2023-03-27 01:12    [W:9.694 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site