mirror of https://gitee.com/openkylin/linux.git
suspend: constify platform_suspend_ops
While at it, fix two checkpatch errors.
Several non-const struct instances constified by this patch were added after
the introduction of platform_suspend_ops in checkpatch.pl's list of "should
be const" structs (79404849e9
).
Patch against mainline.
Inspired by hunks of the grsecurity patch, updated for newer kernels.
Signed-off-by: Lionel Debroux <lionel_debroux@yahoo.fr>
Acked-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
This commit is contained in:
parent
acc2472ed3
commit
2f55ac072f
|
@ -301,7 +301,7 @@ static void at91_pm_end(void)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
static struct platform_suspend_ops at91_pm_ops ={
|
static const struct platform_suspend_ops at91_pm_ops = {
|
||||||
.valid = at91_pm_valid_state,
|
.valid = at91_pm_valid_state,
|
||||||
.begin = at91_pm_begin,
|
.begin = at91_pm_begin,
|
||||||
.enter = at91_pm_enter,
|
.enter = at91_pm_enter,
|
||||||
|
|
|
@ -110,7 +110,7 @@ static int davinci_pm_enter(suspend_state_t state)
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops davinci_pm_ops = {
|
static const struct platform_suspend_ops davinci_pm_ops = {
|
||||||
.enter = davinci_pm_enter,
|
.enter = davinci_pm_enter,
|
||||||
.valid = suspend_valid_only_mem,
|
.valid = suspend_valid_only_mem,
|
||||||
};
|
};
|
||||||
|
|
|
@ -32,7 +32,7 @@ static int mx27_suspend_enter(suspend_state_t state)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops mx27_suspend_ops = {
|
static const struct platform_suspend_ops mx27_suspend_ops = {
|
||||||
.enter = mx27_suspend_enter,
|
.enter = mx27_suspend_enter,
|
||||||
.valid = suspend_valid_only_mem,
|
.valid = suspend_valid_only_mem,
|
||||||
};
|
};
|
||||||
|
|
|
@ -123,7 +123,7 @@ static int lpc32xx_pm_enter(suspend_state_t state)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops lpc32xx_pm_ops = {
|
static const struct platform_suspend_ops lpc32xx_pm_ops = {
|
||||||
.valid = suspend_valid_only_mem,
|
.valid = suspend_valid_only_mem,
|
||||||
.enter = lpc32xx_pm_enter,
|
.enter = lpc32xx_pm_enter,
|
||||||
};
|
};
|
||||||
|
|
|
@ -647,7 +647,7 @@ static struct irqaction omap_wakeup_irq = {
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
static struct platform_suspend_ops omap_pm_ops ={
|
static const struct platform_suspend_ops omap_pm_ops = {
|
||||||
.prepare = omap_pm_prepare,
|
.prepare = omap_pm_prepare,
|
||||||
.enter = omap_pm_enter,
|
.enter = omap_pm_enter,
|
||||||
.finish = omap_pm_finish,
|
.finish = omap_pm_finish,
|
||||||
|
|
|
@ -326,7 +326,7 @@ static void omap2_pm_finish(void)
|
||||||
enable_hlt();
|
enable_hlt();
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops omap_pm_ops = {
|
static const struct platform_suspend_ops omap_pm_ops = {
|
||||||
.prepare = omap2_pm_prepare,
|
.prepare = omap2_pm_prepare,
|
||||||
.enter = omap2_pm_enter,
|
.enter = omap2_pm_enter,
|
||||||
.finish = omap2_pm_finish,
|
.finish = omap2_pm_finish,
|
||||||
|
|
|
@ -594,7 +594,7 @@ static void omap3_pm_end(void)
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops omap_pm_ops = {
|
static const struct platform_suspend_ops omap_pm_ops = {
|
||||||
.begin = omap3_pm_begin,
|
.begin = omap3_pm_begin,
|
||||||
.end = omap3_pm_end,
|
.end = omap3_pm_end,
|
||||||
.prepare = omap3_pm_prepare,
|
.prepare = omap3_pm_prepare,
|
||||||
|
|
|
@ -75,7 +75,7 @@ static void omap4_pm_end(void)
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops omap_pm_ops = {
|
static const struct platform_suspend_ops omap_pm_ops = {
|
||||||
.begin = omap4_pm_begin,
|
.begin = omap4_pm_begin,
|
||||||
.end = omap4_pm_end,
|
.end = omap4_pm_end,
|
||||||
.prepare = omap4_pm_prepare,
|
.prepare = omap4_pm_prepare,
|
||||||
|
|
|
@ -119,7 +119,7 @@ static int pnx4008_pm_valid(suspend_state_t state)
|
||||||
(state == PM_SUSPEND_MEM);
|
(state == PM_SUSPEND_MEM);
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops pnx4008_pm_ops = {
|
static const struct platform_suspend_ops pnx4008_pm_ops = {
|
||||||
.enter = pnx4008_pm_enter,
|
.enter = pnx4008_pm_enter,
|
||||||
.valid = pnx4008_pm_valid,
|
.valid = pnx4008_pm_valid,
|
||||||
};
|
};
|
||||||
|
|
|
@ -96,7 +96,7 @@ void pxa_pm_finish(void)
|
||||||
pxa_cpu_pm_fns->finish();
|
pxa_cpu_pm_fns->finish();
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops pxa_pm_ops = {
|
static const struct platform_suspend_ops pxa_pm_ops = {
|
||||||
.valid = pxa_pm_valid,
|
.valid = pxa_pm_valid,
|
||||||
.enter = pxa_pm_enter,
|
.enter = pxa_pm_enter,
|
||||||
.prepare = pxa_pm_prepare,
|
.prepare = pxa_pm_prepare,
|
||||||
|
|
|
@ -868,7 +868,7 @@ static void sharpsl_apm_get_power_status(struct apm_power_info *info)
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef CONFIG_PM
|
#ifdef CONFIG_PM
|
||||||
static struct platform_suspend_ops sharpsl_pm_ops = {
|
static const struct platform_suspend_ops sharpsl_pm_ops = {
|
||||||
.prepare = pxa_pm_prepare,
|
.prepare = pxa_pm_prepare,
|
||||||
.finish = pxa_pm_finish,
|
.finish = pxa_pm_finish,
|
||||||
.enter = corgi_pxa_pm_enter,
|
.enter = corgi_pxa_pm_enter,
|
||||||
|
|
|
@ -120,7 +120,7 @@ unsigned long sleep_phys_sp(void *sp)
|
||||||
return virt_to_phys(sp);
|
return virt_to_phys(sp);
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops sa11x0_pm_ops = {
|
static const struct platform_suspend_ops sa11x0_pm_ops = {
|
||||||
.enter = sa11x0_pm_enter,
|
.enter = sa11x0_pm_enter,
|
||||||
.valid = suspend_valid_only_mem,
|
.valid = suspend_valid_only_mem,
|
||||||
};
|
};
|
||||||
|
|
|
@ -355,7 +355,7 @@ static void s3c_pm_finish(void)
|
||||||
s3c_pm_check_cleanup();
|
s3c_pm_check_cleanup();
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops s3c_pm_ops = {
|
static const struct platform_suspend_ops s3c_pm_ops = {
|
||||||
.enter = s3c_pm_enter,
|
.enter = s3c_pm_enter,
|
||||||
.prepare = s3c_pm_prepare,
|
.prepare = s3c_pm_prepare,
|
||||||
.finish = s3c_pm_finish,
|
.finish = s3c_pm_finish,
|
||||||
|
|
|
@ -176,7 +176,7 @@ static int avr32_pm_enter(suspend_state_t state)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops avr32_pm_ops = {
|
static const struct platform_suspend_ops avr32_pm_ops = {
|
||||||
.valid = avr32_pm_valid_state,
|
.valid = avr32_pm_valid_state,
|
||||||
.enter = avr32_pm_enter,
|
.enter = avr32_pm_enter,
|
||||||
};
|
};
|
||||||
|
|
|
@ -233,7 +233,7 @@ static int bfin_pm_enter(suspend_state_t state)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
struct platform_suspend_ops bfin_pm_ops = {
|
static const struct platform_suspend_ops bfin_pm_ops = {
|
||||||
.enter = bfin_pm_enter,
|
.enter = bfin_pm_enter,
|
||||||
.valid = bfin_pm_valid,
|
.valid = bfin_pm_valid,
|
||||||
};
|
};
|
||||||
|
|
|
@ -110,7 +110,7 @@ static void db1x_pm_end(void)
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops db1x_pm_ops = {
|
static const struct platform_suspend_ops db1x_pm_ops = {
|
||||||
.valid = suspend_valid_only_mem,
|
.valid = suspend_valid_only_mem,
|
||||||
.begin = db1x_pm_begin,
|
.begin = db1x_pm_begin,
|
||||||
.enter = db1x_pm_enter,
|
.enter = db1x_pm_enter,
|
||||||
|
|
|
@ -42,7 +42,7 @@ static int jz4740_pm_enter(suspend_state_t state)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops jz4740_pm_ops = {
|
static const struct platform_suspend_ops jz4740_pm_ops = {
|
||||||
.valid = suspend_valid_only_mem,
|
.valid = suspend_valid_only_mem,
|
||||||
.enter = jz4740_pm_enter,
|
.enter = jz4740_pm_enter,
|
||||||
};
|
};
|
||||||
|
|
|
@ -147,7 +147,7 @@ static int loongson_pm_valid_state(suspend_state_t state)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops loongson_pm_ops = {
|
static const struct platform_suspend_ops loongson_pm_ops = {
|
||||||
.valid = loongson_pm_valid_state,
|
.valid = loongson_pm_valid_state,
|
||||||
.enter = loongson_pm_enter,
|
.enter = loongson_pm_enter,
|
||||||
};
|
};
|
||||||
|
|
|
@ -232,7 +232,7 @@ static void lite5200_pm_end(void)
|
||||||
lite5200_pm_target_state = PM_SUSPEND_ON;
|
lite5200_pm_target_state = PM_SUSPEND_ON;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops lite5200_pm_ops = {
|
static const struct platform_suspend_ops lite5200_pm_ops = {
|
||||||
.valid = lite5200_pm_valid,
|
.valid = lite5200_pm_valid,
|
||||||
.begin = lite5200_pm_begin,
|
.begin = lite5200_pm_begin,
|
||||||
.prepare = lite5200_pm_prepare,
|
.prepare = lite5200_pm_prepare,
|
||||||
|
|
|
@ -186,7 +186,7 @@ void mpc52xx_pm_finish(void)
|
||||||
iounmap(mbar);
|
iounmap(mbar);
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops mpc52xx_pm_ops = {
|
static const struct platform_suspend_ops mpc52xx_pm_ops = {
|
||||||
.valid = mpc52xx_pm_valid,
|
.valid = mpc52xx_pm_valid,
|
||||||
.prepare = mpc52xx_pm_prepare,
|
.prepare = mpc52xx_pm_prepare,
|
||||||
.enter = mpc52xx_pm_enter,
|
.enter = mpc52xx_pm_enter,
|
||||||
|
|
|
@ -311,7 +311,7 @@ static int mpc83xx_is_pci_agent(void)
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops mpc83xx_suspend_ops = {
|
static const struct platform_suspend_ops mpc83xx_suspend_ops = {
|
||||||
.valid = mpc83xx_suspend_valid,
|
.valid = mpc83xx_suspend_valid,
|
||||||
.begin = mpc83xx_suspend_begin,
|
.begin = mpc83xx_suspend_begin,
|
||||||
.enter = mpc83xx_suspend_enter,
|
.enter = mpc83xx_suspend_enter,
|
||||||
|
|
|
@ -153,7 +153,7 @@ static struct sysdev_class suspend_sysdev_class = {
|
||||||
.name = "power",
|
.name = "power",
|
||||||
};
|
};
|
||||||
|
|
||||||
static struct platform_suspend_ops pseries_suspend_ops = {
|
static const struct platform_suspend_ops pseries_suspend_ops = {
|
||||||
.valid = suspend_valid_only_mem,
|
.valid = suspend_valid_only_mem,
|
||||||
.begin = pseries_suspend_begin,
|
.begin = pseries_suspend_begin,
|
||||||
.prepare_late = pseries_prepare_late,
|
.prepare_late = pseries_prepare_late,
|
||||||
|
|
|
@ -53,7 +53,7 @@ static int pmc_suspend_valid(suspend_state_t state)
|
||||||
return 1;
|
return 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops pmc_suspend_ops = {
|
static const struct platform_suspend_ops pmc_suspend_ops = {
|
||||||
.valid = pmc_suspend_valid,
|
.valid = pmc_suspend_valid,
|
||||||
.enter = pmc_suspend_enter,
|
.enter = pmc_suspend_enter,
|
||||||
};
|
};
|
||||||
|
|
|
@ -143,7 +143,7 @@ static int hp6x0_pm_enter(suspend_state_t state)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops hp6x0_pm_ops = {
|
static const struct platform_suspend_ops hp6x0_pm_ops = {
|
||||||
.enter = hp6x0_pm_enter,
|
.enter = hp6x0_pm_enter,
|
||||||
.valid = suspend_valid_only_mem,
|
.valid = suspend_valid_only_mem,
|
||||||
};
|
};
|
||||||
|
|
|
@ -141,7 +141,7 @@ static int sh_pm_enter(suspend_state_t state)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops sh_pm_ops = {
|
static const struct platform_suspend_ops sh_pm_ops = {
|
||||||
.enter = sh_pm_enter,
|
.enter = sh_pm_enter,
|
||||||
.valid = suspend_valid_only_mem,
|
.valid = suspend_valid_only_mem,
|
||||||
};
|
};
|
||||||
|
|
|
@ -319,7 +319,7 @@ static int acpi_suspend_state_valid(suspend_state_t pm_state)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops acpi_suspend_ops = {
|
static const struct platform_suspend_ops acpi_suspend_ops = {
|
||||||
.valid = acpi_suspend_state_valid,
|
.valid = acpi_suspend_state_valid,
|
||||||
.begin = acpi_suspend_begin,
|
.begin = acpi_suspend_begin,
|
||||||
.prepare_late = acpi_pm_prepare,
|
.prepare_late = acpi_pm_prepare,
|
||||||
|
@ -347,7 +347,7 @@ static int acpi_suspend_begin_old(suspend_state_t pm_state)
|
||||||
* The following callbacks are used if the pre-ACPI 2.0 suspend ordering has
|
* The following callbacks are used if the pre-ACPI 2.0 suspend ordering has
|
||||||
* been requested.
|
* been requested.
|
||||||
*/
|
*/
|
||||||
static struct platform_suspend_ops acpi_suspend_ops_old = {
|
static const struct platform_suspend_ops acpi_suspend_ops_old = {
|
||||||
.valid = acpi_suspend_state_valid,
|
.valid = acpi_suspend_state_valid,
|
||||||
.begin = acpi_suspend_begin_old,
|
.begin = acpi_suspend_begin_old,
|
||||||
.prepare_late = acpi_pm_pre_suspend,
|
.prepare_late = acpi_pm_pre_suspend,
|
||||||
|
|
|
@ -2257,7 +2257,7 @@ static int pmu_sleep_valid(suspend_state_t state)
|
||||||
&& (pmac_call_feature(PMAC_FTR_SLEEP_STATE, NULL, 0, -1) >= 0);
|
&& (pmac_call_feature(PMAC_FTR_SLEEP_STATE, NULL, 0, -1) >= 0);
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct platform_suspend_ops pmu_pm_ops = {
|
static const struct platform_suspend_ops pmu_pm_ops = {
|
||||||
.enter = powerbook_sleep,
|
.enter = powerbook_sleep,
|
||||||
.valid = pmu_sleep_valid,
|
.valid = pmu_sleep_valid,
|
||||||
};
|
};
|
||||||
|
|
|
@ -122,7 +122,7 @@ struct platform_suspend_ops {
|
||||||
* suspend_set_ops - set platform dependent suspend operations
|
* suspend_set_ops - set platform dependent suspend operations
|
||||||
* @ops: The new suspend operations to set.
|
* @ops: The new suspend operations to set.
|
||||||
*/
|
*/
|
||||||
extern void suspend_set_ops(struct platform_suspend_ops *ops);
|
extern void suspend_set_ops(const struct platform_suspend_ops *ops);
|
||||||
extern int suspend_valid_only_mem(suspend_state_t state);
|
extern int suspend_valid_only_mem(suspend_state_t state);
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -147,7 +147,7 @@ extern int pm_suspend(suspend_state_t state);
|
||||||
#else /* !CONFIG_SUSPEND */
|
#else /* !CONFIG_SUSPEND */
|
||||||
#define suspend_valid_only_mem NULL
|
#define suspend_valid_only_mem NULL
|
||||||
|
|
||||||
static inline void suspend_set_ops(struct platform_suspend_ops *ops) {}
|
static inline void suspend_set_ops(const struct platform_suspend_ops *ops) {}
|
||||||
static inline int pm_suspend(suspend_state_t state) { return -ENOSYS; }
|
static inline int pm_suspend(suspend_state_t state) { return -ENOSYS; }
|
||||||
#endif /* !CONFIG_SUSPEND */
|
#endif /* !CONFIG_SUSPEND */
|
||||||
|
|
||||||
|
|
|
@ -30,13 +30,13 @@ const char *const pm_states[PM_SUSPEND_MAX] = {
|
||||||
[PM_SUSPEND_MEM] = "mem",
|
[PM_SUSPEND_MEM] = "mem",
|
||||||
};
|
};
|
||||||
|
|
||||||
static struct platform_suspend_ops *suspend_ops;
|
static const struct platform_suspend_ops *suspend_ops;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* suspend_set_ops - Set the global suspend method table.
|
* suspend_set_ops - Set the global suspend method table.
|
||||||
* @ops: Pointer to ops structure.
|
* @ops: Pointer to ops structure.
|
||||||
*/
|
*/
|
||||||
void suspend_set_ops(struct platform_suspend_ops *ops)
|
void suspend_set_ops(const struct platform_suspend_ops *ops)
|
||||||
{
|
{
|
||||||
mutex_lock(&pm_mutex);
|
mutex_lock(&pm_mutex);
|
||||||
suspend_ops = ops;
|
suspend_ops = ops;
|
||||||
|
|
Loading…
Reference in New Issue