From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 68A9BCCFA00 for ; Tue, 4 Nov 2025 06:42:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:Message-Id:Date:Subject:Cc:To:From:Reply-To: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=ibEwJwxRJUdRY1eR4hz3JT7HtcnamXz9ZfIAYm73UCI=; b=ANZY8xE0IRBWaf5BzwjHydolt+ WIUfzyK9qoxpIsBJJSjVUPUSaPKh5mbkgMPmM5wfcOaTWYd7lZQES0gCKVee2GjeIueMHCBcKI81t XqwOG3us4MrSQ4SRPA9KXJZTuRzmpKeClw0fvGZjkOh+/HrtgM5NR4u4V75k8kJH5wwycUerXCkSw llbQcpuapqvVAWW1X/V8loa+Xb/Mn6P52BUfJ7H5+IZYErYB+Q63Zp7KRGFZav9VYb+t6Mdy9Bbik UGi2z9QsVc7Xyekd7zR4yOOlZOdJd9oHYzHkIm48GGl7wG9+HIdf3IgKeqnwoxnpi/6FQCCi3fDo5 evejkJnw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vGAju-0000000BFM1-24ZM; Tue, 04 Nov 2025 06:41:58 +0000 Received: from mx0a-0031df01.pphosted.com ([205.220.168.131]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vGAjs-0000000BFLg-08uH for linux-arm-kernel@lists.infradead.org; Tue, 04 Nov 2025 06:41:57 +0000 Received: from pps.filterd (m0279864.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 5A43W33a3595027 for ; Tue, 4 Nov 2025 06:41:55 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:message-id :mime-version:subject:to; s=qcppdkim1; bh=ibEwJwxRJUdRY1eR4hz3JT 7HtcnamXz9ZfIAYm73UCI=; b=mkJRlasCMb5aqPK4kb+KTOb1KOrxIilShCrkAT YmoH6r1BoLzt6vA3GD8o9adML8ck/fxCpTXfA6bhPvhvv59k8kLghKrjF+QzaQsw OY7CSdPz1jgvDWPyVgT7D0ifWV3vSO40F0WSQhlk7dnq43jGGZuGdJQQky1fG2vM +Ept8f2bYuBjRq0zIIL0S9vol5Dzq59wESzxFZQeSBnXNnA211oflgCnYOcCIIRs OV3PRzNCsR0pd9xopLIWUGkpxN3DY1fehki5HPcDNX/NPpEqsxOkCNKW8cWS5Z23 IxG35LSW68ukz2WG6PgfCFlQbO4oP3ad27S5nYe5yTlnDCwg== Received: from mail-pj1-f69.google.com (mail-pj1-f69.google.com [209.85.216.69]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4a70exhxay-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Tue, 04 Nov 2025 06:41:55 +0000 (GMT) Received: by mail-pj1-f69.google.com with SMTP id 98e67ed59e1d1-3416dc5754fso1107443a91.1 for ; Mon, 03 Nov 2025 22:41:55 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1762238514; x=1762843314; darn=lists.infradead.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=ibEwJwxRJUdRY1eR4hz3JT7HtcnamXz9ZfIAYm73UCI=; b=jTeiDGq2XIQrJSwkgQAYEXMSFSGZp+sh8CkXOAtO2jwKu1I3Xz7zk6EIk71BRWaOzh N+iY42rH6e/iUivvuXAsV/HxqUH7475ObKhqi5X6a/SroSaOfn+PBJWF9aRtn9qdxBoV t3w6jezM+eK7h0eoN9mQpwUVqyL0fNOZ84HtCmD9U4rpSm8cJFCY6/7tqWYpwturBkZ6 9f8i0j4q23NDnbh46K3pMLIoI651e0gSOqMp6e/8Sd9wypjaXm0IlHIP+hy9Yks6jnl7 ku5oJ4J5XkYXMujL6fZbNqjsB8FelzzY7Floxxw+lOjZ+xfIh2Uh9qix2ZHgTPCWtKAj cz6A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762238514; x=1762843314; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=ibEwJwxRJUdRY1eR4hz3JT7HtcnamXz9ZfIAYm73UCI=; b=OCL3Gw5Rqo4gm6mMxA4dLEXiV3PbFf7wmXQb6CGy5C5ssukFuiAANtga64e7o5zhsD JJsSv/26saU/toR2SQipz3j/NufBBbjRtb3DSpOjxFv/4v+SbHY4Lx/ld1sATthCZ3hj 9Dv4INxnYilrMg6RjmP8cy3ayCqDbGDUmeVyI/+80X3YCNWYOxBXBNIjg4Ccg54p6E45 yVp3cjpykDXO4Fs4qnxWs6BsF15fFB2l5e5nYsO0QtbwtKHZAtFmfhXWFWSJuZEY4tZR YnvqY4lIcG2UqFPYVe50WhekLrrgQTzakzZvrgnzWPanPwBmB6BGeJbLAMwJq0s2O4/b 9sWw== X-Forwarded-Encrypted: i=1; AJvYcCXeB3fOlwPcx6pOcI+xXZ0FFyLYnXy5hFGyXkLm+1O/dpHgPdCXvImQVYCTWkcVZEOPeFzM+uEkIZV3g40nqpTd@lists.infradead.org X-Gm-Message-State: AOJu0Yz7/3zhmslnQRwyueicB+HUBzElSfFNosbT4FzxxrhgXjj0cR6n EO4NEIBJWgyd67HbCc2RgyeLQRLs2cUbmagBz4cW6bOc8lazkD0Gbc1oYebJvYjLGWLbkG/Z2es IpKgfz4d9ruxzeeKc0y0z2azVe+G+U57bnFZD9w+PE5bHkYpr8FFWIQg8VVp5PuWqByKDg+miFf bNCQ== X-Gm-Gg: ASbGncs/1vqjFl3xASB5MMOju1uPRvk2tOgn46E+vmPCsS1UQQH4afB3iGkjwrvaEHW zPL26kofU7efSbdySfjMevbKS2Qfiuf52P6ZX9x0z4I4MdlkT7LKlYt5rh83WT9Uu6r0OfhROJL MNlSjtTrn0QyssXJRCpNiqw6CkDhQlJMRHQUEi91vdXble7MkmziMZmh1Rcjh6VmNC8BFu2IxIo qXP+mhDq+rct0AC+Ifo6T4UFJKnGIG6e9gG9UehC7kIm7Ken+Z/1GD89pAPMPgerkG8VF/iWIcq a9aKDPS2aN/rB9m/I+4txt9nsp5KPenkT2rLjQ9NjL9UCMR8NtwxHWuV2BVBXDi0S8dzWaELszm b20QMXrLSJpTb2L2k4bfoyMS3K3UO7qRxIYjSFMAKlfrt9U9KYp+RvQ== X-Received: by 2002:a17:90b:2789:b0:341:88ba:bdda with SMTP id 98e67ed59e1d1-34188babf0emr408682a91.31.1762238513941; Mon, 03 Nov 2025 22:41:53 -0800 (PST) X-Google-Smtp-Source: AGHT+IGBKvCkpW9aHNOj9emv5P1GELGKEfol3zMzYWEVGCEYS1Bh6AYCS0DUS3V9u6Ui+sDCp/uJsQ== X-Received: by 2002:a17:90b:2789:b0:341:88ba:bdda with SMTP id 98e67ed59e1d1-34188babf0emr408656a91.31.1762238513257; Mon, 03 Nov 2025 22:41:53 -0800 (PST) Received: from hu-songchai-lv.qualcomm.com (Global_NAT1.qualcomm.com. [129.46.96.20]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-34159a0780dsm3294791a91.16.2025.11.03.22.41.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 03 Nov 2025 22:41:52 -0800 (PST) From: Songwei Chai To: andersson@kernel.org, alexander.shishkin@linux.intel.com, kernel@oss.qualcomm.com, mike.leach@linaro.org, suzuki.poulose@arm.com Cc: Songwei Chai , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, coresight@lists.linaro.org, devicetree@vger.kernel.org Subject: [PATCH v7 0/7] Provides support for Trigger Generation Unit Date: Mon, 3 Nov 2025 22:40:36 -0800 Message-Id: <20251104064043.88972-1-songwei.chai@oss.qualcomm.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Authority-Analysis: v=2.4 cv=APHuRV3Y c=1 sm=1 tr=0 ts=6909a033 cx=c_pps a=vVfyC5vLCtgYJKYeQD43oA==:117 a=ouPCqIW2jiPt+lZRy3xVPw==:17 a=IkcTkHD0fZMA:10 a=6UeiqGixMTsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=VwQbUJbxAAAA:8 a=pGLkceISAAAA:8 a=EUspDBNiAAAA:8 a=COk6AnOGAAAA:8 a=Y4MViknzB6XciWjnlzcA:9 a=8KGQzfimsMkVHzi2:21 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=rl5im9kqc5Lf4LNbBjHf:22 a=TjNXssC_j7lpFel5tvFf:22 X-Proofpoint-GUID: h3blnpHqhVcnbg23seuaET3XfCJaxxkx X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMTA0MDA1MyBTYWx0ZWRfX11P4IO+bLGbi hx0C4LIYYcWER9ctpcAzroGnhHaGUoHoB0HWYvSPwIM7Bjdnnc+1hhAPiIgW0wkwCECl9pZX1LG n+M7Rn3WDDxplUCLEeS1TKPm5kxF6nnTGm+Qe4HyARVsVCYhP5P7Wfka+PozVnwjRXZEXFjUk5g zyYHSKvhDWFkJbSbuPMawqSevl2aP1n/EGjcDHwTM+dON2NLADLtkr47sGLrebco8DxeDR2RIEZ lAh7fbxVcRhSoMlY+omHbiPUqY5OwnKZbVzCWJN8vvmM9I0+LwyD6Lz5X3LH/ifjCZ8oaBWcPjl zgcHMQsgopxdMNPZg5t5jc0vsKRFQ6bnmJk4tE8C5+kpIH5IJXES4+b4uwdnad77yCflQ0DzXmw 3pH3uHbZzgcWnOloUH+mNNvZaTVZ0A== X-Proofpoint-ORIG-GUID: h3blnpHqhVcnbg23seuaET3XfCJaxxkx X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.100.49 definitions=2025-11-03_06,2025-11-03_03,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 malwarescore=0 clxscore=1015 impostorscore=0 priorityscore=1501 phishscore=0 suspectscore=0 adultscore=0 bulkscore=0 spamscore=0 lowpriorityscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2510240001 definitions=main-2511040053 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251103_224156_092269_D175D753 X-CRM114-Status: GOOD ( 20.35 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org We propose creating a new qcom directory under drivers/hwtracing to host this TGU driver, as well as additional Qualcomm-specific hwtracing drivers that we plan to submit in the coming months. This structure will help organize vendor-specific implementations and facilitate future development and maintenance. Feedback from the community on this proposal is highly appreciated. - Why we are proposing this: TGU has the ability to monitor signal conditions and trigger debug-related actions, serving as a programmable hardware component that enhances system trace and debug capabilities. Placing it under drivers/hwtracing aligns with its function as a trace generation utility. We previously attempted to push this driver to drivers/hwtracing/coresight, but did not receive support from the maintainers of the CoreSight subsystem. The reason provided was: “This component is primarily a part of the Qualcomm proprietary QPMDA subsystem, and is capable of operating independently from the CoreSight hardware trace generation system.” Chat history : https://lore.kernel.org/all/CAJ9a7ViKxHThyZfFFDV_FkNRimk4uo1NrMtQ-kcaj1qO4ZcGnA@mail.gmail.com/ Given this, we have been considering whether it would be appropriate to create a dedicated drivers/hwtracing/qcom directory for Qualcomm-related hwtracing drivers. This would follow the precedent set by Intel, which maintains its own directory at drivers/hwtracing/intel_th. We believe this structure would significantly facilitate future submissions of related Qualcomm drivers. - Maintenance of drivers/hwtracing/qcom: Bjorn, who maintains linux-arm-msm, will be the maintainer of this directory — we’ve discussed this with him and he’s aware that his task list may grow accordingly. Additionally, Qualcomm engineers familiar with the debug hardware — such as [Tingwei Zhang, Jinlong Mao, Songwei Chai], will be available to review incoming patches and support ongoing development. - Detail for TGU: This component can be utilized to sense a plurality of signals and create a trigger into the CTI or generate interrupts to processors once the input signal meets the conditions. We can treat the TGU’s workflow as a flowsheet, it has some “steps” regions for customization. In each step region, we can set the signals that we want with priority in priority_group, set the conditions in each step via condition_decode, and set the resultant action by condition_select. Meanwhile, some TGUs (not all) also provide timer/counter functionality. Based on the characteristics described above, we consider the TGU as a helper in the CoreSight subsystem. Its master device is the TPDM, which can transmit signals from other subsystems, and we reuse the existing ports mechanism to link the TPDM to the connected TGU. Here is a detailed example to explain how to use the TGU: In this example, the TGU is configured to use 2 conditions, 2 steps, and the timer. The goal is to look for one of two patterns which are generated from TPDM, giving priority to one, and then generate a trigger once the timer reaches a certain value. In other words, two conditions are used for the first step to look for the two patterns, where the one with the highest priority is used in the first condition. Then, in the second step, the timer is enabled and set to be compared to the given value at each clock cycle. These steps are better shown below. |-----------------| | | | TPDM | | | |-----------------| | | --- --- --- --- --- --- --- --- --- --- --- --- --- --- --- --- ------ | | | | | |--------------------| | | |---- ---> | | Go to next steps | | | | | |--- ---> | Enable timer | | | | v | | | | | | |-----------------| | |--------------------| | | | | | Yes | | | | | | inputs==0xB | ----->| | <-------- | | | | | | | No | | | No | |-----------------| | v | | | | | | |-----------------| | | | | | | | | | | | | | | | timer>=3 |-- | | | v | | | | | | |-----------------| | |-----------------| | | | | | Yes | | | | |--- | inputs==0xA | ----->| | Yes | | | | | | | |-----------------| v | | |-----------------| | | | | | | | Trigger | | | | | | | |-----------------| | | TGU | | |--- --- --- --- --- --- --- --- --- --- --- --- --- --- |--- --- -- | | v |-----------------| |The controllers | |which will use | |triggers further | |-----------------| steps: 1. Reset TGU /*it will disable tgu and reset dataset*/ - echo 1 > /sys/bus/coresight/devices//reset_tgu 2. Set the pattern match for priority0 to 0xA = 0b1010 and for priority 1 to 0xB = 0b1011. - echo 0x11113232 > /sys/bus/coresight/devices//step0_priority0/reg0 - echo 0x11113233 > /sys/bus/coresight/devices//step0_priority1/reg0 Note: Bit distribution diagram for each priority register |-------------------------------------------------------------------| | Bits | Field Nam | Description | |-------------------------------------------------------------------| | | | 00 = bypass for OR output | | 29:28 | SEL_BIT7_TYPE2 | 01 = bypass for AND output | | | | 10 = sense input '0' is true| | | | 11 = sense input '1' is true| |-------------------------------------------------------------------| | | | 00 = bypass for OR output | | 25:24 | SEL_BIT6_TYPE2 | 01 = bypass for AND output | | | | 10 = sense input '0' is true| | | | 11 = sense input '1' is true| |-------------------------------------------------------------------| | | | 00 = bypass for OR output | | 21:20 | SEL_BIT5_TYPE2 | 01 = bypass for AND output | | | | 10 = sense input '0' is true| | | | 11 = sense input '1' is true| |-------------------------------------------------------------------| | | | 00 = bypass for OR output | | 17:16 | SEL_BIT4_TYPE2 | 01 = bypass for AND output | | | | 10 = sense input '0' is true| | | | 11 = sense input '1' is true| |-------------------------------------------------------------------| | | | 00 = bypass for OR output | | 13:12 | SEL_BIT3_TYPE2 | 01 = bypass for AND output | | | | 10 = sense input '0' is true| | | | 11 = sense input '1' is true| |-------------------------------------------------------------------| | | | 00 = bypass for OR output | | 9:8 | SEL_BIT2_TYPE2 | 01 = bypass for AND output | | | | 10 = sense input '0' is true| | | | 11 = sense input '1' is true| |-------------------------------------------------------------------| | | | 00 = bypass for OR output | | 5:4 | SEL_BIT1_TYPE2 | 01 = bypass for AND output | | | | 10 = sense input '0' is true| | | | 11 = sense input '1' is true| |-------------------------------------------------------------------| | | | 00 = bypass for OR output | | 1:0 | SEL_BIT0_TYPE2 | 01 = bypass for AND output | | | | 10 = sense input '0' is true| | | | 11 = sense input '1' is true| |-------------------------------------------------------------------| These bits are used to identify the signals we want to sense, with a maximum signal number of 140. For example, to sense the signal 0xA (binary 1010), we set the value of bits 0 to 13 to 3232, which represents 1010. The remaining bits are set to 1, as we want to use AND gate to summarize all the signals we want to sense here. For rising or falling edge detection of any input to the priority, set the remaining bits to 0 to use an OR gate. 3. look for the pattern for priority_i i=0,1. - echo 0x3 > /sys/bus/coresight/devices//step0_condition_decode/reg0 - echo 0x30 > /sys/bus/coresight/devices//step0_condition_decode/reg1 |-------------------------------------------------------------------------------| | Bits | Field Nam | Description | |-------------------------------------------------------------------------------| | | |For each decoded condition, this | | 24 | NOT |inverts the output. If the condition | | | |decodes to true, and the NOT field | | | |is '1', then the output is NOT true. | |-------------------------------------------------------------------------------| | | |When '1' the output from the associated| | 21 | BC0_COMP_ACTIVE |comparator will be actively included in| | | |the decoding of this particular | | | |condition. | |-------------------------------------------------------------------------------| | | |When '1' the output from the associated| | | |comparator will need to be 1 to affect | | 20 | BC0_COMP_HIGH |the decoding of this condition. | | | |Conversely, a '0' here requires a '0' | | | |from the comparator | |-------------------------------------------------------------------------------| | | |When '1' the output from the associated| | 17 | |comparator will be actively included in| | | TC0_COMP_ACTIVE |the decoding of this particular | | | |condition. | |-------------------------------------------------------------------------------| | | |When '1' the output from the associated| | | |comparator will need to be 1 to affect | | 16 | TC0_COMP_HIGH |the decoding of this particular | | | |condition.Conversely, a 0 here | | | |requires a '0' from the comparator | |-------------------------------------------------------------------------------| | | |When '1' the output from Priority_n | | | |OR logic will be actively | | 4n+3 | Priority_n_OR_ACTIVE|included in the decoding of | | | (n=0,1,2,3) |this particular condition. | | | | | |-------------------------------------------------------------------------------| | | |When '1' the output from Priority_n | | | |will need to be '1' to affect the | | 4n+2 | Priority_n_OR_HIGH |decoding of this particular | | | (n=0,1,2,3) |condition. Conversely, a '0' here | | | |requires a '0' from Priority_n OR logic| |-------------------------------------------------------------------------------| | | |When '1' the output from Priority_n | | | |AND logic will be actively | | 4n+1 |Priority_n_AND_ACTIVE|included in the decoding of this | | | (n=0,1,2,3) |particular condition. | | | | | |-------------------------------------------------------------------------------| | | |When '1' the output from Priority_n | | | |AND logic will need to be '1' to | | 4n | Priority_n_AND_HIGH |affect the decoding of this | | | (n=0,1,2,3) |particular condition. Conversely, | | | |a '0' here requires a '0' from | | | |Priority_n AND logic. | |-------------------------------------------------------------------------------| Since we use `priority_0` and `priority_1` with an AND output in step 2, we set `0x3` and `0x30` here to activate them. 4. Set NEXT_STEP = 1 and TC0_ENABLE = 1 so that when the conditions are met then the next step will be step 1 and the timer will be enabled. - echo 0x20008 > /sys/bus/coresight/devices//step0_condition_select/reg0 - echo 0x20008 > /sys/bus/coresight/devices//step0_condition_select/reg1 |-----------------------------------------------------------------------------| | Bits | Field Nam | Description | |-----------------------------------------------------------------------------| | | |This field defines the next step the | | 18:17 | NEXT_STEP |TGU will 'goto' for the associated | | | |Condition and Step. | |-----------------------------------------------------------------------------| | | |For each possible output trigger | | 13 | TRIGGER |available, set a '1' if you want | | | |the trigger to go active for the | | | |associated condition and Step. | |-----------------------------------------------------------------------------| | | |This will cause BC0 to increment if the| | 9 | BC0_INC |associated Condition is decoded for | | | |this step. | |-----------------------------------------------------------------------------| | | |This will cause BC0 to decrement if the| | 8 | BC0_DEC |associated Condition is decoded for | | | |this step. | |-----------------------------------------------------------------------------| | | |This will clear BC0 count value to 0 if| | 7 | BC0_CLEAR |the associated Condition is decoded | | | |for this step. | |-----------------------------------------------------------------------------| | | |This will cause TC0 to increment until | | 3 | TC0_ENABLE |paused or cleared if the associated | | | |Condition is decoded for this step. | |-----------------------------------------------------------------------------| | | |This will cause TC0 to pause until | | 2 | TC0_PAUSE |enabled if the associated Condition | | | |is decoded for this step. | |-----------------------------------------------------------------------------| | | |This will clear TC0 count value to 0 | | 1 | TC0_CLEAR |if the associated Condition is | | | |decoded for this step. | |-----------------------------------------------------------------------------| | | |This will set the done signal to the | | 0 | DONE |TGU FSM if the associated Condition | | | |is decoded for this step. | |-----------------------------------------------------------------------------| Based on the distribution diagram, we set `0x20008` for `priority0` and `priority1` to achieve "jump to step 1 and enable TC0" once the signal is sensed. 5. activate the timer comparison for this step. - echo 0x30000 > /sys/bus/coresight/devices//step1_condition_decode/reg0 |-------------------------------------------------------------------------------| | | |When '1' the output from the associated| | 17 | |comparator will be actively included in| | | TC0_COMP_ACTIVE |the decoding of this particular | | | |condition. | |-------------------------------------------------------------------------------| | | |When '1' the output from the associated| | | |comparator will need to be 1 to affect | | 16 | TC0_COMP_HIGH |the decoding of this particular | | | |condition.Conversely, a 0 here | | | |requires a '0' from the comparator | |-------------------------------------------------------------------------------| Accroding to the decode distribution diagram , we give 0x30000 here to set 16th&17th bit to enable timer comparison. 6. Set the NEXT_STEP = 0 and TC0_PAUSE = 1 and TC0_CLEAR = 1 once the timer has reached the given value. - echo 0x6 > /sys/bus/coresight/devices//step1_condition_select/reg0 7. Enable Trigger 0 for TGU when the condition 0 is met in step1, i.e. when the timer reaches 3. - echo 0x2000 > /sys/bus/coresight/devices//step1_condition_select/default Note: 1. 'default' register allows for establishing the resultant action for the default condition 2. Trigger:For each possible output trigger available from the Design document, there are three triggers: interrupts, CTI, and Cross-TGU mapping.All three triggers can occur, but the choice of which trigger to use depends on the user's needs. 8. Compare the timer to 3 in step 1. - echo 0x3 > /sys/bus/coresight/devices//step1_timer/reg0 9. enale tgu - echo 1 > /sys/bus/coresight/devices//enable_tgu --- Link to V6: https://lore.kernel.org/all/20250709104114.22240-1-songchai@qti.qualcomm.com/ Changes in V7: - Move the TGU code location from 'drivers/hwtracing/coresight/' to 'drivers/hwtracing/qcom/'. - Rename the spinlock used in the code from 'spinlock' to 'lock'. - Perform the 'calculate_array_location' separately, instead of doing it within the function. - Update the sender email address. --- Link to V5: https://lore.kernel.org/all/20250529081949.26493-1-quic_songchai@quicinc.com/ Changes in V6: - Replace spinlock with guard(spinlock) in tgu_enable. - Remove redundant blank line. - Update publish date and contact member's name in "sysfs-bus-coresight-devices-tgu". --- Link to V4: https://patchwork.kernel.org/project/linux-arm-msm/cover/20250423101054.954066-1-quic_songchai@quicinc.com/ Changes in V5: - Update publish date and kernel_version in "sysfs-bus-coresight-devices-tgu" --- Link to V3: https://lore.kernel.org/all/20250227092640.2666894-1-quic_songchai@quicinc.com/ Changes in V4: - Add changlog in coverletter. - Correct 'year' in Copyright in patch1. - Correct port mechansim description in patch1. - Remove 'tgu-steps','tgu-regs','tgu-conditions','tgu-timer-counters' from dt-binding and set them through reading DEVID register as per Mike's suggestion. - Modify tgu_disable func to make it have single return point in patch2 as per Mike's suggestion. - Use sysfs_emit in enable_tgu_show func in ptach2. - Remove redundant judgement in enable_tgu_store in patch2. - Correct typo in description in patch3. - Set default ret as SYSFS_GROUP_INVISIBLE, and returnret at end in pacth3 as per Mike's suggestion. - Remove tgu_dataset_ro definition in patch3 - Use #define constants with explanations of what they are rather than arbitrary magic numbers in patch3 and patch4. - Check -EINVAL before using 'calculate_array_location()' in array in patch4. - Add 'default' in 'tgu_dataset_show''s switch part in patch4. - Document the value needed to initiate the reset in pacth7. - Check "value" in 'reset_tgu_store' and bail out with an error code if 0 in patch7. - Remove dev_dbg in 'reset_tgu_store' in patch7. --- Link to V2: https://lore.kernel.org/all/20241010073917.16023-1-quic_songchai@quicinc.com/ Changes in V3: - Correct typo and format in dt-binding in patch1 - Rebase to the latest kernel version --- Link to V1: https://lore.kernel.org/all/20240830092311.14400-1-quic_songchai@quicinc.com/ Changes in V2: - Use real name instead of login name, - Correct typo and format in dt-binding and code. - Bring order in tgu_prob(declarations with and without assignments) as per Krzysztof's suggestion. - Add module device table in patch2. - Set const for tgu_common_grp and tgu_ids in patch2. - Initialize 'data' in tgu_ids to fix the warning in pacth2. --- Songwei Chai (7): dt-bindings: arm: Add support for Qualcomm TGU trace qcom-tgu: Add TGU driver qcom-tgu: Add signal priority support qcom-tgu: Add TGU decode support qcom-tgu: Add support to configure next action qcom-tgu: Add timer/counter functionality for TGU qcom-tgu: Add reset node to initialize .../testing/sysfs-bus-coresight-devices-tgu | 51 ++ .../devicetree/bindings/arm/qcom,tgu.yaml | 82 ++ drivers/Makefile | 1 + drivers/hwtracing/Kconfig | 2 + drivers/hwtracing/qcom/Kconfig | 18 + drivers/hwtracing/qcom/Makefile | 3 + drivers/hwtracing/qcom/tgu.c | 737 ++++++++++++++++++ drivers/hwtracing/qcom/tgu.h | 252 ++++++ 8 files changed, 1146 insertions(+) create mode 100644 Documentation/ABI/testing/sysfs-bus-coresight-devices-tgu create mode 100644 Documentation/devicetree/bindings/arm/qcom,tgu.yaml create mode 100644 drivers/hwtracing/qcom/Kconfig create mode 100644 drivers/hwtracing/qcom/Makefile create mode 100644 drivers/hwtracing/qcom/tgu.c create mode 100644 drivers/hwtracing/qcom/tgu.h -- 2.34.1