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 2A2C0C4332F for ; Thu, 10 Nov 2022 11:51:07 +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:From:References:Cc:To: Subject: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=vxj2pDTRS+sUCXgyZuFrvuLyT0/wRUwwd03tcNo1hgU=; b=OIvn5O7mL5LOKO EbJ8/lS/KgdyxX15PeAm5SmhF851WoRqgfbtv+bbg4JU8cjL22+R+4FNgh41WaKS78eCvolOhU+6p UKBv8AJEdoDyNGqWOthJVtw0j61sfMGsVxjZZ83+HIERT8M9gR7zODP78IfLCg7Oxnx4AYv2/ZVT5 pCt+ICqFkBLhzV8HJoGIl0LhYc/LS8nrgkBJLPMAgFBWWgOYnvDi7cHf9TngY/uEDGZSPP5GFNhO7 IPXlkyZLgn9OreR7cYKmwImYD9hRTEJL7G69LUmY4lznpyOBlmdRdnkJBaqzkzmy1EjGJhTJFHJTY N/iHLprMBsLfpeNTXtUQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ot641-005W0z-Aa; Thu, 10 Nov 2022 11:49:45 +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 1ot5r3-005PQK-P5 for linux-arm-kernel@lists.infradead.org; Thu, 10 Nov 2022 11:36:23 +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 E001042118; Thu, 10 Nov 2022 11:36:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=marcan.st; s=default; t=1668080166; bh=WOf5UXqWmrE/iTU0u9/p1HHRldeA61BWkQMiReNjNM4=; h=Date:Subject:To:Cc:References:From:In-Reply-To; b=DfFgYDCkl/pz6GkXaASFqMMdsyDkSJxzVvQ9rPyZDRvyS5bEOoF/4qS5WbN/YgGgv E3VJbN5gn5PzLll6yyobEfMZiJeHUbkJGMUIOGp5a0RkTDIwJmehg/4ucRfYvhy/Pi XQySwwKZK6J+gkFoow8CWIMB4iJoMbap/4zuWg1x4LE/q3XvckXbWrabcRfQ/zfTbd F8GHlXaje5W9E9y69WMKh4/c+b9LeJHLkhc0cJ/em5rEXwjROotPO80TrQKmxQF9n2 U+c2lNN7eCsJGBmODyoArSmRKbdAad04i8ow4o7DOkL3HZmlpa/kp6c+efbS5WBkfM JRiPsSiLvGI3Q== Message-ID: <269b4d0a-920e-fe1e-0b6d-e972e494895d@marcan.st> Date: Thu, 10 Nov 2022 20:35:59 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.1 Subject: Re: [PATCH v3 3/7] dt-bindings: mfd: add binding for Apple Mac System Management Controller Content-Language: en-US To: Rob Herring , "Russell King (Oracle)" Cc: Krzysztof Kozlowski , Linus Walleij , Bartosz Golaszewski , Lee Jones , Alyssa Rosenzweig , Andy Shevchenko , asahi@lists.linux.dev, devicetree@vger.kernel.org, Jonathan Corbet , Krzysztof Kozlowski , linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org, linux-gpio@vger.kernel.org, Petr Mladek , Rasmus Villemoes , Sergey Senozhatsky , Steven Rostedt , Sven Peter References: <531d88b8-75db-1d8f-1384-b8d05594e7b3@linaro.org> <20221109221723.GA2948356-robh@kernel.org> From: Hector Martin In-Reply-To: <20221109221723.GA2948356-robh@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221110_033622_036934_B2F6718A X-CRM114-Status: GOOD ( 25.35 ) 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 10/11/2022 07.17, Rob Herring wrote: > On Tue, Nov 08, 2022 at 10:22:31PM +0000, Russell King (Oracle) wrote: >> On Tue, Nov 08, 2022 at 09:55:58PM +0100, Krzysztof Kozlowski wrote: >>> On 08/11/2022 17:33, Russell King (Oracle) wrote: >>>> Add a DT binding for the Apple Mac System Management Controller. >>> >>> Drop the second, redundant "binding" from subject. It's already in prefix. >> >> Yet another thing that's been there from the start... how many more >> things are you going to pick up in subsequent versions of the patch? >> When does this stop? >> >> In any case, taking your comment literally, >> >> "dt-bindings: mfd: add for Apple Mac System Management Controller" >> >> makes no sense, so presumably you want something more than that. >> >> In any case, I see several recent cases already merged which follow >> the pattern that I've used and that you've reviewed. >> >>>> Signed-off-by: Russell King (Oracle) >>>> --- >>>> .../devicetree/bindings/mfd/apple,smc.yaml | 67 +++++++++++++++++++ >>>> 1 file changed, 67 insertions(+) >>>> create mode 100644 Documentation/devicetree/bindings/mfd/apple,smc.yaml >>>> >>>> diff --git a/Documentation/devicetree/bindings/mfd/apple,smc.yaml b/Documentation/devicetree/bindings/mfd/apple,smc.yaml >>>> new file mode 100644 >>>> index 000000000000..014eba5a1bbc >>>> --- /dev/null >>>> +++ b/Documentation/devicetree/bindings/mfd/apple,smc.yaml >>>> @@ -0,0 +1,67 @@ >>>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >>>> +%YAML 1.2 >>>> +--- >>>> +$id: http://devicetree.org/schemas/mfd/apple,smc.yaml# >>>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>>> + >>>> +title: Apple Mac System Management Controller >>>> + >>>> +maintainers: >>>> + - Hector Martin >>>> + >>>> +description: >>>> + Apple Mac System Management Controller implements various functions >>>> + such as GPIO, RTC, power, reboot. >>>> + >>>> +properties: >>>> + compatible: >>>> + items: >>>> + - enum: >>>> + - apple,t6000-smc >>>> + - apple,t8103-smc >>>> + - apple,t8112-smc >>>> + - const: apple,smc >>>> + >>>> + reg: >>>> + items: >>>> + - description: SMC area >>>> + - description: SRAM area >>>> + >>>> + reg-names: >>>> + items: >>>> + - const: smc >>>> + - const: sram >>>> + >>>> + mboxes: >>>> + maxItems: 1 >>>> + >>>> + gpio: >>>> + $ref: /schemas/gpio/gpio-macsmc.yaml >>> >>> So this depends on other patch, so: >>> 1. You need mention the dependency in cover letter (nothing there), >>> 2. Re-order patches. >>> >>> The GPIO cannot go separate tree and this must be explicitly communicated. >> >> Sigh, getting an order that is sensible is really bloody difficult. > > It's not. Sub-devices before the MFD. The only time that doesn't work is > when the sub-devices put the parent MFD in their example. The solution > there is don't do that. Just 1 complete example in the MFD schema and no > examples in the sub-devices. > >> I'm quite sure Lee is only going to want to apply the mfd bits. > > Indeed. I can't seem to make Lee care... All the schemas should really > be applied together. > >> Then >> what do we do with the other bits? GPIO stuff via the GPIO tree, then >> wait a cycle before the rest can be merged. Or what? > > The schemas must be picked up in the same cycle. I don't care so much > if subsystem maintainers' trees have warnings if they don't care, but I > do care for linux-next. If the subsystem bits aren't picked up, then > I'll pick them up if it comes to that. > We can take all the schemas and DT changes via asahi-soc if that works for you. This also lets us move forward with more related DT changes that would apply on top of things already in that branch. Then Lee only has to take the mfd core bits (and possibly the RTKit platform part, or we can figure something else out for that). - Hector _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel