From: Alistair Francis <alistair23@gmail.com> To: Matti Vaittinen <mazziesaccount@gmail.com> Cc: Alistair Francis <alistair@alistair23.me>, s.hauer@pengutronix.de, devicetree@vger.kernel.org, shawnguo@kernel.org, linux-arm-kernel@lists.infradead.org, linux-imx@nxp.com, linux-kernel@vger.kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, arnd@arndb.de, kernel@pengutronix.de, jernej.skrabec@gmail.com, linux@armlinux.org.uk, festevam@gmail.com Subject: Re: [PATCH v3 3/3] ARM: dts: imx7d-remarkable2: Enable the rohm,bd71815 Date: Sun, 26 Feb 2023 15:50:31 +1000 [thread overview] Message-ID: <CAKmqyKPoOX5rg3CyCU=UAsZQaH3AMd-d0f-XT_wuRQdnTFc20w@mail.gmail.com> (raw) In-Reply-To: <52813148-fef6-aac3-09ff-65aac8426528@gmail.com> On Sun, Feb 26, 2023 at 12:22 AM Matti Vaittinen <mazziesaccount@gmail.com> wrote: > > Hi Alistair, > > On 2/25/23 13:37, Alistair Francis wrote: > > Add support for the rohm,bd71815 power controller controller for the > > reMarkable 2. > > > > Signed-off-by: Alistair Francis <alistair@alistair23.me> > > --- > > arch/arm/boot/dts/imx7d-remarkable2.dts | 158 ++++++++++++++++++++++++ > > 1 file changed, 158 insertions(+) > > > > diff --git a/arch/arm/boot/dts/imx7d-remarkable2.dts b/arch/arm/boot/dts/imx7d-remarkable2.dts > > index 288fc8611117..9ecb733545cc 100644 > > --- a/arch/arm/boot/dts/imx7d-remarkable2.dts > > +++ b/arch/arm/boot/dts/imx7d-remarkable2.dts > > @@ -91,6 +91,10 @@ wifi_pwrseq: wifi_pwrseq { > > }; > > }; > > > > +&cpu0 { > > + cpu-supply = <&buck1_reg>; > > +}; > > + > > &clks { > > assigned-clocks = <&clks IMX7D_CLKO2_ROOT_SRC>, > > <&clks IMX7D_CLKO2_ROOT_DIV>; > > @@ -118,6 +122,147 @@ wacom_digitizer: digitizer@9 { > > }; > > }; > > > > +&i2c2 { > > + clock-frequency = <100000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_i2c2>; > > + status = "okay"; > > + > > + bd71815: pmic@4b { > > + compatible = "rohm,bd71815"; > > + reg = <0x4b>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_bd71815>; > > + interrupt-parent = <&gpio6>; /* PMIC_INT_B GPIO6_IO16 */ > > + interrupts = <16 IRQ_TYPE_LEVEL_LOW>; > > + gpio-controller; > > + clocks = <&clks IMX7D_CLKO2_ROOT_SRC>; > > + clock-output-names = "bd71815-32k-out"; > > + #clock-cells = <0>; > > + #gpio-cells = <1>; > > + > > + regulators { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + buck1_reg: regulator@0 { > > + reg = <0>; > > + regulator-compatible = "buck1"; > > I'm sure you have learned to regard my comments with certain care ;) (I > think it was you for whom I gave an advice - which resulted a board to > be bricked :| So, please treat my comment as if I did not know what I am > talking about). That was me! The board wasn't bricked though, luckily as it's my only device. It took a month of full battery discharge cycles but I managed to replace the device tree with a working one :) > > Anyways, I believe the "regulator-compatible" is deprecated and should > no longer be used to match the node with the regulator? Instead, the > regulator node name itself should be used for the matching. > > ref. > https://elixir.bootlin.com/linux/latest/source/drivers/regulator/of_regulator.c#L380 > > I don't think the regulator-compatible can even be found from the > regulator.yaml binding list... You're right. Thanks for that, I have removed them and replaced them with "regulator-name" to match the documentation. > > With this remark - (and what ever it is worth): > Reviewed-by: Matti Vaittinen <mazziesaccount@gmail.com> Thanks! Alistair > > -- > Matti Vaittinen > Linux kernel developer at ROHM Semiconductors > Oulu Finland > > ~~ When things go utterly wrong vim users can always type :help! ~~ >
WARNING: multiple messages have this Message-ID (diff)
From: Alistair Francis <alistair23@gmail.com> To: Matti Vaittinen <mazziesaccount@gmail.com> Cc: Alistair Francis <alistair@alistair23.me>, s.hauer@pengutronix.de, devicetree@vger.kernel.org, shawnguo@kernel.org, linux-arm-kernel@lists.infradead.org, linux-imx@nxp.com, linux-kernel@vger.kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, arnd@arndb.de, kernel@pengutronix.de, jernej.skrabec@gmail.com, linux@armlinux.org.uk, festevam@gmail.com Subject: Re: [PATCH v3 3/3] ARM: dts: imx7d-remarkable2: Enable the rohm,bd71815 Date: Sun, 26 Feb 2023 15:50:31 +1000 [thread overview] Message-ID: <CAKmqyKPoOX5rg3CyCU=UAsZQaH3AMd-d0f-XT_wuRQdnTFc20w@mail.gmail.com> (raw) In-Reply-To: <52813148-fef6-aac3-09ff-65aac8426528@gmail.com> On Sun, Feb 26, 2023 at 12:22 AM Matti Vaittinen <mazziesaccount@gmail.com> wrote: > > Hi Alistair, > > On 2/25/23 13:37, Alistair Francis wrote: > > Add support for the rohm,bd71815 power controller controller for the > > reMarkable 2. > > > > Signed-off-by: Alistair Francis <alistair@alistair23.me> > > --- > > arch/arm/boot/dts/imx7d-remarkable2.dts | 158 ++++++++++++++++++++++++ > > 1 file changed, 158 insertions(+) > > > > diff --git a/arch/arm/boot/dts/imx7d-remarkable2.dts b/arch/arm/boot/dts/imx7d-remarkable2.dts > > index 288fc8611117..9ecb733545cc 100644 > > --- a/arch/arm/boot/dts/imx7d-remarkable2.dts > > +++ b/arch/arm/boot/dts/imx7d-remarkable2.dts > > @@ -91,6 +91,10 @@ wifi_pwrseq: wifi_pwrseq { > > }; > > }; > > > > +&cpu0 { > > + cpu-supply = <&buck1_reg>; > > +}; > > + > > &clks { > > assigned-clocks = <&clks IMX7D_CLKO2_ROOT_SRC>, > > <&clks IMX7D_CLKO2_ROOT_DIV>; > > @@ -118,6 +122,147 @@ wacom_digitizer: digitizer@9 { > > }; > > }; > > > > +&i2c2 { > > + clock-frequency = <100000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_i2c2>; > > + status = "okay"; > > + > > + bd71815: pmic@4b { > > + compatible = "rohm,bd71815"; > > + reg = <0x4b>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_bd71815>; > > + interrupt-parent = <&gpio6>; /* PMIC_INT_B GPIO6_IO16 */ > > + interrupts = <16 IRQ_TYPE_LEVEL_LOW>; > > + gpio-controller; > > + clocks = <&clks IMX7D_CLKO2_ROOT_SRC>; > > + clock-output-names = "bd71815-32k-out"; > > + #clock-cells = <0>; > > + #gpio-cells = <1>; > > + > > + regulators { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + buck1_reg: regulator@0 { > > + reg = <0>; > > + regulator-compatible = "buck1"; > > I'm sure you have learned to regard my comments with certain care ;) (I > think it was you for whom I gave an advice - which resulted a board to > be bricked :| So, please treat my comment as if I did not know what I am > talking about). That was me! The board wasn't bricked though, luckily as it's my only device. It took a month of full battery discharge cycles but I managed to replace the device tree with a working one :) > > Anyways, I believe the "regulator-compatible" is deprecated and should > no longer be used to match the node with the regulator? Instead, the > regulator node name itself should be used for the matching. > > ref. > https://elixir.bootlin.com/linux/latest/source/drivers/regulator/of_regulator.c#L380 > > I don't think the regulator-compatible can even be found from the > regulator.yaml binding list... You're right. Thanks for that, I have removed them and replaced them with "regulator-name" to match the documentation. > > With this remark - (and what ever it is worth): > Reviewed-by: Matti Vaittinen <mazziesaccount@gmail.com> Thanks! Alistair > > -- > Matti Vaittinen > Linux kernel developer at ROHM Semiconductors > Oulu Finland > > ~~ When things go utterly wrong vim users can always type :help! ~~ > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-02-26 5:51 UTC|newest] Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-02-25 11:37 [PATCH v3 0/3] imx7d-remarkable2: Enable cyttsp5 and rohm,bd71815 Alistair Francis 2023-02-25 11:37 ` Alistair Francis 2023-02-25 11:37 ` [PATCH v3 1/3] ARM: dts: imx7d-remarkable2: Enable the cyttsp5 Alistair Francis 2023-02-25 11:37 ` Alistair Francis 2023-02-25 11:37 ` [PATCH v3 2/3] ARM: imx_v6_v7_defconfig: Enable rohm,bd71815 Alistair Francis 2023-02-25 11:37 ` Alistair Francis 2023-02-25 11:37 ` [PATCH v3 3/3] ARM: dts: imx7d-remarkable2: Enable the rohm,bd71815 Alistair Francis 2023-02-25 11:37 ` Alistair Francis 2023-02-25 14:22 ` Matti Vaittinen 2023-02-25 14:22 ` Matti Vaittinen 2023-02-26 5:50 ` Alistair Francis [this message] 2023-02-26 5:50 ` Alistair Francis 2023-02-25 20:37 ` Fabio Estevam 2023-02-25 20:37 ` Fabio Estevam 2023-02-26 5:52 ` Alistair Francis 2023-02-26 5:52 ` Alistair Francis
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAKmqyKPoOX5rg3CyCU=UAsZQaH3AMd-d0f-XT_wuRQdnTFc20w@mail.gmail.com' \ --to=alistair23@gmail.com \ --cc=alistair@alistair23.me \ --cc=arnd@arndb.de \ --cc=devicetree@vger.kernel.org \ --cc=festevam@gmail.com \ --cc=jernej.skrabec@gmail.com \ --cc=kernel@pengutronix.de \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-imx@nxp.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=mazziesaccount@gmail.com \ --cc=robh+dt@kernel.org \ --cc=s.hauer@pengutronix.de \ --cc=shawnguo@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.