All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Deucher, Alexander" <Alexander.Deucher-5C7GfCeVMHo@public.gmane.org>
To: "sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org"
	<sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Mauro Rossi <issor.oruam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Mike Lothian <mike-4+n8WJKc9ve9FHfhHBbuYA@public.gmane.org>,
	"Wentland, Harry" <Harry.Wentland-5C7GfCeVMHo@public.gmane.org>,
	"amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org"
	<amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>
Subject: Re: [RFC] drm/amd/display: add SI support to AMD DC
Date: Mon, 15 Oct 2018 17:53:39 +0000	[thread overview]
Message-ID: <BN6PR12MB18092365083B2653B547057AF7FD0@BN6PR12MB1809.namprd12.prod.outlook.com> (raw)
In-Reply-To: <20181015124549.GA579@freedom>


[-- Attachment #1.1: Type: text/plain, Size: 1652 bytes --]

There are two sets of power management code in amdgpu, the older dpm code which was ported from radeon, and the newer powerplay code which was rewritten to align closer with the power management code for other OSes and the hw teams.  The "powerplay" code has more features than the older dpm code, but at a fundamental level, they are pretty much the same.  Both use the same SMU interfaces to support dynamic clock switching.  SI is a bit tricky because it uses and older version of the SMU and an older power management design so it doesn't really match cleanly with the current APIs in the "powerplay" code.


Alex


________________________________
From: sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org <sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Sent: Monday, October 15, 2018 8:45 AM
To: Mauro Rossi
Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org; Wentland, Harry; Deucher, Alexander; Mike Lothian
Subject: Re: [RFC] drm/amd/display: add SI support to AMD DC

On Mon, Oct 15, 2018 at 07:28:57AM +0200, Mauro Rossi wrote:
> dpm for SI is available, while powerplay for SI is not, but
> display/amdgpu_dm uses some powerplay calls, where get_static_clock
> functions not available and the *ERROR* DM_PPLIB is due to missing handling
> in powerplay

I though powerplay was just more power states for the dpm state machines to
play with. I recall parsing ATOMBIOS pplay tables for records of (voltages/clocks
frequencies).

Maybe the future is to trust the OS kernel and perform all "low speed" 3D asic
functions there and keep only high speed 3D operations in the asic.

--
Sylvain

[-- Attachment #1.2: Type: text/html, Size: 2884 bytes --]

[-- Attachment #2: Type: text/plain, Size: 154 bytes --]

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

  reply	other threads:[~2018-10-15 17:53 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-08  2:23 [RFC] drm/amd/display: add SI support to AMD DC Mauro Rossi
     [not found] ` <20181008022344.10247-1-issor.oruam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2018-10-08  2:23   ` [PATCH 01/10] drm/amd/display: add asics info for SI parts Mauro Rossi
2018-10-08  2:23   ` [PATCH 02/10] drm/amd/display: dc/dce: add DCE6 support Mauro Rossi
2018-10-08  2:23   ` [PATCH 03/10] drm/amd/display: dc/core: " Mauro Rossi
2018-10-08  2:23   ` [PATCH 04/10] drm/amd/display: dc/bios: add support for DCE6 Mauro Rossi
2018-10-08  2:23   ` [PATCH 05/10] drm/amd/display: dc/gpio: " Mauro Rossi
2018-10-08  2:23   ` [PATCH 06/10] drm/amd/display: dc/i2caux: " Mauro Rossi
2018-10-08  2:23   ` [PATCH 07/10] drm/amd/display: dc/irq: " Mauro Rossi
2018-10-08  2:23   ` [PATCH 08/10] drm/amd/display: amdgpu_dm: add SI support Mauro Rossi
2018-10-08  2:23   ` [PATCH 09/10] drm/amdgpu: enable DC support for SI parts Mauro Rossi
2018-10-08  2:23   ` [PATCH 10/10] drm/amd/display: enable SI support in the Kconfig Mauro Rossi
2018-10-08 11:00   ` [RFC] drm/amd/display: add SI support to AMD DC Mike Lothian
     [not found]     ` <CAHbf0-HK4W4xE-hOJPiwr8zhzuuG2GobCTyHHik3mwe1-9_BmQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-10-08 11:22       ` Mauro Rossi
     [not found]         ` <CAEQFVGbWWy7jmcaserbMwANNHei90WX+1AvOfDAY8J=BcsyCrg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-10-08 11:29           ` Christian König
     [not found]             ` <7a8b5d6d-82c2-2b98-b2b2-098baf095aef-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2018-10-08 12:16               ` Mike Lothian
     [not found]                 ` <CAHbf0-FB2GV18igVo-8MHcVGL89KZoXn+O2B4asoe5R4RbgCVw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-10-08 15:47                   ` Deucher, Alexander
2018-10-08 12:04   ` sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w
2018-10-08 12:32     ` sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w
2018-10-08 17:02     ` Mauro Rossi
     [not found]       ` <CAEQFVGahx4U+52uKu20_q0iCPrdzeW8G+viS7p2LJtgF61bf6Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-10-08 20:17         ` sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w
2018-10-08 21:22           ` sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w
2018-10-14 21:47             ` Mauro Rossi
     [not found]               ` <CAEQFVGaErupy3y+sKA+uqQPn7x0oL1T9BKWj6y8EC12Ap2-YDw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-10-15  1:25                 ` sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w
2018-10-15  5:28                   ` Mauro Rossi
     [not found]                     ` <CAEQFVGbB_ezGSGwPu2Ka-4rY9RjB_rJvPL8ZCEG-_rfXxOEN-A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-10-15 12:45                       ` sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w
2018-10-15 17:53                         ` Deucher, Alexander [this message]
2018-10-15 21:06                 ` Harry Wentland
     [not found]                   ` <bef5787e-cc8d-df35-dc55-353ed4443a8c-5C7GfCeVMHo@public.gmane.org>
2018-10-15 21:19                     ` Harry Wentland
     [not found]                       ` <70b01042-3210-dcce-2b9a-a16754db9f10-5C7GfCeVMHo@public.gmane.org>
2018-10-16 12:20                         ` Mauro Rossi
2018-10-16 14:48                     ` Mauro Rossi

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=BN6PR12MB18092365083B2653B547057AF7FD0@BN6PR12MB1809.namprd12.prod.outlook.com \
    --to=alexander.deucher-5c7gfcevmho@public.gmane.org \
    --cc=Harry.Wentland-5C7GfCeVMHo@public.gmane.org \
    --cc=amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
    --cc=issor.oruam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=mike-4+n8WJKc9ve9FHfhHBbuYA@public.gmane.org \
    --cc=sylvain.bertrand-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.