From af68b2d70687e9ee925072617383aa2878a56bfc Mon Sep 17 00:00:00 2001 From: Peter Robinson Date: Sun, 27 Sep 2020 09:09:59 +0100 Subject: A few more Pinebook Pro fixes --- ...p-Keep-enabled-PWMs-running-while-probing.patch | 141 +++++++++++++++++++++ arm64-rockchip-pinebookpro-add-fuel-gauge.patch | 59 +++++++++ kernel.spec | 12 +- 3 files changed, 209 insertions(+), 3 deletions(-) create mode 100644 arm64-pwm-rockchip-Keep-enabled-PWMs-running-while-probing.patch create mode 100644 arm64-rockchip-pinebookpro-add-fuel-gauge.patch diff --git a/arm64-pwm-rockchip-Keep-enabled-PWMs-running-while-probing.patch b/arm64-pwm-rockchip-Keep-enabled-PWMs-running-while-probing.patch new file mode 100644 index 000000000..4bdb5cb76 --- /dev/null +++ b/arm64-pwm-rockchip-Keep-enabled-PWMs-running-while-probing.patch @@ -0,0 +1,141 @@ +From patchwork Sat Sep 19 19:33:06 2020 +Content-Type: text/plain; charset="utf-8" +MIME-Version: 1.0 +Content-Transfer-Encoding: 8bit +X-Patchwork-Submitter: Simon South +X-Patchwork-Id: 11787259 +Return-Path: + +Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org + [172.30.200.123]) + by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id D0780618 + for ; + Sat, 19 Sep 2020 19:33:55 +0000 (UTC) +Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) + (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) + (No client certificate requested) + by mail.kernel.org (Postfix) with ESMTPS id 9DF4C21707 + for ; + Sat, 19 Sep 2020 19:33:55 +0000 (UTC) +Authentication-Results: mail.kernel.org; + dkim=pass (2048-bit key) header.d=lists.infradead.org + header.i=@lists.infradead.org header.b="tXxoe57q" +DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9DF4C21707 +Authentication-Results: mail.kernel.org; + dmarc=none (p=none dis=none) header.from=simonsouth.net +Authentication-Results: mail.kernel.org; + spf=none + smtp.mailfrom=linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org +DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; + d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: + Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: + List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:To:From: + Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender + :Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; + bh=rTFhR4wE/kksRMBhCZanM50Dz/U+ohZU+WFgqGfznMQ=; b=tXxoe57qQlrQ18QBvKs3JZ7wYU + mFyPVtA2yDfFfGESvkL1Xa44HehjqPoeSxNBAyejg6YfozEgeDKKVZsBEex5hw6LE56H8Iofj0pZV + AFM98+vh4tydKA7vxhGl0cm7jMVq3N68wqcikUkcdCEs3bp+5dHpOowzZaj3sm51gw/g6pD3ut1+V + IYlRTF/VuDd3IDW4o27pNrgIqWx9f+/eO1gUdxwCa4i0VNLAHXCt5ysGE+CSwVWMEcp5YN5K1iO8b + sFKtjMiOcg9wrJdEoYpiFbcHETgUGNj5wdbwRvJR2qZ36gSImyUqJobYqSUyVHCJI2JRUch3UDFtt + yDQgQ99A==; +Received: from localhost ([::1] helo=merlin.infradead.org) + by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) + id 1kJic9-0004AL-NX; Sat, 19 Sep 2020 19:33:41 +0000 +Received: from mailout.easymail.ca ([64.68.200.34]) + by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) + id 1kJic6-00049d-Ia; Sat, 19 Sep 2020 19:33:39 +0000 +Received: from localhost (localhost [127.0.0.1]) + by mailout.easymail.ca (Postfix) with ESMTP id C7C54A05B3; + Sat, 19 Sep 2020 19:33:37 +0000 (UTC) +X-Virus-Scanned: Debian amavisd-new at emo05-pco.easydns.vpn +Received: from mailout.easymail.ca ([127.0.0.1]) + by localhost (emo05-pco.easydns.vpn [127.0.0.1]) (amavisd-new, port 10024) + with ESMTP id G6RbiLyf29ms; Sat, 19 Sep 2020 19:33:37 +0000 (UTC) +Received: from jupiter.simonsouth.net (unknown [108.162.141.195]) + (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) + (No client certificate requested) + by mailout.easymail.ca (Postfix) with ESMTPSA id 77C83A025D; + Sat, 19 Sep 2020 19:33:27 +0000 (UTC) +From: Simon South +To: thierry.reding@gmail.com, u.kleine-koenig@pengutronix.de, + lee.jones@linaro.org, heiko@sntech.de, linux-pwm@vger.kernel.org, + linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org +Subject: [PATCH v2] pwm: rockchip: Keep enabled PWMs running while probing +Date: Sat, 19 Sep 2020 15:33:06 -0400 +Message-Id: <20200919193306.1023-1-simon@simonsouth.net> +X-Mailer: git-send-email 2.28.0 +MIME-Version: 1.0 +X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 +X-CRM114-CacheID: sfid-20200919_153338_624271_ABEEE8C4 +X-CRM114-Status: GOOD ( 16.76 ) +X-Spam-Score: -2.3 (--) +X-Spam-Report: SpamAssassin version 3.4.4 on merlin.infradead.org summary: + Content analysis details: (-2.3 points) + pts rule name description + ---- ---------------------- + -------------------------------------------------- + -2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at https://www.dnswl.org/, + medium trust [64.68.200.34 listed in list.dnswl.org] + 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record + -0.0 SPF_PASS SPF: sender matches SPF record +X-BeenThere: linux-arm-kernel@lists.infradead.org +X-Mailman-Version: 2.1.29 +Precedence: list +List-Id: +List-Unsubscribe: + , + +List-Archive: +List-Post: +List-Help: +List-Subscribe: + , + +Cc: Simon South +Sender: "linux-arm-kernel" +Errors-To: + linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org + +Following commit cfc4c189bc70 ("pwm: Read initial hardware state at +request time") the Rockchip PWM driver can no longer assume a device's +pwm_state structure has been populated after a call to pwmchip_add(). +Consequently, the test in rockchip_pwm_probe() intended to prevent the +driver from stopping PWM devices already enabled by the bootloader no +longer functions reliably and this can lead to the kernel hanging +during startup, particularly on devices like the Pinebook Pro that use +a PWM-controlled backlight for their display. + +Avoid this by querying the device directly at probe time to determine +whether or not it is enabled. + +Fixes: cfc4c189bc70 ("pwm: Read initial hardware state at request time") +Signed-off-by: Simon South +Reviewed-by: Uwe Kleine-König +Reviewed-by: Heiko Stuebner +--- + drivers/pwm/pwm-rockchip.c | 5 ++++- + 1 file changed, 4 insertions(+), 1 deletion(-) + +diff --git a/drivers/pwm/pwm-rockchip.c b/drivers/pwm/pwm-rockchip.c +index eb8c9cb645a6..098e94335cb5 100644 +--- a/drivers/pwm/pwm-rockchip.c ++++ b/drivers/pwm/pwm-rockchip.c +@@ -288,6 +288,7 @@ static int rockchip_pwm_probe(struct platform_device *pdev) + const struct of_device_id *id; + struct rockchip_pwm_chip *pc; + struct resource *r; ++ u32 enable_conf, ctrl; + int ret, count; + + id = of_match_device(rockchip_pwm_dt_ids, &pdev->dev); +@@ -362,7 +363,9 @@ static int rockchip_pwm_probe(struct platform_device *pdev) + } + + /* Keep the PWM clk enabled if the PWM appears to be up and running. */ +- if (!pwm_is_enabled(pc->chip.pwms)) ++ enable_conf = pc->data->enable_conf; ++ ctrl = readl_relaxed(pc->base + pc->data->regs.ctrl); ++ if ((ctrl & enable_conf) != enable_conf) + clk_disable(pc->clk); + + return 0; diff --git a/arm64-rockchip-pinebookpro-add-fuel-gauge.patch b/arm64-rockchip-pinebookpro-add-fuel-gauge.patch new file mode 100644 index 000000000..5decda78a --- /dev/null +++ b/arm64-rockchip-pinebookpro-add-fuel-gauge.patch @@ -0,0 +1,59 @@ +From c7c4d698cd2882c4d095aeed43bbad6fc990e998 Mon Sep 17 00:00:00 2001 +From: Tobias Schramm +Date: Thu, 28 May 2020 19:25:50 +0200 +Subject: arm64: dts: rockchip: add fuel gauge to Pinebook Pro dts + +This commit adds cw2015 fuel gauge and battery to the Pinebook Pro dts. + +Signed-off-by: Tobias Schramm +Link: https://lore.kernel.org/r/20200528172550.2324722-2-t.schramm@manjaro.org +Signed-off-by: Heiko Stuebner +--- + .../boot/dts/rockchip/rk3399-pinebook-pro.dts | 25 ++++++++++++++++++++++ + 1 file changed, 25 insertions(+) + +diff --git a/arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dts b/arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dts +index cb0245d2226d5..8f5b2df01560e 100644 +--- a/arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dts ++++ b/arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dts +@@ -28,6 +28,13 @@ + pwms = <&pwm0 0 740740 0>; + }; + ++ bat: battery { ++ compatible = "simple-battery"; ++ charge-full-design-microamp-hours = <9800000>; ++ voltage-max-design-microvolt = <4350000>; ++ voltage-min-design-microvolt = <3000000>; ++ }; ++ + edp_panel: edp-panel { + compatible = "boe,nv140fhmn49"; + backlight = <&backlight>; +@@ -741,6 +748,24 @@ + }; + }; + }; ++ ++ cw2015@62 { ++ compatible = "cellwise,cw2015"; ++ reg = <0x62>; ++ cellwise,battery-profile = /bits/ 8 < ++ 0x17 0x67 0x80 0x73 0x6E 0x6C 0x6B 0x63 ++ 0x77 0x51 0x5C 0x58 0x50 0x4C 0x48 0x36 ++ 0x15 0x0C 0x0C 0x19 0x5B 0x7D 0x6F 0x69 ++ 0x69 0x5B 0x0C 0x29 0x20 0x40 0x52 0x59 ++ 0x57 0x56 0x54 0x4F 0x3B 0x1F 0x7F 0x17 ++ 0x06 0x1A 0x30 0x5A 0x85 0x93 0x96 0x2D ++ 0x48 0x77 0x9C 0xB3 0x80 0x52 0x94 0xCB ++ 0x2F 0x00 0x64 0xA5 0xB5 0x11 0xF0 0x11 ++ >; ++ cellwise,monitor-interval-ms = <5000>; ++ monitored-battery = <&bat>; ++ power-supplies = <&mains_charger>, <&fusb0>; ++ }; + }; + + &i2s1 { +-- +cgit 1.2.3-1.el7 diff --git a/kernel.spec b/kernel.spec index f00c50d3d..21b92bda1 100644 --- a/kernel.spec +++ b/kernel.spec @@ -864,9 +864,6 @@ Patch103: arm64-tegra-Use-valid-PWM-period-for-VDD_GPU-on-Tegra210.patch # Goes away with 5.9 Patch105: 0001-platform-x86-thinkpad_acpi-lap-or-desk-mode-interfac.patch -# https://patchwork.kernel.org/patch/11796255/ -Patch106: arm64-dts-rockchip-disable-USB-type-c-DisplayPort.patch - # https://bugzilla.redhat.com/show_bug.cgi?id=1874117 Patch107: 0001-drivers-perf-xgene_pmu-Fix-uninitialized-resource-st.patch @@ -887,6 +884,15 @@ Patch114: v2-nfs-Fix-security-label-length-not-being-reset.patch # rhbz 1875339 1875828 1876997 Patch115: pdx86-SW_TABLET_MODE-fixes.patch +# https://patchwork.kernel.org/patch/11796255/ +Patch116: arm64-dts-rockchip-disable-USB-type-c-DisplayPort.patch + +# https://patchwork.kernel.org/patch/11787259/ +Patch117: arm64-pwm-rockchip-Keep-enabled-PWMs-running-while-probing.patch + +# Backport from 5.9 +Patch118: arm64-rockchip-pinebookpro-add-fuel-gauge.patch + # END OF PATCH DEFINITIONS %endif -- cgit