On Thu, 28 Sep 2023 16:55:26 +0800, Chen-Yu Tsai wrote: > The MT6358 PMIC allows changing operating modes for the buck regulators, > but not the LDOs. Existing device trees and the Linux implementation > already utilize this through the standard regulator-allowed-modes > property. > > The values currently used in existing device trees are simply raw > numbers. The values in the Linux driver are matching numbers defined > with macros denoting the two supported modes. Turns out these two > modes are common across parts of the larger MT63xx PMIC family. The > MT6397 regulator binding already has macros for the two modes, with > matching numbers. > > Codify the supported values for regulator-allowed-modes for the MT6358 > in the device tree binding: 0 and 1 are supported for buck regulators, > and the property should not be present for LDO regulators. Users should > use the dt-bindings/regulator/mediatek,mt6397-regulator.h header for > the macros, instead of using raw numbers. > > Signed-off-by: Chen-Yu Tsai <wenst@xxxxxxxxxxxx> > --- > Changes since v3: > - adapt to ldo_vxo22 and ldo_vusb movement to properties > > Changes since v2: > - new patch > > .../regulator/mediatek,mt6358-regulator.yaml | 27 +++++++++++++++++++ > 1 file changed, 27 insertions(+) > Reviewed-by: Rob Herring <robh@xxxxxxxxxx>