From deebdf70577582c5914cfbee17f712e83ce54fad Mon Sep 17 00:00:00 2001 From: buxiasen Date: Fri, 29 Nov 2024 23:06:47 +0800 Subject: [PATCH] goldfish: use arch_idle_custom Will benifit on cpuload, and power consume Signed-off-by: buxiasen --- arch/arm/Kconfig | 1 + arch/arm/src/goldfish/CMakeLists.txt | 4 ++ arch/arm/src/goldfish/Make.defs | 4 ++ arch/arm/src/goldfish/goldfish_idle.c | 61 +++++++++++++++++++++++++++ 4 files changed, 70 insertions(+) create mode 100644 arch/arm/src/goldfish/goldfish_idle.c diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index 64328c0006a..269bb599f66 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -693,6 +693,7 @@ config ARCH_CHIP_GOLDFISH_ARM select ARCH_HAVE_RESET select ARM_HAVE_PSCI select ARM_HAVE_NEON + select ARCH_IDLE_CUSTOM ---help--- GOLDFISH virt platform (ARMv7a) diff --git a/arch/arm/src/goldfish/CMakeLists.txt b/arch/arm/src/goldfish/CMakeLists.txt index 229763209b5..26ac836951d 100644 --- a/arch/arm/src/goldfish/CMakeLists.txt +++ b/arch/arm/src/goldfish/CMakeLists.txt @@ -24,4 +24,8 @@ if(CONFIG_SMP) list(APPEND SRCS goldfish_cpuboot.c) endif() +if(CONFIG_ARCH_IDLE_CUSTOM) + list(APPEND SRCS goldfish_idle.c) +endif() + target_sources(arch PRIVATE ${SRCS}) diff --git a/arch/arm/src/goldfish/Make.defs b/arch/arm/src/goldfish/Make.defs index e3d93e34f77..20ff9d3fea3 100644 --- a/arch/arm/src/goldfish/Make.defs +++ b/arch/arm/src/goldfish/Make.defs @@ -28,3 +28,7 @@ CHIP_CSRCS += goldfish_memorymap.c goldfish_serial.c goldfish_timer.c ifeq ($(CONFIG_SMP),y) CHIP_CSRCS += goldfish_cpuboot.c endif + +ifeq ($(CONFIG_ARCH_IDLE_CUSTOM),y) +CHIP_CSRCS += goldfish_idle.c +endif diff --git a/arch/arm/src/goldfish/goldfish_idle.c b/arch/arm/src/goldfish/goldfish_idle.c new file mode 100644 index 00000000000..e7483ac0a37 --- /dev/null +++ b/arch/arm/src/goldfish/goldfish_idle.c @@ -0,0 +1,61 @@ +/**************************************************************************** + * arch/arm/src/goldfish/goldfish_idle.c + * + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. The + * ASF licenses this file to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance with the + * License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations + * under the License. + * + ****************************************************************************/ + +/**************************************************************************** + * Included Files + ****************************************************************************/ + +#include + +#include +#include "arm_internal.h" + +/**************************************************************************** + * Public Functions + ****************************************************************************/ + +/**************************************************************************** + * Name: up_idle + * + * Description: + * up_idle() is the logic that will be executed when there is no other + * ready-to-run task. This is processor idle time and will continue until + * some interrupt occurs to cause a context switch from the idle task. + * + * Processing in this state may be processor-specific. e.g., this is where + * power management operations might be performed. + * + ****************************************************************************/ + +void up_idle(void) +{ +#if defined(CONFIG_SUPPRESS_INTERRUPTS) || defined(CONFIG_SUPPRESS_TIMER_INTS) + /* If the system is idle and there are no timer interrupts, then process + * "fake" timer interrupts. Hopefully, something will wake up. + */ + + nxsched_process_timer(); +#else + + /* Sleep until an interrupt occurs to save power */ + + asm("WFI"); +#endif +}