diff options
| -rw-r--r-- | src/common/settings.cpp | 2 | ||||
| -rw-r--r-- | src/common/settings.h | 2 | ||||
| -rw-r--r-- | src/core/hle/service/am/am.cpp | 12 | ||||
| -rw-r--r-- | src/core/hle/service/vi/vi.cpp | 27 | ||||
| -rw-r--r-- | src/core/telemetry_session.cpp | 2 | ||||
| -rw-r--r-- | src/video_core/video_core.cpp | 9 | ||||
| -rw-r--r-- | src/video_core/video_core.h | 2 | 
7 files changed, 19 insertions, 37 deletions
| diff --git a/src/common/settings.cpp b/src/common/settings.cpp index dd3a3d456..6f3acee79 100644 --- a/src/common/settings.cpp +++ b/src/common/settings.cpp @@ -48,7 +48,6 @@ void LogSettings() {      log_setting("Core_UseMultiCore", values.use_multi_core.GetValue());      log_setting("CPU_Accuracy", values.cpu_accuracy.GetValue());      log_setting("Renderer_UseResolutionScaling", values.resolution_setup.GetValue()); -    log_setting("Renderer_UseResolutionFactor", values.resolution_factor.GetValue());      log_setting("Renderer_UseSpeedLimit", values.use_speed_limit.GetValue());      log_setting("Renderer_SpeedLimit", values.speed_limit.GetValue());      log_setting("Renderer_UseDiskShaderCache", values.use_disk_shader_cache.GetValue()); @@ -155,6 +154,7 @@ void UpdateRescalingInfo() {      info.down_factor = static_cast<f32>(1U << info.down_shift) / info.up_scale;      info.size_up = info.up_scale * info.up_scale;      info.size_shift = info.down_shift * 2; +    info.active = info.up_scale != 1 || info.down_shift != 0;  }  void RestoreGlobalState(bool is_powered_on) { diff --git a/src/common/settings.h b/src/common/settings.h index f4df2fc95..2b11984b4 100644 --- a/src/common/settings.h +++ b/src/common/settings.h @@ -69,6 +69,7 @@ struct ResolutionScalingInfo {      f32 down_factor{1.0f};      u32 size_up{1};      u32 size_shift{0}; +    bool active{};  };  /** The BasicSetting class is a simple resource manager. It defines a label and default value @@ -472,7 +473,6 @@ struct Values {      ResolutionScalingInfo resolution_info{};      Setting<ResolutionSetup> resolution_setup{ResolutionSetup::Res1X, "resolution_setup"}; -    Setting<u16> resolution_factor{1, "resolution_factor"};      // *nix platforms may have issues with the borderless windowed fullscreen mode.      // Default to exclusive fullscreen on these platforms for now.      RangedSetting<FullscreenMode> fullscreen_mode{ diff --git a/src/core/hle/service/am/am.cpp b/src/core/hle/service/am/am.cpp index 50c2ace93..aee8d4f93 100644 --- a/src/core/hle/service/am/am.cpp +++ b/src/core/hle/service/am/am.cpp @@ -797,15 +797,11 @@ void ICommonStateGetter::GetDefaultDisplayResolution(Kernel::HLERequestContext&      rb.Push(ResultSuccess);      if (Settings::values.use_docked_mode.GetValue()) { -        rb.Push(static_cast<u32>(Service::VI::DisplayResolution::DockedWidth) * -                static_cast<u32>(Settings::values.resolution_factor.GetValue())); -        rb.Push(static_cast<u32>(Service::VI::DisplayResolution::DockedHeight) * -                static_cast<u32>(Settings::values.resolution_factor.GetValue())); +        rb.Push(static_cast<u32>(Service::VI::DisplayResolution::DockedWidth)); +        rb.Push(static_cast<u32>(Service::VI::DisplayResolution::DockedHeight));      } else { -        rb.Push(static_cast<u32>(Service::VI::DisplayResolution::UndockedWidth) * -                static_cast<u32>(Settings::values.resolution_factor.GetValue())); -        rb.Push(static_cast<u32>(Service::VI::DisplayResolution::UndockedHeight) * -                static_cast<u32>(Settings::values.resolution_factor.GetValue())); +        rb.Push(static_cast<u32>(Service::VI::DisplayResolution::UndockedWidth)); +        rb.Push(static_cast<u32>(Service::VI::DisplayResolution::UndockedHeight));      }  } diff --git a/src/core/hle/service/vi/vi.cpp b/src/core/hle/service/vi/vi.cpp index 63d5242c4..75ee3e5e4 100644 --- a/src/core/hle/service/vi/vi.cpp +++ b/src/core/hle/service/vi/vi.cpp @@ -541,11 +541,8 @@ private:          switch (transaction) {          case TransactionId::Connect: {              IGBPConnectRequestParcel request{ctx.ReadBuffer()}; -            IGBPConnectResponseParcel response{ -                static_cast<u32>(static_cast<u32>(DisplayResolution::UndockedWidth) * -                                 Settings::values.resolution_factor.GetValue()), -                static_cast<u32>(static_cast<u32>(DisplayResolution::UndockedHeight) * -                                 Settings::values.resolution_factor.GetValue())}; +            IGBPConnectResponseParcel response{static_cast<u32>(DisplayResolution::UndockedWidth), +                                               static_cast<u32>(DisplayResolution::UndockedHeight)};              buffer_queue.Connect(); @@ -775,15 +772,11 @@ private:          rb.Push(ResultSuccess);          if (Settings::values.use_docked_mode.GetValue()) { -            rb.Push(static_cast<u32>(Service::VI::DisplayResolution::DockedWidth) * -                    static_cast<u32>(Settings::values.resolution_factor.GetValue())); -            rb.Push(static_cast<u32>(Service::VI::DisplayResolution::DockedHeight) * -                    static_cast<u32>(Settings::values.resolution_factor.GetValue())); +            rb.Push(static_cast<u32>(Service::VI::DisplayResolution::DockedWidth)); +            rb.Push(static_cast<u32>(Service::VI::DisplayResolution::DockedHeight));          } else { -            rb.Push(static_cast<u32>(Service::VI::DisplayResolution::UndockedWidth) * -                    static_cast<u32>(Settings::values.resolution_factor.GetValue())); -            rb.Push(static_cast<u32>(Service::VI::DisplayResolution::UndockedHeight) * -                    static_cast<u32>(Settings::values.resolution_factor.GetValue())); +            rb.Push(static_cast<u32>(Service::VI::DisplayResolution::UndockedWidth)); +            rb.Push(static_cast<u32>(Service::VI::DisplayResolution::UndockedHeight));          }          rb.PushRaw<float>(60.0f); // This wouldn't seem to be correct for 30 fps games. @@ -1063,10 +1056,8 @@ private:          // This only returns the fixed values of 1280x720 and makes no distinguishing          // between docked and undocked dimensions. We take the liberty of applying          // the resolution scaling factor here. -        rb.Push(static_cast<u64>(DisplayResolution::UndockedWidth) * -                static_cast<u32>(Settings::values.resolution_factor.GetValue())); -        rb.Push(static_cast<u64>(DisplayResolution::UndockedHeight) * -                static_cast<u32>(Settings::values.resolution_factor.GetValue())); +        rb.Push(static_cast<u64>(DisplayResolution::UndockedWidth)); +        rb.Push(static_cast<u64>(DisplayResolution::UndockedHeight));      }      void SetLayerScalingMode(Kernel::HLERequestContext& ctx) { @@ -1099,8 +1090,6 @@ private:          LOG_WARNING(Service_VI, "(STUBBED) called");          DisplayInfo display_info; -        display_info.width *= static_cast<u64>(Settings::values.resolution_factor.GetValue()); -        display_info.height *= static_cast<u64>(Settings::values.resolution_factor.GetValue());          ctx.WriteBuffer(&display_info, sizeof(DisplayInfo));          IPC::ResponseBuilder rb{ctx, 4};          rb.Push(ResultSuccess); diff --git a/src/core/telemetry_session.cpp b/src/core/telemetry_session.cpp index 191475f71..654db0b52 100644 --- a/src/core/telemetry_session.cpp +++ b/src/core/telemetry_session.cpp @@ -229,8 +229,6 @@ void TelemetrySession::AddInitialInfo(Loader::AppLoader& app_loader,      AddField(field_type, "Core_UseMultiCore", Settings::values.use_multi_core.GetValue());      AddField(field_type, "Renderer_Backend",               TranslateRenderer(Settings::values.renderer_backend.GetValue())); -    AddField(field_type, "Renderer_ResolutionFactor", -             Settings::values.resolution_factor.GetValue());      AddField(field_type, "Renderer_UseSpeedLimit", Settings::values.use_speed_limit.GetValue());      AddField(field_type, "Renderer_SpeedLimit", Settings::values.speed_limit.GetValue());      AddField(field_type, "Renderer_UseDiskShaderCache", diff --git a/src/video_core/video_core.cpp b/src/video_core/video_core.cpp index cae543a51..508173db3 100644 --- a/src/video_core/video_core.cpp +++ b/src/video_core/video_core.cpp @@ -53,11 +53,10 @@ std::unique_ptr<Tegra::GPU> CreateGPU(Core::Frontend::EmuWindow& emu_window, Cor      }  } -u16 GetResolutionScaleFactor(const RendererBase& renderer) { -    return static_cast<u16>( -        Settings::values.resolution_factor.GetValue() != 0 -            ? Settings::values.resolution_factor.GetValue() -            : renderer.GetRenderWindow().GetFramebufferLayout().GetScalingRatio()); +float GetResolutionScaleFactor(const RendererBase& renderer) { +    return Settings::values.resolution_info.active +               ? Settings::values.resolution_info.up_factor +               : renderer.GetRenderWindow().GetFramebufferLayout().GetScalingRatio();  }  } // namespace VideoCore diff --git a/src/video_core/video_core.h b/src/video_core/video_core.h index f5c27125d..f86877e86 100644 --- a/src/video_core/video_core.h +++ b/src/video_core/video_core.h @@ -25,6 +25,6 @@ class RendererBase;  /// Creates an emulated GPU instance using the given system context.  std::unique_ptr<Tegra::GPU> CreateGPU(Core::Frontend::EmuWindow& emu_window, Core::System& system); -u16 GetResolutionScaleFactor(const RendererBase& renderer); +float GetResolutionScaleFactor(const RendererBase& renderer);  } // namespace VideoCore | 
