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 A4B2DC433F5 for ; Wed, 6 Apr 2022 11:56:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229769AbiDFL6X (ORCPT ); Wed, 6 Apr 2022 07:58:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56702 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232274AbiDFL5o (ORCPT ); Wed, 6 Apr 2022 07:57:44 -0400 Received: from mail-ej1-x633.google.com (mail-ej1-x633.google.com [IPv6:2a00:1450:4864:20::633]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F37045255D8 for ; Wed, 6 Apr 2022 00:26:54 -0700 (PDT) Received: by mail-ej1-x633.google.com with SMTP id k23so2410837ejd.3 for ; Wed, 06 Apr 2022 00:26:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=yuM6R8v8TiPq5FdXnzgHGP6cxLXx2tfXvc1fPQGyPBI=; b=p6Pc0ixT7qxKXk+mxDu2v4XYCIZeEosUb3J0LRs+z66/uFVsXJwBQcIHLeAJCLEZqm BDP+rEgxhCs10j5/OzaNq75QgoD5JDu54ZWzYoVZGhIEObUWPAbMNWX0eSeBAkgR9QUh HmwNWIexmyxLuiIXhCIcls/oPEWhm6QUWRaIefiArso7wXQJT7x9S1c755ixecS9fAZW Nb56Ykvbmf3dAY/A8PtcXLpyBHu3vy/jboCZEMr++Db2ISpbG7ZILEf4iyLuOqbz9Cgh cfpuxhjQMP26bNGNhfJRM1N1Y+frlLWH7WpI2sGqvy6bqioT9sItLEM9UyNKOafhwh6d NJxA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=yuM6R8v8TiPq5FdXnzgHGP6cxLXx2tfXvc1fPQGyPBI=; b=7gNTH/6GHm+WxTTvHOotbFJlELJxuwUCBXsuuoK+ZS6PGT5me8xT5pExKKHQtXdvBM KRYkxiSIQj1iWxarEaFCU+wAcnh1lhQXjH7MYFcENyh6OCmgzVKN76zh3Lh86gZU6g70 2hTZV+Jou7xd706LiUTX1SDroofTSaIwi1VL1Vtg1LSnNH4H5zm7r+W1t1POPdPZQKJ5 QqCom8ssgZYjSaXE0qQ56fV5yIzPa2VqHmcTzrer0npiD8RTgl1t0ERgr/2e4epeHSvS Vk3Z9RkQjhi4m8JYcAnzueXRWjoff8c6WcXPnMndMyJ4sU8twK1uICFkpo8q6NvkjXMN rH+g== X-Gm-Message-State: AOAM533lL56R6aeomWSaA8tqVUK7w58/YpMj0enWDAuH8UdNJHmxdp/a lkeMlC+A66HT06RZbS59xEjigA== X-Google-Smtp-Source: ABdhPJwkyCtsJH2vwIdQXVVX2u6O4ZU+EN9y3XsOqCwJVzVWy6LcXdfv65/APnX6lC9vhfF9bwaN9Q== X-Received: by 2002:a17:907:c16:b0:6db:682:c8c9 with SMTP id ga22-20020a1709070c1600b006db0682c8c9mr7206485ejc.153.1649230013464; Wed, 06 Apr 2022 00:26:53 -0700 (PDT) Received: from [192.168.0.182] (xdsl-188-155-201-27.adslplus.ch. [188.155.201.27]) by smtp.gmail.com with ESMTPSA id o12-20020a50c90c000000b0041907e62024sm7510979edh.85.2022.04.06.00.26.52 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 06 Apr 2022 00:26:53 -0700 (PDT) Message-ID: <315c957b-8d22-dcef-a5b8-36e92f0897be@linaro.org> Date: Wed, 6 Apr 2022 09:26:52 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.7.0 Subject: Re: [PATCH 1/2] i2c: add binding to prevent device detection Content-Language: en-US To: Vincent Whitchurch Cc: "wsa@kernel.org" , kernel , "linux-i2c@vger.kernel.org" , "devicetree@vger.kernel.org" , "krzk+dt@kernel.org" , "robh+dt@kernel.org" , "linux-kernel@vger.kernel.org" References: <20220405121627.1560949-1-vincent.whitchurch@axis.com> <20220405121627.1560949-2-vincent.whitchurch@axis.com> <20220405141255.GD28574@axis.com> From: Krzysztof Kozlowski In-Reply-To: <20220405141255.GD28574@axis.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 05/04/2022 16:12, Vincent Whitchurch wrote: > On Tue, Apr 05, 2022 at 03:57:16PM +0200, Krzysztof Kozlowski wrote: >> On 05/04/2022 14:16, Vincent Whitchurch wrote: >>> Documentation/devicetree/bindings/i2c/i2c.txt | 4 ++++ >> >> Use the subject matching the subsystem (git log --oneline -- .......). > > I did look at the git log when writing the subject, but there's a bunch > of variation there so I went for the subject prefix which Wolfram > himself used since I assumed he would be the one applying these patches. > > $ git log -n5 --oneline origin/master -- Documentation/devicetree/bindings/i2c/i2c.txt > 6881e493b08f dt-bindings: net: New binding mctp-i2c-controller > 168290fb2675 i2c: add binding to mark a bus as supporting SMBus-Alert > e6277308ac21 i2c: add binding to mark a bus as SMBus > db36e827d876 dt-bindings: i2c: add generic properties for GPIO bus recovery > fad5972a1eca i2c: add 'single-master' property to generic bindings The proper command for a subsystem is: git log --oneline -- Documentation/devicetree/bindings/i2c/ and then you see mostly different pattern. Other way, like for example Mark Brown prefers, is: i2c: dt-bindings: It would be fine as well, but the point is that you did not include the dt-bindings prefix here at all and almost always I ignore all such emails. >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/i2c/i2c.txt b/Documentation/devicetree/bindings/i2c/i2c.txt >>> index fc3dd7ec0445..5f7bdf4851e8 100644 >>> --- a/Documentation/devicetree/bindings/i2c/i2c.txt >>> +++ b/Documentation/devicetree/bindings/i2c/i2c.txt >>> @@ -72,6 +72,10 @@ wants to support one of the below features, it should adapt these bindings. >>> this information to adapt power management to keep the arbitration awake >>> all the time, for example. Can not be combined with 'single-master'. >>> >>> +- no-detect >>> + states that the system should not attempt to automatically detect >>> + devices which are not explicitly specified as child nodes. >> >> Focus on the hardware, not on system and behavior. The hardware property >> is that bus is completely described in Devicetree or detection does not >> work correctly. I guess the property name could stay like this, but the >> description could be rephrased. > > I'm having a hard time drawing the line between hardware and the > "system" here. The bus being completely described in devicetree doesn't > sound like a hardware property to me, but, yes, I can change the > description to say that. Then you need to come up with better description. :) By system I understood here Operating System, therefore a Software piece or pieces. Bindings should not rather focus on hardware instead of referencing software. Software actually not matter for the bindings... There is quite clear distinction between hardware and operating system... Can you explain why do you think "bus being completely described" is not a hardware description? Bindings and DTS describe hardware, so one of parts of such description is whether nodes and other properties are present. If all child nodes are present in DT - no-detect - and this is a purely hardware point of view. No software here. Best regards, Krzysztof