From ab8f75f0003e02b70111284c7fe4d354c299dc1d Mon Sep 17 00:00:00 2001 From: JosJuice Date: Thu, 24 Sep 2015 14:57:25 +0200 Subject: [PATCH] SystemTimers: Don't name non-constant variables as if they are constant --- Source/Core/Core/HW/SystemTimers.cpp | 38 +++++++++++++--------------- 1 file changed, 18 insertions(+), 20 deletions(-) diff --git a/Source/Core/Core/HW/SystemTimers.cpp b/Source/Core/Core/HW/SystemTimers.cpp index e95a704194..2b903ee5e5 100644 --- a/Source/Core/Core/HW/SystemTimers.cpp +++ b/Source/Core/Core/HW/SystemTimers.cpp @@ -67,8 +67,6 @@ IPC_HLE_PERIOD: For the Wiimote this is the call schedule: namespace SystemTimers { -static u32 CPU_CORE_CLOCK = 486000000u; // 486 mhz (its not 485, stop bugging me!) - static int et_Dec; static int et_VI; static int et_SI; @@ -79,23 +77,23 @@ static int et_IPC_HLE; static int et_PatchEngine; // PatchEngine updates every 1/60th of a second by default static int et_Throttle; +static u32 s_cpu_core_clock = 486000000u; // 486 mhz (its not 485, stop bugging me!) static u64 s_last_sync_gpu_tick; -// These are badly educated guesses -// Feel free to experiment. Set these in Init below. -static int - // This is a fixed value, don't change it - AUDIO_DMA_PERIOD, +// These two are badly educated guesses. +// Feel free to experiment. Set them in Init below. - // This is completely arbitrary. If we find that we need lower latency, we can just - // increase this number. - IPC_HLE_PERIOD; +// This is a fixed value, don't change it +static int s_audio_dma_period; +// This is completely arbitrary. If we find that we need lower latency, +// we can just increase this number. +static int s_ipc_hle_period; u32 GetTicksPerSecond() { - return CPU_CORE_CLOCK; + return s_cpu_core_clock; } // DSP/CPU timeslicing. @@ -109,7 +107,7 @@ static void DSPCallback(u64 userdata, int cyclesLate) static void AudioDMACallback(u64 userdata, int cyclesLate) { - int period = CPU_CORE_CLOCK / (AudioInterface::GetAIDSampleRate() * 4 / 32); + int period = s_cpu_core_clock / (AudioInterface::GetAIDSampleRate() * 4 / 32); DSP::UpdateAudioDMA(); // Push audio to speakers. CoreTiming::ScheduleEvent(period - cyclesLate, et_AudioDMA); } @@ -119,7 +117,7 @@ static void IPC_HLE_UpdateCallback(u64 userdata, int cyclesLate) if (SConfig::GetInstance().bWii) { WII_IPC_HLE_Interface::UpdateDevices(); - CoreTiming::ScheduleEvent(IPC_HLE_PERIOD - cyclesLate, et_IPC_HLE); + CoreTiming::ScheduleEvent(s_ipc_hle_period - cyclesLate, et_IPC_HLE); } } @@ -219,9 +217,9 @@ static void ThrottleCallback(u64 last_time, int cyclesLate) void PreInit() { if (SConfig::GetInstance().bWii) - CPU_CORE_CLOCK = 729000000u; + s_cpu_core_clock = 729000000u; else - CPU_CORE_CLOCK = 486000000u; + s_cpu_core_clock = 486000000u; } void Init() @@ -234,15 +232,15 @@ void Init() // FYI, WII_IPC_HLE_Interface::Update is also called in WII_IPCInterface::Write32 const int freq = 1500; - IPC_HLE_PERIOD = GetTicksPerSecond() / freq; + s_ipc_hle_period = GetTicksPerSecond() / freq; } // System internal sample rate is fixed at 32KHz * 4 (16bit Stereo) / 32 bytes DMA - AUDIO_DMA_PERIOD = CPU_CORE_CLOCK / (AudioInterface::GetAIDSampleRate() * 4 / 32); + s_audio_dma_period = s_cpu_core_clock / (AudioInterface::GetAIDSampleRate() * 4 / 32); Common::Timer::IncreaseResolution(); // store and convert localtime at boot to timebase ticks - CoreTiming::SetFakeTBStartValue((u64)(CPU_CORE_CLOCK / TIMER_RATIO) * (u64)CEXIIPL::GetGCTime()); + CoreTiming::SetFakeTBStartValue((u64)(s_cpu_core_clock / TIMER_RATIO) * (u64)CEXIIPL::GetGCTime()); CoreTiming::SetFakeTBStartTicks(CoreTiming::GetTicks()); CoreTiming::SetFakeDecStartValue(0xFFFFFFFF); @@ -262,7 +260,7 @@ void Init() CoreTiming::ScheduleEvent(VideoInterface::GetTicksPerHalfLine(), et_VI); CoreTiming::ScheduleEvent(0, et_DSP); CoreTiming::ScheduleEvent(VideoInterface::GetTicksPerField(), et_SI); - CoreTiming::ScheduleEvent(AUDIO_DMA_PERIOD, et_AudioDMA); + CoreTiming::ScheduleEvent(s_audio_dma_period, et_AudioDMA); CoreTiming::ScheduleEvent(0, et_Throttle, Common::Timer::GetTimeMs()); if (SConfig::GetInstance().bCPUThread && SConfig::GetInstance().bSyncGPU) CoreTiming::ScheduleEvent(0, et_CP); @@ -271,7 +269,7 @@ void Init() CoreTiming::ScheduleEvent(VideoInterface::GetTicksPerField(), et_PatchEngine); if (SConfig::GetInstance().bWii) - CoreTiming::ScheduleEvent(IPC_HLE_PERIOD, et_IPC_HLE); + CoreTiming::ScheduleEvent(s_ipc_hle_period, et_IPC_HLE); } void Shutdown()