From: Sagar Arun Kamble <sagar.a.kamble@intel.com>
To: Ewelina Musial <ewelina.musial@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 08/31] drm/i915: Rename intel_enable_rc6 to intel_rc6_enabled
Date: Thu, 28 Sep 2017 14:41:55 +0530 [thread overview]
Message-ID: <f92a9461-735f-be5c-2fb6-31fdddc30d12@intel.com> (raw)
In-Reply-To: <20170926074101.GB30395@emusial-desk.ger.corp.intel.com>
Thank you Radek, Ewelina for the reviews.
On 9/26/2017 1:11 PM, Ewelina Musial wrote:
> On Tue, Sep 19, 2017 at 11:11:44PM +0530, Sagar Arun Kamble wrote:
>> This function gives the status of RC6, whether disabled or if
>> enabled then which state. intel_enable_rc6 will be used for
>> enabling RC6 in the next patch.
>>
>> Cc: Chris Wilson <chris@chris-wilson.co.uk>
>> Cc: Imre Deak <imre.deak@intel.com>
>> Signed-off-by: Sagar Arun Kamble <sagar.a.kamble@intel.com>
> Reviewed-by: Ewelina Musial <ewelina.musial@intel.com>
>
> - Ewelina
>> ---
>> drivers/gpu/drm/i915/i915_drv.c | 2 +-
>> drivers/gpu/drm/i915/i915_sysfs.c | 2 +-
>> drivers/gpu/drm/i915/intel_drv.h | 2 +-
>> drivers/gpu/drm/i915/intel_guc.c | 3 ++-
>> drivers/gpu/drm/i915/intel_pm.c | 12 ++++++------
>> 5 files changed, 11 insertions(+), 10 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
>> index 6cc1162..a6dbad3 100644
>> --- a/drivers/gpu/drm/i915/i915_drv.c
>> +++ b/drivers/gpu/drm/i915/i915_drv.c
>> @@ -2486,7 +2486,7 @@ static int intel_runtime_suspend(struct device *kdev)
>> struct drm_i915_private *dev_priv = to_i915(dev);
>> int ret;
>>
>> - if (WARN_ON_ONCE(!(dev_priv->pm.rps.enabled && intel_enable_rc6())))
>> + if (WARN_ON_ONCE(!(dev_priv->pm.rps.enabled && intel_rc6_enabled())))
>> return -ENODEV;
>>
>> if (WARN_ON_ONCE(!HAS_RUNTIME_PM(dev_priv)))
>> diff --git a/drivers/gpu/drm/i915/i915_sysfs.c b/drivers/gpu/drm/i915/i915_sysfs.c
>> index c16e907..8add849 100644
>> --- a/drivers/gpu/drm/i915/i915_sysfs.c
>> +++ b/drivers/gpu/drm/i915/i915_sysfs.c
>> @@ -49,7 +49,7 @@ static u32 calc_residency(struct drm_i915_private *dev_priv,
>> static ssize_t
>> show_rc6_mask(struct device *kdev, struct device_attribute *attr, char *buf)
>> {
>> - return snprintf(buf, PAGE_SIZE, "%x\n", intel_enable_rc6());
>> + return snprintf(buf, PAGE_SIZE, "%x\n", intel_rc6_enabled());
>> }
>>
>> static ssize_t
>> diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
>> index f7db720..0cf04eb 100644
>> --- a/drivers/gpu/drm/i915/intel_drv.h
>> +++ b/drivers/gpu/drm/i915/intel_drv.h
>> @@ -1900,7 +1900,7 @@ int skl_check_pipe_max_pixel_rate(struct intel_crtc *intel_crtc,
>> struct intel_crtc_state *cstate);
>> void intel_init_ipc(struct drm_i915_private *dev_priv);
>> void intel_enable_ipc(struct drm_i915_private *dev_priv);
>> -static inline int intel_enable_rc6(void)
>> +static inline int intel_rc6_enabled(void)
>> {
>> return i915.enable_rc6;
>> }
>> diff --git a/drivers/gpu/drm/i915/intel_guc.c b/drivers/gpu/drm/i915/intel_guc.c
>> index c731cff..f4dc708 100644
>> --- a/drivers/gpu/drm/i915/intel_guc.c
>> +++ b/drivers/gpu/drm/i915/intel_guc.c
>> @@ -128,7 +128,8 @@ int intel_guc_sample_forcewake(struct intel_guc *guc)
>>
>> action[0] = INTEL_GUC_ACTION_SAMPLE_FORCEWAKE;
>> /* WaRsDisableCoarsePowerGating:skl,bxt */
>> - if (!intel_enable_rc6() || NEEDS_WaRsDisableCoarsePowerGating(dev_priv))
>> + if (!intel_rc6_enabled() ||
>> + NEEDS_WaRsDisableCoarsePowerGating(dev_priv))
>> action[1] = 0;
>> else
>> /* bit 0 and 1 are for Render and Media domain separately */
>> diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
>> index b89677a..b83751e 100644
>> --- a/drivers/gpu/drm/i915/intel_pm.c
>> +++ b/drivers/gpu/drm/i915/intel_pm.c
>> @@ -6606,7 +6606,7 @@ static void gen9_enable_rc6(struct drm_i915_private *dev_priv)
>> I915_WRITE(GEN9_RENDER_PG_IDLE_HYSTERESIS, 25);
>>
>> /* 3a: Enable RC6 */
>> - if (intel_enable_rc6() & INTEL_RC6_ENABLE)
>> + if (intel_rc6_enabled() & INTEL_RC6_ENABLE)
>> rc6_mask = GEN6_RC_CTL_RC6_ENABLE;
>> DRM_INFO("RC6 %s\n", onoff(rc6_mask & GEN6_RC_CTL_RC6_ENABLE));
>> I915_WRITE(GEN6_RC6_THRESHOLD, 37500); /* 37.5/125ms per EI */
>> @@ -6655,7 +6655,7 @@ static void gen8_enable_rc6(struct drm_i915_private *dev_priv)
>> I915_WRITE(GEN6_RC6_THRESHOLD, 50000); /* 50/125ms per EI */
>>
>> /* 3: Enable RC6 */
>> - if (intel_enable_rc6() & INTEL_RC6_ENABLE)
>> + if (intel_rc6_enabled() & INTEL_RC6_ENABLE)
>> rc6_mask = GEN6_RC_CTL_RC6_ENABLE;
>> intel_print_rc6_info(dev_priv, rc6_mask);
>> I915_WRITE(GEN6_RC_CONTROL, GEN6_RC_CTL_HW_ENABLE |
>> @@ -6749,7 +6749,7 @@ static void gen6_enable_rc6(struct drm_i915_private *dev_priv)
>> I915_WRITE(GEN6_RC6pp_THRESHOLD, 64000); /* unused */
>>
>> /* Check if we are enabling RC6 */
>> - rc6_mode = intel_enable_rc6();
>> + rc6_mode = intel_rc6_enabled();
>> if (rc6_mode & INTEL_RC6_ENABLE)
>> rc6_mask |= GEN6_RC_CTL_RC6_ENABLE;
>>
>> @@ -7251,7 +7251,7 @@ static void cherryview_enable_rc6(struct drm_i915_private *dev_priv)
>> pcbr = I915_READ(VLV_PCBR);
>>
>> /* 3: Enable RC6 */
>> - if ((intel_enable_rc6() & INTEL_RC6_ENABLE) &&
>> + if ((intel_rc6_enabled() & INTEL_RC6_ENABLE) &&
>> (pcbr >> VLV_PCBR_ADDR_SHIFT))
>> rc6_mode = GEN7_RC_CTL_TO_MODE;
>>
>> @@ -7345,7 +7345,7 @@ static void valleyview_enable_rc6(struct drm_i915_private *dev_priv)
>> VLV_MEDIA_RC6_COUNT_EN |
>> VLV_RENDER_RC6_COUNT_EN));
>>
>> - if (intel_enable_rc6() & INTEL_RC6_ENABLE)
>> + if (intel_rc6_enabled() & INTEL_RC6_ENABLE)
>> rc6_mode = GEN7_RC_CTL_TO_MODE | VLV_RC_CTL_CTX_RST_PARALLEL;
>>
>> intel_print_rc6_info(dev_priv, rc6_mode);
>> @@ -9418,7 +9418,7 @@ u64 intel_rc6_residency_us(struct drm_i915_private *dev_priv,
>> {
>> u64 time_hw, units, div;
>>
>> - if (!intel_enable_rc6())
>> + if (!intel_rc6_enabled())
>> return 0;
>>
>> intel_runtime_pm_get(dev_priv);
>> --
>> 1.9.1
>>
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2017-09-28 9:11 UTC|newest]
Thread overview: 58+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-19 17:41 [PATCH 00/31] Add support for GuC-based SLPC Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 01/31] drm/i915/debugfs: Create generic string tokenize function and update CRC control parsing Sagar Arun Kamble
2017-09-21 15:12 ` Michal Wajdeczko
2017-09-28 9:10 ` Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 02/31] drm/i915: Separate RPS and RC6 handling for gen6+ Sagar Arun Kamble
2017-09-20 12:29 ` Szwichtenberg, Radoslaw
2017-09-19 17:41 ` [PATCH 03/31] drm/i915: Separate RPS and RC6 handling for BDW Sagar Arun Kamble
2017-09-20 11:14 ` Szwichtenberg, Radoslaw
2017-09-20 12:31 ` Szwichtenberg, Radoslaw
2017-09-19 17:41 ` [PATCH 04/31] drm/i915: Separate RPS and RC6 handling for VLV Sagar Arun Kamble
2017-09-20 12:30 ` Szwichtenberg, Radoslaw
2017-09-19 17:41 ` [PATCH 05/31] drm/i915: Separate RPS and RC6 handling for CHV Sagar Arun Kamble
2017-09-20 12:32 ` Szwichtenberg, Radoslaw
2017-09-19 17:41 ` [PATCH 06/31] drm/i915: Name i915_runtime_pm structure in dev_priv as "rpm" Sagar Arun Kamble
2017-09-20 12:34 ` Szwichtenberg, Radoslaw
2017-09-19 17:41 ` [PATCH 07/31] drm/i915: Name structure in dev_priv that contains RPS/RC6 state as "pm" Sagar Arun Kamble
2017-09-21 8:23 ` Szwichtenberg, Radoslaw
2017-09-19 17:41 ` [PATCH 08/31] drm/i915: Rename intel_enable_rc6 to intel_rc6_enabled Sagar Arun Kamble
2017-09-21 8:26 ` Szwichtenberg, Radoslaw
2017-09-26 7:41 ` Ewelina Musial
2017-09-28 9:11 ` Sagar Arun Kamble [this message]
2017-09-19 17:41 ` [PATCH 09/31] drm/i915: Create generic function to setup ring frequency table Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 10/31] drm/i915: Create generic functions to control RC6, RPS Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 11/31] drm/i915: Introduce separate status variable for RC6 and Ring frequency setup Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 12/31] drm/i915: Define RPS idle, busy, boost function pointers Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 13/31] drm/i915/slpc: Add has_slpc capability flag Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 14/31] drm/i915/slpc: Add enable_slpc module parameter Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 15/31] drm/i915/slpc: Sanitize GuC version Sagar Arun Kamble
2017-09-21 12:52 ` Michal Wajdeczko
2017-09-28 9:20 ` Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 16/31] drm/i915/slpc: Lay out SLPC init/enable/disable/cleanup helpers Sagar Arun Kamble
2017-09-21 13:00 ` Michal Wajdeczko
2017-09-28 9:29 ` Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 17/31] drm/i915/slpc: Enable SLPC in GuC if supported Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 18/31] drm/i915/slpc: Add SLPC communication interfaces Sagar Arun Kamble
2017-09-21 13:14 ` Michal Wajdeczko
2017-09-28 9:48 ` Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 19/31] drm/i915/slpc: Allocate/Release/Initialize SLPC shared data Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 20/31] drm/i915/slpc: Add parameter set/unset/get, task control/status functions Sagar Arun Kamble
2017-09-21 13:47 ` Michal Wajdeczko
2017-09-28 9:55 ` Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 21/31] drm/i915/slpc: Send RESET event to enable SLPC during Load/TDR Sagar Arun Kamble
2017-09-21 14:06 ` Michal Wajdeczko
2017-09-28 10:10 ` Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 22/31] drm/i915/slpc: Send SHUTDOWN event Sagar Arun Kamble
2017-09-19 17:41 ` [PATCH 23/31] drm/i915/slpc: Add support for min/max frequency control Sagar Arun Kamble
2017-09-19 17:42 ` [PATCH 24/31] drm/i915/slpc: Add debugfs support to read/write/revert the parameters Sagar Arun Kamble
2017-09-21 15:07 ` Michal Wajdeczko
2017-09-28 10:18 ` Sagar Arun Kamble
2017-09-19 17:42 ` [PATCH 25/31] drm/i915/slpc: Add enable/disable controls for SLPC tasks Sagar Arun Kamble
2017-09-19 17:42 ` [PATCH 26/31] drm/i915/slpc: Add i915_slpc_info to debugfs Sagar Arun Kamble
2017-09-21 15:13 ` Michal Wajdeczko
2017-09-28 10:20 ` Sagar Arun Kamble
2017-09-19 17:42 ` [PATCH 27/31] drm/i915/slpc: Add SLPC banner to RPS debugfs interfaces Sagar Arun Kamble
2017-09-19 17:42 ` [PATCH 28/31] drm/i915/slpc: Add SKL SLPC Support Sagar Arun Kamble
2017-09-19 17:42 ` [PATCH 29/31] drm/i915/slpc: Add Broxton SLPC support Sagar Arun Kamble
2017-09-19 17:42 ` [PATCH 30/31] drm/i915/slpc: Add Kabylake " Sagar Arun Kamble
2017-09-19 17:42 ` [PATCH 31/31] drm/i915/slpc: Add Geminilake " Sagar Arun Kamble
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f92a9461-735f-be5c-2fb6-31fdddc30d12@intel.com \
--to=sagar.a.kamble@intel.com \
--cc=ewelina.musial@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox