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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 C713AC54E71 for ; Fri, 22 Mar 2024 08:16:37 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 76C7610F125; Fri, 22 Mar 2024 08:16:36 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.b="pdWUkciI"; dkim-atps=neutral Received: from sin.source.kernel.org (sin.source.kernel.org [145.40.73.55]) by gabe.freedesktop.org (Postfix) with ESMTPS id CDF0010F11E for ; Fri, 22 Mar 2024 08:16:34 +0000 (UTC) Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id 96880CE10C8; Fri, 22 Mar 2024 08:16:31 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8D9CCC433F1; Fri, 22 Mar 2024 08:16:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1711095390; bh=78e84KjAWSryCmLw73e2JyPKfWstRQ6SUGWADdV5MAo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=pdWUkciI1p6OLe7Mp+iUUUkevTieQn7eyS3YHgIIgT29n/AUCtVaV+k3ToslsobLD lArtEjjMUZxjohcYrlq9gO56WY3BqCAjKewuAlOdOcZOX03agHSPh2BogaSFjvipuC +whc6ZZT8m2UIH0EXapTRZPPKyo1/LnDjofMwLPneA5XidcN3nV0FhbFu3vEZ06Ed3 6zrwBFzERX/CGcGElK8A0aVPjvGignBrRL46+7bh/T0SnLmXDshmUXNTbyLaT+VR+U i7dmG5Y44ciYvgfW1R3EZQXguHUN5VgNbV/YyogWtkQ5G3Nd3Z1mTjq5HAjrQQz5DW rEwtYnZcPA7xA== Date: Fri, 22 Mar 2024 08:16:25 +0000 From: Lee Jones To: andy@kernel.org, daniel.thompson@linaro.org, jingoohan1@gmail.com, deller@gmx.de, robin@protonic.nl, javierm@redhat.com, Thomas Zimmermann Cc: dri-devel@lists.freedesktop.org, linux-fbdev@vger.kernel.org, linux-input@vger.kernel.org, linux-pwm@vger.kernel.org Subject: Re: [PATCH v4 00/10] backlight: Replace struct fb_info in interfaces Message-ID: <20240322081625.GM13211@google.com> References: <20240305162425.23845-1-tzimmermann@suse.de> <171103756721.89062.17090257592751026195.b4-ty@kernel.org> <20240321161358.GB13211@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20240321161358.GB13211@google.com> X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On Thu, 21 Mar 2024, Lee Jones wrote: > On Thu, 21 Mar 2024, Lee Jones wrote: > > > On Tue, 05 Mar 2024 17:22:33 +0100, Thomas Zimmermann wrote: > > > Backlight drivers implement struct backlight_ops.check_fb, which > > > uses struct fb_info in its interface. Replace the callback with one > > > that does not use fb_info. > > > > > > In DRM, we have several drivers that implement backlight support. By > > > including these drivers depend on . > > > At the same time, fbdev is deprecated for new drivers and likely to > > > be replaced on many systems. > > > > > > [...] > > > > Applied, thanks! > > > > [01/10] backlight: Match backlight device against struct fb_info.bl_dev > > commit: f1ecddf747f0d734682152b37c927aa958a51497 > > [02/10] auxdisplay/ht16k33: Remove struct backlight_ops.check_fb > > commit: dddfda7d5f12a7b48aeca6c3840167529c8cd34a > > [03/10] hid/hid-picolcd: Fix initialization order > > commit: a951a15002da620871d8f3d8218c043cdc4c2471 > > [04/10] hid/hid-picolcd: Remove struct backlight_ops.check_fb > > commit: b3c52552f8d8a816bda2bda984411c73f4dd0b87 > > [05/10] backlight/aat2870-backlight: Remove struct backlight.check_fb > > commit: 0e03c96046405281fb072c05a7810d2661a2f334 > > [06/10] backlight/pwm-backlight: Remove struct backlight_ops.check_fb > > commit: 78534967e7cb3c2fbfcb2d37820b51a80c570f90 > > [07/10] fbdev/sh_mobile_lcdc_fb: Remove struct backlight_ops.check_fb > > commit: b853c08cd6598b3b3ff91cb2bba336bfef9c0ac4 > > [08/10] fbdev/ssd1307fb: Init backlight before registering framebuffer > > commit: d5ae81e965953da27cf46db6281d6a6a28eaaccb > > [09/10] fbdev/ssd1307fb: Remove struct backlight_ops.check_fb > > commit: ec5925ef4a2dfd7ee060f4fd2a2e8036f8a94e8e > > [10/10] backlight: Add controls_device callback to struct backlight_ops > > commit: 2e427743de015c1ac047036ef495c3f004105439 > > Okay, let's try this again. > > Send off for more build testing based on v6.8. > > Will report back once complete. So far, so good. Will rebase these and sent out an immutable branch PR once v6.9-rc1 is out. Note to self: ib-backlight-auxdisplay-hid-fb-6.9 -- Lee Jones [李琼斯]