lib: sbi_hsm: Rename 'priv' argument to 'arg1'
authorBin Meng <bmeng@tinylab.org>
Fri, 30 Dec 2022 05:07:23 +0000 (13:07 +0800)
committerAnup Patel <anup@brainfault.org>
Mon, 9 Jan 2023 11:22:34 +0000 (16:52 +0530)
'priv' argument of sbi_hsm_hart_start() and sbi_hsm_hart_suspend()
may mislead people to think it stands for 'privilege mode', but it
is not. Change it to 'arg1' to clearly indicate the a1 register.

Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Samuel Holland <samuel@sholland.org>
Tested-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Atish Patra <atishp@rivosinc.com>
Reviewed-by: Anup Patel <anup@brainfault.org>
include/sbi/sbi_hsm.h
lib/sbi/sbi_hsm.c

index d6cc468..c2a728a 100644 (file)
@@ -59,12 +59,12 @@ void __noreturn sbi_hsm_exit(struct sbi_scratch *scratch);
 
 int sbi_hsm_hart_start(struct sbi_scratch *scratch,
                       const struct sbi_domain *dom,
-                      u32 hartid, ulong saddr, ulong smode, ulong priv);
+                      u32 hartid, ulong saddr, ulong smode, ulong arg1);
 int sbi_hsm_hart_stop(struct sbi_scratch *scratch, bool exitnow);
 void sbi_hsm_hart_resume_start(struct sbi_scratch *scratch);
 void sbi_hsm_hart_resume_finish(struct sbi_scratch *scratch);
 int sbi_hsm_hart_suspend(struct sbi_scratch *scratch, u32 suspend_type,
-                        ulong raddr, ulong rmode, ulong priv);
+                        ulong raddr, ulong rmode, ulong arg1);
 int sbi_hsm_hart_get_state(const struct sbi_domain *dom, u32 hartid);
 int sbi_hsm_hart_interruptible_mask(const struct sbi_domain *dom,
                                    ulong hbase, ulong *out_hmask);
index 836008f..c0a5505 100644 (file)
@@ -248,7 +248,7 @@ fail_exit:
 
 int sbi_hsm_hart_start(struct sbi_scratch *scratch,
                       const struct sbi_domain *dom,
-                      u32 hartid, ulong saddr, ulong smode, ulong priv)
+                      u32 hartid, ulong saddr, ulong smode, ulong arg1)
 {
        unsigned long init_count;
        unsigned int hstate;
@@ -281,7 +281,7 @@ int sbi_hsm_hart_start(struct sbi_scratch *scratch,
                return SBI_EINVAL;
 
        init_count = sbi_init_count(hartid);
-       rscratch->next_arg1 = priv;
+       rscratch->next_arg1 = arg1;
        rscratch->next_addr = saddr;
        rscratch->next_mode = smode;
 
@@ -402,7 +402,7 @@ void sbi_hsm_hart_resume_finish(struct sbi_scratch *scratch)
 }
 
 int sbi_hsm_hart_suspend(struct sbi_scratch *scratch, u32 suspend_type,
-                        ulong raddr, ulong rmode, ulong priv)
+                        ulong raddr, ulong rmode, ulong arg1)
 {
        int oldstate, ret;
        const struct sbi_domain *dom = sbi_domain_thishart_ptr();
@@ -433,7 +433,7 @@ int sbi_hsm_hart_suspend(struct sbi_scratch *scratch, u32 suspend_type,
        }
 
        /* Save the resume address and resume mode */
-       scratch->next_arg1 = priv;
+       scratch->next_arg1 = arg1;
        scratch->next_addr = raddr;
        scratch->next_mode = rmode;