From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755651AbeASLSW (ORCPT ); Fri, 19 Jan 2018 06:18:22 -0500 Received: from mail-wr0-f181.google.com ([209.85.128.181]:34311 "EHLO mail-wr0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755596AbeASLSN (ORCPT ); Fri, 19 Jan 2018 06:18:13 -0500 X-Google-Smtp-Source: ACJfBovkwXeb77ljS60eCPrW0eY42EcY7QadVTWp721LTuEjH7rpFFZs4PaBGkS/kJFLg62zUgCN/A== Date: Fri, 19 Jan 2018 11:18:08 +0000 From: Daniel Thompson To: Meghana Madhyastha Cc: Lee Jones , Jingoo Han , Thierry Reding , Noralf =?iso-8859-1?Q?Tr=F8nnes?= , Tomi Valkeinen , Daniel Vetter , Sean Paul , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v17 00/10] Add backlight helper functions Message-ID: <20180119111808.jpm4erds7o3stwso@oak.lan> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 19, 2018 at 10:37:58AM +0000, Meghana Madhyastha wrote: > Move drm helper functions from tinydrm-helpers to linux/backlight for > ease of use by callers in other drivers. > > changes in v17: > -set fb_blank along with clearing the BL_CORE_FBBLANK bit > -rebase with drm-misc-next > -fix checkpath errors/warnings > -Make the gpio_backlight issue appear as a comment in the > code and not as part of the docs > -rebase with drm-misc-next > -convert st7735r callers from tinydrm specific helpers > to new generic backlight helpers > -remove select BACKLIGHT_LCD_SUPPORT and select > BACKLIGHT_CLASS_DEVICE from tinydrm/Kconfig > -fix checkpath errors/warnings > -rename devm_backlight_put to > devm_backlight_release > -remove redundant NULL check > -remove put_device() to avoid double put > as we are using the devm version > > Meghana Madhyastha (10): > video: backlight: Add helpers to enable and disable backlight > drm/tinydrm: Convert tinydrm_enable/disable_backlight to > backlight_enable/disable > video: backlight: Add of_find_backlight helper in backlight.c > drm/tinydrm: Replace tinydrm_of_find_backlight with of_find_backlight > video: backlight: Add devres versions of of_find_backlight > drm/tinydrm: Call devres version of of_find_backlight > drm/panel: Use backlight_enable/disable helpers > drm/omapdrm: Use backlight_enable/disable helpers > drm/panel: Use of_find_backlight helper > drm/omapdrm: Use of_find_backlight helper So, I've acked all the backlight bits (if there's a v18 please include the acks in the posting). I've read the other patches and I *like* them. Switching to backlight_enable()/disable() looks like a good way to ensure drivers don't interact with all those flags in unexpected ways (so its a good contribution to eventually cleaning them up). Daniel. > > drivers/gpu/drm/omapdrm/displays/panel-dpi.c | 32 ++------- > drivers/gpu/drm/panel/panel-innolux-p079zca.c | 29 ++------ > drivers/gpu/drm/panel/panel-jdi-lt070me05000.c | 6 +- > drivers/gpu/drm/panel/panel-sharp-lq101r1sx01.c | 35 ++------- > drivers/gpu/drm/panel/panel-sharp-ls043t1le01.c | 35 ++------- > drivers/gpu/drm/tinydrm/Kconfig | 2 - > drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c | 95 ------------------------- > drivers/gpu/drm/tinydrm/mi0283qt.c | 3 +- > drivers/gpu/drm/tinydrm/mipi-dbi.c | 4 +- > drivers/gpu/drm/tinydrm/st7735r.c | 3 +- > drivers/video/backlight/backlight.c | 73 +++++++++++++++++++ > include/drm/tinydrm/tinydrm-helpers.h | 4 -- > include/linux/backlight.h | 58 +++++++++++++++ > 13 files changed, 164 insertions(+), 215 deletions(-) > > -- > 2.11.0 >