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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 97E8EC36010 for ; Fri, 11 Apr 2025 15:05:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=RPrkADoo7VdRQpJb2qtFQrnqFAJ1uoUlYLatfqn8EX0=; b=kQOSmZU7avJA2T AY+b/1t/Y7X5msksVmg1RfBZeksIH/1ZFelElgbi5+slNqgeiWbOPSZpa4bb03QC9jbmtJCxfVOlm UXoqnvecNcpcFh8LR7ThIDxpbuDJbi3nPSCe0J8KoAO0Hj4sBjotB6cOBbhIlcBBHis9QEOw8oyPj UjxrnPHs37OazzLtZegkuBtYqrIzDION5RLCwKUOzvAI26g9P/MScvYkobEvnesh3b5GPVsxET1bn Ntk2Z2JnI2HqTgTRtTzAI60ncfoCqnfc8XEBnTdUMJr3wrjGpA57GcMhbHY3DdliY/cMGtRaeabJy IaYCX4stSHho54D5XVpQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1u3FwK-0000000EBVC-02pl; Fri, 11 Apr 2025 15:05:08 +0000 Received: from dev.gentoo.org ([2001:470:ea4a:1:5054:ff:fec7:86e4] helo=smtp.gentoo.org) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1u3Fdm-0000000E8uK-2M42 for linux-riscv@lists.infradead.org; Fri, 11 Apr 2025 14:45:59 +0000 Received: from localhost (unknown [116.232.27.72]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (prime256v1) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: dlan) by smtp.gentoo.org (Postfix) with ESMTPSA id DC77E3436BC; Fri, 11 Apr 2025 14:45:55 +0000 (UTC) Date: Fri, 11 Apr 2025 14:45:49 +0000 From: Yixun Lan To: Alex Elder Cc: Guodong Xu , ukleinek@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, paul.walmsley@sifive.com, palmer@dabbelt.com, aou@eecs.berkeley.edu, alex@ghiti.fr, p.zabel@pengutronix.de, drew@pdp7.com, inochiama@gmail.com, geert+renesas@glider.be, heylenay@4d2.org, tglx@linutronix.de, hal.feng@starfivetech.com, unicorn_wang@outlook.com, duje.mihanovic@skole.hr, linux-pwm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, spacemit@lists.linux.dev Subject: Re: [PATCH 7/9] riscv: dts: spacemit: Add PWM14 backlight support for BPI-F3 Message-ID: <20250411144549-GYA22568@gentoo> References: <20250411131423.3802611-1-guodong@riscstar.com> <20250411131423.3802611-8-guodong@riscstar.com> <20250411140510-GYA22364@gentoo> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250411_074558_634499_48312012 X-CRM114-Status: GOOD ( 26.51 ) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org Hi Alex, On 09:23 Fri 11 Apr , Alex Elder wrote: > On 4/11/25 9:05 AM, Yixun Lan wrote: > > > > On 21:14 Fri 11 Apr , Guodong Xu wrote: > >> Add a PWM-based backlight node for the Banana Pi BPI-F3 board, > >> using PWM14. The backlight is defined as a 'pwm-backlight' device with > >> brightness levels and a default brightness setting. PWM14 is assigned > >> a period length of 2000 nanoseconds. > >> > >> This configuration was used to verify PWM driver changes, with PWM14 > >> tested and its waveform confirmed as correct. > >> > >> The node status is set to "disabled", and should be enabled when the > >> display driver is ready. > >> > > .. see comments below > >> Signed-off-by: Guodong Xu > >> --- > >> .../boot/dts/spacemit/k1-bananapi-f3.dts | 32 +++++++++++++++++++ > >> 1 file changed, 32 insertions(+) > >> > >> diff --git a/arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dts b/arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dts > >> index 816ef1bc358e..d04b57ddeb46 100644 > >> --- a/arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dts > >> +++ b/arch/riscv/boot/dts/spacemit/k1-bananapi-f3.dts > >> @@ -28,6 +28,32 @@ led1 { > >> default-state = "on"; > >> }; > >> }; > >> + > >> + pwm_bl: lcd_backlight { > >> + compatible = "pwm-backlight"; > >> + > >> + pwms = <&pwm14 2000>; > >> + brightness-levels = < > >> + 0 40 40 40 40 40 40 40 40 40 40 40 40 40 40 40 > >> + 40 40 40 40 40 40 40 40 40 40 40 40 40 40 40 40 > >> + 40 40 40 40 40 40 40 40 40 41 42 43 44 45 46 47 > >> + 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 > >> + 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 > >> + 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 > >> + 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 > >> + 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 > >> + 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 > >> + 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 > >> + 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 > >> + 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 > >> + 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 > >> + 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 > >> + 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 > >> + 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 > >> + >; > >> + default-brightness-level = <100>; > >> + status = "disabled"; > > I'm confused, has DT in board file with disabled status doesn't make sense? > > it doesn't really useful for placeholder, even worse that functionality may not > > verified, so I'd suggest sending along with display driver while at it.. > > I think I suggested he include this. Guodong tested PWM using > a backlight on a display connected to a Banana Pi PBI-F3 board. > The above numbers come directly from the downstream code, which > uses this PWM consistently as a display back light. > > But you're right, the exact set of numbers to use is dependent > on the display used, so it's better to add them when the display > gets integrated. > Ok > The pwm14 node could update still be added here, but that too > might as well wait until there's something to use it. So I > think this patch can just be dropped. > thanks for the explanation -- Yixun Lan (dlan) Gentoo Linux Developer GPG Key ID AABEFD55 _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv