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 ACAAAC636CD for ; Mon, 30 Jan 2023 22:25:25 +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-Transfer-Encoding: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-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=crKvfTvuiYFZL0tu1pbnJPttCnAbkPl5gmSYPtAVKnU=; b=R9dCayLDHARMeh VwNFupeo/m5xnAOmZqdtgwLu+IZwlBhLJ2qMgHFfWAUtEFW9Kf+MVPJZMw78NIDhbMv5WPUAaqebl nJPtnlPKpFnTvN4btiB1yGBuHmh/1OLNs/1J4ZKKK29CGKOrfCR+pkecuVmgvctXfWXE1bVu+RIsc OU9vR+XKnuTVSJUxTILSDFBTixrTKYiinQx0At+nl7yTJKpPv1+SaL8LmPyc5df1UbuDsTyEQ4KVC 0AqoGVsnn+bcHi0ykscHqZoMUGCOMR5Bv6RC7NYKNKztjLV7EiEUlQArxCPSKP/gKhourNUKaaLZZ l9RYw2ejQrfjw8ru8MwQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pMcZZ-005dYA-UF; Mon, 30 Jan 2023 22:24:22 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pMcZW-005dWt-Aa for linux-arm-kernel@lists.infradead.org; Mon, 30 Jan 2023 22:24:19 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 06A50612BC; Mon, 30 Jan 2023 22:24:17 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4BA79C433EF; Mon, 30 Jan 2023 22:24:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1675117456; bh=rUrt/K553BODk14r+M79yVWYoxK3CXD/QWYVmFgW8H4=; h=Date:From:To:List-Id:Cc:Subject:References:In-Reply-To:From; b=INYURyC402rJE9eFtLJ2mi3AfyMHJoi21Ly+fHxMnwjSj6Ab5BJ4C/sRzjTeE6KbV tKIMgq9SYTMaceBndfOS63/2sLlNvTAyVZ1n/XQzhIHMSInujC0AR8k+H66LqftLRc hGPb6Y0frzkv8XvzwKmFTUybIC0lIOvbzQlbMEmS+ZNpIzxd9t2MrzUzZ1Tuh/j/zA uEW+yPvGYgOZ0/Sm/9718sEbZRZ03PYroGhJnTQY3W6F7sHaGIXfWhcgKsAv9ra+w+ cSQV+qkg+OlEb1vez7WT/fRqsBkRe7PXcDckWcLwNNAx2EjsOb/zawA2gyQ88qXsEp 3ZbTuMHyKkhbQ== Date: Mon, 30 Jan 2023 16:24:12 -0600 From: Bjorn Andersson To: Arnd Bergmann Cc: arm , soc@kernel.org, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Andy Gross , Olof Johansson , Kevin Hilman , Konrad Dybcio , Stephan Gerhold , Souradeep Chowdhury , AngeloGioacchino Del Regno , Bagas Sanjaya , Bartosz Golaszewski , Guru Das Srinagesh , Krzysztof Kozlowski , Bjorn Andersson , Bryan O'Donoghue , Dawei Li , Loic Poulain , Melody Olvera , Naman Jain , Neil Armstrong , Yang Li , Alexander Shishkin , coresight@lists.linaro.org, Mathieu Poirier , Suzuki K Poulose , Mike Leach , Leo Yan Subject: Re: [GIT PULL] Qualcomm driver updates for v6.3 Message-ID: <20230130222412.t2tkkhqfep2orxf2@builder.lan> References: <20230126163008.3676950-1-andersson@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230130_142418_516555_213688E0 X-CRM114-Status: GOOD ( 35.31 ) 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: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Jan 30, 2023 at 04:18:45PM +0100, Arnd Bergmann wrote: > On Thu, Jan 26, 2023, at 17:30, Bjorn Andersson wrote: > > The following changes since commit 6049aae52392539e505bfb8ccbcff3c26f1d2f0b: > > > > ---------------------------------------------------------------- > > Qualcomm driver updates for v6.3 > > > > This introduces a new driver for the Data Capture and Compare block, > > which provides a mechanism for capturing hardware state (access MMIO > > registers) either upon request of triggered automatically e.g. upon a > > watchdog bite, for post mortem analysis. > > > > The remote filesystem memory share driver gains support for having its > > memory bound to more than a single VMID. > > > > The SCM driver gains the minimal support needed to support a new > > mechanism where secure world can put calls on hold and later request > > them to be retried. > > > > Support for the new SA8775P platform is added to rpmhpd, QDU1000 is > > added to the SCM driver and a long list of platforms are added to the > > socinfo driver. Support for socinfo data revision 16 is also introduced. > > > > Lastly a driver to program the ramp controller in MSM8976 is introduced. > > Hi Bjorn, > > I don't feel comfortable merging the DCC driver through drivers/soc/ > at this point: This is the first time I see the driver and it introduces > a complex user space ABI that I have no time to review as part of the > merge process. > The DCC driver has made 22 versions over the last 23 months, but now that I look back I do agree that the recipients list has been too limited. Further more, due to the complexity of the ABI I steered this towards debugfs, with the explicit mentioning that we will change the interface if needed - in particular since not a lot of review interest has been shown... > I usually try to avoid adding any custom user space interfaces > in drivers/soc, as these tend to be things that end up being > similar to other chips and need a generic interface. > I have no concern with that, but I'm not able to suggest an existing subsystem where this would fit. > In particular I don't see an explanation about how the new interface > relates to the established drivers/hwtracing/ subsystem and why it > shouldn't be part of that (adding the hwtracing and coresight > maintainers to Cc in case they have looked at this already). > To my knowledge the hwtracing framework is an interface for enabling/disabling traces and then you get a stream of trace data out of it. With DCC you essentially write a small "program" to be run at the time of an exception (or triggered manually). When the "program" is run it acquire data from mmio interfaces and stores data in sram, which can then be retrieved - possibly after the fatal reset of the system. Perhaps I've misunderstood the hwtracing framework, please help me steer Souradeep towards a subsystem you find suitable for this functionality. > Can you send an updated pull request that leaves out the > DCC driver until we have clarified these points? > I will send a new pull request, with the driver addition reverted. I don't think there's anything controversial with the DT binding, so let's keep that and the dts nodes (we can move the yaml if a better home is found...). Regards, Bjorn _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel