On Tue, Aug 11, 2026 at 06:02:42PM +0100, Leo Yan wrote:
On Sat, Jul 25, 2026 at 12:36:42PM +0100, Yeoreum Yun wrote:
The purpose of TRCSSCSRn register is to show status of the corresponding Single-shot Comparator Control and input supports.
I vaguely remember that we discussed the issue with always clearing drvdata::ss_status in hw_enable(), which might result in single-shot does not work properly in a perf session.
This issue is not introduced by this patch, and I think we agreed to address it later. Could you document it in the commit log so that we can understand and track the issue later.
Okay. I'll add this in commit message.
That means writable field's purpose for reset or restore from idle status not for configuration.
Therefore, exclude ss_status from drvdata->config and move it to drvdata.
Signed-off-by: Yeoreum Yun yeoreum.yun@arm.com
With above amending:
Reviewed-by: Leo Yan leo.yan@arm.com
Thanks!