lib: Simplify sbi_platform ipi_init() hooks

Instead of having separate ipi_init() hooks for cold and warm boot,
this patch updates struct sbi_platform to have just one ipi_init()
hook. The type of boot (cold or warm) is now a boolean flag parameter
for the updated ipi_init() hook.

Signed-off-by: Anup Patel <anup.patel@wdc.com>
This commit is contained in:
Anup Patel
2018-12-26 18:36:11 +05:30
committed by Anup Patel
parent e34aa8a671
commit c1b6200653
8 changed files with 60 additions and 48 deletions

View File

@@ -26,8 +26,6 @@ void sbi_ipi_clear_smode(struct sbi_scratch *scratch, u32 hartid);
void sbi_ipi_process(struct sbi_scratch *scratch, u32 hartid);
int sbi_ipi_warm_init(struct sbi_scratch *scratch, u32 hartid);
int sbi_ipi_cold_init(struct sbi_scratch *scratch);
int sbi_ipi_init(struct sbi_scratch *scratch, u32 hartid, bool cold_boot);
#endif

View File

@@ -46,8 +46,7 @@ struct sbi_platform {
void (*ipi_inject)(u32 target_hart, u32 source_hart);
void (*ipi_sync)(u32 target_hart, u32 source_hart);
void (*ipi_clear)(u32 target_hart);
int (*cold_ipi_init)(void);
int (*warm_ipi_init)(u32 target_hart);
int (*ipi_init)(u32 hartid, bool cold_boot);
u64 (*timer_value)(void);
void (*timer_event_stop)(u32 target_hart);
void (*timer_event_start)(u32 target_hart, u64 next_event);
@@ -187,18 +186,11 @@ static inline void sbi_platform_ipi_clear(struct sbi_platform *plat,
plat->ipi_clear(target_hart);
}
static inline int sbi_platform_warm_ipi_init(struct sbi_platform *plat,
u32 target_hart)
static inline int sbi_platform_ipi_init(struct sbi_platform *plat,
u32 hartid, bool cold_boot)
{
if (plat && plat->warm_ipi_init)
return plat->warm_ipi_init(target_hart);
return 0;
}
static inline int sbi_platform_cold_ipi_init(struct sbi_platform *plat)
{
if (plat && plat->cold_ipi_init)
return plat->cold_ipi_init();
if (plat && plat->ipi_init)
return plat->ipi_init(hartid, cold_boot);
return 0;
}

View File

@@ -50,11 +50,7 @@ static void __attribute__((noreturn)) init_coldboot(struct sbi_scratch *scratch,
if (rc)
sbi_hart_hang();
rc = sbi_ipi_cold_init(scratch);
if (rc)
sbi_hart_hang();
rc = sbi_ipi_warm_init(scratch, hartid);
rc = sbi_ipi_init(scratch, hartid, TRUE);
if (rc)
sbi_hart_hang();
@@ -125,7 +121,7 @@ static void __attribute__((noreturn)) init_warmboot(struct sbi_scratch *scratch,
if (rc)
sbi_hart_hang();
rc = sbi_ipi_warm_init(scratch, hartid);
rc = sbi_ipi_init(scratch, hartid, FALSE);
if (rc)
sbi_hart_hang();

View File

@@ -68,15 +68,11 @@ void sbi_ipi_process(struct sbi_scratch *scratch, u32 hartid)
scratch->ipi_type = 0;
}
int sbi_ipi_warm_init(struct sbi_scratch *scratch, u32 hartid)
int sbi_ipi_init(struct sbi_scratch *scratch, u32 hartid, bool cold_boot)
{
/* Enable software interrupts */
csr_set(mie, MIP_MSIP);
return sbi_platform_warm_ipi_init(sbi_platform_ptr(scratch), hartid);
}
int sbi_ipi_cold_init(struct sbi_scratch *scratch)
{
return sbi_platform_cold_ipi_init(sbi_platform_ptr(scratch));
return sbi_platform_ipi_init(sbi_platform_ptr(scratch),
hartid, cold_boot);
}

View File

@@ -52,9 +52,18 @@ static int k210_irqchip_init(u32 hartid, bool cold_boot)
(2 * hartid + 1));
}
static int k210_cold_ipi_init(void)
static int k210_ipi_init(u32 hartid, bool cold_boot)
{
return clint_cold_ipi_init(CLINT_BASE_ADDR, K210_HART_COUNT);
int rc;
if (cold_boot) {
rc = clint_cold_ipi_init(CLINT_BASE_ADDR,
K210_HART_COUNT);
if (rc)
return rc;
}
return clint_warm_ipi_init(hartid);
}
static int k210_cold_timer_init(void)
@@ -93,8 +102,7 @@ struct sbi_platform platform = {
.irqchip_init = k210_irqchip_init,
.cold_ipi_init = k210_cold_ipi_init,
.warm_ipi_init = clint_warm_ipi_init,
.ipi_init = k210_ipi_init,
.ipi_inject = clint_ipi_inject,
.ipi_sync = clint_ipi_sync,
.ipi_clear = clint_ipi_clear,

View File

@@ -91,10 +91,18 @@ static int sifive_u_irqchip_init(u32 hartid, bool cold_boot)
(2 * hartid + 1));
}
static int sifive_u_cold_ipi_init(void)
static int sifive_u_ipi_init(u32 hartid, bool cold_boot)
{
return clint_cold_ipi_init(SIFIVE_U_CLINT_ADDR,
int rc;
if (cold_boot) {
rc = clint_cold_ipi_init(SIFIVE_U_CLINT_ADDR,
SIFIVE_U_HART_COUNT);
if (rc)
return rc;
}
return clint_warm_ipi_init(hartid);
}
static int sifive_u_cold_timer_init(void)
@@ -125,8 +133,7 @@ struct sbi_platform platform = {
.ipi_inject = clint_ipi_inject,
.ipi_sync = clint_ipi_sync,
.ipi_clear = clint_ipi_clear,
.warm_ipi_init = clint_warm_ipi_init,
.cold_ipi_init = sifive_u_cold_ipi_init,
.ipi_init = sifive_u_ipi_init,
.timer_value = clint_timer_value,
.timer_event_stop = clint_timer_event_stop,
.timer_event_start = clint_timer_event_start,

View File

@@ -92,10 +92,18 @@ static int virt_irqchip_init(u32 hartid, bool cold_boot)
(2 * hartid + 1));
}
static int virt_cold_ipi_init(void)
static int virt_ipi_init(u32 hartid, bool cold_boot)
{
return clint_cold_ipi_init(VIRT_CLINT_ADDR,
int rc;
if (cold_boot) {
rc = clint_cold_ipi_init(VIRT_CLINT_ADDR,
VIRT_HART_COUNT);
if (rc)
return rc;
}
return clint_warm_ipi_init(hartid);
}
static int virt_cold_timer_init(void)
@@ -126,8 +134,7 @@ struct sbi_platform platform = {
.ipi_inject = clint_ipi_inject,
.ipi_sync = clint_ipi_sync,
.ipi_clear = clint_ipi_clear,
.warm_ipi_init = clint_warm_ipi_init,
.cold_ipi_init = virt_cold_ipi_init,
.ipi_init = virt_ipi_init,
.timer_value = clint_timer_value,
.timer_event_stop = clint_timer_event_stop,
.timer_event_start = clint_timer_event_start,

View File

@@ -111,10 +111,19 @@ static int sifive_u_irqchip_init(u32 hartid, bool cold_boot)
(hartid) ? (2 * hartid) : -1);
}
static int sifive_u_cold_ipi_init(void)
static int sifive_u_ipi_init(u32 hartid, bool cold_boot)
{
return clint_cold_ipi_init(SIFIVE_U_CLINT_ADDR,
int rc;
if (cold_boot) {
rc = clint_cold_ipi_init(SIFIVE_U_CLINT_ADDR,
SIFIVE_U_HART_COUNT);
if (rc)
return rc;
}
return clint_warm_ipi_init(hartid);
}
static int sifive_u_cold_timer_init(void)
@@ -145,8 +154,7 @@ struct sbi_platform platform = {
.ipi_inject = clint_ipi_inject,
.ipi_sync = clint_ipi_sync,
.ipi_clear = clint_ipi_clear,
.warm_ipi_init = clint_warm_ipi_init,
.cold_ipi_init = sifive_u_cold_ipi_init,
.ipi_init = sifive_u_ipi_init,
.timer_value = clint_timer_value,
.timer_event_stop = clint_timer_event_stop,
.timer_event_start = clint_timer_event_start,