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 A0012C46467 for ; Mon, 16 Jan 2023 15:20:49 +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:Subject:Cc:To:From:Date:References: In-Reply-To:Message-Id:Mime-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=yzp21d1Y01K+UhE4pBIj1R191uMVujVpOy4gf9w56ho=; b=lHQA4UgXQyx4qw pwhtWMHDCqsyxGmpwqrsvE3lnMhQO1uO6rHGcfflSeQhCXOzHZKtInVsyXvgeuSBfeGJAAiWofqBp 3Qc3DPvQp3Klh4MFJopaQ6FtAoVjQf/YDWULRPbualZtqxoEBYUdk9JDUKB08rAe4jjVnPnL82sB2 5az4/jriIXEOFVs+g0zL0ihcUMQWJL5eMLaOlwltuyOqZXpkQLGT9JOBGYb2Bddg1iuTc+KhrG349 LcUxie10FUIbbaltBIIie1IK21H8aZIQGLHF8UXOkx5fWCjDnBMkuqs8a6C7Zfd6wyc0qDNYA8mYm jCVfXC7I9NMv8eHvnlFQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pHRGh-00B4jY-1M; Mon, 16 Jan 2023 15:19:27 +0000 Received: from out1-smtp.messagingengine.com ([66.111.4.25]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pHRGd-00B4in-9W for linux-arm-kernel@lists.infradead.org; Mon, 16 Jan 2023 15:19:25 +0000 Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id B82B35C01D1; Mon, 16 Jan 2023 10:19:19 -0500 (EST) Received: from imap51 ([10.202.2.101]) by compute6.internal (MEProxy); Mon, 16 Jan 2023 10:19:19 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-type:date:date:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to; s=fm2; t=1673882359; x=1673968759; bh=N2AaqpF3HW MCVXZ2qUPffZma9rjZbSafx8M41yLmuoE=; b=jHj/gDQGNBpfYh0yCyZRtlXclm ueThgrKdnuCdyJROmmqJVs6lW2kLgffXl6PYuqJfF1RBj70yqhA3tx4Ref9hc1PT DNVjtiQNkycJDb3OF4zN7AFfYtsOOAPI9lma+4vvaRzK7WegrNxshPqRgA8/Ulo5 iOohCEiDC36NdhQk2xF+hrZjr0pTWvz8RJeQpaEWoYdr5gQiThcsfA6JYNBNQhMy SBt/gEIncnUX66ysNEYo4gQufb4lfXw+b0lkT2qk3/h2FtakuDtmVmn8ZDk79djx CRC8t+5Bk0DkjYs7swPM1kzJ8i4U9us/VwnmPpI9kUAdqI1Yu8Bzj4ELT/WQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:date:date:feedback-id :feedback-id:from:from:in-reply-to:in-reply-to:message-id :mime-version:references:reply-to:sender:subject:subject:to:to :x-me-proxy:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s= fm3; t=1673882359; x=1673968759; bh=N2AaqpF3HWMCVXZ2qUPffZma9rjZ bSafx8M41yLmuoE=; b=I33uRZ+2TxBvz+HDJrERzoGN8v/hpeQGeyNS1upZxtwv 70WFNq/uW4axz/MT2Ju0pIbR5XlYi9pC6JTVfNvfCFErCPiRRHGz2fuMa6tO9t69 XAI6zTJd3FZLzRqSQgb3tAI/PLB7z8jDg2iZdTgdKQzrB89UK0JYOiFA9cpDtPgy ojbOthdU36Kgq5pYirKmzAWRjFQ6WnWPSgq5Hpr1trHyyAHl35L9Q6Gp21fzn9NP FAA7liEhYpBtQLkPft2TnziqOHNTS/pn2NoMEmwZHvJY25E1S5nxyXjxXxMxoypN 71UNW09Svhu0UiZ+RjnbzpQ/unrmo2SHzaYy2G+sNg== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedruddtgedgjeeiucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepofgfggfkjghffffhvfevufgtsehttdertderredtnecuhfhrohhmpedftehr nhguuceuvghrghhmrghnnhdfuceorghrnhgusegrrhhnuggsrdguvgeqnecuggftrfgrth htvghrnhepffehueegteeihfegtefhjefgtdeugfegjeelheejueethfefgeeghfektdek teffnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomheprg hrnhgusegrrhhnuggsrdguvg X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 3BA93B60086; Mon, 16 Jan 2023 10:19:19 -0500 (EST) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.7.0-alpha0-1187-g678636ba0d-fm-20230113.001-g678636ba Mime-Version: 1.0 Message-Id: <55f09599-b553-4429-aa79-ca99ccf95cda@app.fastmail.com> In-Reply-To: References: <20230110042533.12894-1-clayc@hpe.com> <20230110042533.12894-3-clayc@hpe.com> <110af65f-9c18-524d-a073-ef7be60d1f5a@linaro.org> <48cd3e28-f1db-487d-8971-473dc8c12c09@app.fastmail.com> Date: Mon, 16 Jan 2023 16:18:59 +0100 From: "Arnd Bergmann" To: "Clay Chang" Cc: "Krzysztof Kozlowski" , linux-kernel@vger.kernel.org, soc@kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, "Verdun, Jean-Marie" , "Hawkins, Nick" , "Rob Herring" , krzysztof.kozlowski+dt@linaro.org, "Russell King" , "Olof Johansson" Subject: Re: [PATCH 2/5] dt-bindings: soc: hpe: hpe,gxp-srom.yaml X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230116_071923_724221_3A65FC3A X-CRM114-Status: GOOD ( 19.95 ) 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 16, 2023, at 14:42, Clay Chang wrote: > On Thu, Jan 12, 2023 at 02:37:53PM +0100, Arnd Bergmann wrote: >> On Thu, Jan 12, 2023, at 14:16, Clay Chang wrote: >> For the user interface side, I don't really like the idea of >> having a hardware register directly exposed as driver in >> drivers/soc, this generally makes it impossible to have portable >> userspace that works across implementations of multiple SoC >> vendors, and it makes it too easy to come up with an ad-hoc >> interface to make a chip work for a particular use case when >> a more general solution would be better. >> > > I agree with you. I have one question though: if we create a 'hpe' > directory under drivers/soc, and put all HPE BMC specific drivers there, > do you think this proper? It certainly wouldn't be right to put "all HPE BMC specific drivers" in there. Most drivers will fit into some existing subsystem, and should be moved there instead. drivers/soc is used primarily for drivers using soc_device_register() to provide information about the soc, and we also use it as a place for drivers that just export soc-specific helper functions that can be used by other drivers. >> Again, it's hard for me to tell why this even needs to be runtime >> configurable, please try to describe what type of application >> would access the sysfs interface here, and why this can't just >> be set to a fixed value by bootloader or kernel without user >> interaction. > > The register is used for communication and synchronization between the > BMC and the host. During runtime, user-space daemons configures the > value of the register for interactions. That does not sound very specific. What is the subsystem on the host that this communicates with? Can you put the driver into the same subsystem? Arnd _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel