Add device tree bindings for Monolithic Power Systems MPQ8785, MPM82504
and MPM3695 PMBus-compliant voltage regulators.
These bindings also documents the optional "voltage-scale-loop" property.
Signed-off-by: Pawel Dembicki <paweldembicki@gmail.com>
---
.../bindings/hwmon/pmbus/mps,mpq8785.yaml | 54 +++++++++++++++++++
1 file changed, 54 insertions(+)
create mode 100644 Documentation/devicetree/bindings/hwmon/pmbus/mps,mpq8785.yaml
diff --git a/Documentation/devicetree/bindings/hwmon/pmbus/mps,mpq8785.yaml b/Documentation/devicetree/bindings/hwmon/pmbus/mps,mpq8785.yaml
new file mode 100644
index 000000000000..e2a3958a61fa
--- /dev/null
+++ b/Documentation/devicetree/bindings/hwmon/pmbus/mps,mpq8785.yaml
@@ -0,0 +1,54 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+
+$id: http://devicetree.org/schemas/hwmon/pmbus/mps,mpq8785.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+allOf:
+ - $ref: /schemas/i2c/i2c-device.yaml#
+
+title: Monolithic Power Systems Multiphase Voltage Regulators with PMBus
+
+maintainers:
+ - Charles Hsu <ythsu0511@gmail.com>
+
+description: |
+ Monolithic Power Systems digital multiphase voltage regulators with PMBus.
+
+properties:
+ compatible:
+ enum:
+ - mps,mpq8785
+ - mps,mpm82504
+ - mps,mpm3695-10
+
+ reg:
+ maxItems: 1
+
+ voltage-scale-loop:
+ description:
+ Voltage scale factor for the VOUT_SCALE_LOOP register.
+ Value expressed in mili-units (1/1000th of a unit).
+ The simplest way to calculate it is
+ VOUT_SCALE_LOOP = VFB / VOUT * 1000
+ $ref: /schemas/types.yaml#/definitions/uint32
+
+required:
+ - compatible
+ - reg
+
+additionalProperties: false
+
+examples:
+ - |
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ pmic@30 {
+ compatible = "mps,mpm82504";
+ reg = <0x30>;
+ voltage-scale-loop = <600>;
+ };
+ };
--
2.43.0
Hi Pawel, kernel test robot noticed the following build warnings: [auto build test WARNING on groeck-staging/hwmon-next] [also build test WARNING on linus/master v6.15-rc4 next-20250428] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Pawel-Dembicki/hwmon-pmbus-mpq8785-Prepare-driver-for-multiple-device-support/20250429-061658 base: https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-next patch link: https://lore.kernel.org/r/20250428221420.2077697-6-paweldembicki%40gmail.com patch subject: [PATCH 5/5] dt-bindings: hwmon: Add bindings for mpq8785 driver config: csky-randconfig-052-20250429 (https://download.01.org/0day-ci/archive/20250429/202504291853.nDOvzGEJ-lkp@intel.com/config) compiler: csky-linux-gcc (GCC) 11.5.0 dtschema version: 2025.3.dev21+ge6ea659 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250429/202504291853.nDOvzGEJ-lkp@intel.com/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp@intel.com> | Closes: https://lore.kernel.org/oe-kbuild-all/202504291853.nDOvzGEJ-lkp@intel.com/ All warnings (new ones prefixed by >>): >> Warning: Duplicate compatible "mps,mpq8785" found in schemas matching "$id": http://devicetree.org/schemas/trivial-devices.yaml# http://devicetree.org/schemas/hwmon/pmbus/mps,mpq8785.yaml# -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
On Tue, Apr 29, 2025 at 12:13:35AM GMT, Pawel Dembicki wrote: > Add device tree bindings for Monolithic Power Systems MPQ8785, MPM82504 > and MPM3695 PMBus-compliant voltage regulators. > > These bindings also documents the optional "voltage-scale-loop" property. > > Signed-off-by: Pawel Dembicki <paweldembicki@gmail.com> > --- > .../bindings/hwmon/pmbus/mps,mpq8785.yaml | 54 +++++++++++++++++++ > 1 file changed, 54 insertions(+) > create mode 100644 Documentation/devicetree/bindings/hwmon/pmbus/mps,mpq8785.yaml > > diff --git a/Documentation/devicetree/bindings/hwmon/pmbus/mps,mpq8785.yaml b/Documentation/devicetree/bindings/hwmon/pmbus/mps,mpq8785.yaml > new file mode 100644 > index 000000000000..e2a3958a61fa > --- /dev/null > +++ b/Documentation/devicetree/bindings/hwmon/pmbus/mps,mpq8785.yaml > @@ -0,0 +1,54 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > + Drop blank line. > +$id: http://devicetree.org/schemas/hwmon/pmbus/mps,mpq8785.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +allOf: > + - $ref: /schemas/i2c/i2c-device.yaml# From where did you take such code? Drop. > + > +title: Monolithic Power Systems Multiphase Voltage Regulators with PMBus > + > +maintainers: > + - Charles Hsu <ythsu0511@gmail.com> > + > +description: | Drop | > + Monolithic Power Systems digital multiphase voltage regulators with PMBus. > + > +properties: > + compatible: > + enum: > + - mps,mpq8785 > + - mps,mpm82504 > + - mps,mpm3695-10 Keep alphabetical order > + > + reg: > + maxItems: 1 > + > + voltage-scale-loop: Missing vendor prefix, missing tests, missing property unit suffix (percent? bpp?) Or this should be just output voltage in microvolts. Best regards, Krzysztof
On Tue, 29 Apr 2025 00:13:35 +0200, Pawel Dembicki wrote:
> Add device tree bindings for Monolithic Power Systems MPQ8785, MPM82504
> and MPM3695 PMBus-compliant voltage regulators.
>
> These bindings also documents the optional "voltage-scale-loop" property.
>
> Signed-off-by: Pawel Dembicki <paweldembicki@gmail.com>
> ---
> .../bindings/hwmon/pmbus/mps,mpq8785.yaml | 54 +++++++++++++++++++
> 1 file changed, 54 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/hwmon/pmbus/mps,mpq8785.yaml
>
My bot found errors running 'make dt_binding_check' on your patch:
yamllint warnings/errors:
dtschema/dtc warnings/errors:
Warning: Duplicate compatible "mps,mpq8785" found in schemas matching "$id":
http://devicetree.org/schemas/hwmon/pmbus/mps,mpq8785.yaml#
http://devicetree.org/schemas/trivial-devices.yaml#
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/hwmon/pmbus/mps,mpq8785.example.dtb: pmic@30 (mps,mpm82504): False schema does not allow {'compatible': ['mps,mpm82504'], 'reg': [[48]], 'voltage-scale-loop': 600, '$nodename': ['pmic@30']}
from schema $id: http://devicetree.org/schemas/hwmon/pmbus/mps,mpq8785.yaml#
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250428221420.2077697-6-paweldembicki@gmail.com
The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.
© 2016 - 2026 Red Hat, Inc.