From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko =?ISO-8859-1?Q?St=FCbner?= Subject: Re: [PATCH v4 2/2] arm64: dts: rockchip: Add initial support for Pinebook Pro Date: Fri, 06 Mar 2020 20:09:05 +0100 Message-ID: <1783248.GgDFSgCcj8@diego> References: <20200304213023.689983-2-t.schramm@manjaro.org> <6168222.Wuk326WHQK@phil> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Tobias Schramm Cc: Mark Rutland , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Emmanuel Vadot , Alexis Ballier , Katsuhiro Suzuki , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Douglas Anderson , Kever Yang , Markus Reichl , linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Rob Herring , Jagan Teki , Nick Xie , Andy Yan , Johan Jonker , Matthias Kaehlcke , Vivek Unune , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: linux-rockchip.vger.kernel.org Hi Tobias, Am Freitag, 6. März 2020, 19:58:45 CET schrieb Tobias Schramm: > Hi Heiko, > > >> This commit adds initial dt support for the rk3399 based Pinebook Pro. > >> > >> Signed-off-by: Tobias Schramm > > > > applied for 5.7 > > > > any chance you can squeeze in another small fix? Somewhere in the > process the vmcc and vqmmc supplies of the sdmmc controller were removed > in error. Those should be added to the sdmmc controller like this: > > &sdmmc { > pinctrl-names = "default"; > pinctrl-0 = <&sdmmc_clk &sdmmc_cmd &sdmmc_bus4>; > sd-uhs-sdr104; > + vmmc-supply = <&vcc3v0_sd>; > + vqmmc-supply = <&vcc_sdio>; > status = "okay"; > }; I've amended the commit with these two properties. Heiko