Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp195745pxb; Fri, 29 Oct 2021 08:12:36 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyOl/lBpl1ndlCjgsTDgIK/MYry2yqqjxA4A+EFwaIG88yzwPCG8mLdRD3h9oKq5Ma2Lbis X-Received: by 2002:a92:6408:: with SMTP id y8mr8519415ilb.34.1635520356431; Fri, 29 Oct 2021 08:12:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1635520356; cv=none; d=google.com; s=arc-20160816; b=lkeissSjuQLjylIpeBsiKL+ARfO7w/LUAYeOIQNS/bgilUHEMmo17lG+eFz6um/W5M JWdWWZnmTeesmmXE1H67NUHyUINmTJBAFdpjrfquZkF568orujw723ZlgvaudGBBdMou WUhmd+NpoxfoEj7PHTFBsNDzvhly09G+O2lpUk4lfoD7/cOC6fg73FxyVRLc+7aabau3 IXHSquc4vSVUca3yQFfnBHplKvbVM7NEUUvJX4VwPjgBoyN3vxb16qfDF25jih82o9Aa ddUkE6vBsg03VpqGfm0QTlOirLZxeL6k6NtInaaD3dHAE2pRLv0x182jVNIMp7z9wXA7 Fvvw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=faQ2ykjv+znBZqcR3N4GdcbHIMEaYM0Hj3gqOcXIfys=; b=q4Azs+qX8T9zhza8fHOzSDGXnu2Dv3py+oG5CirukrN01Ig80F6HDFLQuO2UK5LXrX mgo8qAmF89bnbc+Qcp/fj/1e810iyiX60520Awm4r7adBt1854pU27kwcIvXg3ab/zgz nE3T+XCdiFI8AeV3VcB02bbwja2wCDqbpIe2SBbgo9PwCLaIiWStx/X+9u2UBND02f2U 5Bs8ifvzgLTtYteGl4g6vnKKQAo/Hm4PAUUsk61eB2dedq5OPVW3+hOu65AkJH7++Wah /1j9l9Ock3uqw3Ux/dqr4jP9h1iHDvKCWKTxrfADMJEakugu1y7gQDiz4d4+GqNT6cIn LG0Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcdkim header.b=HnTkKPfL; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id y13si10418994jat.0.2021.10.29.08.12.19; Fri, 29 Oct 2021 08:12:36 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcdkim header.b=HnTkKPfL; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229807AbhJ2PNr (ORCPT + 99 others); Fri, 29 Oct 2021 11:13:47 -0400 Received: from alexa-out.qualcomm.com ([129.46.98.28]:64068 "EHLO alexa-out.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229542AbhJ2PNq (ORCPT ); Fri, 29 Oct 2021 11:13:46 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; i=@quicinc.com; q=dns/txt; s=qcdkim; t=1635520278; x=1667056278; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=faQ2ykjv+znBZqcR3N4GdcbHIMEaYM0Hj3gqOcXIfys=; b=HnTkKPfLylGeV+5bGX9TOraOXdD/ms+2ghDpFLwk7pYQOfwWjj6VX62t fWeDINuD+dMn8CqwaYp3zjQ+udWMvm2ZC2/ozdJTGnJqF4ORUpKY34jFB Oi8pDmoXewdy+Jqa2v3wVav/Lt3uHOCa3D+/WokBFa9iLB0CyDDXSvuOE 0=; Received: from ironmsg09-lv.qualcomm.com ([10.47.202.153]) by alexa-out.qualcomm.com with ESMTP; 29 Oct 2021 08:11:18 -0700 X-QCInternal: smtphost Received: from nasanex01c.na.qualcomm.com ([10.47.97.222]) by ironmsg09-lv.qualcomm.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Oct 2021 08:11:17 -0700 Received: from nalasex01c.na.qualcomm.com (10.47.97.35) by nasanex01c.na.qualcomm.com (10.47.97.222) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.922.7; Fri, 29 Oct 2021 08:11:17 -0700 Received: from taozha-gv.ap.qualcomm.com (10.80.80.8) by nalasex01c.na.qualcomm.com (10.47.97.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.922.7; Fri, 29 Oct 2021 08:11:13 -0700 Date: Fri, 29 Oct 2021 23:11:10 +0800 From: Tao Zhang To: Mathieu Poirier CC: Suzuki K Poulose , Alexander Shishkin , Mike Leach , Leo Yan , Greg Kroah-Hartman , , , , Tingwei Zhang , Mao Jinlong , Yuanfang Zhang , Trilok Soni Subject: Re: [PATCH 00/10] Add support for TPDM and TPDA Message-ID: <20211029151110.GA5081@taozha-gv.ap.qualcomm.com> References: <1634801936-15080-1-git-send-email-quic_taozha@quicinc.com> <20211028171615.GB4045120@p14s> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20211028171615.GB4045120@p14s> User-Agent: Mutt/1.5.24 (2015-08-30) X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: nasanex01a.na.qualcomm.com (10.52.223.231) To nalasex01c.na.qualcomm.com (10.47.97.35) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Oct 28, 2021 at 11:16:15AM -0600, Mathieu Poirier wrote: > I have started to review this set. It is substantial in size and as such will > take me several days, most likely weeks. > > Did you run checkpatch before sending this set out? If so, any reason why there > are still errors? > > Thanks, > Mathieu > > On Thu, Oct 21, 2021 at 03:38:46PM +0800, Tao Zhang wrote: Hi Mathieu, Sorry, Mathieu, I didn't run checkpatch script after adding DT binding docs to the patches. We will check the errors and fix them in the next version. Thanks, Tao > > This series adds support for the trace performance monitoring and > > diagnostics hardware (TPDM and TPDA). It is composed of two major > > elements. > > a) Changes for original coresight framework to support for TPDM and TPDA. > > b) Add driver code for TPDM and TPDA. > > > > Introduction of changes for original coresight framework > > a) Support TPDM as new coresight source. > > Since only STM and ETM are supported as coresight source originally. > > TPDM is a newly added coresight source. We need to change > > the original way of saving coresight path to support more types source > > for coresight driver. > > The following patch is to add support more coresight sources. > > coresight: add support to enable more coresight paths > > > > b) To support multi-port input and multi-port output for funnels > > In some cases, different TPDM hardware will be connected to the same > > funnel, but eventually they need to be linked to different TPDAs or > > funnels. This requires funnel to support multi-port input and multi-port > > output, and can specify which input port corresponds to which output port. > > Use property ?label? in the funnel?s configuration to point out from > > which input port to which output port. > > The following patch is to support multi-port input and multi-port output > > for funnels. > > coresight: funnel: add support for multiple output ports > > > > Introduction of TPDM and TPDA > > TPDM - The trace performance monitoring and diagnostics monitor or TPDM in > > short serves as data collection component for various dataset types > > specified in the QPMDA(Qualcomm performance monitoring and diagnostics > > architecture) spec. The primary use case of the TPDM is to collect data > > from different data sources and send it to a TPDA for packetization, > > timestamping and funneling. > > The following patch is to add driver for TPDM. > > Coresight: Add driver to support Coresight device TPDM > > Coresight: Enable BC and GPR for TPDM driver > > Coresight: Add interface for TPDM BC subunit > > Coresight: Enable and add interface for TPDM TC subunit > > Coresight: Enable DSB subunit for TPDM > > Coresight: Enable CMB subunit for TPDM > > > > TPDA - The trace performance monitoring and diagnostics aggregator or > > TPDA in short serves as an arbitration and packetization engine for the > > performance monitoring and diagnostics network as specified in the QPMDA > > (Qualcomm performance monitoring and diagnostics architecture) > > specification. The primary use case of the TPDA is to provide > > packetization, funneling and timestamping of Monitor data as specified > > in the QPMDA specification. > > The following patch is to add driver for TPDA. > > coresight: Add driver to support Coresight device TPDA > > > > The last patch of this series is a device tree modification, which add > > the TPDM and TPDA configuration to device tree for validating. > > ARM: dts: msm: Add TPDA and TPDM configuration to device > > > > Once this series patches are applied properly, the tpdm and tpda nodes > > should be observed at the coresight path /sys/bus/coresight/devices > > e.g. > > /sys/bus/coresight/devices # ls -l | grep tpd > > tpda0 -> ../../../devices/platform/soc@0/6004000.tpda/tpda0 > > tpdm0 -> ../../../devices/platform/soc@0/6844000.lpass.tpdm/tpdm0 > > > > We can use the commands are similar to the below to validate TPDMs. > > Enable coresight sink first. > > echo 1 > /sys/bus/coresight/devices/tpdm0/enable_source > > echo 1 > /sys/bus/coresight/devices/tpdm0/integration_test > > echo 2 > /sys/bus/coresight/devices/tpdm0/integration_test > > The test data will be collected in the coresight sink which is enabled. > > > > This series applies to coresight/next > > https://git.kernel.org/pub/scm/linux/kernel/git/coresight/linux.git?h=next > > > > Tao Zhang (10): > > coresight: add support to enable more coresight paths > > coresight: funnel: add support for multiple output ports > > Coresight: Add driver to support Coresight device TPDM > > Coresight: Enable BC and GPR for TPDM driver > > Coresight: Add interface for TPDM BC subunit > > Coresight: Enable and add interface for TPDM TC subunit > > Coresight: Enable DSB subunit for TPDM > > Coresight: Enable CMB subunit for TPDM > > coresight: Add driver to support Coresight device TPDA > > ARM: dts: msm: Add TPDA and TPDM support to DTS for RB5 > > > > .../bindings/arm/coresight-tpda.yaml | 169 + > > .../bindings/arm/coresight-tpdm.yaml | 86 + > > MAINTAINERS | 6 + > > arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 439 ++ > > drivers/hwtracing/coresight/Kconfig | 18 + > > drivers/hwtracing/coresight/Makefile | 2 + > > drivers/hwtracing/coresight/coresight-core.c | 175 +- > > .../hwtracing/coresight/coresight-platform.c | 8 + > > drivers/hwtracing/coresight/coresight-tpda.c | 828 ++++ > > drivers/hwtracing/coresight/coresight-tpdm.c | 4253 +++++++++++++++++ > > include/linux/coresight.h | 2 + > > 11 files changed, 5928 insertions(+), 58 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/arm/coresight-tpda.yaml > > create mode 100644 Documentation/devicetree/bindings/arm/coresight-tpdm.yaml > > create mode 100644 drivers/hwtracing/coresight/coresight-tpda.c > > create mode 100644 drivers/hwtracing/coresight/coresight-tpdm.c > > > > -- > > 2.17.1 > >