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 5A712ECAAD5 for ; Tue, 6 Sep 2022 11:37:50 +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:Subject:From: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=tG7JDS9xOrOI6k6NeKbDcJxKHsqgTigrLMkUsjbFqc0=; b=kG9kumQaMuajAm h1OT5E+NLkGAWWHYxPMU8tY44HevkxjBXWD+r3MctIFNPpOepNtvtElwz1JUeAZ5noMsSTq1NiWNJ 5kTpHEc1EJj+mQLmsewIB5VNlkDOJRr5ws2XhxXzNMVfVEYbYjB9XIHUGsN9knzf4EPZ+ETPkZQdD 7Ukpo4ZqpgE92FEr3mFygCe5OBjD3IdGuYCpLYpjCsCkpqMZ6A33CPhDY08AHVUWkHcVtobEqbafU MxqB54bBEEaV4Kzpvb08wKGwjULYgN06+vT1V8uNHtvWz9K19mBgLsuzc9w+oZceDdU7vOKln+1gN Ac692uisP3MNO8A9x9Jw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oVWso-00CwZV-Vb; Tue, 06 Sep 2022 11:36:47 +0000 Received: from marcansoft.com ([212.63.210.85] helo=mail.marcansoft.com) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oVWse-00CwU6-Tn for linux-arm-kernel@lists.infradead.org; Tue, 06 Sep 2022 11:36:38 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: marcan@marcan.st) by mail.marcansoft.com (Postfix) with ESMTPSA id D82EB41F12; Tue, 6 Sep 2022 11:36:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=marcan.st; s=default; t=1662464191; bh=iZaovA3utTa1MRTvB6tlYyIRYCuVcLY9Hsc4sBAEXHA=; h=Date:To:Cc:References:From:Subject:In-Reply-To; b=gaJghwidE54NgcQlYlv0xIM81NWChh/4fnOJRbbzs/VRr8HNzQ+WaJtZkHSV1tH8u 81Vth9//yauZpcTBB4TgSuAuAo09pCRsJvYfzYsJiDD6Guai6KQ0tp/452hyfGdi3/ /cbMpAKzpFLy58hDgpKnppEl7OT58u11d6/2pVPtXgP6RbhvMQQgeIO+c4XiseVZu5 a8hgRWBghjlRPgcLXpaeJT6nwkrmbdBxcUVIIacVl3z5JEPQplCbjV13euzaopq+Gc xDs3MqsdVMSni2cvqh2cKVoTWllkCHOrgCDQNVGxohTU5YDF0lZWqi3/RGqFhq/Ck3 d8zQBxImKX+wQ== Message-ID: <909bb4e7-5bd2-2903-5bba-87ae37f3448a@marcan.st> Date: Tue, 6 Sep 2022 20:36:25 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.1 Content-Language: es-ES To: Linus Walleij , Mark Kettenis Cc: "Russell King (Oracle)" , robh@kernel.org, krzysztof.kozlowski@linaro.org, arnd@arndb.de, lee@kernel.org, alyssa@rosenzweig.io, asahi@lists.linux.dev, brgl@bgdev.pl, linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org, sven@svenpeter.dev, krzysztof.kozlowski+dt@linaro.org, devicetree@vger.kernel.org References: <928ddeff-efac-920c-7bbf-dda35a942b93@linaro.org> <2fedff34-6a20-f1ce-a756-2bd8671fcd52@linaro.org> <20220902172808.GB52527-robh@kernel.org> From: Hector Martin Subject: Re: [PATCH 1/6] dt-bindings: mfd: add binding for Apple Mac System Management Controller In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220906_043637_410235_AF8F918B X-CRM114-Status: GOOD ( 24.82 ) 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 06/09/2022 20.22, Linus Walleij wrote: > On Tue, Sep 6, 2022 at 11:31 AM Mark Kettenis wrote: > >> Another argument for having sub-nodes is that the firmware actually >> exposes *two* GPIO controllers. For now we only support the "master" >> PMU GPIOs, but there also is a "slave" PMU GPIO controller that uses a >> separate set of SMC "keys". We currently don't need any of the pins >> on the "slave", so we don't expose it in the DT yet. > > That sounds backward, like we don't expose device X as DT node > because $OS doesn't use it yet. DT should just expose (by nodes or > other ways) all hardware that exist or at least all hardware we know > about no matter what $OS is using. How so? The are piles and piles of unused hardware not exposed in the DT, and piles and piles of hardware that will be used but we haven't figured out how to do it yet, so it's not exposed. For example, we know there are like 8 or so UARTs, but we don't define them in the DT because they are not connected to anything on any existing device and we don't need them. Apple does the same thing in their DTs (only used hardware is defined). I don't really see the point of exposing a GPIO controller when we don't actually do anything with the pins yet, and might never do so. Having drivers bind and stay unused just increases the amount of code running without any ultimate purpose, so why do it? It's not like any other OS would use the hardware either - GPIOs are only useful if they are referenced in the DT for something, and we don't have anything that would reference these. For SMC in particular, there's a huge amount of functionality we don't have drivers for yet, and I don't see the point of trying to conjure up DT bindings for it until someone writes a driver (and has a reason to do so) :) > FWIW I think nodes makes most sense because no doubt for example > the RTC is a separate hardware unit somewhere, and so is the > GPIO. The fact that it is hidden behind a software abstraction doesn't > change the fact that the HW definitely has these discrete units. The RTC and the GPIO happen to be part of the same physical IC (PMU), but yes, I agree. - Hector _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel