Re: [v2 PATCH] cpufreq: powernv: Correctly parse the sign of pstates on POWER8 vs POWER9

2017-12-10 Thread Gautham R Shenoy
Hi Balbir, On Fri, Dec 08, 2017 at 02:44:40PM +1100, Balbir Singh wrote: > On Thu, Dec 7, 2017 at 4:59 PM, Gautham R. Shenoy > wrote: > > From: "Gautham R. Shenoy" > > > > On POWERNV platform, Pstates are 8-bit values. On POWER8 they are > > negatively numbered while on POWER9 they are positivel

Re: [v2 PATCH] cpufreq: powernv: Correctly parse the sign of pstates on POWER8 vs POWER9

2017-12-08 Thread Rafael J. Wysocki
On Fri, Dec 8, 2017 at 12:47 PM, Michael Ellerman wrote: > "Rafael J. Wysocki" writes: > >> On Thu, Dec 7, 2017 at 6:59 AM, Gautham R. Shenoy >> wrote: >>> From: "Gautham R. Shenoy" >>> >>> On POWERNV platform, Pstates are 8-bit values. On POWER8 they are >>> negatively numbered while on POWER9

Re: [v2 PATCH] cpufreq: powernv: Correctly parse the sign of pstates on POWER8 vs POWER9

2017-12-08 Thread Michael Ellerman
"Rafael J. Wysocki" writes: > On Thu, Dec 7, 2017 at 6:59 AM, Gautham R. Shenoy > wrote: >> From: "Gautham R. Shenoy" >> >> On POWERNV platform, Pstates are 8-bit values. On POWER8 they are >> negatively numbered while on POWER9 they are positively >> numbered. Thus, on POWER9, the maximum numb

Re: [v2 PATCH] cpufreq: powernv: Correctly parse the sign of pstates on POWER8 vs POWER9

2017-12-07 Thread Balbir Singh
On Thu, Dec 7, 2017 at 4:59 PM, Gautham R. Shenoy wrote: > From: "Gautham R. Shenoy" > > On POWERNV platform, Pstates are 8-bit values. On POWER8 they are > negatively numbered while on POWER9 they are positively > numbered. Thus, on POWER9, the maximum number of pstates could be as > high as 256

Re: [v2 PATCH] cpufreq: powernv: Correctly parse the sign of pstates on POWER8 vs POWER9

2017-12-07 Thread Rafael J. Wysocki
On Thu, Dec 7, 2017 at 6:59 AM, Gautham R. Shenoy wrote: > From: "Gautham R. Shenoy" > > On POWERNV platform, Pstates are 8-bit values. On POWER8 they are > negatively numbered while on POWER9 they are positively > numbered. Thus, on POWER9, the maximum number of pstates could be as > high as 256

[v2 PATCH] cpufreq: powernv: Correctly parse the sign of pstates on POWER8 vs POWER9

2017-12-06 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" On POWERNV platform, Pstates are 8-bit values. On POWER8 they are negatively numbered while on POWER9 they are positively numbered. Thus, on POWER9, the maximum number of pstates could be as high as 256. The current code interprets pstates as a signed 8-bit value. This