Message ID | 20221031173933.936147-2-luca@z3ntu.xyz |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp2449074wru; Mon, 31 Oct 2022 10:42:23 -0700 (PDT) X-Google-Smtp-Source: AMsMyM4u68kgobvnwzCzNgvymE7yY2EMex2rILzcx2tb/rxc+Vdvh1Ot7Eoy4CuB11SZxxBUbMhq X-Received: by 2002:a65:5242:0:b0:46e:9bac:178 with SMTP id q2-20020a655242000000b0046e9bac0178mr13697337pgp.300.1667238143173; Mon, 31 Oct 2022 10:42:23 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1667238143; cv=none; d=google.com; s=arc-20160816; b=s+T1PNsJ33t2k2uS7cdYRzNgKnA3EhNMWQz0IxuFjF1TxIaEyWXz1dYSYqDgqZCkcq mzyA7ttHGdWF2jGAAOTzHpjtuiaYbppo1a8Nf6SNmu9TFd3DD8GbARXHXNDmkKdfqTQq AXa7vJY3C3tT1sXUXYJBT8ZEr1nALDRKveUK5h7lUIeA5MyFTyJUNPcUsKkfNiNjUSaA VL57+5lCNBjtxEBR+n5FmGBQ9Ho7Nyq2bhSqqXTyBuD5Y11ZOtD32eiUjOThrGe3vYDP cXXJPd91s8UjzVB/1YBT59MLAZ7E6cE30Dpz4cCg6r1aHKlJ4/D/zqIq5uFTbbbLGI3J 16oA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=hCg6A83bWmAelcxHHXn+jKlObbgQYBFVPtUxnda4trI=; b=00neos3irKeRtT1PPitUkRAwxDBMLfMB+Mcl12MgdsUOelPT2558TGzLb8gQuUcAcb mpXx+PrvOV7RHv/1hcefMS+uDTmrjzC0mwEmioqbiYwd4R++Z4GS61zi5z7PsIwJgLy2 1nMiBaArQOS86Wgsx23SVsPfQXc6pl+iAGvHFTmkShRAf0NPQj+mvcAcLN0/T3+MYn+T C2QH6sCK+SyidjiRtD8rem7wzUgCVtOq2IZntfSTe4I9GIld1ZlKCt8c6UIdCcfmLT4f NGvRIktUozku+M9BCLKNaKI4NgHEjABP5UwUF/3fleW2j3lEz/0CmosQT3TzqgTMBzSD gLMA== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@z3ntu.xyz header.s=z3ntu header.b="JoSqj/1W"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=z3ntu.xyz Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id c18-20020a6566d2000000b0046b158da2d9si9204430pgw.528.2022.10.31.10.42.09; Mon, 31 Oct 2022 10:42:23 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=fail header.i=@z3ntu.xyz header.s=z3ntu header.b="JoSqj/1W"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=z3ntu.xyz Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231724AbiJaRkf (ORCPT <rfc822;kartikey406@gmail.com> + 99 others); Mon, 31 Oct 2022 13:40:35 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39610 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229626AbiJaRkT (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Mon, 31 Oct 2022 13:40:19 -0400 Received: from mail.z3ntu.xyz (mail.z3ntu.xyz [128.199.32.197]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 52E1610554; Mon, 31 Oct 2022 10:40:19 -0700 (PDT) Received: from g550jk.arnhem.chello.nl (31-151-115-246.dynamic.upc.nl [31.151.115.246]) by mail.z3ntu.xyz (Postfix) with ESMTPSA id ABBB2D01B5; Mon, 31 Oct 2022 17:39:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=z3ntu.xyz; s=z3ntu; t=1667237987; bh=GDhGOeBuz53WBcObyjQCoYK9JeJOabBRag+IKc2FGBI=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=JoSqj/1W/uDUg2GQrW5U0IcaduRAVV2IIybMw2Q0K3xncFjfNry+DXhvploM3T1rv UWIxFDcoV6w/He1lSk5Q17aA6t4PZd+Vs8SKDmlZ3Ma/2PZ0YzKsnKwlw8XDG+UBco 2HmUxGqyUykXfbhXa7aJ8e09zxP0Q3CHmU6wRDlA= From: Luca Weiss <luca@z3ntu.xyz> To: linux-arm-msm@vger.kernel.org Cc: ~postmarketos/upstreaming@lists.sr.ht, phone-devel@vger.kernel.org, afd@ti.com, Luca Weiss <luca@z3ntu.xyz>, Andy Gross <agross@kernel.org>, Bjorn Andersson <andersson@kernel.org>, Konrad Dybcio <konrad.dybcio@somainline.org>, Liam Girdwood <lgirdwood@gmail.com>, Mark Brown <broonie@kernel.org>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Wesley Cheng <quic_wcheng@quicinc.com>, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: [PATCH v2 2/3] regulator: dt-bindings: qcom,usb-vbus-regulator: change node name Date: Mon, 31 Oct 2022 18:39:32 +0100 Message-Id: <20221031173933.936147-2-luca@z3ntu.xyz> X-Mailer: git-send-email 2.38.1 In-Reply-To: <20221031173933.936147-1-luca@z3ntu.xyz> References: <20221031173933.936147-1-luca@z3ntu.xyz> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=0.3 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FROM_SUSPICIOUS_NTLD, PDS_OTHER_BAD_TLD,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1748225903131336982?= X-GMAIL-MSGID: =?utf-8?q?1748225903131336982?= |
Series |
[v2,1/3] dt-bindings: mfd: qcom,spmi-pmic: support more types
|
|
Commit Message
Luca Weiss
Oct. 31, 2022, 5:39 p.m. UTC
usb-vbus-regulator is a better generic node name than dcdc to change the
example to match.
Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
Changes in v2:
* New patch
.../devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On Mon, Oct 31, 2022 at 06:39:32PM +0100, Luca Weiss wrote: > usb-vbus-regulator is a better generic node name than dcdc to change the > example to match. You've not copied me on the rest of the series so I've no idea what's going on with dependencies. When sending a patch series it is important to ensure that all the various maintainers understand what the relationship between the patches as the expecation is that there will be interdependencies. Either copy everyone on the whole series or at least copy them on the cover letter and explain what's going on. If there are no strong interdependencies then it's generally simplest to just send the patches separately to avoid any possible confusion.
Hi, On 31/10/2022 18:39, Luca Weiss wrote: > usb-vbus-regulator is a better generic node name than dcdc to change the > example to match. Subject is wrong, should be something like: dt-bindings: regulator: qcom,usb-vbus-regulator: update example node name > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > --- > Changes in v2: > * New patch > > .../devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml b/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml > index dbe78cd4adba..b1cff3adb21b 100644 > --- a/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml > +++ b/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml > @@ -33,7 +33,7 @@ examples: > pm8150b { > #address-cells = <1>; > #size-cells = <0>; > - pm8150b_vbus: dcdc@1100 { > + pm8150b_vbus: usb-vbus-regulator@1100 { > compatible = "qcom,pm8150b-vbus-reg"; > reg = <0x1100>; > }; Thanks, Neil
Hi Neil, On Mittwoch, 2. November 2022 11:38:29 CET Neil Armstrong wrote: > Hi, > > On 31/10/2022 18:39, Luca Weiss wrote: > > usb-vbus-regulator is a better generic node name than dcdc to change the > > example to match. > > Subject is wrong, should be something like: > dt-bindings: regulator: qcom,usb-vbus-regulator: update example node name I've followed the prefix style of recent commits in Documentation/devicetree/bindings/regulator/ 475043fabe8c regulator: Fix qcom,spmi-regulator schema b0de7fa70650 regulator: pca9450: Remove restrictions for regulator-name 4c7da3741804 regulator: Add missing type for 'regulator-microvolt-offset' 91de5eb17c10 regulator: pwm: Update Lee Jones' email address 044750573903 regulator: qcom,spmi-regulator: add PMP8074 PMIC 0b3bbd7646b0 regulator: qcom,spmi-regulator: Convert to dtschema 8cbb948a7cc2 regulator: dt-bindings: qcom,smd-rpm: Add PM8909 a9369fd316cf regulator: dt-bindings: Convert pwm-regulator to DT schema Regards Luca
On Mon, 31 Oct 2022 18:39:32 +0100, Luca Weiss wrote: > usb-vbus-regulator is a better generic node name than dcdc to change the > example to match. > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > --- > Changes in v2: > * New patch > > .../devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Acked-by: Rob Herring <robh@kernel.org>
On 02/11/2022 06:38, Neil Armstrong wrote: > Hi, > > On 31/10/2022 18:39, Luca Weiss wrote: >> usb-vbus-regulator is a better generic node name than dcdc to change the >> example to match. > > Subject is wrong, should be something like: > dt-bindings: regulator: qcom,usb-vbus-regulator: update example node name > No, subject is correct. Best regards, Krzysztof
On Montag, 31. Oktober 2022 18:39:32 CET Luca Weiss wrote: > usb-vbus-regulator is a better generic node name than dcdc to change the > example to match. > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> Bump? Can this patch be picked up please? > --- > Changes in v2: > * New patch > > .../devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git > a/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml > b/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml > index dbe78cd4adba..b1cff3adb21b 100644 > --- > a/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml > +++ > b/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml > @@ -33,7 +33,7 @@ examples: > pm8150b { > #address-cells = <1>; > #size-cells = <0>; > - pm8150b_vbus: dcdc@1100 { > + pm8150b_vbus: usb-vbus-regulator@1100 { > compatible = "qcom,pm8150b-vbus-reg"; > reg = <0x1100>; > };
On Wed, Dec 28, 2022 at 10:30:47PM +0100, Luca Weiss wrote: > On Montag, 31. Oktober 2022 18:39:32 CET Luca Weiss wrote: > > usb-vbus-regulator is a better generic node name than dcdc to change the > > example to match. > > > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > > Bump? Can this patch be picked up please? Please don't send content free pings and please allow a reasonable time for review. People get busy, go on holiday, attend conferences and so on so unless there is some reason for urgency (like critical bug fixes) please allow at least a couple of weeks for review. If there have been review comments then people may be waiting for those to be addressed. Sending content free pings adds to the mail volume (if they are seen at all) which is often the problem and since they can't be reviewed directly if something has gone wrong you'll have to resend the patches anyway, so sending again is generally a better approach though there are some other maintainers who like them - if in doubt look at how patches for the subsystem are normally handled.
On Donnerstag, 29. Dezember 2022 18:57:15 CET Mark Brown wrote: > On Wed, Dec 28, 2022 at 10:30:47PM +0100, Luca Weiss wrote: > > On Montag, 31. Oktober 2022 18:39:32 CET Luca Weiss wrote: > > > usb-vbus-regulator is a better generic node name than dcdc to change the > > > example to match. > > > > > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > > > > Bump? Can this patch be picked up please? > > Please don't send content free pings and please allow a reasonable time > for review. People get busy, go on holiday, attend conferences and so > on so unless there is some reason for urgency (like critical bug fixes) > please allow at least a couple of weeks for review. If there have been > review comments then people may be waiting for those to be addressed. The patch was sent end of October, that is two months ago. I don't think two months of waiting is an unreasonable amount of time and is also more than "a couple of weeks"... And as far as I am aware there's no open review comments, which is why I did send the email to ask what the status is so that it can go into 6.3 and I can remove it from my inbox. Regards Luca > > Sending content free pings adds to the mail volume (if they are seen at > all) which is often the problem and since they can't be reviewed > directly if something has gone wrong you'll have to resend the patches > anyway, so sending again is generally a better approach though there are > some other maintainers who like them - if in doubt look at how patches > for the subsystem are normally handled.
On Thu, Dec 29, 2022 at 07:11:10PM +0100, Luca Weiss wrote: > The patch was sent end of October, that is two months ago. I don't think two > months of waiting is an unreasonable amount of time and is also more than "a > couple of weeks"... > And as far as I am aware there's no open review comments, which is why I did > send the email to ask what the status is so that it can go into 6.3 and I can > remove it from my inbox. Please read the rest of the mail: > > directly if something has gone wrong you'll have to resend the patches > > anyway, so sending again is generally a better approach though there are > > some other maintainers who like them - if in doubt look at how patches > > for the subsystem are normally handled. I can't do anything with your content free ping on patch 2 of some series...
On Donnerstag, 29. Dezember 2022 19:43:08 CET Mark Brown wrote: > On Thu, Dec 29, 2022 at 07:11:10PM +0100, Luca Weiss wrote: > > The patch was sent end of October, that is two months ago. I don't think > > two months of waiting is an unreasonable amount of time and is also more > > than "a couple of weeks"... > > > > And as far as I am aware there's no open review comments, which is why I > > did send the email to ask what the status is so that it can go into 6.3 > > and I can remove it from my inbox. > > Please read the rest of the mail: > > > directly if something has gone wrong you'll have to resend the patches > > > anyway, so sending again is generally a better approach though there are > > > some other maintainers who like them - if in doubt look at how patches > > > for the subsystem are normally handled. > > I can't do anything with your content free ping on patch 2 of > some series... I'll just resend, that should alleviate all problems.
diff --git a/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml b/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml index dbe78cd4adba..b1cff3adb21b 100644 --- a/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml +++ b/Documentation/devicetree/bindings/regulator/qcom,usb-vbus-regulator.yaml @@ -33,7 +33,7 @@ examples: pm8150b { #address-cells = <1>; #size-cells = <0>; - pm8150b_vbus: dcdc@1100 { + pm8150b_vbus: usb-vbus-regulator@1100 { compatible = "qcom,pm8150b-vbus-reg"; reg = <0x1100>; };