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 5F642C3DA42 for ; Mon, 8 Jul 2024 12:56:59 +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:In-Reply-To:From:Subject:References:Cc:To:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=AFsq21pzBB3I32VUxpbDobQ6NOFvMJ5Jnfv4RTcn+lg=; b=H631LUokA2u1WPqcPvrn20n34J sNfow3q9lrGSZ60gOZ3VLIOAAUYkrnBpk4lFafmJuD1BNg1xvg1tZ6zm1rCngj2V5+CV4juZYocX0 pQfQlfai/80BQ8DufxqzPObXmAnRUoXxVYWAMFydja4PzSJgDfB2jEA9uPNw2c1RBSKVoByLaEzZG WXvUbjQV32FV6RERPaRowr438m4Bur3NOHLKOOzX5EBGzXmnru/YmQUNT5mLYKgfI6aMwzDDytFLk 31re4Rm/hA0Zjc7NYqGcYBnZ7uRCwSRho2xAOKlJHKvp43z/TGEYVHmllQJt62s8viMe6IszQBgUB Gy5nn+gg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sQnvB-00000003rLp-2n2d; Mon, 08 Jul 2024 12:56:45 +0000 Received: from mout.web.de ([217.72.192.78]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sQnuv-00000003rKc-2zLj for linux-arm-kernel@lists.infradead.org; Mon, 08 Jul 2024 12:56:31 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=web.de; s=s29768273; t=1720443379; x=1721048179; i=markus.elfring@web.de; bh=AFsq21pzBB3I32VUxpbDobQ6NOFvMJ5Jnfv4RTcn+lg=; h=X-UI-Sender-Class:Message-ID:Date:MIME-Version:To:Cc:References: Subject:From:In-Reply-To:Content-Type:Content-Transfer-Encoding: cc:content-transfer-encoding:content-type:date:from:message-id: mime-version:reply-to:subject:to; b=haGMLV7WpePPC/OB/uZldXBOHXTKR3gqAyQDLwdoq6fbbozzGgARqQYKRydYr0iX R1maclf+OdSvT7riBMHknUBSLmzeSxPNVwfTuxH9+dCYNbDgDKkNmX/7IQZwkxUCu M9hERWCh9Z6rl5xCNPXkYvFZ3vHBXVn6d//RYyGGx4ye5+xqswhnXaZ4qIoJiGD1p XjqVnDTmSflDxPT2e24T+MpD246oB8xX5d2FNqzn5/L15ucHJc2QfbX3Jl8hWaOIg VaIn8/9XDzEJRR/Jpbr820mOw3oXk+rUPvUUYFMtlWbq5ln4VU1U4sS0HKdKLGGfW B/8M1VajjyrFWi1IlA== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.91.95]) by smtp.web.de (mrweb105 [213.165.67.124]) with ESMTPSA (Nemesis) id 1M5QqN-1sRdGI0vH3-00GwZf; Mon, 08 Jul 2024 14:56:19 +0200 Message-ID: <10cff397-af4f-4649-994f-309203376f42@web.de> Date: Mon, 8 Jul 2024 14:56:17 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird To: Jie Gan , coresight@lists.linaro.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, Alexander Shishkin , James Clark , Krzysztof Kozlowski , Mathieu Poirier , Mike Leach , Rob Herring , Suzuki Poulouse Cc: LKML , Jinlong Mao , Leo Yan , Song Chai , Tao Zhang , Tingwei Zhang , Trilok Soni , Yuanfang Zhang References: <20240705090049.1656986-4-quic_jiegan@quicinc.com> Subject: Re: [PATCH v2 3/4] Coresight: Add Coresight Control Unit driver Content-Language: en-GB From: Markus Elfring In-Reply-To: <20240705090049.1656986-4-quic_jiegan@quicinc.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:7WkijsB2gTSSPyJ7H+MItEif0hriT4lLW7yGijm+/aiCZZ0MUCs CVuqlTCqrCvtbBTPXc4qM1lmzEGG0MV+QgDYiB/ROORRAgwLVMRhlWSDXgfGdaeXZ3uye8n Q7ZZaC3LDw2SRpFL3V+KTfU3MEhnZdT5h7XCJ9vHD9GkpqmOasEfAEzPHzIrInpZFn8hIm7 6n8csU9n3ACg1eahAkANQ== UI-OutboundReport: notjunk:1;M01:P0:ugSciB4Nclo=;HBZRS/QjTtROUiFS1uPzclUgXBO bQawFdj4sGC8o2TxnSx7i+CwMJatJL0KOh1EWSPY0K2g6+ptGqRN3Xgv9PTOAlqkDgIVcBWe6 pqJG4ytUqen+IBiZIIVKnZSpqznGE6AidIG0mQ/Lxk5ychVXCbcmz/Kg1jEu04mzfaS9CBrx+ HaNdkScGkcZi38+nTfOJeCBEth0nIcHMHCo88Yo3hfAoU0+fMgOK7kpQ003qXUCPqT5HX5kCh b49jARzMuzpEhy+NawjP7UjjiuJvm3JE5+w5VlpBFJCFEabTNfTIBlSehXR1lav6Uhbd3bT52 RlpBK1b/3miVDAzELi195hZTiWZYAk0NZ0T7Oe46cl/+dpmEXn8qR0fEWw6pO46HhyC/WMpDc ViyiM1tv1cSSS6YIUBfLH/OQ+JGQCVi/wmkY9kmXtQxI94rPTXhMxRWxSUherpKJLrClDdu/d ro+EyCWOOqw5PUXjwFrdxA1ws2cTD6oNI6wQCDy8K0Zmov8oO9a9ylg2ISmPtrLYAhrF+mRiP v0Capt05sowIs12rj7DehW9QMVrZnRnbao+qotoO3/C5E1Uz8JnNuCYAsOVljFRh4IZus4FdD Tapl/46huPsogCUdFU4K3dkPcGbvz5pC9bEsn6X8F9zhGEHQKRO1JlWCcxMGpfDMUjeN49U4Q rLC7S7A/h2jKEt+JpDBS17ttpUkWPuXvn+fXp8ZZ8NZ8xZY5h6yM2JBoQMN881aPVtDeJ8UkR CUxYagVE+GegjVGbt7Mc1vKan7zFwNbKCnt7vKAxTAnuJR3E9Sq5l8Jvc2/ae9+pnrMLJpSB1 mXzdodw6gXGlPV0abqJyEA/QyYF7Bb2bSZaTEWHTPqPUY= X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240708_055630_064227_97E3D5BD X-CRM114-Status: UNSURE ( 7.68 ) X-CRM114-Notice: Please train this message. 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 =E2=80=A6 > Disabling source device resets the bit according to the source device's = trace ID. How do you think about to improve such a change description with imperativ= e wordings? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Do= cumentation/process/submitting-patches.rst?h=3Dv6.10-rc7#n94 =E2=80=A6 > +++ b/drivers/hwtracing/coresight/coresight-ccu.c > @@ -0,0 +1,290 @@ =E2=80=A6 > +static int __ccu_set_etr_traceid(struct coresight_device *csdev, > + uint32_t traceid, bool enable) > +{ =E2=80=A6 > + spin_lock_irqsave(&drvdata->spin_lock, flags); > + CS_UNLOCK(drvdata->base); =E2=80=A6 > + ccu_writel(drvdata, val, reg_offset); > + > + CS_LOCK(drvdata->base); > + spin_unlock_irqrestore(&drvdata->spin_lock, flags); > + return 0; > +} =E2=80=A6 Under which circumstances would you become interested to apply a statement like =E2=80=9Cguard(spinlock_irqsave)(&drvdata->spin_lock);=E2=80=9D? https://elixir.bootlin.com/linux/v6.10-rc7/source/include/linux/spinlock.h= #L574 Regards, Markus