2022-04-22 20:39:55

by Quan Nguyen

Subject: [PATCH v7 2/3] bindings: ipmi: Add binding for SSIF BMC driver

Add device tree binding document for the SSIF BMC driver.

Signed-off-by: Quan Nguyen <[email protected]>
---
v7:
+ Change compatible string from "ampere,ssif-bmc" to "ssif-bmc" [Jae]

v6:
+ None

v5:
+ None

v4:
+ Fix warning with dt_binding_check [Rob]
+ Change aspeed-ssif-bmc.yaml to ssif-bmc.yaml [Quan]

v3:
+ Switched to use DT schema format [Rob]

v2:
+ None

.../devicetree/bindings/ipmi/ssif-bmc.yaml | 38 +++++++++++++++++++
1 file changed, 38 insertions(+)
create mode 100644 Documentation/devicetree/bindings/ipmi/ssif-bmc.yaml

diff --git a/Documentation/devicetree/bindings/ipmi/ssif-bmc.yaml b/Documentation/devicetree/bindings/ipmi/ssif-bmc.yaml
new file mode 100644
index 000000000000..02b662d780bb
--- /dev/null
+++ b/Documentation/devicetree/bindings/ipmi/ssif-bmc.yaml
@@ -0,0 +1,38 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/ipmi/ssif-bmc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: SSIF IPMI BMC interface
+
+description: SSIF IPMI BMC device bindings
+
+maintainers:
+ - Quan Nguyen <[email protected]>
+
+properties:
+ compatible:
+ enum:
+ - ssif-bmc
+
+ reg:
+ maxItems: 1
+
+required:
+ - compatible
+ - reg
+
+additionalProperties: false
+
+examples:
+ - |
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ ssif-bmc@10 {
+ compatible = "ssif-bmc";
+ reg = <0x10>;
+ };
+ };
--
2.35.1

2022-04-22 21:07:12

by Quan Nguyen

Subject: Re: [PATCH v7 2/3] bindings: ipmi: Add binding for SSIF BMC driver

Added Krzysztof Kozlowski <[email protected]>
as I'm not aware of the email change
- Quan

On 22/04/2022 11:08, Quan Nguyen wrote:
> Add device tree binding document for the SSIF BMC driver.
>
> Signed-off-by: Quan Nguyen <[email protected]>
> ---
> v7:
> + Change compatible string from "ampere,ssif-bmc" to "ssif-bmc" [Jae]
>
> v6:
> + None
>
> v5:
> + None
>
> v4:
> + Fix warning with dt_binding_check [Rob]
> + Change aspeed-ssif-bmc.yaml to ssif-bmc.yaml [Quan]
>
> v3:
> + Switched to use DT schema format [Rob]
>
> v2:
> + None
>
> .../devicetree/bindings/ipmi/ssif-bmc.yaml | 38 +++++++++++++++++++
> 1 file changed, 38 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/ipmi/ssif-bmc.yaml
>
> diff --git a/Documentation/devicetree/bindings/ipmi/ssif-bmc.yaml b/Documentation/devicetree/bindings/ipmi/ssif-bmc.yaml
> new file mode 100644
> index 000000000000..02b662d780bb
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/ipmi/ssif-bmc.yaml
> @@ -0,0 +1,38 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/ipmi/ssif-bmc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: SSIF IPMI BMC interface
> +
> +description: SSIF IPMI BMC device bindings
> +
> +maintainers:
> + - Quan Nguyen <[email protected]>
> +
> +properties:
> + compatible:
> + enum:
> + - ssif-bmc
> +
> + reg:
> + maxItems: 1
> +
> +required:
> + - compatible
> + - reg
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + i2c {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + ssif-bmc@10 {
> + compatible = "ssif-bmc";
> + reg = <0x10>;
> + };
> + };

2022-04-25 22:58:01

by Rob Herring (Arm)

Subject: Re: [PATCH v7 2/3] bindings: ipmi: Add binding for SSIF BMC driver

On Fri, 22 Apr 2022 11:08:02 +0700, Quan Nguyen wrote:
> Add device tree binding document for the SSIF BMC driver.
>
> Signed-off-by: Quan Nguyen <[email protected]>
> ---
> v7:
> + Change compatible string from "ampere,ssif-bmc" to "ssif-bmc" [Jae]
>
> v6:
> + None
>
> v5:
> + None
>
> v4:
> + Fix warning with dt_binding_check [Rob]
> + Change aspeed-ssif-bmc.yaml to ssif-bmc.yaml [Quan]
>
> v3:
> + Switched to use DT schema format [Rob]
>
> v2:
> + None
>
> .../devicetree/bindings/ipmi/ssif-bmc.yaml | 38 +++++++++++++++++++
> 1 file changed, 38 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/ipmi/ssif-bmc.yaml
>

Reviewed-by: Rob Herring <[email protected]>