2021-10-20 08:51:48

by Maulik Shah

Subject: [PATCH] soc: qcom: qcom_stats: Fix client votes offset

Client votes starts at 0x20 offset. Correct the offset.

Reported-and-suggested-by: Shawn Guo <[email protected]>
Fixes: 1d7724690344 ("soc: qcom: Add Sleep stats driver")
Signed-off-by: Maulik Shah <[email protected]>
---
drivers/soc/qcom/qcom_stats.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/soc/qcom/qcom_stats.c b/drivers/soc/qcom/qcom_stats.c
index 817505b..131d24c 100644
--- a/drivers/soc/qcom/qcom_stats.c
+++ b/drivers/soc/qcom/qcom_stats.c
@@ -22,7 +22,7 @@
#define LAST_ENTERED_AT_OFFSET 0x8
#define LAST_EXITED_AT_OFFSET 0x10
#define ACCUMULATED_OFFSET 0x18
-#define CLIENT_VOTES_OFFSET 0x1c
+#define CLIENT_VOTES_OFFSET 0x20

struct subsystem_data {
const char *name;
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation

2021-10-20 13:11:40

by Shawn Guo

Subject: Re: [PATCH] soc: qcom: qcom_stats: Fix client votes offset

On Wed, Oct 20, 2021 at 02:19:13PM +0530, Maulik Shah wrote:
> Client votes starts at 0x20 offset. Correct the offset.
>
> Reported-and-suggested-by: Shawn Guo <[email protected]>
> Fixes: 1d7724690344 ("soc: qcom: Add Sleep stats driver")
> Signed-off-by: Maulik Shah <[email protected]>

Reviewed-by: Shawn Guo <[email protected]>

> ---
> drivers/soc/qcom/qcom_stats.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/soc/qcom/qcom_stats.c b/drivers/soc/qcom/qcom_stats.c
> index 817505b..131d24c 100644
> --- a/drivers/soc/qcom/qcom_stats.c
> +++ b/drivers/soc/qcom/qcom_stats.c
> @@ -22,7 +22,7 @@
> #define LAST_ENTERED_AT_OFFSET 0x8
> #define LAST_EXITED_AT_OFFSET 0x10
> #define ACCUMULATED_OFFSET 0x18
> -#define CLIENT_VOTES_OFFSET 0x1c
> +#define CLIENT_VOTES_OFFSET 0x20
>
> struct subsystem_data {
> const char *name;
> --
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
> of Code Aurora Forum, hosted by The Linux Foundation
>