Hey Sudeep, On 8/14/23 14:55, Sudeep Holla wrote:
On Sat, Aug 12, 2023 at 02:18:18AM +0530, Sibi Sankar wrote:The perf power scale value would currently be reported as bogowatts if the platform firmware supports microwatt power scale and meets the perf major version requirements. Fix this by populating version information in the driver private data before the call to protocol attributes is made. CC: Chandra Sekhar Lingutla <quic_lingutla@xxxxxxxxxxx> Fixes: 3630cd8130ce ("firmware: arm_scmi: Add SCMI v3.1 perf power-cost in microwatts") Signed-off-by: Sibi Sankar <quic_sibis@xxxxxxxxxxx> --- drivers/firmware/arm_scmi/perf.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/firmware/arm_scmi/perf.c b/drivers/firmware/arm_scmi/perf.c index c0cd556fbaae..30dedd6ebfde 100644 --- a/drivers/firmware/arm_scmi/perf.c +++ b/drivers/firmware/arm_scmi/perf.c @@ -1080,6 +1080,8 @@ static int scmi_perf_protocol_init(const struct scmi_protocol_handle *ph)Please rebase any patch when posting upstream on upstream kernel tree. This definitely looks like some downstream tree. I will adjust and apply this time. This file never crossed 1000 line and this patch indicates otherwise, so I am sure Qcom has some downstream changes in this file now.
Thanks for taking time to review the patch but zero points for the ^^ deduction lol. The correct phrasing would be v6.5 rc6 hasn't crossed 1k lines but the patch was based on next-20230809 which has the perf level indexing mode support in addition :) - Sibi
