* [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
Hi,
The max77843 shares some parts with max77693 but it lacked bindings. All
its compatibles were undocumented. Add basic bindings for max77843,
matching existing devicetree. These are not complete bindings, but
rather reverse-engineered to match current state.
I do not have access to device with MAX77843, so if anyone else
volunteers to be the bindings maintainer, please join.
Changes since v1
================
1. MFD: Use absolute path to schemas.
2. Regulator: mention all allowed properties,
additionalProperties=false, add min/max values for voltages and
current, don't use patternProperties when not needed.
3. extcon: Use absolute path to schemas.
Dependencies
============
1. Patch 1/5 (dts): nothing depends on it, sending here so Rob's automatic
checker won't complain about DTS.
I will take it via Samsung SoC tree.
2. The patch 4/5 (mfd bindings) depends on regulator and extcon, so they
should come together (2+3+4+5).
Best regards,
Krzysztof
Krzysztof Kozlowski (5):
arm64: dts: exynos: Align MAX77843 nodes with dtschema on TM2
dt-bindings: extcon: maxim,max77843: add MAX77843 bindings
regulator: dt-bindings: maxim,max77843: add MAX77843 bindings
dt-bindings: mfd: maxim,max77843: add MAX77843 bindings
MAINTAINERS: mfd: cover MAX77843 by Maxim PMIC/MUIC for Exynos boards
entry
.../bindings/extcon/maxim,max77843.yaml | 40 +++++
.../bindings/mfd/maxim,max77843.yaml | 144 ++++++++++++++++++
.../bindings/regulator/maxim,max77843.yaml | 65 ++++++++
MAINTAINERS | 2 +
.../dts/exynos/exynos5433-tm2-common.dtsi | 17 ++-
5 files changed, 265 insertions(+), 3 deletions(-)
create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
--
2.32.0
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
Hi,
The max77843 shares some parts with max77693 but it lacked bindings. All
its compatibles were undocumented. Add basic bindings for max77843,
matching existing devicetree. These are not complete bindings, but
rather reverse-engineered to match current state.
I do not have access to device with MAX77843, so if anyone else
volunteers to be the bindings maintainer, please join.
Changes since v1
================
1. MFD: Use absolute path to schemas.
2. Regulator: mention all allowed properties,
additionalProperties=false, add min/max values for voltages and
current, don't use patternProperties when not needed.
3. extcon: Use absolute path to schemas.
Dependencies
============
1. Patch 1/5 (dts): nothing depends on it, sending here so Rob's automatic
checker won't complain about DTS.
I will take it via Samsung SoC tree.
2. The patch 4/5 (mfd bindings) depends on regulator and extcon, so they
should come together (2+3+4+5).
Best regards,
Krzysztof
Krzysztof Kozlowski (5):
arm64: dts: exynos: Align MAX77843 nodes with dtschema on TM2
dt-bindings: extcon: maxim,max77843: add MAX77843 bindings
regulator: dt-bindings: maxim,max77843: add MAX77843 bindings
dt-bindings: mfd: maxim,max77843: add MAX77843 bindings
MAINTAINERS: mfd: cover MAX77843 by Maxim PMIC/MUIC for Exynos boards
entry
.../bindings/extcon/maxim,max77843.yaml | 40 +++++
.../bindings/mfd/maxim,max77843.yaml | 144 ++++++++++++++++++
.../bindings/regulator/maxim,max77843.yaml | 65 ++++++++
MAINTAINERS | 2 +
.../dts/exynos/exynos5433-tm2-common.dtsi | 17 ++-
5 files changed, 265 insertions(+), 3 deletions(-)
create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
--
2.32.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* [PATCH v2 1/5] arm64: dts: exynos: Align MAX77843 nodes with dtschema on TM2
2022-01-11 17:48 ` Krzysztof Kozlowski
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
The newly introduced dtschema for MAX77843 MUIC require the children to
have proper naming and a port@0 property.
This should not have actual impact on MFD children driver binding,
because the max77843 MFD driver uses compatibles. The port@0 is
disabled to avoid any impact.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
Please kindly test or even better - fix the DTS and extcon driver.
---
.../boot/dts/exynos/exynos5433-tm2-common.dtsi | 17 ++++++++++++++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi
index cbcc01a66aab..03f7c9acaacb 100644
--- a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi
@@ -858,10 +858,10 @@ pmic@66 {
interrupts = <5 IRQ_TYPE_EDGE_FALLING>;
reg = <0x66>;
- muic: max77843-muic {
+ muic: extcon {
compatible = "maxim,max77843-muic";
- musb_con: musb-connector {
+ musb_con: connector {
compatible = "samsung,usb-connector-11pin",
"usb-b-connector";
label = "micro-USB";
@@ -871,6 +871,17 @@ ports {
#address-cells = <1>;
#size-cells = <0>;
+ port@0 {
+ /*
+ * TODO: The DTS this is based on does not have
+ * port@0 which is a required property. The ports
+ * look incomplete and need fixing.
+ * Add a disabled port just to satisfy dtschema.
+ */
+ reg = <0>;
+ status = "disabled";
+ };
+
port@3 {
reg = <3>;
musb_con_to_mhl: endpoint {
@@ -910,7 +921,7 @@ charger_reg: CHARGER {
};
};
- haptic: max77843-haptic {
+ haptic: motor-driver {
compatible = "maxim,max77843-haptic";
haptic-supply = <&ldo38_reg>;
pwms = <&pwm 0 33670 0>;
--
2.32.0
^ permalink raw reply related [flat|nested] 42+ messages in thread
* [PATCH v2 1/5] arm64: dts: exynos: Align MAX77843 nodes with dtschema on TM2
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
The newly introduced dtschema for MAX77843 MUIC require the children to
have proper naming and a port@0 property.
This should not have actual impact on MFD children driver binding,
because the max77843 MFD driver uses compatibles. The port@0 is
disabled to avoid any impact.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
Please kindly test or even better - fix the DTS and extcon driver.
---
.../boot/dts/exynos/exynos5433-tm2-common.dtsi | 17 ++++++++++++++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi
index cbcc01a66aab..03f7c9acaacb 100644
--- a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi
@@ -858,10 +858,10 @@ pmic@66 {
interrupts = <5 IRQ_TYPE_EDGE_FALLING>;
reg = <0x66>;
- muic: max77843-muic {
+ muic: extcon {
compatible = "maxim,max77843-muic";
- musb_con: musb-connector {
+ musb_con: connector {
compatible = "samsung,usb-connector-11pin",
"usb-b-connector";
label = "micro-USB";
@@ -871,6 +871,17 @@ ports {
#address-cells = <1>;
#size-cells = <0>;
+ port@0 {
+ /*
+ * TODO: The DTS this is based on does not have
+ * port@0 which is a required property. The ports
+ * look incomplete and need fixing.
+ * Add a disabled port just to satisfy dtschema.
+ */
+ reg = <0>;
+ status = "disabled";
+ };
+
port@3 {
reg = <3>;
musb_con_to_mhl: endpoint {
@@ -910,7 +921,7 @@ charger_reg: CHARGER {
};
};
- haptic: max77843-haptic {
+ haptic: motor-driver {
compatible = "maxim,max77843-haptic";
haptic-supply = <&ldo38_reg>;
pwms = <&pwm 0 33670 0>;
--
2.32.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 42+ messages in thread
* [PATCH v2 2/5] dt-bindings: extcon: maxim,max77843: add MAX77843 bindings
2022-01-11 17:48 ` Krzysztof Kozlowski
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
Cc: Rob Herring
Document the bindings for MAX77843 MUIC/extcon driver, based on
Exynos5433 TM2 devicetree. These are neither accurate nor finished
bindings but at least allow parsing existing DTS files.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
.../bindings/extcon/maxim,max77843.yaml | 40 +++++++++++++++++++
1 file changed, 40 insertions(+)
create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
diff --git a/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
new file mode 100644
index 000000000000..f9ffe3d6f957
--- /dev/null
+++ b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
@@ -0,0 +1,40 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/extcon/maxim,max77843.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim MAX77843 MicroUSB and Companion Power Management IC Extcon
+
+maintainers:
+ - Chanwoo Choi <cw00.choi@samsung.com>
+ - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
+
+description: |
+ This is a part of device tree bindings for Maxim MAX77843 MicroUSB
+ Integrated Circuit (MUIC).
+
+ See also Documentation/devicetree/bindings/mfd/maxim,max77843.yaml for
+ additional information and example.
+
+properties:
+ compatible:
+ const: maxim,max77843-muic
+
+ connector:
+ $ref: /schemas/connector/usb-connector.yaml#
+
+ ports:
+ $ref: /schemas/graph.yaml#/properties/port
+ description:
+ Any connector to the data bus of this controller should be modelled using
+ the OF graph bindings specified
+ properties:
+ port:
+ $ref: /schemas/graph.yaml#/properties/port
+
+required:
+ - compatible
+ - connector
+
+additionalProperties: false
--
2.32.0
^ permalink raw reply related [flat|nested] 42+ messages in thread
* [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: add MAX77843 bindings
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
Cc: Rob Herring
Document the bindings for MAX77843 MUIC/extcon driver, based on
Exynos5433 TM2 devicetree. These are neither accurate nor finished
bindings but at least allow parsing existing DTS files.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
.../bindings/extcon/maxim,max77843.yaml | 40 +++++++++++++++++++
1 file changed, 40 insertions(+)
create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
diff --git a/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
new file mode 100644
index 000000000000..f9ffe3d6f957
--- /dev/null
+++ b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
@@ -0,0 +1,40 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/extcon/maxim,max77843.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim MAX77843 MicroUSB and Companion Power Management IC Extcon
+
+maintainers:
+ - Chanwoo Choi <cw00.choi@samsung.com>
+ - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
+
+description: |
+ This is a part of device tree bindings for Maxim MAX77843 MicroUSB
+ Integrated Circuit (MUIC).
+
+ See also Documentation/devicetree/bindings/mfd/maxim,max77843.yaml for
+ additional information and example.
+
+properties:
+ compatible:
+ const: maxim,max77843-muic
+
+ connector:
+ $ref: /schemas/connector/usb-connector.yaml#
+
+ ports:
+ $ref: /schemas/graph.yaml#/properties/port
+ description:
+ Any connector to the data bus of this controller should be modelled using
+ the OF graph bindings specified
+ properties:
+ port:
+ $ref: /schemas/graph.yaml#/properties/port
+
+required:
+ - compatible
+ - connector
+
+additionalProperties: false
--
2.32.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 42+ messages in thread
* [PATCH v2 3/5] regulator: dt-bindings: maxim,max77843: add MAX77843 bindings
2022-01-11 17:48 ` Krzysztof Kozlowski
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
Document the bindings for MAX77843 regulator driver. The bindings are
almost identical to MAX77693 bindings, except the actual names of
regulators.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
.../bindings/regulator/maxim,max77843.yaml | 65 +++++++++++++++++++
1 file changed, 65 insertions(+)
create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
diff --git a/Documentation/devicetree/bindings/regulator/maxim,max77843.yaml b/Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
new file mode 100644
index 000000000000..a963025e96c1
--- /dev/null
+++ b/Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
@@ -0,0 +1,65 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/regulator/maxim,max77843.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim MAX77843 MicroUSB and Companion Power Management IC regulators
+
+maintainers:
+ - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
+
+description: |
+ This is a part of device tree bindings for Maxim MAX77843 MicroUSB Integrated
+ Circuit (MUIC).
+
+ See also Documentation/devicetree/bindings/mfd/maxim,max77843.yaml for
+ additional information and example.
+
+properties:
+ compatible:
+ const: maxim,max77843-regulator
+
+ CHARGER:
+ type: object
+ $ref: regulator.yaml#
+ additionalProperties: false
+ description: |
+ Current regulator.
+
+ properties:
+ regulator-name: true
+ regulator-always-on: true
+ regulator-boot-on: true
+ regulator-min-microamp:
+ minimum: 100000
+ regulator-max-microamp:
+ maximum: 3150000
+
+ required:
+ - regulator-name
+
+patternProperties:
+ "^SAFEOUT[12]$":
+ type: object
+ $ref: regulator.yaml#
+ additionalProperties: false
+ description: |
+ Safeout LDO regulator.
+
+ properties:
+ regulator-name: true
+ regulator-always-on: true
+ regulator-boot-on: true
+ regulator-min-microvolt:
+ minimum: 3300000
+ regulator-max-microvolt:
+ maximum: 4950000
+
+ required:
+ - regulator-name
+
+required:
+ - compatible
+
+additionalProperties: false
--
2.32.0
^ permalink raw reply related [flat|nested] 42+ messages in thread
* [PATCH v2 3/5] regulator: dt-bindings: maxim, max77843: add MAX77843 bindings
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
Document the bindings for MAX77843 regulator driver. The bindings are
almost identical to MAX77693 bindings, except the actual names of
regulators.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
.../bindings/regulator/maxim,max77843.yaml | 65 +++++++++++++++++++
1 file changed, 65 insertions(+)
create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
diff --git a/Documentation/devicetree/bindings/regulator/maxim,max77843.yaml b/Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
new file mode 100644
index 000000000000..a963025e96c1
--- /dev/null
+++ b/Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
@@ -0,0 +1,65 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/regulator/maxim,max77843.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim MAX77843 MicroUSB and Companion Power Management IC regulators
+
+maintainers:
+ - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
+
+description: |
+ This is a part of device tree bindings for Maxim MAX77843 MicroUSB Integrated
+ Circuit (MUIC).
+
+ See also Documentation/devicetree/bindings/mfd/maxim,max77843.yaml for
+ additional information and example.
+
+properties:
+ compatible:
+ const: maxim,max77843-regulator
+
+ CHARGER:
+ type: object
+ $ref: regulator.yaml#
+ additionalProperties: false
+ description: |
+ Current regulator.
+
+ properties:
+ regulator-name: true
+ regulator-always-on: true
+ regulator-boot-on: true
+ regulator-min-microamp:
+ minimum: 100000
+ regulator-max-microamp:
+ maximum: 3150000
+
+ required:
+ - regulator-name
+
+patternProperties:
+ "^SAFEOUT[12]$":
+ type: object
+ $ref: regulator.yaml#
+ additionalProperties: false
+ description: |
+ Safeout LDO regulator.
+
+ properties:
+ regulator-name: true
+ regulator-always-on: true
+ regulator-boot-on: true
+ regulator-min-microvolt:
+ minimum: 3300000
+ regulator-max-microvolt:
+ maximum: 4950000
+
+ required:
+ - regulator-name
+
+required:
+ - compatible
+
+additionalProperties: false
--
2.32.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 42+ messages in thread
* [PATCH v2 4/5] dt-bindings: mfd: maxim,max77843: add MAX77843 bindings
2022-01-11 17:48 ` Krzysztof Kozlowski
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
Cc: Rob Herring
Document the bindings for MAX77843 MFD driver, based on Exynos5433 TM2
devicetree. These are neither accurate nor finished bindings but at
least allow parsing existing DTS files.
The example DTS was copied from existing DTS
(exynos5433-tm2-common.dtsi), so keep the license as GPL-2.0-only.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
.../bindings/mfd/maxim,max77843.yaml | 144 ++++++++++++++++++
1 file changed, 144 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
diff --git a/Documentation/devicetree/bindings/mfd/maxim,max77843.yaml b/Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
new file mode 100644
index 000000000000..61a0f9dcb983
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
@@ -0,0 +1,144 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mfd/maxim,max77843.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim MAX77843 MicroUSB and Companion Power Management IC
+
+maintainers:
+ - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
+
+description: |
+ This is a part of device tree bindings for Maxim MAX77843 MicroUSB
+ Integrated Circuit (MUIC).
+
+ The Maxim MAX77843 is a MicroUSB and Companion Power Management IC which
+ includes voltage current regulators, charger, fuel-gauge, haptic motor driver
+ and MicroUSB management IC.
+
+properties:
+ compatible:
+ const: maxim,max77843
+
+ interrupts:
+ maxItems: 1
+
+ reg:
+ maxItems: 1
+
+ extcon:
+ $ref: /schemas/extcon/maxim,max77843.yaml
+
+ motor-driver:
+ type: object
+ properties:
+ compatible:
+ const: maxim,max77843-haptic
+
+ haptic-supply:
+ description: Power supply to the haptic motor
+
+ pwms:
+ maxItems: 1
+
+ required:
+ - compatible
+ - haptic-supply
+ - pwms
+
+ regulators:
+ $ref: /schemas/regulator/maxim,max77843.yaml
+
+required:
+ - compatible
+ - interrupts
+ - reg
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/irq.h>
+
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ pmic@66 {
+ compatible = "maxim,max77843";
+ interrupt-parent = <&gpa1>;
+ interrupts = <5 IRQ_TYPE_EDGE_FALLING>;
+ reg = <0x66>;
+
+ extcon {
+ compatible = "maxim,max77843-muic";
+
+ connector {
+ compatible = "samsung,usb-connector-11pin",
+ "usb-b-connector";
+ label = "micro-USB";
+ type = "micro";
+
+ ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@0 {
+ /*
+ * TODO: The DTS this is based on does not have
+ * port@0 which is a required property. The ports
+ * look incomplete and need fixing.
+ * Add a disabled port just to satisfy dtschema.
+ */
+ reg = <0>;
+ status = "disabled";
+ };
+
+ port@3 {
+ reg = <3>;
+ endpoint {
+ remote-endpoint = <&mhl_to_musb_con>;
+ };
+ };
+ };
+ };
+
+ ports {
+ port {
+ endpoint {
+ remote-endpoint = <&usb_to_muic>;
+ };
+ };
+ };
+ };
+
+ regulators {
+ compatible = "maxim,max77843-regulator";
+
+ SAFEOUT1 {
+ regulator-name = "SAFEOUT1";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <4950000>;
+ };
+
+ SAFEOUT2 {
+ regulator-name = "SAFEOUT2";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <4950000>;
+ };
+
+ CHARGER {
+ regulator-name = "CHARGER";
+ regulator-min-microamp = <100000>;
+ regulator-max-microamp = <3150000>;
+ };
+ };
+
+ motor-driver {
+ compatible = "maxim,max77843-haptic";
+ haptic-supply = <&ldo38_reg>;
+ pwms = <&pwm 0 33670 0>;
+ };
+ };
+ };
--
2.32.0
^ permalink raw reply related [flat|nested] 42+ messages in thread
* [PATCH v2 4/5] dt-bindings: mfd: maxim, max77843: add MAX77843 bindings
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
Cc: Rob Herring
Document the bindings for MAX77843 MFD driver, based on Exynos5433 TM2
devicetree. These are neither accurate nor finished bindings but at
least allow parsing existing DTS files.
The example DTS was copied from existing DTS
(exynos5433-tm2-common.dtsi), so keep the license as GPL-2.0-only.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
.../bindings/mfd/maxim,max77843.yaml | 144 ++++++++++++++++++
1 file changed, 144 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
diff --git a/Documentation/devicetree/bindings/mfd/maxim,max77843.yaml b/Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
new file mode 100644
index 000000000000..61a0f9dcb983
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
@@ -0,0 +1,144 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mfd/maxim,max77843.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim MAX77843 MicroUSB and Companion Power Management IC
+
+maintainers:
+ - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
+
+description: |
+ This is a part of device tree bindings for Maxim MAX77843 MicroUSB
+ Integrated Circuit (MUIC).
+
+ The Maxim MAX77843 is a MicroUSB and Companion Power Management IC which
+ includes voltage current regulators, charger, fuel-gauge, haptic motor driver
+ and MicroUSB management IC.
+
+properties:
+ compatible:
+ const: maxim,max77843
+
+ interrupts:
+ maxItems: 1
+
+ reg:
+ maxItems: 1
+
+ extcon:
+ $ref: /schemas/extcon/maxim,max77843.yaml
+
+ motor-driver:
+ type: object
+ properties:
+ compatible:
+ const: maxim,max77843-haptic
+
+ haptic-supply:
+ description: Power supply to the haptic motor
+
+ pwms:
+ maxItems: 1
+
+ required:
+ - compatible
+ - haptic-supply
+ - pwms
+
+ regulators:
+ $ref: /schemas/regulator/maxim,max77843.yaml
+
+required:
+ - compatible
+ - interrupts
+ - reg
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/irq.h>
+
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ pmic@66 {
+ compatible = "maxim,max77843";
+ interrupt-parent = <&gpa1>;
+ interrupts = <5 IRQ_TYPE_EDGE_FALLING>;
+ reg = <0x66>;
+
+ extcon {
+ compatible = "maxim,max77843-muic";
+
+ connector {
+ compatible = "samsung,usb-connector-11pin",
+ "usb-b-connector";
+ label = "micro-USB";
+ type = "micro";
+
+ ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@0 {
+ /*
+ * TODO: The DTS this is based on does not have
+ * port@0 which is a required property. The ports
+ * look incomplete and need fixing.
+ * Add a disabled port just to satisfy dtschema.
+ */
+ reg = <0>;
+ status = "disabled";
+ };
+
+ port@3 {
+ reg = <3>;
+ endpoint {
+ remote-endpoint = <&mhl_to_musb_con>;
+ };
+ };
+ };
+ };
+
+ ports {
+ port {
+ endpoint {
+ remote-endpoint = <&usb_to_muic>;
+ };
+ };
+ };
+ };
+
+ regulators {
+ compatible = "maxim,max77843-regulator";
+
+ SAFEOUT1 {
+ regulator-name = "SAFEOUT1";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <4950000>;
+ };
+
+ SAFEOUT2 {
+ regulator-name = "SAFEOUT2";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <4950000>;
+ };
+
+ CHARGER {
+ regulator-name = "CHARGER";
+ regulator-min-microamp = <100000>;
+ regulator-max-microamp = <3150000>;
+ };
+ };
+
+ motor-driver {
+ compatible = "maxim,max77843-haptic";
+ haptic-supply = <&ldo38_reg>;
+ pwms = <&pwm 0 33670 0>;
+ };
+ };
+ };
--
2.32.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 42+ messages in thread
* [PATCH v2 5/5] MAINTAINERS: mfd: cover MAX77843 by Maxim PMIC/MUIC for Exynos boards entry
2022-01-11 17:48 ` Krzysztof Kozlowski
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
The MAX77843 is used in Exynos5433-based TM2 boards and shares some
parts of code with MAX77693 (regulator and haptic motor drivers).
Include all MAX77843 drivers in the entry for Maxim PMIC/MUIC drivers
for Exynos boards, so they will receive some dedicated review coverage.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index e5f2758531bc..d1f8f312f322 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11693,8 +11693,10 @@ L: linux-kernel@vger.kernel.org
S: Supported
F: Documentation/devicetree/bindings/*/maxim,max77686.yaml
F: Documentation/devicetree/bindings/*/maxim,max77693.yaml
+F: Documentation/devicetree/bindings/*/maxim,max77843.yaml
F: Documentation/devicetree/bindings/clock/maxim,max77686.txt
F: Documentation/devicetree/bindings/mfd/max14577.txt
+F: drivers/*/*max77843.c
F: drivers/*/max14577*.c
F: drivers/*/max77686*.c
F: drivers/*/max77693*.c
--
2.32.0
^ permalink raw reply related [flat|nested] 42+ messages in thread
* [PATCH v2 5/5] MAINTAINERS: mfd: cover MAX77843 by Maxim PMIC/MUIC for Exynos boards entry
@ 2022-01-11 17:48 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-11 17:48 UTC (permalink / raw)
To: Chanwoo Choi, Krzysztof Kozlowski, MyungJoo Ham, Rob Herring,
Lee Jones, Liam Girdwood, Mark Brown, linux-kernel, devicetree,
linux-arm-kernel, linux-samsung-soc
The MAX77843 is used in Exynos5433-based TM2 boards and shares some
parts of code with MAX77693 (regulator and haptic motor drivers).
Include all MAX77843 drivers in the entry for Maxim PMIC/MUIC drivers
for Exynos boards, so they will receive some dedicated review coverage.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index e5f2758531bc..d1f8f312f322 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11693,8 +11693,10 @@ L: linux-kernel@vger.kernel.org
S: Supported
F: Documentation/devicetree/bindings/*/maxim,max77686.yaml
F: Documentation/devicetree/bindings/*/maxim,max77693.yaml
+F: Documentation/devicetree/bindings/*/maxim,max77843.yaml
F: Documentation/devicetree/bindings/clock/maxim,max77686.txt
F: Documentation/devicetree/bindings/mfd/max14577.txt
+F: drivers/*/*max77843.c
F: drivers/*/max14577*.c
F: drivers/*/max77686*.c
F: drivers/*/max77693*.c
--
2.32.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 42+ messages in thread
* Re: [PATCH v2 3/5] regulator: dt-bindings: maxim,max77843: add MAX77843 bindings
2022-01-11 17:48 ` [PATCH v2 3/5] regulator: dt-bindings: maxim, max77843: " Krzysztof Kozlowski
@ 2022-01-11 22:21 ` Rob Herring
-1 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2022-01-11 22:21 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Liam Girdwood, linux-samsung-soc, linux-arm-kernel, Rob Herring,
MyungJoo Ham, Chanwoo Choi, Mark Brown, devicetree, Lee Jones,
linux-kernel
On Tue, 11 Jan 2022 18:48:03 +0100, Krzysztof Kozlowski wrote:
> Document the bindings for MAX77843 regulator driver. The bindings are
> almost identical to MAX77693 bindings, except the actual names of
> regulators.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
> .../bindings/regulator/maxim,max77843.yaml | 65 +++++++++++++++++++
> 1 file changed, 65 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 3/5] regulator: dt-bindings: maxim,max77843: add MAX77843 bindings
@ 2022-01-11 22:21 ` Rob Herring
0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2022-01-11 22:21 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Liam Girdwood, linux-samsung-soc, linux-arm-kernel, Rob Herring,
MyungJoo Ham, Chanwoo Choi, Mark Brown, devicetree, Lee Jones,
linux-kernel
On Tue, 11 Jan 2022 18:48:03 +0100, Krzysztof Kozlowski wrote:
> Document the bindings for MAX77843 regulator driver. The bindings are
> almost identical to MAX77693 bindings, except the actual names of
> regulators.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
> .../bindings/regulator/maxim,max77843.yaml | 65 +++++++++++++++++++
> 1 file changed, 65 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
>
Reviewed-by: Rob Herring <robh@kernel.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: (subset) [PATCH v2 1/5] arm64: dts: exynos: Align MAX77843 nodes with dtschema on TM2
2022-01-11 17:48 ` Krzysztof Kozlowski
@ 2022-01-23 17:10 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-23 17:10 UTC (permalink / raw)
To: Krzysztof Kozlowski, linux-samsung-soc, Rob Herring,
Liam Girdwood, Chanwoo Choi, Mark Brown, MyungJoo Ham, Lee Jones,
devicetree, linux-arm-kernel, linux-kernel
On Tue, 11 Jan 2022 18:48:01 +0100, Krzysztof Kozlowski wrote:
> The newly introduced dtschema for MAX77843 MUIC require the children to
> have proper naming and a port@0 property.
>
> This should not have actual impact on MFD children driver binding,
> because the max77843 MFD driver uses compatibles. The port@0 is
> disabled to avoid any impact.
>
> [...]
Applied, thanks!
[1/5] arm64: dts: exynos: Align MAX77843 nodes with dtschema on TM2
commit: 7638d3c945beb6c781acf5dd0a78e04c76f1c32f
Best regards,
--
Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: (subset) [PATCH v2 1/5] arm64: dts: exynos: Align MAX77843 nodes with dtschema on TM2
@ 2022-01-23 17:10 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-01-23 17:10 UTC (permalink / raw)
To: Krzysztof Kozlowski, linux-samsung-soc, Rob Herring,
Liam Girdwood, Chanwoo Choi, Mark Brown, MyungJoo Ham, Lee Jones,
devicetree, linux-arm-kernel, linux-kernel
On Tue, 11 Jan 2022 18:48:01 +0100, Krzysztof Kozlowski wrote:
> The newly introduced dtschema for MAX77843 MUIC require the children to
> have proper naming and a port@0 property.
>
> This should not have actual impact on MFD children driver binding,
> because the max77843 MFD driver uses compatibles. The port@0 is
> disabled to avoid any impact.
>
> [...]
Applied, thanks!
[1/5] arm64: dts: exynos: Align MAX77843 nodes with dtschema on TM2
commit: 7638d3c945beb6c781acf5dd0a78e04c76f1c32f
Best regards,
--
Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
2022-01-11 17:48 ` Krzysztof Kozlowski
@ 2022-02-06 17:39 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-02-06 17:39 UTC (permalink / raw)
To: Lee Jones
Cc: Rob Herring, Krzysztof Kozlowski, linux-kernel, Mark Brown,
MyungJoo Ham, Liam Girdwood, linux-samsung-soc, linux-arm-kernel,
devicetree, Chanwoo Choi
On 11/01/2022 18:48, Krzysztof Kozlowski wrote:
> Hi,
>
> The max77843 shares some parts with max77693 but it lacked bindings. All
> its compatibles were undocumented. Add basic bindings for max77843,
> matching existing devicetree. These are not complete bindings, but
> rather reverse-engineered to match current state.
>
> I do not have access to device with MAX77843, so if anyone else
> volunteers to be the bindings maintainer, please join.
>
> Changes since v1
> ================
> 1. MFD: Use absolute path to schemas.
> 2. Regulator: mention all allowed properties,
> additionalProperties=false, add min/max values for voltages and
> current, don't use patternProperties when not needed.
> 3. extcon: Use absolute path to schemas.
>
> Dependencies
> ============
> 1. Patch 1/5 (dts): nothing depends on it, sending here so Rob's automatic
> checker won't complain about DTS.
> I will take it via Samsung SoC tree.
>
> 2. The patch 4/5 (mfd bindings) depends on regulator and extcon, so they
> should come together (2+3+4+5).
>
Dear Lee,
This patchset was reviewed and there are no outstanding issues. Could
you pick up patches 2-5 (skipping DTS patch) via MFD tree?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
@ 2022-02-06 17:39 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-02-06 17:39 UTC (permalink / raw)
To: Lee Jones
Cc: Rob Herring, Krzysztof Kozlowski, linux-kernel, Mark Brown,
MyungJoo Ham, Liam Girdwood, linux-samsung-soc, linux-arm-kernel,
devicetree, Chanwoo Choi
On 11/01/2022 18:48, Krzysztof Kozlowski wrote:
> Hi,
>
> The max77843 shares some parts with max77693 but it lacked bindings. All
> its compatibles were undocumented. Add basic bindings for max77843,
> matching existing devicetree. These are not complete bindings, but
> rather reverse-engineered to match current state.
>
> I do not have access to device with MAX77843, so if anyone else
> volunteers to be the bindings maintainer, please join.
>
> Changes since v1
> ================
> 1. MFD: Use absolute path to schemas.
> 2. Regulator: mention all allowed properties,
> additionalProperties=false, add min/max values for voltages and
> current, don't use patternProperties when not needed.
> 3. extcon: Use absolute path to schemas.
>
> Dependencies
> ============
> 1. Patch 1/5 (dts): nothing depends on it, sending here so Rob's automatic
> checker won't complain about DTS.
> I will take it via Samsung SoC tree.
>
> 2. The patch 4/5 (mfd bindings) depends on regulator and extcon, so they
> should come together (2+3+4+5).
>
Dear Lee,
This patchset was reviewed and there are no outstanding issues. Could
you pick up patches 2-5 (skipping DTS patch) via MFD tree?
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
2022-02-06 17:39 ` Krzysztof Kozlowski
@ 2022-02-07 9:33 ` Lee Jones
-1 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-02-07 9:33 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, linux-kernel, Mark Brown, MyungJoo Ham,
Liam Girdwood, linux-samsung-soc, linux-arm-kernel, devicetree,
Chanwoo Choi
On Sun, 06 Feb 2022, Krzysztof Kozlowski wrote:
> On 11/01/2022 18:48, Krzysztof Kozlowski wrote:
> > Hi,
> >
> > The max77843 shares some parts with max77693 but it lacked bindings. All
> > its compatibles were undocumented. Add basic bindings for max77843,
> > matching existing devicetree. These are not complete bindings, but
> > rather reverse-engineered to match current state.
> >
> > I do not have access to device with MAX77843, so if anyone else
> > volunteers to be the bindings maintainer, please join.
> >
> > Changes since v1
> > ================
> > 1. MFD: Use absolute path to schemas.
> > 2. Regulator: mention all allowed properties,
> > additionalProperties=false, add min/max values for voltages and
> > current, don't use patternProperties when not needed.
> > 3. extcon: Use absolute path to schemas.
> >
> > Dependencies
> > ============
> > 1. Patch 1/5 (dts): nothing depends on it, sending here so Rob's automatic
> > checker won't complain about DTS.
> > I will take it via Samsung SoC tree.
> >
> > 2. The patch 4/5 (mfd bindings) depends on regulator and extcon, so they
> > should come together (2+3+4+5).
> >
> Dear Lee,
>
> This patchset was reviewed and there are no outstanding issues. Could
> you pick up patches 2-5 (skipping DTS patch) via MFD tree?
Are the subsystem maintainers not going to review/ack?
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
@ 2022-02-07 9:33 ` Lee Jones
0 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-02-07 9:33 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, linux-kernel, Mark Brown, MyungJoo Ham,
Liam Girdwood, linux-samsung-soc, linux-arm-kernel, devicetree,
Chanwoo Choi
On Sun, 06 Feb 2022, Krzysztof Kozlowski wrote:
> On 11/01/2022 18:48, Krzysztof Kozlowski wrote:
> > Hi,
> >
> > The max77843 shares some parts with max77693 but it lacked bindings. All
> > its compatibles were undocumented. Add basic bindings for max77843,
> > matching existing devicetree. These are not complete bindings, but
> > rather reverse-engineered to match current state.
> >
> > I do not have access to device with MAX77843, so if anyone else
> > volunteers to be the bindings maintainer, please join.
> >
> > Changes since v1
> > ================
> > 1. MFD: Use absolute path to schemas.
> > 2. Regulator: mention all allowed properties,
> > additionalProperties=false, add min/max values for voltages and
> > current, don't use patternProperties when not needed.
> > 3. extcon: Use absolute path to schemas.
> >
> > Dependencies
> > ============
> > 1. Patch 1/5 (dts): nothing depends on it, sending here so Rob's automatic
> > checker won't complain about DTS.
> > I will take it via Samsung SoC tree.
> >
> > 2. The patch 4/5 (mfd bindings) depends on regulator and extcon, so they
> > should come together (2+3+4+5).
> >
> Dear Lee,
>
> This patchset was reviewed and there are no outstanding issues. Could
> you pick up patches 2-5 (skipping DTS patch) via MFD tree?
Are the subsystem maintainers not going to review/ack?
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
2022-02-07 9:33 ` Lee Jones
@ 2022-02-07 9:47 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-02-07 9:47 UTC (permalink / raw)
To: Lee Jones
Cc: Rob Herring, linux-kernel, Mark Brown, MyungJoo Ham,
Liam Girdwood, linux-samsung-soc, linux-arm-kernel, devicetree,
Chanwoo Choi
On 07/02/2022 10:33, Lee Jones wrote:
> On Sun, 06 Feb 2022, Krzysztof Kozlowski wrote:
>
>> On 11/01/2022 18:48, Krzysztof Kozlowski wrote:
>>> Hi,
>>>
>>> The max77843 shares some parts with max77693 but it lacked bindings. All
>>> its compatibles were undocumented. Add basic bindings for max77843,
>>> matching existing devicetree. These are not complete bindings, but
>>> rather reverse-engineered to match current state.
>>>
>>> I do not have access to device with MAX77843, so if anyone else
>>> volunteers to be the bindings maintainer, please join.
>>>
>>> Changes since v1
>>> ================
>>> 1. MFD: Use absolute path to schemas.
>>> 2. Regulator: mention all allowed properties,
>>> additionalProperties=false, add min/max values for voltages and
>>> current, don't use patternProperties when not needed.
>>> 3. extcon: Use absolute path to schemas.
>>>
>>> Dependencies
>>> ============
>>> 1. Patch 1/5 (dts): nothing depends on it, sending here so Rob's automatic
>>> checker won't complain about DTS.
>>> I will take it via Samsung SoC tree.
>>>
>>> 2. The patch 4/5 (mfd bindings) depends on regulator and extcon, so they
>>> should come together (2+3+4+5).
>>>
>> Dear Lee,
>>
>> This patchset was reviewed and there are no outstanding issues. Could
>> you pick up patches 2-5 (skipping DTS patch) via MFD tree?
>
> Are the subsystem maintainers not going to review/ack?
>
Would be nice to have them, but maybe since these are dt-bindings maybe
they were skipped? Or maybe it was the end-of-year holidays?
I can resend hoping it will trigger more acks. Do you wish me to resend?
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
@ 2022-02-07 9:47 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-02-07 9:47 UTC (permalink / raw)
To: Lee Jones
Cc: Rob Herring, linux-kernel, Mark Brown, MyungJoo Ham,
Liam Girdwood, linux-samsung-soc, linux-arm-kernel, devicetree,
Chanwoo Choi
On 07/02/2022 10:33, Lee Jones wrote:
> On Sun, 06 Feb 2022, Krzysztof Kozlowski wrote:
>
>> On 11/01/2022 18:48, Krzysztof Kozlowski wrote:
>>> Hi,
>>>
>>> The max77843 shares some parts with max77693 but it lacked bindings. All
>>> its compatibles were undocumented. Add basic bindings for max77843,
>>> matching existing devicetree. These are not complete bindings, but
>>> rather reverse-engineered to match current state.
>>>
>>> I do not have access to device with MAX77843, so if anyone else
>>> volunteers to be the bindings maintainer, please join.
>>>
>>> Changes since v1
>>> ================
>>> 1. MFD: Use absolute path to schemas.
>>> 2. Regulator: mention all allowed properties,
>>> additionalProperties=false, add min/max values for voltages and
>>> current, don't use patternProperties when not needed.
>>> 3. extcon: Use absolute path to schemas.
>>>
>>> Dependencies
>>> ============
>>> 1. Patch 1/5 (dts): nothing depends on it, sending here so Rob's automatic
>>> checker won't complain about DTS.
>>> I will take it via Samsung SoC tree.
>>>
>>> 2. The patch 4/5 (mfd bindings) depends on regulator and extcon, so they
>>> should come together (2+3+4+5).
>>>
>> Dear Lee,
>>
>> This patchset was reviewed and there are no outstanding issues. Could
>> you pick up patches 2-5 (skipping DTS patch) via MFD tree?
>
> Are the subsystem maintainers not going to review/ack?
>
Would be nice to have them, but maybe since these are dt-bindings maybe
they were skipped? Or maybe it was the end-of-year holidays?
I can resend hoping it will trigger more acks. Do you wish me to resend?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
2022-02-07 9:47 ` Krzysztof Kozlowski
@ 2022-02-07 10:11 ` Lee Jones
-1 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-02-07 10:11 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, linux-kernel, Mark Brown, MyungJoo Ham,
Liam Girdwood, linux-samsung-soc, linux-arm-kernel, devicetree,
Chanwoo Choi
On Mon, 07 Feb 2022, Krzysztof Kozlowski wrote:
> On 07/02/2022 10:33, Lee Jones wrote:
> > On Sun, 06 Feb 2022, Krzysztof Kozlowski wrote:
> >
> >> On 11/01/2022 18:48, Krzysztof Kozlowski wrote:
> >>> Hi,
> >>>
> >>> The max77843 shares some parts with max77693 but it lacked bindings. All
> >>> its compatibles were undocumented. Add basic bindings for max77843,
> >>> matching existing devicetree. These are not complete bindings, but
> >>> rather reverse-engineered to match current state.
> >>>
> >>> I do not have access to device with MAX77843, so if anyone else
> >>> volunteers to be the bindings maintainer, please join.
> >>>
> >>> Changes since v1
> >>> ================
> >>> 1. MFD: Use absolute path to schemas.
> >>> 2. Regulator: mention all allowed properties,
> >>> additionalProperties=false, add min/max values for voltages and
> >>> current, don't use patternProperties when not needed.
> >>> 3. extcon: Use absolute path to schemas.
> >>>
> >>> Dependencies
> >>> ============
> >>> 1. Patch 1/5 (dts): nothing depends on it, sending here so Rob's automatic
> >>> checker won't complain about DTS.
> >>> I will take it via Samsung SoC tree.
> >>>
> >>> 2. The patch 4/5 (mfd bindings) depends on regulator and extcon, so they
> >>> should come together (2+3+4+5).
> >>>
> >> Dear Lee,
> >>
> >> This patchset was reviewed and there are no outstanding issues. Could
> >> you pick up patches 2-5 (skipping DTS patch) via MFD tree?
> >
> > Are the subsystem maintainers not going to review/ack?
> >
>
> Would be nice to have them, but maybe since these are dt-bindings maybe
> they were skipped?
Agreed, not required, but nice to have to avoid possible conflict.
> Or maybe it was the end-of-year holidays?
> I can resend hoping it will trigger more acks. Do you wish me to resend?
That shouldn't be necessary.
I'll give the guys a little while to protest/ack, then I'll merge.
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
@ 2022-02-07 10:11 ` Lee Jones
0 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-02-07 10:11 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, linux-kernel, Mark Brown, MyungJoo Ham,
Liam Girdwood, linux-samsung-soc, linux-arm-kernel, devicetree,
Chanwoo Choi
On Mon, 07 Feb 2022, Krzysztof Kozlowski wrote:
> On 07/02/2022 10:33, Lee Jones wrote:
> > On Sun, 06 Feb 2022, Krzysztof Kozlowski wrote:
> >
> >> On 11/01/2022 18:48, Krzysztof Kozlowski wrote:
> >>> Hi,
> >>>
> >>> The max77843 shares some parts with max77693 but it lacked bindings. All
> >>> its compatibles were undocumented. Add basic bindings for max77843,
> >>> matching existing devicetree. These are not complete bindings, but
> >>> rather reverse-engineered to match current state.
> >>>
> >>> I do not have access to device with MAX77843, so if anyone else
> >>> volunteers to be the bindings maintainer, please join.
> >>>
> >>> Changes since v1
> >>> ================
> >>> 1. MFD: Use absolute path to schemas.
> >>> 2. Regulator: mention all allowed properties,
> >>> additionalProperties=false, add min/max values for voltages and
> >>> current, don't use patternProperties when not needed.
> >>> 3. extcon: Use absolute path to schemas.
> >>>
> >>> Dependencies
> >>> ============
> >>> 1. Patch 1/5 (dts): nothing depends on it, sending here so Rob's automatic
> >>> checker won't complain about DTS.
> >>> I will take it via Samsung SoC tree.
> >>>
> >>> 2. The patch 4/5 (mfd bindings) depends on regulator and extcon, so they
> >>> should come together (2+3+4+5).
> >>>
> >> Dear Lee,
> >>
> >> This patchset was reviewed and there are no outstanding issues. Could
> >> you pick up patches 2-5 (skipping DTS patch) via MFD tree?
> >
> > Are the subsystem maintainers not going to review/ack?
> >
>
> Would be nice to have them, but maybe since these are dt-bindings maybe
> they were skipped?
Agreed, not required, but nice to have to avoid possible conflict.
> Or maybe it was the end-of-year holidays?
> I can resend hoping it will trigger more acks. Do you wish me to resend?
That shouldn't be necessary.
I'll give the guys a little while to protest/ack, then I'll merge.
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
2022-02-07 9:47 ` Krzysztof Kozlowski
@ 2022-02-07 16:25 ` Mark Brown
-1 siblings, 0 replies; 42+ messages in thread
From: Mark Brown @ 2022-02-07 16:25 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Rob Herring, linux-kernel, MyungJoo Ham,
Liam Girdwood, linux-samsung-soc, linux-arm-kernel, devicetree,
Chanwoo Choi
[-- Attachment #1: Type: text/plain, Size: 736 bytes --]
On Mon, Feb 07, 2022 at 10:47:00AM +0100, Krzysztof Kozlowski wrote:
> On 07/02/2022 10:33, Lee Jones wrote:
> > Are the subsystem maintainers not going to review/ack?
> Would be nice to have them, but maybe since these are dt-bindings maybe
> they were skipped? Or maybe it was the end-of-year holidays?
You've been sending out a large number of very similar serieses, each of
which is going through multiple revisions - it's hard to keep track of
what's been reviewed, what's in a state where it's useful to review
> I can resend hoping it will trigger more acks. Do you wish me to resend?
It's probably going to add more noise, on the other hand some things
might've been dropped on the floor due to the similarity and whatnot.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings
@ 2022-02-07 16:25 ` Mark Brown
0 siblings, 0 replies; 42+ messages in thread
From: Mark Brown @ 2022-02-07 16:25 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Rob Herring, linux-kernel, MyungJoo Ham,
Liam Girdwood, linux-samsung-soc, linux-arm-kernel, devicetree,
Chanwoo Choi
[-- Attachment #1.1: Type: text/plain, Size: 736 bytes --]
On Mon, Feb 07, 2022 at 10:47:00AM +0100, Krzysztof Kozlowski wrote:
> On 07/02/2022 10:33, Lee Jones wrote:
> > Are the subsystem maintainers not going to review/ack?
> Would be nice to have them, but maybe since these are dt-bindings maybe
> they were skipped? Or maybe it was the end-of-year holidays?
You've been sending out a large number of very similar serieses, each of
which is going through multiple revisions - it's hard to keep track of
what's been reviewed, what's in a state where it's useful to review
> I can resend hoping it will trigger more acks. Do you wish me to resend?
It's probably going to add more noise, on the other hand some things
might've been dropped on the floor due to the similarity and whatnot.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 176 bytes --]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 3/5] regulator: dt-bindings: maxim,max77843: add MAX77843 bindings
2022-01-11 17:48 ` [PATCH v2 3/5] regulator: dt-bindings: maxim, max77843: " Krzysztof Kozlowski
@ 2022-02-11 11:59 ` Mark Brown
-1 siblings, 0 replies; 42+ messages in thread
From: Mark Brown @ 2022-02-11 11:59 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Lee Jones,
Liam Girdwood, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc
[-- Attachment #1: Type: text/plain, Size: 270 bytes --]
On Tue, Jan 11, 2022 at 06:48:03PM +0100, Krzysztof Kozlowski wrote:
> Document the bindings for MAX77843 regulator driver. The bindings are
> almost identical to MAX77693 bindings, except the actual names of
> regulators.
Reviewed-by: Mark Brown <broonie@kernel.org>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 3/5] regulator: dt-bindings: maxim,max77843: add MAX77843 bindings
@ 2022-02-11 11:59 ` Mark Brown
0 siblings, 0 replies; 42+ messages in thread
From: Mark Brown @ 2022-02-11 11:59 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Lee Jones,
Liam Girdwood, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc
[-- Attachment #1.1: Type: text/plain, Size: 270 bytes --]
On Tue, Jan 11, 2022 at 06:48:03PM +0100, Krzysztof Kozlowski wrote:
> Document the bindings for MAX77843 regulator driver. The bindings are
> almost identical to MAX77693 bindings, except the actual names of
> regulators.
Reviewed-by: Mark Brown <broonie@kernel.org>
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 176 bytes --]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 2/5] dt-bindings: extcon: maxim,max77843: add MAX77843 bindings
2022-01-11 17:48 ` [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: " Krzysztof Kozlowski
@ 2022-03-07 9:08 ` Lee Jones
-1 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-03-07 9:08 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Liam Girdwood,
Mark Brown, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc, Rob Herring
On Tue, 11 Jan 2022, Krzysztof Kozlowski wrote:
> Document the bindings for MAX77843 MUIC/extcon driver, based on
> Exynos5433 TM2 devicetree. These are neither accurate nor finished
> bindings but at least allow parsing existing DTS files.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
> .../bindings/extcon/maxim,max77843.yaml | 40 +++++++++++++++++++
> 1 file changed, 40 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
Applied, thanks.
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: add MAX77843 bindings
@ 2022-03-07 9:08 ` Lee Jones
0 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-03-07 9:08 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Liam Girdwood,
Mark Brown, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc, Rob Herring
On Tue, 11 Jan 2022, Krzysztof Kozlowski wrote:
> Document the bindings for MAX77843 MUIC/extcon driver, based on
> Exynos5433 TM2 devicetree. These are neither accurate nor finished
> bindings but at least allow parsing existing DTS files.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
> .../bindings/extcon/maxim,max77843.yaml | 40 +++++++++++++++++++
> 1 file changed, 40 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
Applied, thanks.
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 3/5] regulator: dt-bindings: maxim,max77843: add MAX77843 bindings
2022-01-11 17:48 ` [PATCH v2 3/5] regulator: dt-bindings: maxim, max77843: " Krzysztof Kozlowski
@ 2022-03-07 9:11 ` Lee Jones
-1 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-03-07 9:11 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Liam Girdwood,
Mark Brown, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc
On Tue, 11 Jan 2022, Krzysztof Kozlowski wrote:
> Document the bindings for MAX77843 regulator driver. The bindings are
> almost identical to MAX77693 bindings, except the actual names of
> regulators.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
> .../bindings/regulator/maxim,max77843.yaml | 65 +++++++++++++++++++
> 1 file changed, 65 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
Applied, thanks.
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 3/5] regulator: dt-bindings: maxim,max77843: add MAX77843 bindings
@ 2022-03-07 9:11 ` Lee Jones
0 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-03-07 9:11 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Liam Girdwood,
Mark Brown, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc
On Tue, 11 Jan 2022, Krzysztof Kozlowski wrote:
> Document the bindings for MAX77843 regulator driver. The bindings are
> almost identical to MAX77693 bindings, except the actual names of
> regulators.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
> .../bindings/regulator/maxim,max77843.yaml | 65 +++++++++++++++++++
> 1 file changed, 65 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
Applied, thanks.
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 4/5] dt-bindings: mfd: maxim,max77843: add MAX77843 bindings
2022-01-11 17:48 ` [PATCH v2 4/5] dt-bindings: mfd: maxim, max77843: " Krzysztof Kozlowski
@ 2022-03-07 9:13 ` Lee Jones
-1 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-03-07 9:13 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Liam Girdwood,
Mark Brown, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc, Rob Herring
On Tue, 11 Jan 2022, Krzysztof Kozlowski wrote:
> Document the bindings for MAX77843 MFD driver, based on Exynos5433 TM2
> devicetree. These are neither accurate nor finished bindings but at
> least allow parsing existing DTS files.
>
> The example DTS was copied from existing DTS
> (exynos5433-tm2-common.dtsi), so keep the license as GPL-2.0-only.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
> .../bindings/mfd/maxim,max77843.yaml | 144 ++++++++++++++++++
> 1 file changed, 144 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
Applied, thanks.
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 4/5] dt-bindings: mfd: maxim,max77843: add MAX77843 bindings
@ 2022-03-07 9:13 ` Lee Jones
0 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-03-07 9:13 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Liam Girdwood,
Mark Brown, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc, Rob Herring
On Tue, 11 Jan 2022, Krzysztof Kozlowski wrote:
> Document the bindings for MAX77843 MFD driver, based on Exynos5433 TM2
> devicetree. These are neither accurate nor finished bindings but at
> least allow parsing existing DTS files.
>
> The example DTS was copied from existing DTS
> (exynos5433-tm2-common.dtsi), so keep the license as GPL-2.0-only.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
> .../bindings/mfd/maxim,max77843.yaml | 144 ++++++++++++++++++
> 1 file changed, 144 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
Applied, thanks.
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 5/5] MAINTAINERS: mfd: cover MAX77843 by Maxim PMIC/MUIC for Exynos boards entry
2022-01-11 17:48 ` Krzysztof Kozlowski
@ 2022-03-07 9:16 ` Lee Jones
-1 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-03-07 9:16 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Liam Girdwood,
Mark Brown, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc
On Tue, 11 Jan 2022, Krzysztof Kozlowski wrote:
> The MAX77843 is used in Exynos5433-based TM2 boards and shares some
> parts of code with MAX77693 (regulator and haptic motor drivers).
> Include all MAX77843 drivers in the entry for Maxim PMIC/MUIC drivers
> for Exynos boards, so they will receive some dedicated review coverage.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
> MAINTAINERS | 2 ++
> 1 file changed, 2 insertions(+)
Applied, thanks.
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 5/5] MAINTAINERS: mfd: cover MAX77843 by Maxim PMIC/MUIC for Exynos boards entry
@ 2022-03-07 9:16 ` Lee Jones
0 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-03-07 9:16 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Liam Girdwood,
Mark Brown, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc
On Tue, 11 Jan 2022, Krzysztof Kozlowski wrote:
> The MAX77843 is used in Exynos5433-based TM2 boards and shares some
> parts of code with MAX77693 (regulator and haptic motor drivers).
> Include all MAX77843 drivers in the entry for Maxim PMIC/MUIC drivers
> for Exynos boards, so they will receive some dedicated review coverage.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
> MAINTAINERS | 2 ++
> 1 file changed, 2 insertions(+)
Applied, thanks.
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* [GIT PULL] Immutable branch between MFD, SPI and DT due for the v5.18 merge window
2022-01-11 17:48 ` Krzysztof Kozlowski
@ 2022-03-07 13:04 ` Lee Jones
-1 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-03-07 13:04 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Liam Girdwood,
Mark Brown, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc
Enjoy!
The following changes since commit e783362eb54cd99b2cac8b3a9aeac942e6f6ac07:
Linux 5.17-rc1 (2022-01-23 10:12:53 +0200)
are available in the Git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git ib-mfd-spi-dt-v5.18
for you to fetch changes up to 172e611b54e813c49a35b6b74bccaa99f27bf566:
dt-bindings: mfd: maxim,max77802: Convert to dtschema (2022-03-07 12:55:59 +0000)
----------------------------------------------------------------
Immutable branch between MFD, SPI and DT due for the v5.18 merge window
----------------------------------------------------------------
Krzysztof Kozlowski (13):
dt-bindings: extcon: maxim,max77843: Add MAX77843 bindings
regulator: dt-bindings: maxim,max77843: Add MAX77843 bindings
dt-bindings: mfd: maxim,max77843: Add MAX77843 bindings
MAINTAINERS: mfd: Cover MAX77843 by Maxim PMIC/MUIC for Exynos boards entry
spi: dt-bindings: samsung: Convert to dtschema
mfd: dt-bindings: google,cros-ec: Reference Samsung SPI bindings
mfd: dt-bindings: google,cros-ec: Fix indentation in example
spi: s3c64xx: Allow controller-data to be optional
dt-bindings: power: supply: maxim,max14577: Convert to dtschema
regulator: dt-bindings: maxim,max14577: Convert to dtschema
dt-bindings: mfd: maxim,max14577: Convert to dtschema
regulator: dt-bindings: maxim,max77802: Convert to dtschema
dt-bindings: mfd: maxim,max77802: Convert to dtschema
.../devicetree/bindings/extcon/maxim,max77843.yaml | 40 +++++
.../devicetree/bindings/mfd/google,cros-ec.yaml | 31 ++--
Documentation/devicetree/bindings/mfd/max14577.txt | 147 ----------------
Documentation/devicetree/bindings/mfd/max77802.txt | 25 ---
.../devicetree/bindings/mfd/maxim,max14577.yaml | 195 +++++++++++++++++++++
.../devicetree/bindings/mfd/maxim,max77802.yaml | 194 ++++++++++++++++++++
.../devicetree/bindings/mfd/maxim,max77843.yaml | 144 +++++++++++++++
.../bindings/power/supply/maxim,max14577.yaml | 84 +++++++++
.../devicetree/bindings/regulator/max77802.txt | 111 ------------
.../bindings/regulator/maxim,max14577.yaml | 78 +++++++++
.../bindings/regulator/maxim,max77802.yaml | 85 +++++++++
.../bindings/regulator/maxim,max77843.yaml | 65 +++++++
.../bindings/soc/samsung/exynos-usi.yaml | 2 +-
.../bindings/spi/samsung,spi-peripheral-props.yaml | 33 ++++
.../devicetree/bindings/spi/samsung,spi.yaml | 187 ++++++++++++++++++++
.../bindings/spi/spi-peripheral-props.yaml | 1 +
.../devicetree/bindings/spi/spi-samsung.txt | 122 -------------
MAINTAINERS | 9 +-
drivers/spi/spi-s3c64xx.c | 14 +-
19 files changed, 1136 insertions(+), 431 deletions(-)
create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
delete mode 100644 Documentation/devicetree/bindings/mfd/max14577.txt
delete mode 100644 Documentation/devicetree/bindings/mfd/max77802.txt
create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max14577.yaml
create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77802.yaml
create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
create mode 100644 Documentation/devicetree/bindings/power/supply/maxim,max14577.yaml
delete mode 100644 Documentation/devicetree/bindings/regulator/max77802.txt
create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max14577.yaml
create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
create mode 100644 Documentation/devicetree/bindings/spi/samsung,spi-peripheral-props.yaml
create mode 100644 Documentation/devicetree/bindings/spi/samsung,spi.yaml
delete mode 100644 Documentation/devicetree/bindings/spi/spi-samsung.txt
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 42+ messages in thread
* [GIT PULL] Immutable branch between MFD, SPI and DT due for the v5.18 merge window
@ 2022-03-07 13:04 ` Lee Jones
0 siblings, 0 replies; 42+ messages in thread
From: Lee Jones @ 2022-03-07 13:04 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Rob Herring, Liam Girdwood,
Mark Brown, linux-kernel, devicetree, linux-arm-kernel,
linux-samsung-soc
Enjoy!
The following changes since commit e783362eb54cd99b2cac8b3a9aeac942e6f6ac07:
Linux 5.17-rc1 (2022-01-23 10:12:53 +0200)
are available in the Git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git ib-mfd-spi-dt-v5.18
for you to fetch changes up to 172e611b54e813c49a35b6b74bccaa99f27bf566:
dt-bindings: mfd: maxim,max77802: Convert to dtschema (2022-03-07 12:55:59 +0000)
----------------------------------------------------------------
Immutable branch between MFD, SPI and DT due for the v5.18 merge window
----------------------------------------------------------------
Krzysztof Kozlowski (13):
dt-bindings: extcon: maxim,max77843: Add MAX77843 bindings
regulator: dt-bindings: maxim,max77843: Add MAX77843 bindings
dt-bindings: mfd: maxim,max77843: Add MAX77843 bindings
MAINTAINERS: mfd: Cover MAX77843 by Maxim PMIC/MUIC for Exynos boards entry
spi: dt-bindings: samsung: Convert to dtschema
mfd: dt-bindings: google,cros-ec: Reference Samsung SPI bindings
mfd: dt-bindings: google,cros-ec: Fix indentation in example
spi: s3c64xx: Allow controller-data to be optional
dt-bindings: power: supply: maxim,max14577: Convert to dtschema
regulator: dt-bindings: maxim,max14577: Convert to dtschema
dt-bindings: mfd: maxim,max14577: Convert to dtschema
regulator: dt-bindings: maxim,max77802: Convert to dtschema
dt-bindings: mfd: maxim,max77802: Convert to dtschema
.../devicetree/bindings/extcon/maxim,max77843.yaml | 40 +++++
.../devicetree/bindings/mfd/google,cros-ec.yaml | 31 ++--
Documentation/devicetree/bindings/mfd/max14577.txt | 147 ----------------
Documentation/devicetree/bindings/mfd/max77802.txt | 25 ---
.../devicetree/bindings/mfd/maxim,max14577.yaml | 195 +++++++++++++++++++++
.../devicetree/bindings/mfd/maxim,max77802.yaml | 194 ++++++++++++++++++++
.../devicetree/bindings/mfd/maxim,max77843.yaml | 144 +++++++++++++++
.../bindings/power/supply/maxim,max14577.yaml | 84 +++++++++
.../devicetree/bindings/regulator/max77802.txt | 111 ------------
.../bindings/regulator/maxim,max14577.yaml | 78 +++++++++
.../bindings/regulator/maxim,max77802.yaml | 85 +++++++++
.../bindings/regulator/maxim,max77843.yaml | 65 +++++++
.../bindings/soc/samsung/exynos-usi.yaml | 2 +-
.../bindings/spi/samsung,spi-peripheral-props.yaml | 33 ++++
.../devicetree/bindings/spi/samsung,spi.yaml | 187 ++++++++++++++++++++
.../bindings/spi/spi-peripheral-props.yaml | 1 +
.../devicetree/bindings/spi/spi-samsung.txt | 122 -------------
MAINTAINERS | 9 +-
drivers/spi/spi-s3c64xx.c | 14 +-
19 files changed, 1136 insertions(+), 431 deletions(-)
create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
delete mode 100644 Documentation/devicetree/bindings/mfd/max14577.txt
delete mode 100644 Documentation/devicetree/bindings/mfd/max77802.txt
create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max14577.yaml
create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77802.yaml
create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
create mode 100644 Documentation/devicetree/bindings/power/supply/maxim,max14577.yaml
delete mode 100644 Documentation/devicetree/bindings/regulator/max77802.txt
create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max14577.yaml
create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
create mode 100644 Documentation/devicetree/bindings/regulator/maxim,max77843.yaml
create mode 100644 Documentation/devicetree/bindings/spi/samsung,spi-peripheral-props.yaml
create mode 100644 Documentation/devicetree/bindings/spi/samsung,spi.yaml
delete mode 100644 Documentation/devicetree/bindings/spi/spi-samsung.txt
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 2/5] dt-bindings: extcon: maxim,max77843: add MAX77843 bindings
2022-01-11 17:48 ` [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: " Krzysztof Kozlowski
@ 2022-03-09 21:03 ` Rob Herring
-1 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2022-03-09 21:03 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Lee Jones, Liam Girdwood, Mark Brown,
linux-kernel, devicetree, linux-arm-kernel, linux-samsung-soc
On Tue, Jan 11, 2022 at 11:48 AM Krzysztof Kozlowski
<krzysztof.kozlowski@canonical.com> wrote:
>
> Document the bindings for MAX77843 MUIC/extcon driver, based on
> Exynos5433 TM2 devicetree. These are neither accurate nor finished
> bindings but at least allow parsing existing DTS files.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
> .../bindings/extcon/maxim,max77843.yaml | 40 +++++++++++++++++++
> 1 file changed, 40 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
>
> diff --git a/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
> new file mode 100644
> index 000000000000..f9ffe3d6f957
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
> @@ -0,0 +1,40 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/extcon/maxim,max77843.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Maxim MAX77843 MicroUSB and Companion Power Management IC Extcon
> +
> +maintainers:
> + - Chanwoo Choi <cw00.choi@samsung.com>
> + - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> +
> +description: |
> + This is a part of device tree bindings for Maxim MAX77843 MicroUSB
> + Integrated Circuit (MUIC).
> +
> + See also Documentation/devicetree/bindings/mfd/maxim,max77843.yaml for
> + additional information and example.
> +
> +properties:
> + compatible:
> + const: maxim,max77843-muic
> +
> + connector:
> + $ref: /schemas/connector/usb-connector.yaml#
> +
> + ports:
> + $ref: /schemas/graph.yaml#/properties/port
Now failing in -next:
Documentation/devicetree/bindings/mfd/maxim,max77843.example.dt.yaml:
pmic@66: extcon:ports: Unevaluated properties are not allowed ('port'
was unexpected)
From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
s/port/ports/
> + description:
> + Any connector to the data bus of this controller should be modelled using
> + the OF graph bindings specified
> + properties:
> + port:
> + $ref: /schemas/graph.yaml#/properties/port
> +
> +required:
> + - compatible
> + - connector
> +
> +additionalProperties: false
> --
> 2.32.0
>
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: add MAX77843 bindings
@ 2022-03-09 21:03 ` Rob Herring
0 siblings, 0 replies; 42+ messages in thread
From: Rob Herring @ 2022-03-09 21:03 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Chanwoo Choi, MyungJoo Ham, Lee Jones, Liam Girdwood, Mark Brown,
linux-kernel, devicetree, linux-arm-kernel, linux-samsung-soc
On Tue, Jan 11, 2022 at 11:48 AM Krzysztof Kozlowski
<krzysztof.kozlowski@canonical.com> wrote:
>
> Document the bindings for MAX77843 MUIC/extcon driver, based on
> Exynos5433 TM2 devicetree. These are neither accurate nor finished
> bindings but at least allow parsing existing DTS files.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
> .../bindings/extcon/maxim,max77843.yaml | 40 +++++++++++++++++++
> 1 file changed, 40 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
>
> diff --git a/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
> new file mode 100644
> index 000000000000..f9ffe3d6f957
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
> @@ -0,0 +1,40 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/extcon/maxim,max77843.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Maxim MAX77843 MicroUSB and Companion Power Management IC Extcon
> +
> +maintainers:
> + - Chanwoo Choi <cw00.choi@samsung.com>
> + - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> +
> +description: |
> + This is a part of device tree bindings for Maxim MAX77843 MicroUSB
> + Integrated Circuit (MUIC).
> +
> + See also Documentation/devicetree/bindings/mfd/maxim,max77843.yaml for
> + additional information and example.
> +
> +properties:
> + compatible:
> + const: maxim,max77843-muic
> +
> + connector:
> + $ref: /schemas/connector/usb-connector.yaml#
> +
> + ports:
> + $ref: /schemas/graph.yaml#/properties/port
Now failing in -next:
Documentation/devicetree/bindings/mfd/maxim,max77843.example.dt.yaml:
pmic@66: extcon:ports: Unevaluated properties are not allowed ('port'
was unexpected)
From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
s/port/ports/
> + description:
> + Any connector to the data bus of this controller should be modelled using
> + the OF graph bindings specified
> + properties:
> + port:
> + $ref: /schemas/graph.yaml#/properties/port
> +
> +required:
> + - compatible
> + - connector
> +
> +additionalProperties: false
> --
> 2.32.0
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 2/5] dt-bindings: extcon: maxim,max77843: add MAX77843 bindings
2022-03-09 21:03 ` [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: " Rob Herring
@ 2022-03-10 7:27 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-03-10 7:27 UTC (permalink / raw)
To: Rob Herring
Cc: Chanwoo Choi, MyungJoo Ham, Lee Jones, Liam Girdwood, Mark Brown,
linux-kernel, devicetree, linux-arm-kernel, linux-samsung-soc
On 09/03/2022 22:03, Rob Herring wrote:
> On Tue, Jan 11, 2022 at 11:48 AM Krzysztof Kozlowski
> <krzysztof.kozlowski@canonical.com> wrote:
>>
>> Document the bindings for MAX77843 MUIC/extcon driver, based on
>> Exynos5433 TM2 devicetree. These are neither accurate nor finished
>> bindings but at least allow parsing existing DTS files.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>> ---
>> .../bindings/extcon/maxim,max77843.yaml | 40 +++++++++++++++++++
>> 1 file changed, 40 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
>> new file mode 100644
>> index 000000000000..f9ffe3d6f957
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
>> @@ -0,0 +1,40 @@
>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/extcon/maxim,max77843.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Maxim MAX77843 MicroUSB and Companion Power Management IC Extcon
>> +
>> +maintainers:
>> + - Chanwoo Choi <cw00.choi@samsung.com>
>> + - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>> +
>> +description: |
>> + This is a part of device tree bindings for Maxim MAX77843 MicroUSB
>> + Integrated Circuit (MUIC).
>> +
>> + See also Documentation/devicetree/bindings/mfd/maxim,max77843.yaml for
>> + additional information and example.
>> +
>> +properties:
>> + compatible:
>> + const: maxim,max77843-muic
>> +
>> + connector:
>> + $ref: /schemas/connector/usb-connector.yaml#
>> +
>> + ports:
>> + $ref: /schemas/graph.yaml#/properties/port
>
> Now failing in -next:
>
> Documentation/devicetree/bindings/mfd/maxim,max77843.example.dt.yaml:
> pmic@66: extcon:ports: Unevaluated properties are not allowed ('port'
> was unexpected)
> From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
>
> s/port/ports/
I'll fix it up. I wonder why I did not see warning before...
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 42+ messages in thread
* Re: [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: add MAX77843 bindings
@ 2022-03-10 7:27 ` Krzysztof Kozlowski
0 siblings, 0 replies; 42+ messages in thread
From: Krzysztof Kozlowski @ 2022-03-10 7:27 UTC (permalink / raw)
To: Rob Herring
Cc: Chanwoo Choi, MyungJoo Ham, Lee Jones, Liam Girdwood, Mark Brown,
linux-kernel, devicetree, linux-arm-kernel, linux-samsung-soc
On 09/03/2022 22:03, Rob Herring wrote:
> On Tue, Jan 11, 2022 at 11:48 AM Krzysztof Kozlowski
> <krzysztof.kozlowski@canonical.com> wrote:
>>
>> Document the bindings for MAX77843 MUIC/extcon driver, based on
>> Exynos5433 TM2 devicetree. These are neither accurate nor finished
>> bindings but at least allow parsing existing DTS files.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>> ---
>> .../bindings/extcon/maxim,max77843.yaml | 40 +++++++++++++++++++
>> 1 file changed, 40 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
>> new file mode 100644
>> index 000000000000..f9ffe3d6f957
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/extcon/maxim,max77843.yaml
>> @@ -0,0 +1,40 @@
>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/extcon/maxim,max77843.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Maxim MAX77843 MicroUSB and Companion Power Management IC Extcon
>> +
>> +maintainers:
>> + - Chanwoo Choi <cw00.choi@samsung.com>
>> + - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>> +
>> +description: |
>> + This is a part of device tree bindings for Maxim MAX77843 MicroUSB
>> + Integrated Circuit (MUIC).
>> +
>> + See also Documentation/devicetree/bindings/mfd/maxim,max77843.yaml for
>> + additional information and example.
>> +
>> +properties:
>> + compatible:
>> + const: maxim,max77843-muic
>> +
>> + connector:
>> + $ref: /schemas/connector/usb-connector.yaml#
>> +
>> + ports:
>> + $ref: /schemas/graph.yaml#/properties/port
>
> Now failing in -next:
>
> Documentation/devicetree/bindings/mfd/maxim,max77843.example.dt.yaml:
> pmic@66: extcon:ports: Unevaluated properties are not allowed ('port'
> was unexpected)
> From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/mfd/maxim,max77843.yaml
>
> s/port/ports/
I'll fix it up. I wonder why I did not see warning before...
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 42+ messages in thread
end of thread, other threads:[~2022-03-10 7:28 UTC | newest]
Thread overview: 42+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-11 17:48 [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings Krzysztof Kozlowski
2022-01-11 17:48 ` Krzysztof Kozlowski
2022-01-11 17:48 ` [PATCH v2 1/5] arm64: dts: exynos: Align MAX77843 nodes with dtschema on TM2 Krzysztof Kozlowski
2022-01-11 17:48 ` Krzysztof Kozlowski
2022-01-23 17:10 ` (subset) " Krzysztof Kozlowski
2022-01-23 17:10 ` Krzysztof Kozlowski
2022-01-11 17:48 ` [PATCH v2 2/5] dt-bindings: extcon: maxim,max77843: add MAX77843 bindings Krzysztof Kozlowski
2022-01-11 17:48 ` [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: " Krzysztof Kozlowski
2022-03-07 9:08 ` [PATCH v2 2/5] dt-bindings: extcon: maxim,max77843: " Lee Jones
2022-03-07 9:08 ` [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: " Lee Jones
2022-03-09 21:03 ` [PATCH v2 2/5] dt-bindings: extcon: maxim,max77843: " Rob Herring
2022-03-09 21:03 ` [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: " Rob Herring
2022-03-10 7:27 ` [PATCH v2 2/5] dt-bindings: extcon: maxim,max77843: " Krzysztof Kozlowski
2022-03-10 7:27 ` [PATCH v2 2/5] dt-bindings: extcon: maxim, max77843: " Krzysztof Kozlowski
2022-01-11 17:48 ` [PATCH v2 3/5] regulator: dt-bindings: maxim,max77843: " Krzysztof Kozlowski
2022-01-11 17:48 ` [PATCH v2 3/5] regulator: dt-bindings: maxim, max77843: " Krzysztof Kozlowski
2022-01-11 22:21 ` [PATCH v2 3/5] regulator: dt-bindings: maxim,max77843: " Rob Herring
2022-01-11 22:21 ` Rob Herring
2022-02-11 11:59 ` Mark Brown
2022-02-11 11:59 ` Mark Brown
2022-03-07 9:11 ` Lee Jones
2022-03-07 9:11 ` Lee Jones
2022-01-11 17:48 ` [PATCH v2 4/5] dt-bindings: mfd: " Krzysztof Kozlowski
2022-01-11 17:48 ` [PATCH v2 4/5] dt-bindings: mfd: maxim, max77843: " Krzysztof Kozlowski
2022-03-07 9:13 ` [PATCH v2 4/5] dt-bindings: mfd: maxim,max77843: " Lee Jones
2022-03-07 9:13 ` Lee Jones
2022-01-11 17:48 ` [PATCH v2 5/5] MAINTAINERS: mfd: cover MAX77843 by Maxim PMIC/MUIC for Exynos boards entry Krzysztof Kozlowski
2022-01-11 17:48 ` Krzysztof Kozlowski
2022-03-07 9:16 ` Lee Jones
2022-03-07 9:16 ` Lee Jones
2022-02-06 17:39 ` [PATCH v2 0/5] mfd/extcon/regulators: max77843: add dtschema/bindings Krzysztof Kozlowski
2022-02-06 17:39 ` Krzysztof Kozlowski
2022-02-07 9:33 ` Lee Jones
2022-02-07 9:33 ` Lee Jones
2022-02-07 9:47 ` Krzysztof Kozlowski
2022-02-07 9:47 ` Krzysztof Kozlowski
2022-02-07 10:11 ` Lee Jones
2022-02-07 10:11 ` Lee Jones
2022-02-07 16:25 ` Mark Brown
2022-02-07 16:25 ` Mark Brown
2022-03-07 13:04 ` [GIT PULL] Immutable branch between MFD, SPI and DT due for the v5.18 merge window Lee Jones
2022-03-07 13:04 ` Lee Jones
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.