lib: utils: Rename the prefix in PMU DT properties
authorAtish Patra <atish.patra@wdc.com>
Mon, 8 Nov 2021 18:53:06 +0000 (10:53 -0800)
committerAnup Patel <anup@brainfault.org>
Thu, 11 Nov 2021 12:24:46 +0000 (17:54 +0530)
As per the DT schema rules, the prefix should be vendor. As the PMU
properties are generic for all vendors, change the prefix to riscv
instead of pmu.

Reviewed-by: Anup Patel <anup.patel@wdc.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Signed-off-by: Atish Patra <atish.patra@wdc.com>
docs/pmu_support.md
lib/utils/fdt/fdt_pmu.c

index 6533c99..a60b75c 100644 (file)
@@ -31,7 +31,7 @@ bindings in details.
 * **compatible** (Mandatory) - The compatible string of SBI PMU device tree node.
 This DT property must have the value **riscv,pmu**.
 
-* **pmu,event-to-mhpmevent**(Optional) - It represents an ONE-to-ONE mapping
+* **riscv,event-to-mhpmevent**(Optional) - It represents an ONE-to-ONE mapping
 between a PMU event and the event selector value that platform expects to be
 written to the MHPMEVENTx CSR for that event. The mapping is encoded in a
 table format where each row represents an event. The first column represent the
@@ -39,7 +39,7 @@ event idx where the 2nd & 3rd column represent the event selector value that
 should be encoded in the expected value to be written in MHPMEVENTx.
 This property shouldn't encode any raw hardware event.
 
-* **pmu,event-to-mhpmcounters**(Optional) - It represents a MANY-to-MANY
+* **riscv,event-to-mhpmcounters**(Optional) - It represents a MANY-to-MANY
 mapping between a range of events and all the MHPMCOUNTERx in a bitmap format
 that can be used to monitor these range of events. The information is encoded in
 a table format where each row represent a certain range of events and
@@ -49,7 +49,7 @@ represent a bitmap of all the MHPMCOUNTERx. This property is mandatory if
 event-to-mhpmevent is present. Otherwise, it can be omitted. This property
 shouldn't encode any raw event.
 
-* **pmu,raw-event-to-mhpmcounters**(Optional) - It represents an ONE-to-MANY
+* **riscv,raw-event-to-mhpmcounters**(Optional) - It represents an ONE-to-MANY
 mapping between a raw event and all the MHPMCOUNTERx in a bitmap format that can
 be used to monitor that raw event. The information is encoded in a table format
 where each raw represent a specific raw event. The first column stores the
@@ -67,12 +67,12 @@ pmu {
        compatible                      = "riscv,pmu";
        interrupts                      = <0x100>;
        interrupt-parent                        = <&plic>
-       pmu,event-to-mhpmevent          = <0x0000B  0x0000 0x0001>,
-       pmu,event-to-mhpmcounters       = <0x00001 0x00001 0x00000001>,
+       riscv,event-to-mhpmevent                = <0x0000B  0x0000 0x0001>,
+       riscv,event-to-mhpmcounters     = <0x00001 0x00001 0x00000001>,
                                                  <0x00002 0x00002 0x00000004>,
                                                  <0x00003 0x0000A 0x00000ff8>,
                                                  <0x10000 0x10033 0x000ff000>,
-       pmu,raw-event-to-mhpmcounters   = <0x0000 0x0002 0x00000f8>,
+       riscv,raw-event-to-mhpmcounters         = <0x0000 0x0002 0x00000f8>,
                                          <0x0000 0x0003 0x00000ff0>,
 };
 
index 09c83c8..529ee42 100644 (file)
@@ -50,9 +50,9 @@ int fdt_pmu_fixup(void *fdt)
        if (pmu_offset < 0)
                return SBI_EFAIL;
 
-       fdt_delprop(fdt, pmu_offset, "pmu,event-to-mhpmcounters");
-       fdt_delprop(fdt, pmu_offset, "pmu,event-to-mhpmevent");
-       fdt_delprop(fdt, pmu_offset, "pmu,raw-event-to-mhpmcounters");
+       fdt_delprop(fdt, pmu_offset, "riscv,event-to-mhpmcounters");
+       fdt_delprop(fdt, pmu_offset, "riscv,event-to-mhpmevent");
+       fdt_delprop(fdt, pmu_offset, "riscv,raw-event-to-mhpmcounters");
        if (!sbi_hart_has_feature(scratch, SBI_HART_HAS_SSCOFPMF))
                fdt_delprop(fdt, pmu_offset, "interrupts-extended");
 
@@ -75,7 +75,7 @@ int fdt_pmu_setup(void *fdt)
        if (pmu_offset < 0)
                return SBI_EFAIL;
 
-       event_ctr_map = fdt_getprop(fdt, pmu_offset, "pmu,event-to-mhpmcounters", &len);
+       event_ctr_map = fdt_getprop(fdt, pmu_offset, "riscv,event-to-mhpmcounters", &len);
        if (!event_ctr_map || len < 8)
                return SBI_EFAIL;
        len = len / (sizeof(u32) * 3);
@@ -86,7 +86,7 @@ int fdt_pmu_setup(void *fdt)
                sbi_pmu_add_hw_event_counter_map(event_idx_start, event_idx_end, ctr_map);
        }
 
-       event_val = fdt_getprop(fdt, pmu_offset, "pmu,event-to-mhpmevent", &len);
+       event_val = fdt_getprop(fdt, pmu_offset, "riscv,event-to-mhpmevent", &len);
        if (!event_val || len < 8)
                return SBI_EFAIL;
        len = len / (sizeof(u32) * 3);
@@ -98,7 +98,7 @@ int fdt_pmu_setup(void *fdt)
                hw_event_count++;
        }
 
-       event_val = fdt_getprop(fdt, pmu_offset, "pmu,raw-event-to-mhpmcounters", &len);
+       event_val = fdt_getprop(fdt, pmu_offset, "riscv,raw-event-to-mhpmcounters", &len);
        if (!event_val || len < 8)
                return SBI_EFAIL;
        len = len / (sizeof(u32) * 3);