platform: andes: Drop andes_pmu_setup()

andes_pmu_setup() [1] was intended to populate event mapping from
hardcoded arrays, however, this increases firmware size and we should
just use PMU DT node [2] instead.

Link: https://lists.infradead.org/pipermail/opensbi/2023-November/006032.html
Link: https://github.com/riscv-software-src/opensbi/blob/v1.4/docs/pmu_support.md#example-3
Signed-off-by: Yu Chien Peter Lin <peterlin@andestech.com>
Reviewed-by: Anup Patel <anup@brainfault.org>

diff --git a/platform/generic/andes/andes_pmu.c b/platform/generic/andes/andes_pmu.c
index 4b2d45b062e5ba8021781f1d6776c0a46918dece..479fb9a6cd2becbdbaa988097b7c8988ecfb7a77 100644 (file)
--- a/platform/generic/andes/andes_pmu.c
+++ b/platform/generic/andes/andes_pmu.c
@@ -6,7 +6,6 @@
  */
 
 #include <andes/andes45.h>
-#include <andes/andes_hpm.h>
 #include <andes/andes_pmu.h>
 #include <sbi/sbi_bitops.h>
 #include <sbi/sbi_error.h>
@@ -86,20 +85,9 @@ int andes_pmu_extensions_init(const struct fdt_match *match,
 int andes_pmu_init(const struct fdt_match *match)
 {
        struct sbi_scratch *scratch = sbi_scratch_thishart_ptr();
-       void *fdt = fdt_get_address();
-       int pmu_offset;
 
        if (sbi_hart_has_extension(scratch, SBI_HART_EXT_XANDESPMU))
                sbi_pmu_set_device(&andes_pmu);
 
-       /*
-        * Populate default mappings if device-tree doesn't
-        * provide a valid pmu node.
-        */
-       pmu_offset = fdt_node_offset_by_compatible(fdt, -1, "riscv,pmu");
-       if (pmu_offset < 0)
-               return (pmu_offset == -FDT_ERR_NOTFOUND) ? andes_pmu_setup()
-                                                        : SBI_EFAIL;
-
        return 0;
 }

diff --git a/platform/generic/include/andes/andes_hpm.h b/platform/generic/include/andes/andes_hpm.h
deleted file mode 100644 (file)
index b4d71b9..0000000
--- a/platform/generic/include/andes/andes_hpm.h
+++ /dev/null
@@ -1,12 +0,0 @@
-/*
- * SPDX-License-Identifier: BSD-2-Clause
- *
- * Copyright (c) 2023 Andes Technology Corporation
- */
-
-#ifndef _ANDES_HPM_H_
-#define _ANDES_HPM_H_
-
-static inline int andes_pmu_setup(void) { return 0; }
-
-#endif /* _ANDES_HPM_H_ */