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 X-Spam-Level: X-Spam-Status: No, score=-6.5 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1B70EC3F2D3 for ; Fri, 28 Feb 2020 14:34:26 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id E75E82469F for ; Fri, 28 Feb 2020 14:34:25 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=crapouillou.net header.i=@crapouillou.net header.b="WpX2Sf49" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727022AbgB1OeZ (ORCPT ); Fri, 28 Feb 2020 09:34:25 -0500 Received: from outils.crapouillou.net ([89.234.176.41]:36694 "EHLO crapouillou.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726875AbgB1OeZ (ORCPT ); Fri, 28 Feb 2020 09:34:25 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=crapouillou.net; s=mail; t=1582900463; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=2iYh4GJzCLrpeJtYfBs2xAG0SYopNEWP83AbHh4thW0=; b=WpX2Sf49Pg3NYVfFNkB1+sdFRtdvG7324DYZQJC+r0R8jOEbxPd9Sl1beMCcybT3g3qE/4 7VhUhSA40280rvkxb0YZeXo0Zi7SfuwjfGkva1nut9xwr5CRXaAJ+Zvr+MidI847N5jK8l /XvuqKBvVyA3Z0yMJA0rHGE/f2P3TMo= Date: Fri, 28 Feb 2020 11:34:04 -0300 From: Paul Cercueil Subject: Re: [PATCH v3 3/6] MIPS: DTS: CI20: fix PMU definitions for ACT8600 To: "H. Nikolaus Schaller" Cc: Paul Boddie , Rob Herring , Mark Rutland , Ralf Baechle , Paul Burton , Miquel Raynal , Andi Kleen , Kees Cook , Krzysztof Kozlowski , devicetree@vger.kernel.org, linux-mips@vger.kernel.org, linux-kernel@vger.kernel.org, letux-kernel@openphoenux.org, kernel@pyra-handheld.com, stable@vger.kernel.org Message-Id: <1582900444.3.1@crapouillou.net> In-Reply-To: <36aa1e80153fbb29eeb56f65cac9e3672165f7b7.1581884459.git.hns@goldelico.com> References: <36aa1e80153fbb29eeb56f65cac9e3672165f7b7.1581884459.git.hns@goldelico.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1; format=flowed Content-Transfer-Encoding: quoted-printable Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Hi Nikolaus, Le dim., f=E9vr. 16, 2020 at 21:20, H. Nikolaus Schaller=20 a =E9crit : > There is a ACT8600 on the CI20 board and the bindings of the > ACT8865 driver have changed without updating the CI20 device > tree. Therefore the PMU can not be probed successfully and > is running in power-on reset state. >=20 > Fix DT to match the latest act8865-regulator bindings. >=20 > Fixes: 73f2b940474d ("MIPS: CI20: DTS: Add I2C nodes") > Cc: stable@vger.kernel.org > Signed-off-by: H. Nikolaus Schaller > --- > arch/mips/boot/dts/ingenic/ci20.dts | 48=20 > ++++++++++++++++++++--------- > 1 file changed, 33 insertions(+), 15 deletions(-) >=20 > diff --git a/arch/mips/boot/dts/ingenic/ci20.dts=20 > b/arch/mips/boot/dts/ingenic/ci20.dts > index 59c104289ece..4f48bc16fb52 100644 > --- a/arch/mips/boot/dts/ingenic/ci20.dts > +++ b/arch/mips/boot/dts/ingenic/ci20.dts > @@ -153,6 +153,8 @@ > pinctrl-0 =3D <&pins_uart4>; > }; >=20 > +#include Includes at the beginning of the file please. Keeps it tidy. > + > &i2c0 { > status =3D "okay"; >=20 > @@ -166,65 +168,81 @@ > reg =3D <0x5a>; > status =3D "okay"; >=20 > +/* > +Optional input supply properties: > +- for act8600: > + - vp1-supply: The input supply for DCDC_REG1 > + - vp2-supply: The input supply for DCDC_REG2 > + - vp3-supply: The input supply for DCDC_REG3 > + - inl-supply: The input supply for LDO_REG5, LDO_REG6, LDO_REG7=20 > and LDO_REG8 > + SUDCDC_REG4, LDO_REG9 and LDO_REG10 do not have separate supplies. > +*/ > + > regulators { > vddcore: SUDCDC1 { > - regulator-name =3D "VDDCORE"; > + regulator-name =3D "DCDC_REG1"; > regulator-min-microvolt =3D <1100000>; > regulator-max-microvolt =3D <1100000>; > regulator-always-on; > }; > vddmem: SUDCDC2 { > - regulator-name =3D "VDDMEM"; > + regulator-name =3D "DCDC_REG2"; > regulator-min-microvolt =3D <1500000>; > regulator-max-microvolt =3D <1500000>; > regulator-always-on; > }; > vcc_33: SUDCDC3 { > - regulator-name =3D "VCC33"; > + regulator-name =3D "DCDC_REG3"; > regulator-min-microvolt =3D <3300000>; > regulator-max-microvolt =3D <3300000>; > regulator-always-on; > }; > vcc_50: SUDCDC4 { > - regulator-name =3D "VCC50"; > + regulator-name =3D "SUDCDC_REG4"; > regulator-min-microvolt =3D <5000000>; > regulator-max-microvolt =3D <5000000>; > regulator-always-on; > }; > vcc_25: LDO_REG5 { > - regulator-name =3D "VCC25"; > + regulator-name =3D "LDO_REG5"; > regulator-min-microvolt =3D <2500000>; > regulator-max-microvolt =3D <2500000>; > regulator-always-on; > }; > wifi_io: LDO_REG6 { > - regulator-name =3D "WIFIIO"; > + regulator-name =3D "LDO_REG6"; > regulator-min-microvolt =3D <2500000>; > regulator-max-microvolt =3D <2500000>; > regulator-always-on; > }; > vcc_28: LDO_REG7 { > - regulator-name =3D "VCC28"; > + regulator-name =3D "LDO_REG7"; > regulator-min-microvolt =3D <2800000>; > regulator-max-microvolt =3D <2800000>; > regulator-always-on; > }; > vcc_15: LDO_REG8 { > - regulator-name =3D "VCC15"; > + regulator-name =3D "LDO_REG8"; > regulator-min-microvolt =3D <1500000>; > regulator-max-microvolt =3D <1500000>; > regulator-always-on; > }; > - vcc_18: LDO_REG9 { > - regulator-name =3D "VCC18"; > - regulator-min-microvolt =3D <1800000>; > - regulator-max-microvolt =3D <1800000>; > + vrtc_18: LDO_REG9 { > + regulator-name =3D "LDO_REG9"; > + /* Despite the datasheet stating 3.3V for REG9 and > + driver expecting that, REG9 outputs 1.8V. > + Likely the CI20 uses a chip variant. > + Since it is a simple on/off LDO the exact values > + do not matter. > + */ > + regulator-min-microvolt =3D <3300000>; > + regulator-max-microvolt =3D <3300000>; > regulator-always-on; > }; > vcc_11: LDO_REG10 { > - regulator-name =3D "VCC11"; > - regulator-min-microvolt =3D <1100000>; > - regulator-max-microvolt =3D <1100000>; > + regulator-name =3D "LDO_REG10"; > + regulator-min-microvolt =3D <1200000>; > + regulator-max-microvolt =3D <1200000>; > regulator-always-on; > }; > }; > -- > 2.23.0 >=20 =