On 11/8/2023 8:02 AM, Krishna Chaitanya Chundru wrote:
On 11/3/2023 10:42 AM, Viresh Kumar wrote:
On 02-11-23, 07:09, Bjorn Helgaas wrote:
On Thu, Nov 02, 2023 at 11:00:13AM +0530, Viresh Kumar wrote:
On 01-11-23, 17:17, Bjorn Helgaas wrote:
Can you expand "OPP" somewhere so we know what it stands for? I'm
sure everybody knows except me :)
It is "Operating Performance Points", defined here:
Documentation/power/opp.rst
Thanks; I meant in the subject or commit log of the next revision, of
course.
Yeah, I understood that. Krishna shall do it in next version I believe.
Hi All,
I will do this in my next patch both commit message and ICC voting
through OPP
got stuck in some other work, will try to send new series as soon as
possible.
- Krishna Chaitanya.
Hi Viresh,
Sorry for late response.
We calculate ICC BW voting based up on PCIe speed and PCIe width.
Right now we are adding the opp table based up on PCIe speed.
Each PCIe controller can support multiple lane configurations like x1,
x2, x4, x8, x16 based up on controller capability.
So for each GEN speed we need up to 5 entries in OPP table. This will
make OPP table very long.
It is best to calculate the ICC BW voting in the driver itself and apply
them through ICC driver.
Let me know your opinion on this.
Thanks & Regards,
Krishna Chaitanya.