Received: by 2002:a05:7412:251c:b0:e2:908c:2ebd with SMTP id w28csp2432779rda; Wed, 25 Oct 2023 02:32:56 -0700 (PDT) X-Google-Smtp-Source: AGHT+IFrEGAmFwV9PvmbDjc0feCYvzCdYm110mW7gs+oXtn8jFHq/K/Cm0TiD1QyASjyNGtvNglI X-Received: by 2002:a9d:63d7:0:b0:6b9:62f5:ac62 with SMTP id e23-20020a9d63d7000000b006b962f5ac62mr12354014otl.35.1698226376464; Wed, 25 Oct 2023 02:32:56 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1698226376; cv=none; d=google.com; s=arc-20160816; b=L5uEFWCbUVl817xp6pLKUSJ+nstbT10sAVGwuTBIikAdxR5UhCguIjciNpyuQD6zec ma45xboFbFsvJ/otVZGZrScEhBDcwQlWtkM+KfX2AdajrQdUsSP434tzbNEnVhRLm7mz 2xZHm7tjw+ItkWohUgAY1CEWF68NNtud1LWM0b7Y72lhsupbyoGK39jz+5H1b+NspUlR nt6X2nsfhe2YLlpYik74yWJ65p91O+9mu0+7xrF9zrmSy3IvJUaqDzofoM2KSacw/Krr qkej8Ujqqh2Co+9/c19EWKujv0m1U9GFruI+m5UBYJyF/Fq/6edBCq1UIvYX3z5h2oGO aOAQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=JleXfSUkp+S9wegLsWLcm92sHQy5rJ9h3rjZTaRBvnc=; fh=sYEXsaPdh6O0wHn4cPOnFv0fZZZDCkKw67KbMz5E/5s=; b=lkYwmmq0Dg2ncmCK/VnYzOvyLd/MC09i95WMnpmA8YQIVvLaSM/p+pbyEoxlFrcQQN cKdaPsHr+Yczj/dUju2TlK3AkOKjQ5ByQGtQL9qVIFxdUEVvx/aoqfBg1CcKIz+A3v4T EttKtZBxGHHrdSyE5/5WVwNqvEFF0zh4Oyl9rPNbbpouAsB1xyRjwaZOfqybXAEZzDMD LuS4KQ1yoc94XX2H2L6TezTcTHJnlOvvtEw8KZahf4WarVvQXEaEWkMTJZ7rtFCeTjFE Og6yLvTHa8x5YnuXBFTIBrgAi+oYSOLuZpJ/ljDhrnRUGxPx2rYFSVbk8Cu5msnT3I1i kc6w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sifive.com header.s=google header.b=OYKclmbK; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=sifive.com Return-Path: Received: from agentk.vger.email (agentk.vger.email. [23.128.96.32]) by mx.google.com with ESMTPS id u197-20020a8184ce000000b005a8aa020effsi10208983ywf.72.2023.10.25.02.32.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 25 Oct 2023 02:32:56 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) client-ip=23.128.96.32; Authentication-Results: mx.google.com; dkim=pass header.i=@sifive.com header.s=google header.b=OYKclmbK; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=sifive.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by agentk.vger.email (Postfix) with ESMTP id 3713C80AD9BB; Wed, 25 Oct 2023 02:32:53 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at agentk.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231794AbjJYJci (ORCPT + 99 others); Wed, 25 Oct 2023 05:32:38 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50546 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233761AbjJYJcg (ORCPT ); Wed, 25 Oct 2023 05:32:36 -0400 Received: from mail-oo1-xc33.google.com (mail-oo1-xc33.google.com [IPv6:2607:f8b0:4864:20::c33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 241FF132 for ; Wed, 25 Oct 2023 02:32:34 -0700 (PDT) Received: by mail-oo1-xc33.google.com with SMTP id 006d021491bc7-581ed744114so2193841eaf.0 for ; Wed, 25 Oct 2023 02:32:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sifive.com; s=google; t=1698226353; x=1698831153; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=JleXfSUkp+S9wegLsWLcm92sHQy5rJ9h3rjZTaRBvnc=; b=OYKclmbKyWktjRtAkSdXs0Ix/x913g4zDh6w1aAKWYDyoETuShoaQDTwZRVDQHd+Gw CnHsjqRK5LaLMAA8AlAAcnqAXfYx0sFYrsPuas4L1MzkwZY3PmBU5IOkJ8p5M++phScJ ITOfRQhfbkJTnv+Oq4YLbJpNqucExlUPOnfKEmGjyjOJkvE4dYT20QTGojo+x4O9cTH6 TbS/bwLi+ER1idQxkedtbhJPOf2Y2It4ZfI+IRbAWOLXa5VUC8V10HjOjZziDRdDEkzg joB/EEcelncRdN+3/xlF9Bsj5ZyUdITd8RbjnbinHjPDMYfCgRmoOPJ5dUMDX+zeVQfK vLLQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1698226353; x=1698831153; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=JleXfSUkp+S9wegLsWLcm92sHQy5rJ9h3rjZTaRBvnc=; b=nSorzgdXlu3KFwWURh/AxABHxKTyBs7JYktBpvaTUNZJS6srht40LekcEP0jcFR97y XKkUTlnsskGdqoUbj8ZrHlwEq+PqfH4d1y0TpJuYt946kV08Rce1KB57dWAiv4MxNRS2 rMX3bZ4SZD4WmUUhZW9s7j4ceEYxMq8L2TWKwzsR4zFyQ2OfRl9Ypvwz/jwCj6/8LNWP bqWqS25xaGbCmPNnAHgSItz7oMHKprz4nu2clHzdnmk1cQ6NlczykLVqIz9Isjb1HrAR rXnYr+OoeOk0TDy2gUlR1nwXp6tY1xsPrJQR1TRvmQZ2fPIl/XhyrH+2ZVCG8kUrmDVM WpBw== X-Gm-Message-State: AOJu0YxCz6j/yMvrT1Lv6IJP81iC7wKzkf7BchVtnugydVCeuGRcVSB5 anqMDrocBqZctfaTBa2S3EsbFtkoIdHdAz+fD0zULg== X-Received: by 2002:a05:6870:1250:b0:1ea:fca:8ca2 with SMTP id 16-20020a056870125000b001ea0fca8ca2mr12157627oao.56.1698226353289; Wed, 25 Oct 2023 02:32:33 -0700 (PDT) MIME-Version: 1.0 References: <20231024101902.6689-1-nylon.chen@sifive.com> <20231024101902.6689-2-nylon.chen@sifive.com> <20231024-yin-coliseum-11f5e06fec14@spud> In-Reply-To: <20231024-yin-coliseum-11f5e06fec14@spud> From: Nylon Chen Date: Wed, 25 Oct 2023 17:32:21 +0800 Message-ID: Subject: Re: [v5 1/2] riscv: dts: sifive: unleashed/unmatched: Remove PWM controlled LED's active-low properties To: Conor Dooley Cc: linux-pwm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, devicetree@vger.kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, paul.walmsley@sifive.com, palmer@dabbelt.com, aou@eecs.berkeley.edu, thierry.reding@gmail.com, u.kleine-koenig@pengutronix.de, emil.renner.berthing@canonical.com, vincent.chen@sifive.com, greentime.hu@sifive.com, zong.li@sifive.com, nylon7717@gmail.com Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Wed, 25 Oct 2023 02:32:53 -0700 (PDT) Hi Conor, Conor Dooley =E6=96=BC 2023=E5=B9=B410=E6=9C=8824=E6=97= =A5 =E9=80=B1=E4=BA=8C =E4=B8=8B=E5=8D=8810:55=E5=AF=AB=E9=81=93=EF=BC=9A > > Hey, > > On Tue, Oct 24, 2023 at 06:19:01PM +0800, Nylon Chen wrote: > > This removes the active-low properties of the PWM-controlled LEDs in > > the HiFive Unmatched device tree. > > > > The reference is hifive-unleashed-a00.pdf[0] and hifive-unmatched-schem= atics-v3.pdf[1]. > > > > Link: https://sifive.cdn.prismic.io/sifive/c52a8e32-05ce-4aaf-95c8-7bf8= 453f8698_hifive-unleashed-a00-schematics-1.pdf [0] > > Link: https://sifive.cdn.prismic.io/sifive/6a06d6c0-6e66-49b5-8e9e-e68c= e76f4192_hifive-unmatched-schematics-v3.pdf [1] > > > > > This blank line should be removed if there is a follow-up. thanks, I got it. > > > Signed-off-by: Vincent Chen > > What did Vincent contribute to this patch? Are you missing a > co-developed-by tag, perhaps? Yes, Vincent was the first person to find the PWM driver problem, and Zong Li helped me with the relevant review internally. so in the next version, I will add the correct tags for these two developer= s. Thank you again for your time. > > > Signed-off-by: Nylon Chen > > Reviewed-by: Conor Dooley > Acked-by: Conor Dooley > > I expect this to go via the pwm tree since this is going to "break" (in > the loosest possible sense) existing systems if merged separately. > > Cheers, > Conor. > > > --- > > arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts | 8 ++++---- > > arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts | 12 ++++-------- > > 2 files changed, 8 insertions(+), 12 deletions(-) > > > > diff --git a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts b/arch= /riscv/boot/dts/sifive/hifive-unleashed-a00.dts > > index 900a50526d77..11e7ac1c54bb 100644 > > --- a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts > > +++ b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts > > @@ -49,7 +49,7 @@ led-controller { > > compatible =3D "pwm-leds"; > > > > led-d1 { > > - pwms =3D <&pwm0 0 7812500 PWM_POLARITY_INVERTED>; > > + pwms =3D <&pwm0 0 7812500 0>; > > active-low; > > color =3D ; > > max-brightness =3D <255>; > > @@ -57,7 +57,7 @@ led-d1 { > > }; > > > > led-d2 { > > - pwms =3D <&pwm0 1 7812500 PWM_POLARITY_INVERTED>; > > + pwms =3D <&pwm0 1 7812500 0>; > > active-low; > > color =3D ; > > max-brightness =3D <255>; > > @@ -65,7 +65,7 @@ led-d2 { > > }; > > > > led-d3 { > > - pwms =3D <&pwm0 2 7812500 PWM_POLARITY_INVERTED>; > > + pwms =3D <&pwm0 2 7812500 0>; > > active-low; > > color =3D ; > > max-brightness =3D <255>; > > @@ -73,7 +73,7 @@ led-d3 { > > }; > > > > led-d4 { > > - pwms =3D <&pwm0 3 7812500 PWM_POLARITY_INVERTED>; > > + pwms =3D <&pwm0 3 7812500 0>; > > active-low; > > color =3D ; > > max-brightness =3D <255>; > > diff --git a/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts b/arch= /riscv/boot/dts/sifive/hifive-unmatched-a00.dts > > index 07387f9c135c..b328ee80693f 100644 > > --- a/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts > > +++ b/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts > > @@ -51,8 +51,7 @@ led-controller-1 { > > compatible =3D "pwm-leds"; > > > > led-d12 { > > - pwms =3D <&pwm0 0 7812500 PWM_POLARITY_INVERTED>; > > - active-low; > > + pwms =3D <&pwm0 0 7812500 0>; > > color =3D ; > > max-brightness =3D <255>; > > label =3D "d12"; > > @@ -68,20 +67,17 @@ multi-led { > > label =3D "d2"; > > > > led-red { > > - pwms =3D <&pwm0 2 7812500 PWM_POLARITY_IN= VERTED>; > > - active-low; > > + pwms =3D <&pwm0 2 7812500 0>; > > color =3D ; > > }; > > > > led-green { > > - pwms =3D <&pwm0 1 7812500 PWM_POLARITY_IN= VERTED>; > > - active-low; > > + pwms =3D <&pwm0 1 7812500 0>; > > color =3D ; > > }; > > > > led-blue { > > - pwms =3D <&pwm0 3 7812500 PWM_POLARITY_IN= VERTED>; > > - active-low; > > + pwms =3D <&pwm0 3 7812500 0>; > > color =3D ; > > }; > > }; > > -- > > 2.42.0 > >