From patchwork Sun May 17 19:01:23 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sam Ravnborg X-Patchwork-Id: 11554259 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 48987912 for ; Sun, 17 May 2020 19:01:50 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 2CC6A20801 for ; Sun, 17 May 2020 19:01:50 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="SyPwK9wj" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726522AbgEQTBt (ORCPT ); Sun, 17 May 2020 15:01:49 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45482 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726269AbgEQTBt (ORCPT ); Sun, 17 May 2020 15:01:49 -0400 Received: from mail-lj1-x242.google.com (mail-lj1-x242.google.com [IPv6:2a00:1450:4864:20::242]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E3C58C061A0C; Sun, 17 May 2020 12:01:48 -0700 (PDT) Received: by mail-lj1-x242.google.com with SMTP id e25so7530912ljg.5; Sun, 17 May 2020 12:01:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=+vWFegYYLZQwCEOaLXtrEOXjzYIe1aYsXrabMwiuRn8=; b=SyPwK9wjsty7a+yolMNcUlz/fyBtkp6kYIO/DY626WHRKcSEHm1CWooE563AfyccvY eYgQj9HBx6SSxE6az5TtZb6as24wi3zqIKTmVlR8QDWn+ZTricaiHUUggjfiNytSqA3s qW8307TF58lrfmfAQwEse5dAfpSq+RC0guP11lsbK5mhXOvY3Gs+O4cp/7zoB9shCU/b 6yGX34tFmairsC7E/m7gxyccSZV0Rfe5FVWKXA8CKFYUiV8D8hygu5muTXsosXKg2xKQ 7Bqy4S8feujVHpBD+Vpn+hXi7L+3kyDqRZgKgQ0efSFsTft3QPdJ0cooYV7J3LrhJToJ OYvQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :mime-version:content-transfer-encoding; bh=+vWFegYYLZQwCEOaLXtrEOXjzYIe1aYsXrabMwiuRn8=; b=F6H6Uwh4pX+driSXG5sfMB1GqS64wJcOs5ggrnspqOoDDmg4T2lgKQ52kq4mM51Xn2 c8RE5I7Qud7mapPJSD29ixdkerhWzc/RQwsaSrxHkoz9uoBHCBA/oOCrQ0bC9AzzZtVD 8+UULfnMJQAmXA9NTYbvrIHyL07Xs2ZqKTQchfwi7RDxhFIPft+zR3w3b4A25WH626jo DEaoErF9WNH2YZ2KW8Wt5at+m9Mz3jBDpoSwRhggDv8g3pjGFHTp1bw+az1DeBth7jXW IsYUEhLaKlp9NgqiwvCQGTmJZVaYwyel9VDCbmonCUtsd1d+6aoNYIeJJ4sjfb2njC3g wVEA== X-Gm-Message-State: AOAM532rMOymcYSr3LtoI2dtFdH++qcYNV9YeTZ3YdzX1d3scK6HW5JX hMVCtIxiT7i/ZFx220nEIRg= X-Google-Smtp-Source: ABdhPJzcTXFuU0suV+4pnM5YfobMc0J5sej97g1AnOHfjJxoY0fPxI5WyNS2BgDLet6YtDr6pB+e7w== X-Received: by 2002:a2e:97d3:: with SMTP id m19mr7889928ljj.136.1589742107096; Sun, 17 May 2020 12:01:47 -0700 (PDT) Received: from saturn.lan ([2a00:fd00:805f:db00:5d55:b3eb:397b:9086]) by smtp.gmail.com with ESMTPSA id t20sm3282535lfl.17.2020.05.17.12.01.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 17 May 2020 12:01:46 -0700 (PDT) From: Sam Ravnborg To: dri-devel@lists.freedesktop.org, Jingoo Han , Lee Jones , Daniel Thompson , Peter Ujfalusi , Tomi Valkeinen Cc: Andy Gross , Bartlomiej Zolnierkiewicz , Bjorn Andersson , Daniel Vetter , David Airlie , Douglas Anderson , Jani Nikula , Jonathan Corbet , Linus Walleij , linux-arm-msm@vger.kernel.org, linux-pwm@vger.kernel.org, Maarten Lankhorst , Maxime Ripard , Michael Hennerich , patches@opensource.cirrus.com, Russell King , Sam Ravnborg , Support Opensource , Thierry Reding , Thomas Zimmermann , Uwe Kleine-Konig Subject: [PATCH v2 0/16] backlight updates Date: Sun, 17 May 2020 21:01:23 +0200 Message-Id: <20200517190139.740249-1-sam@ravnborg.org> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org v2: - Dropped drm patches that was reviewed and thus applied (Thanks Tomi) - Updated backlight_is_blank() based on Daniel's feedback - Dropped EXPORT_SYMBOL that was no longer relevant - Reordered patches, so patches with no external dependencies comes first - Updated the description that follows. This following series touches a lot of backlight things. It starts by migrating the last user of of_find_backlight_by_node() over to devm_of_find_backlight(). *Tomi*/*Peter* review feedback would be great - as this allows a smooth removal of the export of of_find_backlight_by_node(). Then a small refactoring in backligth.c to remove some indents. This increases the readability and no functional changes. Then a new helper backlight_is_blank() is added. This helper will simplify the implementation of update_status() in almost all backlight drivers. Then while surfing the code I missed some documentation. So I got a bit carried away and updated the documentation for the backlight core and added it to kernel-doc. The documentation express my current understanding. Everything from spelling errors to outright wrong content shall be anticipated - so please review! We are all best helped if the documentation is correct and up-to-date and it is readable. In this process I identified that the backlight_bl was no longer is use - so drop it. All the functions that is no longer used by any drivers are then marked static to avoid adding new users. There are dependencies to the omap patch in drivers/video/fbdev/ so these patches needs to wait until that one is applied. The last patch is for now just an RFC patch that shows the potential simplifications by introducing the use of the backlight_is_blank() helper. I have local patches to introduce backlight_is_blank() in the remaining backlight drivers. But they will await that this patch set matures a bit. Everything builds, but so far no run-time testing. Sam Sam Ravnborg (16): video: amba-clcd: use devm_of_find_backlight backlight: refactor fb_notifier_callback() backlight: add backlight_is_blank() backlight: improve backlight_ops documentation backlight: improve backlight_properties documentation backlight: improve backlight_device documentation backlight: document inline functions in backlight.h backlight: document enums in backlight.h backlight: remove the unused backlight_bl driver backlight: drop extern from prototypes backlight: add overview and update existing doc backlight: wire up kernel-doc documentation backlight: make of_find_backlight static backlight: drop backlight_put() backlight: make of_find_backlight_by_node() static backlight: use backlight_is_blank() in all backlight drivers Documentation/gpu/backlight.rst | 12 + Documentation/gpu/index.rst | 1 + drivers/video/backlight/88pm860x_bl.c | 8 +- drivers/video/backlight/Kconfig | 8 - drivers/video/backlight/Makefile | 1 - drivers/video/backlight/adp5520_bl.c | 5 +- drivers/video/backlight/adp8860_bl.c | 5 +- drivers/video/backlight/adp8870_bl.c | 5 +- drivers/video/backlight/as3711_bl.c | 8 +- drivers/video/backlight/backlight.c | 235 +++++++++-------- drivers/video/backlight/bd6107.c | 4 +- drivers/video/backlight/corgi_lcd.c | 5 +- drivers/video/backlight/cr_bllcd.c | 22 +- drivers/video/backlight/da903x_bl.c | 8 +- drivers/video/backlight/ep93xx_bl.c | 3 +- drivers/video/backlight/generic_bl.c | 110 -------- drivers/video/backlight/gpio_backlight.c | 4 +- drivers/video/backlight/hp680_bl.c | 4 +- drivers/video/backlight/jornada720_bl.c | 2 +- drivers/video/backlight/kb3886_bl.c | 4 +- drivers/video/backlight/led_bl.c | 4 +- drivers/video/backlight/lm3533_bl.c | 4 +- drivers/video/backlight/locomolcd.c | 4 +- drivers/video/backlight/lv5207lp.c | 4 +- drivers/video/backlight/max8925_bl.c | 8 +- drivers/video/backlight/pwm_bl.c | 4 +- drivers/video/backlight/qcom-wled.c | 4 +- drivers/video/backlight/tps65217_bl.c | 4 +- drivers/video/backlight/wm831x_bl.c | 8 +- drivers/video/fbdev/amba-clcd.c | 19 +- include/linux/backlight.h | 417 ++++++++++++++++++++++++------- 31 files changed, 522 insertions(+), 412 deletions(-) Reviewed-by: Emil Velikov