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 040CEC433F5 for ; Thu, 5 May 2022 08:33:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1349421AbiEEIhF (ORCPT ); Thu, 5 May 2022 04:37:05 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48606 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1343493AbiEEIhE (ORCPT ); Thu, 5 May 2022 04:37:04 -0400 Received: from mail-ed1-x52e.google.com (mail-ed1-x52e.google.com [IPv6:2a00:1450:4864:20::52e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B27F0488B2 for ; Thu, 5 May 2022 01:33:25 -0700 (PDT) Received: by mail-ed1-x52e.google.com with SMTP id be20so4326034edb.12 for ; Thu, 05 May 2022 01:33:25 -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=EkR961EY1CXiCzNgowxxrUhXg4TMH8OHqi0oexOXXrs=; b=Mt+y+lKLyTAGxGx+Cc8jLbXo6P9hfMiUjbZCuTTCCuaNtNZJs2Ce2i/GQ7BtCCHYIu kmIjpU6HkEkOQhKTFMtrIHmp/6F0WkvZgVqBhEBLoHfGsnfcdYc6OJEpRvrC+I19872y NNng+PNxrz3aeiuobZAz/CQDLOFpy+qAjrQnt9+0ADz8tY4uO7l7zCX1lmiSFzAhXV6h 7QAUe6emtYXxAMjuNg1xlwVbIZT8ypOOtq6sWGhvT5P3Br3PWVoSSfmmMFTQbxAjwDjb fYsno5UpExANJUOCyBpGZF3XN9ENrq5LKZAxlJdoWtxQ4v5ocISQqhQnwCkOoxS8T+mD lFDQ== 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=EkR961EY1CXiCzNgowxxrUhXg4TMH8OHqi0oexOXXrs=; b=VdJdwUChYL4XaDecPYWTmWV2AS8hXn21AheqXNJg4RG4iDRtORG2DgYbxziUIwff6Y 2bb+d9m8o2ttUHZnp26uGK1E28vfaVLMPy3OtlL0bPcGIg4Hma4TgWTYMP/eZNuQUPHR eUotbUBZoKS0TRteDFDMq+38Z8PPJGFR8I4Cf9Ek0o2CDdqZIsINTWqOJ8crx73oyST6 w/onWAMNBwEFI3tVUopoRJ7q05i3E/sREbvyyT0dqXqqFjtdD0yltDYuI0PDSpzXKj+D oEadpsiHsIysNoCqLDSg6CWjeSSpp+THySe1iuh6wWc2ZcjOoifYke1XP+bgcyxS/Gvv G7Ew== X-Gm-Message-State: AOAM530brvrlDbHP/IMShYffekvcKzaxS5IDrXz+8dGtL+PaPOekf0xe bfWzOP7OWgKJSSFG4ka64enF+g== X-Google-Smtp-Source: ABdhPJyF/rcmBBgiWodOMfkAv7sPsey9xQxskdtkaMwmULm+fFtWEOEYwxBPAE/k1fOsSJoVMBZAzA== X-Received: by 2002:a50:8dca:0:b0:425:e8a3:a79f with SMTP id s10-20020a508dca000000b00425e8a3a79fmr27875839edh.155.1651739604291; Thu, 05 May 2022 01:33:24 -0700 (PDT) Received: from [192.168.0.217] (xdsl-188-155-176-92.adslplus.ch. [188.155.176.92]) by smtp.gmail.com with ESMTPSA id f24-20020a170906391800b006f3ef214df7sm490116eje.93.2022.05.05.01.33.23 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 05 May 2022 01:33:23 -0700 (PDT) Message-ID: <2dbea4b4-9387-6676-8648-94bae6df3d8a@linaro.org> Date: Thu, 5 May 2022 10:33:22 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.1 Subject: Re: [PATCH 2/6] dt-bindings: regulator: Add reg-external-output binding Content-Language: en-US To: Zev Weiss , Mark Brown Cc: Liam Girdwood , Rob Herring , Krzysztof Kozlowski , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Greg Kroah-Hartman , openbmc@lists.ozlabs.org References: <20220504065252.6955-1-zev@bewilderbeest.net> <20220504065252.6955-2-zev@bewilderbeest.net> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 04/05/2022 22:33, Zev Weiss wrote: > On Wed, May 04, 2022 at 05:55:53AM PDT, Mark Brown wrote: >> On Tue, May 03, 2022 at 11:52:48PM -0700, Zev Weiss wrote: >>> This describes an external output supplied by a regulator, such as a >>> power outlet on a power distribution unit (PDU). >> >> OK, so this is that represnetation of the connection - which raises the >> question about why the regulator needs a property? This remained unanswered... the binding looks like approach the problem from wrong side. It's like defining consumers of clock inside clock controller. It does not scale, it's not in Devicetree style/intentions. You should have a dedicated port or connector which takes a regulator as a supply (and for supply we already have bindings). We already have only one connector: git grep connector -- Documentation/devicetree/ (extcon is not the best choice for bindings, it's tied to Linux implementation, not to hardware) Best regards, Krzysztof