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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B92B6C4321E for ; Thu, 10 Nov 2022 14:15:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229561AbiKJOPY (ORCPT ); Thu, 10 Nov 2022 09:15:24 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46752 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229533AbiKJOOr (ORCPT ); Thu, 10 Nov 2022 09:14:47 -0500 Received: from pandora.armlinux.org.uk (pandora.armlinux.org.uk [IPv6:2001:4d48:ad52:32c8:5054:ff:fe00:142]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 953BEFD3F; Thu, 10 Nov 2022 06:14:41 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=/UUmUYw0sYdCPPlbtaNPjw6NAnPNJqCMa+TeidAlxTc=; b=jiZdQN60nA5lItR4t6I2jQH0VH XW7UTD3nIJ5tOXxg4UKZ0KtLbzJ9TMqwzy6K9BZHYqu0DJPbPGVF2YMPaKGwTtuGCej4gtDE6POyG rcfRJTYGBW+ohh8aGcZMg7JSqQ667INwRfyJZwGagHY/SuAxqQyqBvxeGS9WlW0AvsXQco33J8tKf R8DpmSZd7ukckVb7VBjTM+hMUL2knzsUO4jSdTxyPRcGmFNMahnvidlDMgLfmRINxkK8lUYEMp8gH zIbKgonVWz7EOrvA/g0QEp2XAma9v7agSeMDVRYaYVh0utC1g4/VInumPJGQd8lz7yNjFlDCtjtDJ BW7lGXHA==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:35202) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1ot8Jy-0005i1-A0; Thu, 10 Nov 2022 14:14:22 +0000 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1ot8Jt-0006hv-FB; Thu, 10 Nov 2022 14:14:17 +0000 Date: Thu, 10 Nov 2022 14:14:17 +0000 From: "Russell King (Oracle)" To: Krzysztof Kozlowski Cc: Rob Herring , Linus Walleij , Bartosz Golaszewski , Lee Jones , Alyssa Rosenzweig , Andy Shevchenko , asahi@lists.linux.dev, devicetree@vger.kernel.org, Hector Martin , 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 Subject: Re: [PATCH v3 3/7] dt-bindings: mfd: add binding for Apple Mac System Management Controller Message-ID: References: <531d88b8-75db-1d8f-1384-b8d05594e7b3@linaro.org> <20221109221723.GA2948356-robh@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: Russell King (Oracle) Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org On Thu, Nov 10, 2022 at 03:00:16PM +0100, Krzysztof Kozlowski wrote: > On 10/11/2022 12:48, Russell King (Oracle) wrote: > > On Wed, Nov 09, 2022 at 04:17:23PM -0600, 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. > > > > Meanwhile, I was told by Krzysztof that DT schemas must always have an > > example. So, different person, different story. > > Hm, where do you see a message I told you to always have examples? Maybe > in some discussion I mentioned that examples are desired, but not > always. There is no point in having example in MFD child device schema > if it is already part of the parent MFD binding, where it is actually > required for complete picture. 35ed6e48-40e6-eb14-72de-9a0a4f5b38f8@linaro.org and 2e2356f2-ded1-3cbf-4456-20054a8defda@linaro.org For the GPIO macsec binding. So I'm getting contradictory information. First you say that I need an example in the gpio macsec DT binding yaml document. Now I'm told it should go in the parent. Make up your bloody minds and stop pissing me about. This is why I've given up trying to get this in. Getting a consistent message would be nice, but it seems impossible. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!