All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v8 0/2] dt-bindings: mips: add CPU bindings for MIPS architecture
@ 2022-10-06  4:29 Sergio Paracuellos
  2022-10-06  4:29 ` [PATCH v8 1/2] " Sergio Paracuellos
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Sergio Paracuellos @ 2022-10-06  4:29 UTC (permalink / raw)
  To: devicetree
  Cc: robh+dt, krzysztof.kozlowski+dt, hauke, zajec5, tsbogend,
	zhouyanjie, linux-mips, arinc.unal, f.fainelli

Hi all,

This series tries to make the correct thing to represent in schema
all the current documentation related with MIPS CPUs.

Broadcom cpus node is a bit special and need to use the property
'mips-hpt-frequency' in the cpus node. Because of this I have introduced
brcm/soc.yaml schema with current broadcom SoCs and making this property
required as per Rob's v3 review comments.

Ingenic cpus have also its schema already mainlined. To unify things I
have also put this information in this mips/cpus.yaml schema and remove
the 'mips/ingenic/ingenic,cpu.yaml' schema. I have also added current
maintainer as maintainer for this 'mips/cpus.yaml' file.

Thomas, are these patches going through the mips git tree?

Thanks,
    Sergio Paracuellos

Changes in v8:
  - Remove Hauke Mehrtens from brcm/soc.yaml maintainers as per Hauke Mehrtens's
    requested in v7 response.

Changes in v7:
  - Remove Rafał Miłecki from brcm/soc.yaml maintainers as per Rafał Miłecki's
    requested in v6 response.

Changes in v6:
  - Avoid 'convert' in SoC's patch commit message since this is just an
    addition and nothing is removed in this patch.
  - Add Florian Fainelli to maintainers of Broadcom SoC bindings
  - Add Florian's Acked-by for broadcom SoC PATCH.
  - Collect Rob's Reviewed-by for both patches.

Changes in v5:
  - Address Krzysztof comments in v4:
      - change BMIPS SoC compatibles into an enum and drop descriptions,
      - Add blank line.
      - Redo commit message since it is not a conversion to schema.
      - Drop 'device tree bindings' from description.
      - Properly describe 'mips-hpt-frequency'.
  - Review cpus node and add a sample to check schema correctness.    

Changes in v4:
- Address Rob's v3 review comments:
    * Drop PATCH introducing special brcm,cpus.yaml only because properyu
    'mips-hpt-frequency' is required and move this property to brcm/soc.yaml
    schema.
    * Remove 'Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml'
    and add that information to general mips/cpus.yaml schema.
    * Define all properties in mips/cpus.yaml and set 'additionalProperties'
    to false.

Changes in v3:
- cpus.yaml:
    * address Krzysztof comment in v2:
    * add Thomas as maintainer since this is architecture binding.
    * s/cpu/CPU
    * compatible goes first
    * Add compatible 'mips,mips4Kc' as per Thomas request.
- Introduce two new schemas for Broadcom BMIPS:
    * soc.yaml: describing the BRCM SoC's
    * brcm,bmips-cpus.yaml: describing the cpu nodes for this platform.

Previous series:
v1: https://lore.kernel.org/all/CAMhs-H-eUTOHjAXAbywOXQJgc_j5Ex-1sB7eBZU_bWt1fpNVzA@mail.gmail.com/T/
v2: https://lore.kernel.org/all/20220918112245.GA5555@alpha.franken.de/T/
v3: https://lore.kernel.org/linux-devicetree/20220929072004.874795-1-sergio.paracuellos@gmail.com/T/#t
v4: https://lore.kernel.org/linux-devicetree/20221001043855.933528-1-sergio.paracuellos@gmail.com/T/#t
v5: https://lore.kernel.org/linux-devicetree/20221002091611.946210-1-sergio.paracuellos@gmail.com/T/#t
v6: https://lore.kernel.org/linux-devicetree/20221004050924.986211-1-sergio.paracuellos@gmail.com/T/#t

Sergio Paracuellos (2):
  dt-bindings: mips: add CPU bindings for MIPS architecture
  dt-bindings: mips: brcm: add Broadcom SoCs bindings

 .../bindings/mips/brcm/brcm,bmips.txt         |   8 --
 .../devicetree/bindings/mips/brcm/soc.yaml    |  97 +++++++++++++++
 .../devicetree/bindings/mips/cpus.yaml        | 115 ++++++++++++++++++
 .../bindings/mips/ingenic/ingenic,cpu.yaml    |  69 -----------
 4 files changed, 212 insertions(+), 77 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt
 create mode 100644 Documentation/devicetree/bindings/mips/brcm/soc.yaml
 create mode 100644 Documentation/devicetree/bindings/mips/cpus.yaml
 delete mode 100644 Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml

-- 
2.25.1


^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH v8 1/2] dt-bindings: mips: add CPU bindings for MIPS architecture
  2022-10-06  4:29 [PATCH v8 0/2] dt-bindings: mips: add CPU bindings for MIPS architecture Sergio Paracuellos
@ 2022-10-06  4:29 ` Sergio Paracuellos
  2022-10-06 11:23   ` Philippe Mathieu-Daudé
  2022-10-06  4:29 ` [PATCH v8 2/2] dt-bindings: mips: brcm: add Broadcom SoCs bindings Sergio Paracuellos
  2022-11-21 17:06 ` [PATCH v8 0/2] dt-bindings: mips: add CPU bindings for MIPS architecture Thomas Bogendoerfer
  2 siblings, 1 reply; 9+ messages in thread
From: Sergio Paracuellos @ 2022-10-06  4:29 UTC (permalink / raw)
  To: devicetree
  Cc: robh+dt, krzysztof.kozlowski+dt, hauke, zajec5, tsbogend,
	zhouyanjie, linux-mips, arinc.unal, f.fainelli, Rob Herring

Add the yaml binding for available CPUs in MIPS architecture.

Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
 .../bindings/mips/brcm/brcm,bmips.txt         |   8 --
 .../devicetree/bindings/mips/cpus.yaml        | 115 ++++++++++++++++++
 .../bindings/mips/ingenic/ingenic,cpu.yaml    |  69 -----------
 3 files changed, 115 insertions(+), 77 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt
 create mode 100644 Documentation/devicetree/bindings/mips/cpus.yaml
 delete mode 100644 Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml

diff --git a/Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt b/Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt
deleted file mode 100644
index 8ef71b4085ca..000000000000
--- a/Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt
+++ /dev/null
@@ -1,8 +0,0 @@
-* Broadcom MIPS (BMIPS) CPUs
-
-Required properties:
-- compatible: "brcm,bmips3300", "brcm,bmips4350", "brcm,bmips4380",
-  "brcm,bmips5000"
-
-- mips-hpt-frequency: This is common to all CPUs in the system so it lives
-  under the "cpus" node.
diff --git a/Documentation/devicetree/bindings/mips/cpus.yaml b/Documentation/devicetree/bindings/mips/cpus.yaml
new file mode 100644
index 000000000000..e991f4c6668d
--- /dev/null
+++ b/Documentation/devicetree/bindings/mips/cpus.yaml
@@ -0,0 +1,115 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mips/cpus.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: MIPS CPUs bindings
+
+maintainers:
+  - Thomas Bogendoerfer <tsbogend@alpha.franken.de>
+  - 周琰杰 (Zhou Yanjie) <zhouyanjie@wanyeetech.com>
+
+description: |
+  The device tree allows to describe the layout of CPUs in a system through
+  the "cpus" node, which in turn contains a number of subnodes (ie "cpu")
+  defining properties for every CPU.
+
+properties:
+  compatible:
+    enum:
+      - brcm,bmips3300
+      - brcm,bmips4350
+      - brcm,bmips4380
+      - brcm,bmips5000
+      - brcm,bmips5200
+      - ingenic,xburst-mxu1.0
+      - ingenic,xburst-fpu1.0-mxu1.1
+      - ingenic,xburst-fpu2.0-mxu2.0
+      - ingenic,xburst2-fpu2.1-mxu2.1-smt
+      - loongson,gs264
+      - mips,m14Kc
+      - mips,mips4Kc
+      - mips,mips4KEc
+      - mips,mips24Kc
+      - mips,mips24KEc
+      - mips,mips74Kc
+      - mips,mips1004Kc
+      - mti,interaptiv
+      - mti,mips24KEc
+      - mti,mips14KEc
+      - mti,mips14Kc
+
+  reg:
+    maxItems: 1
+
+  clocks:
+    maxItems: 1
+
+  device_type: true
+
+allOf:
+  - if:
+      properties:
+        compatible:
+          contains:
+            enum:
+              - ingenic,xburst-mxu1.0
+              - ingenic,xburst-fpu1.0-mxu1.1
+              - ingenic,xburst-fpu2.0-mxu2.0
+              - ingenic,xburst2-fpu2.1-mxu2.1-smt
+    then:
+      required:
+        - device_type
+        - clocks
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    cpus {
+      #size-cells = <0>;
+      #address-cells = <1>;
+
+      cpu@0 {
+        compatible = "mips,mips1004Kc";
+        device_type = "cpu";
+        reg = <0>;
+      };
+
+      cpu@1 {
+        compatible = "mips,mips1004Kc";
+        device_type = "cpu";
+        reg = <1>;
+      };
+    };
+
+  - |
+    // Example 2 (Ingenic CPU)
+    #include <dt-bindings/clock/ingenic,jz4780-cgu.h>
+
+    cpus {
+      #address-cells = <1>;
+      #size-cells = <0>;
+
+      cpu@0 {
+        compatible = "ingenic,xburst-fpu1.0-mxu1.1";
+        device_type = "cpu";
+        reg = <0>;
+
+        clocks = <&cgu JZ4780_CLK_CPU>;
+      };
+
+      cpu@1 {
+        compatible = "ingenic,xburst-fpu1.0-mxu1.1";
+        device_type = "cpu";
+        reg = <1>;
+
+        clocks = <&cgu JZ4780_CLK_CORE1>;
+      };
+    };
+...
diff --git a/Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml b/Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml
deleted file mode 100644
index b7e7fa715437..000000000000
--- a/Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml
+++ /dev/null
@@ -1,69 +0,0 @@
-# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
-%YAML 1.2
----
-$id: http://devicetree.org/schemas/mips/ingenic/ingenic,cpu.yaml#
-$schema: http://devicetree.org/meta-schemas/core.yaml#
-
-title: Bindings for Ingenic XBurst family CPUs
-
-maintainers:
-  - 周琰杰 (Zhou Yanjie) <zhouyanjie@wanyeetech.com>
-
-description:
-  Ingenic XBurst family CPUs shall have the following properties.
-
-properties:
-  compatible:
-    oneOf:
-
-      - description: Ingenic XBurst®1 CPU Cores
-        enum:
-          - ingenic,xburst-mxu1.0
-          - ingenic,xburst-fpu1.0-mxu1.1
-          - ingenic,xburst-fpu2.0-mxu2.0
-
-      - description: Ingenic XBurst®2 CPU Cores
-        enum:
-          - ingenic,xburst2-fpu2.1-mxu2.1-smt
-
-  reg:
-    maxItems: 1
-
-  clocks:
-    maxItems: 1
-
-  device_type: true
-
-required:
-  - device_type
-  - compatible
-  - reg
-  - clocks
-
-additionalProperties: false
-
-examples:
-  - |
-    #include <dt-bindings/clock/ingenic,jz4780-cgu.h>
-
-    cpus {
-        #address-cells = <1>;
-        #size-cells = <0>;
-
-        cpu0: cpu@0 {
-                device_type = "cpu";
-                compatible = "ingenic,xburst-fpu1.0-mxu1.1";
-                reg = <0>;
-
-                clocks = <&cgu JZ4780_CLK_CPU>;
-        };
-
-        cpu1: cpu@1 {
-                device_type = "cpu";
-                compatible = "ingenic,xburst-fpu1.0-mxu1.1";
-                reg = <1>;
-
-                clocks = <&cgu JZ4780_CLK_CORE1>;
-        };
-    };
-...
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH v8 2/2] dt-bindings: mips: brcm: add Broadcom SoCs bindings
  2022-10-06  4:29 [PATCH v8 0/2] dt-bindings: mips: add CPU bindings for MIPS architecture Sergio Paracuellos
  2022-10-06  4:29 ` [PATCH v8 1/2] " Sergio Paracuellos
@ 2022-10-06  4:29 ` Sergio Paracuellos
  2022-11-21 17:06 ` [PATCH v8 0/2] dt-bindings: mips: add CPU bindings for MIPS architecture Thomas Bogendoerfer
  2 siblings, 0 replies; 9+ messages in thread
From: Sergio Paracuellos @ 2022-10-06  4:29 UTC (permalink / raw)
  To: devicetree
  Cc: robh+dt, krzysztof.kozlowski+dt, hauke, zajec5, tsbogend,
	zhouyanjie, linux-mips, arinc.unal, f.fainelli, Rob Herring

Add the yaml binding for MIPS Broadcom cable/DSL/settop platforms.

Acked-by: Florian Fainelli <f.fainelli@gmail.com>
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
 .../devicetree/bindings/mips/brcm/soc.yaml    | 96 +++++++++++++++++++
 1 file changed, 96 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/mips/brcm/soc.yaml

diff --git a/Documentation/devicetree/bindings/mips/brcm/soc.yaml b/Documentation/devicetree/bindings/mips/brcm/soc.yaml
new file mode 100644
index 000000000000..975945ca2888
--- /dev/null
+++ b/Documentation/devicetree/bindings/mips/brcm/soc.yaml
@@ -0,0 +1,96 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mips/brcm/soc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Broadcom cable/DSL/settop platforms
+
+maintainers:
+  - Florian Fainelli <f.fainelli@gmail.com>
+
+description: |
+    Boards Broadcom cable/DSL/settop SoC shall have the following properties.
+    The experimental -viper variants are for running Linux on the 3384's
+    BMIPS4355 cable modem CPU instead of the BMIPS5000 application processor.
+
+properties:
+  $nodename:
+    const: '/'
+
+  compatible:
+    enum:
+      - brcm,bcm3368
+      - brcm,bcm3384
+      - brcm,bcm33843
+      - brcm,bcm3384-viper
+      - brcm,bcm33843-viper
+      - brcm,bcm6328
+      - brcm,bcm6358
+      - brcm,bcm6362
+      - brcm,bcm6368
+      - brcm,bcm63168
+      - brcm,bcm63268
+      - brcm,bcm7125
+      - brcm,bcm7346
+      - brcm,bcm7358
+      - brcm,bcm7360
+      - brcm,bcm7362
+      - brcm,bcm7420
+      - brcm,bcm7425
+
+  cpus:
+    type: object
+    additionalProperties: false
+    properties:
+      '#address-cells':
+        const: 1
+
+      '#size-cells':
+        const: 0
+
+      mips-hpt-frequency:
+        description: MIPS counter high precision timer frequency.
+         This is common to all CPUs in the system so it lives
+         under the "cpus" node.
+        $ref: /schemas/types.yaml#/definitions/uint32
+
+    patternProperties:
+      "^cpu@[0-9]$":
+        type: object
+        $ref: /schemas/mips/cpus.yaml#
+        unevaluatedProperties: false
+
+    required:
+      - mips-hpt-frequency
+
+additionalProperties: true
+
+examples:
+  - |
+     / {
+         compatible = "brcm,bcm3368";
+         #address-cells = <1>;
+         #size-cells = <1>;
+         model = "Broadcom 3368";
+
+         cpus {
+           #address-cells = <1>;
+           #size-cells = <0>;
+
+           mips-hpt-frequency = <150000000>;
+
+           cpu@0 {
+             compatible = "brcm,bmips4350";
+             device_type = "cpu";
+             reg = <0>;
+           };
+
+           cpu@1 {
+             compatible = "brcm,bmips4350";
+             device_type = "cpu";
+             reg = <1>;
+           };
+         };
+       };
+...
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH v8 1/2] dt-bindings: mips: add CPU bindings for MIPS architecture
  2022-10-06  4:29 ` [PATCH v8 1/2] " Sergio Paracuellos
@ 2022-10-06 11:23   ` Philippe Mathieu-Daudé
  2022-10-17  5:53     ` Sergio Paracuellos
  0 siblings, 1 reply; 9+ messages in thread
From: Philippe Mathieu-Daudé @ 2022-10-06 11:23 UTC (permalink / raw)
  To: Sergio Paracuellos, devicetree
  Cc: robh+dt, krzysztof.kozlowski+dt, hauke, zajec5, tsbogend,
	zhouyanjie, linux-mips, arinc.unal, f.fainelli, Rob Herring

On 6/10/22 06:29, Sergio Paracuellos wrote:
> Add the yaml binding for available CPUs in MIPS architecture.
> 
> Reviewed-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> ---
>   .../bindings/mips/brcm/brcm,bmips.txt         |   8 --
>   .../devicetree/bindings/mips/cpus.yaml        | 115 ++++++++++++++++++
>   .../bindings/mips/ingenic/ingenic,cpu.yaml    |  69 -----------
>   3 files changed, 115 insertions(+), 77 deletions(-)
>   delete mode 100644 Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt
>   create mode 100644 Documentation/devicetree/bindings/mips/cpus.yaml
>   delete mode 100644 Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml

Reviewed-by: Philippe Mathieu-Daudé <philmd@fungible.com>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH v8 1/2] dt-bindings: mips: add CPU bindings for MIPS architecture
  2022-10-06 11:23   ` Philippe Mathieu-Daudé
@ 2022-10-17  5:53     ` Sergio Paracuellos
  2022-10-21  8:47       ` Thomas Bogendoerfer
  0 siblings, 1 reply; 9+ messages in thread
From: Sergio Paracuellos @ 2022-10-17  5:53 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé
  Cc: devicetree, robh+dt, krzysztof.kozlowski+dt, hauke, zajec5,
	tsbogend, zhouyanjie, linux-mips, arinc.unal, f.fainelli,
	Rob Herring

On Thu, Oct 6, 2022 at 1:24 PM Philippe Mathieu-Daudé
<philmd@fungible.com> wrote:
>
> On 6/10/22 06:29, Sergio Paracuellos wrote:
> > Add the yaml binding for available CPUs in MIPS architecture.
> >
> > Reviewed-by: Rob Herring <robh@kernel.org>
> > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > ---
> >   .../bindings/mips/brcm/brcm,bmips.txt         |   8 --
> >   .../devicetree/bindings/mips/cpus.yaml        | 115 ++++++++++++++++++
> >   .../bindings/mips/ingenic/ingenic,cpu.yaml    |  69 -----------
> >   3 files changed, 115 insertions(+), 77 deletions(-)
> >   delete mode 100644 Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt
> >   create mode 100644 Documentation/devicetree/bindings/mips/cpus.yaml
> >   delete mode 100644 Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml
>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@fungible.com>

Thanks!

Thomas, are this series going through your tree?

Thanks in advance for clarification.

Best regards,
    Sergio Paracuellos

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH v8 1/2] dt-bindings: mips: add CPU bindings for MIPS architecture
  2022-10-17  5:53     ` Sergio Paracuellos
@ 2022-10-21  8:47       ` Thomas Bogendoerfer
  2022-10-21  9:05         ` Sergio Paracuellos
  0 siblings, 1 reply; 9+ messages in thread
From: Thomas Bogendoerfer @ 2022-10-21  8:47 UTC (permalink / raw)
  To: Sergio Paracuellos
  Cc: Philippe Mathieu-Daudé,
	devicetree, robh+dt, krzysztof.kozlowski+dt, hauke, zajec5,
	zhouyanjie, linux-mips, arinc.unal, f.fainelli, Rob Herring

On Mon, Oct 17, 2022 at 07:53:00AM +0200, Sergio Paracuellos wrote:
> On Thu, Oct 6, 2022 at 1:24 PM Philippe Mathieu-Daudé
> <philmd@fungible.com> wrote:
> >
> > On 6/10/22 06:29, Sergio Paracuellos wrote:
> > > Add the yaml binding for available CPUs in MIPS architecture.
> > >
> > > Reviewed-by: Rob Herring <robh@kernel.org>
> > > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > > ---
> > >   .../bindings/mips/brcm/brcm,bmips.txt         |   8 --
> > >   .../devicetree/bindings/mips/cpus.yaml        | 115 ++++++++++++++++++
> > >   .../bindings/mips/ingenic/ingenic,cpu.yaml    |  69 -----------
> > >   3 files changed, 115 insertions(+), 77 deletions(-)
> > >   delete mode 100644 Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt
> > >   create mode 100644 Documentation/devicetree/bindings/mips/cpus.yaml
> > >   delete mode 100644 Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml
> >
> > Reviewed-by: Philippe Mathieu-Daudé <philmd@fungible.com>
> 
> Thanks!
> 
> Thomas, are this series going through your tree?

I'll take them.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH v8 1/2] dt-bindings: mips: add CPU bindings for MIPS architecture
  2022-10-21  8:47       ` Thomas Bogendoerfer
@ 2022-10-21  9:05         ` Sergio Paracuellos
  2022-11-11 16:33           ` Sergio Paracuellos
  0 siblings, 1 reply; 9+ messages in thread
From: Sergio Paracuellos @ 2022-10-21  9:05 UTC (permalink / raw)
  To: Thomas Bogendoerfer
  Cc: Philippe Mathieu-Daudé,
	devicetree, robh+dt, krzysztof.kozlowski+dt, hauke, zajec5,
	zhouyanjie, linux-mips, arinc.unal, f.fainelli, Rob Herring

On Fri, Oct 21, 2022 at 11:02 AM Thomas Bogendoerfer
<tsbogend@alpha.franken.de> wrote:
>
> On Mon, Oct 17, 2022 at 07:53:00AM +0200, Sergio Paracuellos wrote:
> > On Thu, Oct 6, 2022 at 1:24 PM Philippe Mathieu-Daudé
> > <philmd@fungible.com> wrote:
> > >
> > > On 6/10/22 06:29, Sergio Paracuellos wrote:
> > > > Add the yaml binding for available CPUs in MIPS architecture.
> > > >
> > > > Reviewed-by: Rob Herring <robh@kernel.org>
> > > > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > > > ---
> > > >   .../bindings/mips/brcm/brcm,bmips.txt         |   8 --
> > > >   .../devicetree/bindings/mips/cpus.yaml        | 115 ++++++++++++++++++
> > > >   .../bindings/mips/ingenic/ingenic,cpu.yaml    |  69 -----------
> > > >   3 files changed, 115 insertions(+), 77 deletions(-)
> > > >   delete mode 100644 Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt
> > > >   create mode 100644 Documentation/devicetree/bindings/mips/cpus.yaml
> > > >   delete mode 100644 Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml
> > >
> > > Reviewed-by: Philippe Mathieu-Daudé <philmd@fungible.com>
> >
> > Thanks!
> >
> > Thomas, are this series going through your tree?
>
> I'll take them.

Thanks for letting me know!!

>
> Thomas.
>

Best regards,
    Sergio Paracuellos


> --
> Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
> good idea.                                                [ RFC1925, 2.3 ]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH v8 1/2] dt-bindings: mips: add CPU bindings for MIPS architecture
  2022-10-21  9:05         ` Sergio Paracuellos
@ 2022-11-11 16:33           ` Sergio Paracuellos
  0 siblings, 0 replies; 9+ messages in thread
From: Sergio Paracuellos @ 2022-11-11 16:33 UTC (permalink / raw)
  To: Thomas Bogendoerfer
  Cc: Philippe Mathieu-Daudé,
	devicetree, robh+dt, krzysztof.kozlowski+dt, hauke, zajec5,
	zhouyanjie, linux-mips, arinc.unal, f.fainelli, Rob Herring

On Fri, Oct 21, 2022 at 11:05 AM Sergio Paracuellos
<sergio.paracuellos@gmail.com> wrote:
>
> On Fri, Oct 21, 2022 at 11:02 AM Thomas Bogendoerfer
> <tsbogend@alpha.franken.de> wrote:
> >
> > On Mon, Oct 17, 2022 at 07:53:00AM +0200, Sergio Paracuellos wrote:
> > > On Thu, Oct 6, 2022 at 1:24 PM Philippe Mathieu-Daudé
> > > <philmd@fungible.com> wrote:
> > > >
> > > > On 6/10/22 06:29, Sergio Paracuellos wrote:
> > > > > Add the yaml binding for available CPUs in MIPS architecture.
> > > > >
> > > > > Reviewed-by: Rob Herring <robh@kernel.org>
> > > > > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > > > > ---
> > > > >   .../bindings/mips/brcm/brcm,bmips.txt         |   8 --
> > > > >   .../devicetree/bindings/mips/cpus.yaml        | 115 ++++++++++++++++++
> > > > >   .../bindings/mips/ingenic/ingenic,cpu.yaml    |  69 -----------
> > > > >   3 files changed, 115 insertions(+), 77 deletions(-)
> > > > >   delete mode 100644 Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt
> > > > >   create mode 100644 Documentation/devicetree/bindings/mips/cpus.yaml
> > > > >   delete mode 100644 Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml
> > > >
> > > > Reviewed-by: Philippe Mathieu-Daudé <philmd@fungible.com>
> > >
> > > Thanks!
> > >
> > > Thomas, are this series going through your tree?
> >
> > I'll take them.
>
> Thanks for letting me know!!

It’s been a while and this series is still not added to the mips tree…
Gentle ping :)

Thanks!!

Best regards,
    Sergio Paracuellos

>
> >
> > Thomas.
> >
>
> Best regards,
>     Sergio Paracuellos
>
>
> > --
> > Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
> > good idea.                                                [ RFC1925, 2.3 ]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH v8 0/2] dt-bindings: mips: add CPU bindings for MIPS architecture
  2022-10-06  4:29 [PATCH v8 0/2] dt-bindings: mips: add CPU bindings for MIPS architecture Sergio Paracuellos
  2022-10-06  4:29 ` [PATCH v8 1/2] " Sergio Paracuellos
  2022-10-06  4:29 ` [PATCH v8 2/2] dt-bindings: mips: brcm: add Broadcom SoCs bindings Sergio Paracuellos
@ 2022-11-21 17:06 ` Thomas Bogendoerfer
  2 siblings, 0 replies; 9+ messages in thread
From: Thomas Bogendoerfer @ 2022-11-21 17:06 UTC (permalink / raw)
  To: Sergio Paracuellos
  Cc: devicetree, robh+dt, krzysztof.kozlowski+dt, hauke, zajec5,
	zhouyanjie, linux-mips, arinc.unal, f.fainelli

On Thu, Oct 06, 2022 at 06:29:43AM +0200, Sergio Paracuellos wrote:
> Hi all,
> 
> This series tries to make the correct thing to represent in schema
> all the current documentation related with MIPS CPUs.
> 
> Broadcom cpus node is a bit special and need to use the property
> 'mips-hpt-frequency' in the cpus node. Because of this I have introduced
> brcm/soc.yaml schema with current broadcom SoCs and making this property
> required as per Rob's v3 review comments.
> 
> Ingenic cpus have also its schema already mainlined. To unify things I
> have also put this information in this mips/cpus.yaml schema and remove
> the 'mips/ingenic/ingenic,cpu.yaml' schema. I have also added current
> maintainer as maintainer for this 'mips/cpus.yaml' file.
> 
> Thomas, are these patches going through the mips git tree?
> 
> Thanks,
>     Sergio Paracuellos
> 
> Changes in v8:
>   - Remove Hauke Mehrtens from brcm/soc.yaml maintainers as per Hauke Mehrtens's
>     requested in v7 response.
> 
> Changes in v7:
>   - Remove Rafał Miłecki from brcm/soc.yaml maintainers as per Rafał Miłecki's
>     requested in v6 response.
> 
> Changes in v6:
>   - Avoid 'convert' in SoC's patch commit message since this is just an
>     addition and nothing is removed in this patch.
>   - Add Florian Fainelli to maintainers of Broadcom SoC bindings
>   - Add Florian's Acked-by for broadcom SoC PATCH.
>   - Collect Rob's Reviewed-by for both patches.
> 
> Changes in v5:
>   - Address Krzysztof comments in v4:
>       - change BMIPS SoC compatibles into an enum and drop descriptions,
>       - Add blank line.
>       - Redo commit message since it is not a conversion to schema.
>       - Drop 'device tree bindings' from description.
>       - Properly describe 'mips-hpt-frequency'.
>   - Review cpus node and add a sample to check schema correctness.    
> 
> Changes in v4:
> - Address Rob's v3 review comments:
>     * Drop PATCH introducing special brcm,cpus.yaml only because properyu
>     'mips-hpt-frequency' is required and move this property to brcm/soc.yaml
>     schema.
>     * Remove 'Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml'
>     and add that information to general mips/cpus.yaml schema.
>     * Define all properties in mips/cpus.yaml and set 'additionalProperties'
>     to false.
> 
> Changes in v3:
> - cpus.yaml:
>     * address Krzysztof comment in v2:
>     * add Thomas as maintainer since this is architecture binding.
>     * s/cpu/CPU
>     * compatible goes first
>     * Add compatible 'mips,mips4Kc' as per Thomas request.
> - Introduce two new schemas for Broadcom BMIPS:
>     * soc.yaml: describing the BRCM SoC's
>     * brcm,bmips-cpus.yaml: describing the cpu nodes for this platform.
> 
> Previous series:
> v1: https://lore.kernel.org/all/CAMhs-H-eUTOHjAXAbywOXQJgc_j5Ex-1sB7eBZU_bWt1fpNVzA@mail.gmail.com/T/
> v2: https://lore.kernel.org/all/20220918112245.GA5555@alpha.franken.de/T/
> v3: https://lore.kernel.org/linux-devicetree/20220929072004.874795-1-sergio.paracuellos@gmail.com/T/#t
> v4: https://lore.kernel.org/linux-devicetree/20221001043855.933528-1-sergio.paracuellos@gmail.com/T/#t
> v5: https://lore.kernel.org/linux-devicetree/20221002091611.946210-1-sergio.paracuellos@gmail.com/T/#t
> v6: https://lore.kernel.org/linux-devicetree/20221004050924.986211-1-sergio.paracuellos@gmail.com/T/#t
> 
> Sergio Paracuellos (2):
>   dt-bindings: mips: add CPU bindings for MIPS architecture
>   dt-bindings: mips: brcm: add Broadcom SoCs bindings
> 
>  .../bindings/mips/brcm/brcm,bmips.txt         |   8 --
>  .../devicetree/bindings/mips/brcm/soc.yaml    |  97 +++++++++++++++
>  .../devicetree/bindings/mips/cpus.yaml        | 115 ++++++++++++++++++
>  .../bindings/mips/ingenic/ingenic,cpu.yaml    |  69 -----------
>  4 files changed, 212 insertions(+), 77 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/mips/brcm/brcm,bmips.txt
>  create mode 100644 Documentation/devicetree/bindings/mips/brcm/soc.yaml
>  create mode 100644 Documentation/devicetree/bindings/mips/cpus.yaml
>  delete mode 100644 Documentation/devicetree/bindings/mips/ingenic/ingenic,cpu.yaml
> 
> -- 
> 2.25.1

series applied to mips-next.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2022-11-21 17:06 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-06  4:29 [PATCH v8 0/2] dt-bindings: mips: add CPU bindings for MIPS architecture Sergio Paracuellos
2022-10-06  4:29 ` [PATCH v8 1/2] " Sergio Paracuellos
2022-10-06 11:23   ` Philippe Mathieu-Daudé
2022-10-17  5:53     ` Sergio Paracuellos
2022-10-21  8:47       ` Thomas Bogendoerfer
2022-10-21  9:05         ` Sergio Paracuellos
2022-11-11 16:33           ` Sergio Paracuellos
2022-10-06  4:29 ` [PATCH v8 2/2] dt-bindings: mips: brcm: add Broadcom SoCs bindings Sergio Paracuellos
2022-11-21 17:06 ` [PATCH v8 0/2] dt-bindings: mips: add CPU bindings for MIPS architecture Thomas Bogendoerfer

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.