On Tue, 22 Oct 2024, Colin Ian King wrote:
> Don't populate the read-only arrays on the stack at run time, instead
> make them static const.
>
> Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx>
Reviewed-by: Nicolas Pitre <npitre@xxxxxxxxxxxx>
> ---
> drivers/thermal/mediatek/lvts_thermal.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/thermal/mediatek/lvts_thermal.c b/drivers/thermal/mediatek/lvts_thermal.c
> index 1997e91bb3be..ce223bab6b55 100644
> --- a/drivers/thermal/mediatek/lvts_thermal.c
> +++ b/drivers/thermal/mediatek/lvts_thermal.c
> @@ -329,7 +329,7 @@ static int lvts_get_temp(struct thermal_zone_device *tz, int *temp)
>
> static void lvts_update_irq_mask(struct lvts_ctrl *lvts_ctrl)
> {
> - u32 masks[] = {
> + static const u32 masks[] = {
> LVTS_MONINT_OFFSET_SENSOR0,
> LVTS_MONINT_OFFSET_SENSOR1,
> LVTS_MONINT_OFFSET_SENSOR2,
> @@ -424,7 +424,7 @@ static irqreturn_t lvts_ctrl_irq_handler(struct lvts_ctrl *lvts_ctrl)
> {
> irqreturn_t iret = IRQ_NONE;
> u32 value;
> - u32 masks[] = {
> + static const u32 masks[] = {
> LVTS_INT_SENSOR0,
> LVTS_INT_SENSOR1,
> LVTS_INT_SENSOR2,
> --
> 2.39.5
>
>
