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 88B13C4332F for ; Mon, 14 Nov 2022 13:59:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=dCAHT8+w2L8iJw7BVKFCklWDrD2T2pVBUsXmzaOxUkU=; b=yYI8xLnAsFpQbESJFwHYhgdEfj sxWWy9DLCsnrt9EUgynn52FyUaLjynJtKTlE57Qk1WRMxnGLAs1twiMp/o0NKC9/eCNYlonKCiOVY JL7NBL16jodJDGf576KQU+9yagnhSl27SRF3fnZekXF0w4HoXYiEEfzE7Q3r5FGqX9WzPXU08UW9X rWl7NLipSwNCIfSkwL4FUYS0JUT5X1Cjm80lF+2uRknsTAsIwJJC8pc/OfII+Iemox6iDowUTfF+t ZfSo+DOI18cEH6icXBEj+eTaLU4ZQoZGMac8xW94KAr9WjfLCdh+J/6gZqJtDdJGN/uAHT36/9bn5 h70WHxiA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ouZxy-001TVt-V3; Mon, 14 Nov 2022 13:57:39 +0000 Received: from mail-pg1-x536.google.com ([2607:f8b0:4864:20::536]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ouZxu-001TTi-CQ for linux-arm-kernel@lists.infradead.org; Mon, 14 Nov 2022 13:57:36 +0000 Received: by mail-pg1-x536.google.com with SMTP id r18so10290991pgr.12 for ; Mon, 14 Nov 2022 05:57:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=8MC3h3qKaAq/POnx65xnRJ3C4NU/+8KjvTe7E05rjRU=; b=UHnHc6SNEKeAoxVofC95gqomBI8THBwyUZaxrtaJ7vtzBNyJZCaP/rJd1gemcrs+bX 1Rx/j7wYW4hFH9SrofOJS4Un2ZqfmmPP2c370T/zhmystqj3VPKQPtqGBKTd8VsEBqzO 6UZSpBFEU42oFz+rCUxKCEtyVKFn5lk5Ox9MXGi71tKv+g8jEARCUHcsgRrIF+aoX/tk Gmc96awJeQ+w3/i5fqXL/t4061UivqhfZUDqlnIYUskZt24+UKXqNyNS/aQ3jWU2kSgD ol+x6Ho3RXm1+AaenYR3aKen6xItOgW8ersfvjEccPS3ujQs2YtInUvmXYLUnhDMxurr ZABA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=8MC3h3qKaAq/POnx65xnRJ3C4NU/+8KjvTe7E05rjRU=; b=wNQ7znVwpLu53WmuwJt3S2CshGrLu/MquhM7BLi7jTJXveFCCThBFNjZmzFR4yuQrS KqpoW3pAFGKjYjM3orjViShasCm0UVraTpjxIBRiOt03aSCLxISZYclN8bS5r7+57bjr XVu+kDyk1ZxAs+ngrkPmKq6Fafbp/c8tGqhJAeHaF3xqG1lSsbOReECz3SPu5T345V+J Yw1cLSk1y7qjTmvOtEhLTKKcSFxjqbN9Jr2ZCBjDTGFJeMb9NJ9FE8jGVNtzKjaVeEdp K/7BAv4ArJdW5vFNph6rkLOjXH5vr4E3/wj4MTY1EUKPcK8XUzQm13SEYYHQlzREqH7+ dJAQ== X-Gm-Message-State: ANoB5pnyqGwrFJvU/oHkzDvTqY9O0hmFfoVw4/aLuSOQBSBa04VTyUMO JycLJBKK4fDojp0lYct+hxY= X-Google-Smtp-Source: AA0mqf7scsGfCFUG7z2vXhK08/UVTuUCikOxvkq4nKtJtqN0mEbylzyx0c4URy3Xe1lzIvuCleSO3A== X-Received: by 2002:a63:d0b:0:b0:438:a750:99b7 with SMTP id c11-20020a630d0b000000b00438a75099b7mr11652778pgl.605.1668434252783; Mon, 14 Nov 2022 05:57:32 -0800 (PST) Received: from debian.me (subs02-180-214-232-86.three.co.id. [180.214.232.86]) by smtp.gmail.com with ESMTPSA id i7-20020a170902c94700b00186dcc37e17sm7510781pla.210.2022.11.14.05.57.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 14 Nov 2022 05:57:32 -0800 (PST) Received: by debian.me (Postfix, from userid 1000) id 5F833103ED3; Mon, 14 Nov 2022 20:57:29 +0700 (WIB) Date: Mon, 14 Nov 2022 20:57:29 +0700 From: Bagas Sanjaya To: Junhao He Cc: mathieu.poirier@linaro.org, suzuki.poulose@arm.com, mike.leach@linaro.org, leo.yan@linaro.org, jonathan.cameron@huawei.com, john.garry@huawei.com, coresight@lists.linaro.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org, lpieralisi@kernel.org, linuxarm@huawei.com, yangyicong@huawei.com, liuqi115@huawei.com, f.fangjian@huawei.com, prime.zeng@hisilicon.com Subject: Re: [PATCH v13 2/2] Documentation: Add document for UltraSoc SMB drivers Message-ID: References: <20221114090316.63157-1-hejunhao3@huawei.com> <20221114090316.63157-3-hejunhao3@huawei.com> MIME-Version: 1.0 In-Reply-To: <20221114090316.63157-3-hejunhao3@huawei.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221114_055734_491279_18F104F1 X-CRM114-Status: GOOD ( 25.92 ) 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: , Content-Type: multipart/mixed; boundary="===============8013455174781780581==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============8013455174781780581== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="EIcrB5LfjrdPKhUG" Content-Disposition: inline --EIcrB5LfjrdPKhUG Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Nov 14, 2022 at 05:03:16PM +0800, Junhao He wrote: > diff --git a/Documentation/trace/coresight/ultrasoc-smb.rst b/Documentati= on/trace/coresight/ultrasoc-smb.rst > new file mode 100644 > index 000000000000..b7fe3f5c7f53 > --- /dev/null > +++ b/Documentation/trace/coresight/ultrasoc-smb.rst > @@ -0,0 +1,82 @@ > +.. SPDX-License-Identifier: GPL-2.0 > + > +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > +UltraSoc - HW Assisted Tracing on SoC > +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > + :Author: Qi Liu > + :Date: March 2022 > + > +Introduction > +------------ > + > +UltraSoc SMB is a per SCCL(Super CPU Cluster) hardware, and it provides a > +way to buffer and store CPU trace messages in a region of shared system > +memory. SMB is plugged as a coresight sink device and the corresponding > +trace generators (ETM) are plugged in as source devices. > + > +Sysfs files and directories > +--------------------------- > + > +The SMB devices appear on the existing coresight bus alongside the other > +coresight devices:: > + > + $# ls /sys/bus/coresight/devices/ > + ultra_smb0 ultra_smb1 ultra_smb2 ultra_smb3 > + > +The ``ultra_smb`` named SMB associated with SCCL.:: > + > + $# ls /sys/bus/coresight/devices/ultra_smb0 > + enable_sink mgmt > + $# ls /sys/bus/coresight/devices/ultra_smb0/mgmt > + buf_size buf_status read_pos write_pos > + > +*Key file items are:-* > + * ``read_pos``: Shows the value held by UltraSoc SMB Read Pointer reg= ister. > + * ``write_pos``: Shows the value held by UltraSoc SMB Write Pointer r= egister. > + * ``buf_status``: Shows the value held by UltraSoc SMB status registe= r. > + BIT(0) is zero means buffer is empty. > + * ``buf_size``: Shows the buffer size of each UltraSoc SMB device. The key list above doesn't look right, so I have applied the fixup: ---- >8 ---- diff --git a/Documentation/trace/coresight/ultrasoc-smb.rst b/Documentation= /trace/coresight/ultrasoc-smb.rst index b7fe3f5c7f53f7..5d0fa1a76b04d1 100644 --- a/Documentation/trace/coresight/ultrasoc-smb.rst +++ b/Documentation/trace/coresight/ultrasoc-smb.rst @@ -30,11 +30,12 @@ The ``ultra_smb`` named SMB associated with SCCL.:: $# ls /sys/bus/coresight/devices/ultra_smb0/mgmt buf_size buf_status read_pos write_pos =20 -*Key file items are:-* +Key file items are: + * ``read_pos``: Shows the value held by UltraSoc SMB Read Pointer regis= ter. * ``write_pos``: Shows the value held by UltraSoc SMB Write Pointer reg= ister. * ``buf_status``: Shows the value held by UltraSoc SMB status register. - BIT(0) is zero means buffer is empty. + BIT(0) is zero means buffer is empty. * ``buf_size``: Shows the buffer size of each UltraSoc SMB device. =20 Firmware Bindings > + > +Firmware Bindings > +--------------------------- > + > +SMB device is only supported with ACPI, and ACPI binding of SMB device > +describes SMB device indentifier, resource information and graph structu= re. > + > +SMB is identified by ACPI HID "HISI03A1", resource of device is declared= using > +the _CRS method. Each SMB must present two base address, the first one i= s the > +configuration base address of SMB device, the second one is the 32bits b= ase > +address of shared system memory. > + > +examples:: > + > + Device(USMB) { \ > + Name(_HID, "HISI03A1") \ > + Name(_CRS, ResourceTemplate() { \ > + QWordMemory (ResourceConsumer, , MinFixed, MaxFixed, NonCachea= ble, \ > + ReadWrite, 0x0, 0x95100000, 0x951FFFFF, 0x0, 0x100000) \ > + QWordMemory (ResourceConsumer, , MinFixed, MaxFixed, Cacheable= , \ > + ReadWrite, 0x0, 0x50000000, 0x53FFFFFF, 0x0, 0x4000000) \ > + }) \ > + Name(_DSD, Package() { \ > + ToUUID("ab02a46b-74c7-45a2-bd68-f7d344ef2153"), \ > + /* Use CoreSight Graph ACPI bindings to describe connections topology */ > + Package() { \ > + 0, \ > + 1, \ > + Package() { \ > + 1, \ > + ToUUID("3ecbc8b6-1d0e-4fb3-8107-e627f805c6cd"), \ > + 8, \ > + Package() {0x8, 0, \_SB.S00.SL11.CL28.F008, 0}, \ > + Package() {0x9, 0, \_SB.S00.SL11.CL29.F009, 0}, \ > + Package() {0xa, 0, \_SB.S00.SL11.CL2A.F010, 0}, \ > + Package() {0xb, 0, \_SB.S00.SL11.CL2B.F011, 0}, \ > + Package() {0xc, 0, \_SB.S00.SL11.CL2C.F012, 0}, \ > + Package() {0xd, 0, \_SB.S00.SL11.CL2D.F013, 0}, \ > + Package() {0xe, 0, \_SB.S00.SL11.CL2E.F014, 0}, \ > + Package() {0xf, 0, \_SB.S00.SL11.CL2F.F015, 0}, \ > + } \ > + } \ > + }) \ > + } The rest of wordings also read a rather weird. What about below instead? ---- >8 ----=20 diff --git a/Documentation/trace/coresight/ultrasoc-smb.rst b/Documentation= /trace/coresight/ultrasoc-smb.rst index 5d0fa1a76b04d1..eee32cbf90d2ea 100644 --- a/Documentation/trace/coresight/ultrasoc-smb.rst +++ b/Documentation/trace/coresight/ultrasoc-smb.rst @@ -9,21 +9,21 @@ UltraSoc - HW Assisted Tracing on SoC Introduction ------------ =20 -UltraSoc SMB is a per SCCL(Super CPU Cluster) hardware, and it provides a +UltraSoc SMB is a per SCCL (Super CPU Cluster) hardware. It provides a way to buffer and store CPU trace messages in a region of shared system -memory. SMB is plugged as a coresight sink device and the corresponding -trace generators (ETM) are plugged in as source devices. +memory. The device acts as a coresight sink device and the +corresponding trace generators (ETM) are attached as source devices. =20 Sysfs files and directories --------------------------- =20 -The SMB devices appear on the existing coresight bus alongside the other -coresight devices:: +The SMB devices appear on the existing coresight bus alongside other +devices:: =20 $# ls /sys/bus/coresight/devices/ ultra_smb0 ultra_smb1 ultra_smb2 ultra_smb3 =20 -The ``ultra_smb`` named SMB associated with SCCL.:: +The ``ultra_smb`` names SMB device associated with SCCL.:: =20 $# ls /sys/bus/coresight/devices/ultra_smb0 enable_sink mgmt @@ -32,24 +32,23 @@ The ``ultra_smb`` named SMB associated with SCCL.:: =20 Key file items are: =20 - * ``read_pos``: Shows the value held by UltraSoc SMB Read Pointer regis= ter. - * ``write_pos``: Shows the value held by UltraSoc SMB Write Pointer reg= ister. - * ``buf_status``: Shows the value held by UltraSoc SMB status register. - BIT(0) is zero means buffer is empty. - * ``buf_size``: Shows the buffer size of each UltraSoc SMB device. + * ``read_pos``: Shows the value on the read pointer register. + * ``write_pos``: Shows the value on the write pointer register. + * ``buf_status``: Shows the value on the status register. + BIT(0) is zero value which means the buffer is empty. + * ``buf_size``: Shows the buffer size of each device. =20 Firmware Bindings ---------------------------- +----------------- =20 -SMB device is only supported with ACPI, and ACPI binding of SMB device -describes SMB device indentifier, resource information and graph structure. +The device is only supported with ACPI. Its binding describes device +identifier, resource information and graph structure. =20 -SMB is identified by ACPI HID "HISI03A1", resource of device is declared u= sing -the _CRS method. Each SMB must present two base address, the first one is = the -configuration base address of SMB device, the second one is the 32bits base -address of shared system memory. +The device is identified as ACPI HID "HISI03A1". Device resources are allo= cated +using the _CRS method. Each device must present two base address; the firs= t one is the configuration base address of the device, the second one is th= e 32-bit +base address of shared system memory. =20 -examples:: +Example:: =20 Device(USMB) { \ Name(_HID, "HISI03A1") \ Thanks. --=20 An old man doll... just what I always wanted! - Clara --EIcrB5LfjrdPKhUG Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQSSYQ6Cy7oyFNCHrUH2uYlJVVFOowUCY3JJRQAKCRD2uYlJVVFO o59oAQCu05hsQT4RBR3HOXLdqbkEwDeu51isPmeDh7K3Uj02+wD8C8hED7b8KiCo LOcau2nKAJfOME0U/UmiiQuRk2RiFQM= =nCZV -----END PGP SIGNATURE----- --EIcrB5LfjrdPKhUG-- --===============8013455174781780581== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============8013455174781780581==--