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 lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (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 9A276C5479D for ; Mon, 9 Jan 2023 21:51:43 +0000 (UTC) Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4NrSMK6NBTz3cCF for ; Tue, 10 Jan 2023 08:51:41 +1100 (AEDT) Authentication-Results: lists.ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; secure) header.d=protonic.nl header.i=@protonic.nl header.a=rsa-sha256 header.s=202111 header.b=I1ZQK8QX; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=protonic.nl (client-ip=2a02:9e0:8000::27; helo=smtp16.bhosted.nl; envelope-from=robin@protonic.nl; receiver=) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; secure) header.d=protonic.nl header.i=@protonic.nl header.a=rsa-sha256 header.s=202111 header.b=I1ZQK8QX; dkim-atps=neutral X-Greylist: delayed 11717 seconds by postgrey-1.36 at boromir; Tue, 10 Jan 2023 00:28:31 AEDT Received: from smtp16.bhosted.nl (smtp16.bhosted.nl [IPv6:2a02:9e0:8000::27]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4NrFBl3hdKz2yZv for ; Tue, 10 Jan 2023 00:28:29 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=protonic.nl; s=202111; h=message-id:references:in-reply-to:reply-to:subject:cc:to:from:date: content-transfer-encoding:content-type:mime-version:from; bh=DpADeRqnpOK1INfFn7yqaglpmahJbhY8EX582Bmd3SY=; b=I1ZQK8QX3QZlT2tEUahNaplPFpvXCyeS9V/mlZDOYwtUI7pmVX0sulujmgh+iZCm0yNHQ1CTJAwCO 2UD84ZVaPxKibdiTFen6jJJA2ngMcKk/YHCQvNir+vGDtmvCc+hbDtExXjzCYs9bVKLFKyFV1eEJ5e H2GU3TJKh5rN41xWmbjpij5cdo5TqQRuuL92xrGuPFof4+3q3YAS+D54SAV1xmAjAY/OoBcZQ1gtVH KGf56rbaDfz4I/xH6cWK5TwxYW5NoBO4LWgqyeST3CQWbwwlCFWMjUa04Elb/KRVqS6BfSBrc4Vy70 Ca3+4vIXUqVfvBYHz0i1DjTMhZyLfmQ== X-MSG-ID: 0f6a649e-9006-11ed-8e79-0050569d2c73 MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Mon, 09 Jan 2023 11:12:02 +0100 From: Robin van der Gracht To: Sam Ravnborg Subject: Re: [PATCH 12/15] auxdisplay: ht16k33: Introduce backlight_get_brightness() Organization: Protonic Holland Mail-Reply-To: robin@protonic.nl In-Reply-To: References: <20230107-sam-video-backlight-drop-fb_blank-v1-0-1bd9bafb351f@ravnborg.org> <20230107-sam-video-backlight-drop-fb_blank-v1-12-1bd9bafb351f@ravnborg.org> Message-ID: <2857575f6ec206f79cc21d423fde7d17@protonic.nl> X-Sender: robin@protonic.nl User-Agent: Roundcube Webmail/1.3.1 X-Mailman-Approved-At: Tue, 10 Jan 2023 08:50:48 +1100 X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: robin@protonic.nl Cc: Alexandre Belloni , linux-staging@lists.linux.dev, Stephen Kitt , linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Paul Mackerras , Daniel Thompson , Helge Deller , Lee Jones , Geert Uytterhoeven , Miguel Ojeda , Antonino Daplas , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Jingoo Han , Nicolas Ferre , Miguel Ojeda , Greg Kroah-Hartman , linuxppc-dev@lists.ozlabs.org, Claudiu Beznea Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" Hi Sam, On 2023-01-08 10:29, Sam Ravnborg wrote: > Hi Robin. > > On Sat, Jan 07, 2023 at 10:02:38PM +0100, Miguel Ojeda wrote: >> On Sat, Jan 7, 2023 at 7:26 PM Sam Ravnborg via B4 Submission Endpoint >> wrote: >> > >> > Introduce backlight_get_brightness() to simplify logic >> > and avoid direct access to backlight properties. >> >> Note: Stephen sent this one too a while ago (with some more details in >> the commit message, which is always nice); and then he sent yesterday >> v2 [1] (to mention the functional change with `BL_CORE_SUSPENDED` >> [2]). > Thanks for the pointers. I will try to move forward with Stephen's > patches. >> >> Anyway, if it goes via drm-misc, feel free to have my: >> >> Acked-by: Miguel Ojeda >> >> Though it would be nice to have Robin test the change. > > Robin - can I get your ack to apply Stephen's original v2 patch to > drm-misc? done! see: https://lore.kernel.org/lkml/0b16391f997e6ed005a326e4e48f2033@protonic.nl/ - Robin