Received: by 2002:a05:6a10:f3d0:0:0:0:0 with SMTP id a16csp969006pxv; Fri, 9 Jul 2021 13:45:03 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzlOEx48TBzCDZvLRhsgqck9rgIZgGvwNwN0Id98qsFdbTO/i39pqnad+O6y1EuHSG9u+aJ X-Received: by 2002:a05:6402:3481:: with SMTP id v1mr30741014edc.235.1625863503518; Fri, 09 Jul 2021 13:45:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1625863503; cv=none; d=google.com; s=arc-20160816; b=iy0INHqbGBDw49XGUBDEjr4Z52Sh7Q+yMx/OoqGxu40HHiKQcGfhKaBR0SKoFhHcnn xuQs3h/KuGcOn0v2qSKyeu9VKBnqc6l3XaYg0ulWisej45SGd0ZwqHnBbDYWnngQc8mG 7qNOo7eYhxxMkM/1qcfZlBT0gLHceY5bhp8tBZKoTno38paZbim34riikldq33Eq0jvC 2wHIIZCEkts/yvXycdfkU+tZVNCHGec8mfIzjXIfGpY1BsE4njdXNaNpsh+/ey7KHI/W zaziwnVaJYrlpnjufl9uFbjJrZzbxWcN5OBXEJHG8dKspKFu73IQVlux0LybOigbLBGm hHyQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-transfer-encoding :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=Uho+valZy1qDzmcqnnZ14HNYUswTjUV5R40i0lEpA00=; b=ZZW9KBD/lLegfXJ+XlC5Wkb550IDpPo2BEYlwd1zW8Z+hUn49BEfvWUAjHxt009qS1 FLIgVxL1ZtE6aiF+/hN+/ZKHtD9QR90wByivv8BBD7ffNsIsiJnbzko9VFykq3+/AU/g 4dlyxEL40Nlbjq3p3lJb163aOZ4oaeZEh/e8qYLK7WCO/TYEsZRp6yB9dXvYg46xr30q OxkAgw52PF865W5m1BSvWXmIE9uWqjvBA5S2Hem+ZR5GMhBzxkj48AlZpyHpJ6m79KOI +8RqX5ZirdsllKI3XvLB4zI9+c7uuy4hBaegBj1Hoe3eLsd+gcl6rAySdZDdzWuwkgSP sr/A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id b7si7573378edr.480.2021.07.09.13.44.40; Fri, 09 Jul 2021 13:45:03 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230316AbhGIUny (ORCPT + 99 others); Fri, 9 Jul 2021 16:43:54 -0400 Received: from mga09.intel.com ([134.134.136.24]:57984 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229506AbhGIUnx (ORCPT ); Fri, 9 Jul 2021 16:43:53 -0400 X-IronPort-AV: E=McAfee;i="6200,9189,10040"; a="209735562" X-IronPort-AV: E=Sophos;i="5.84,227,1620716400"; d="scan'208";a="209735562" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Jul 2021 13:41:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.84,227,1620716400"; d="scan'208";a="450411488" Received: from stinkbox.fi.intel.com (HELO stinkbox) ([10.237.72.171]) by orsmga007.jf.intel.com with SMTP; 09 Jul 2021 13:40:58 -0700 Received: by stinkbox (sSMTP sendmail emulation); Fri, 09 Jul 2021 23:40:57 +0300 Date: Fri, 9 Jul 2021 23:40:57 +0300 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Doug Anderson Cc: Rajeev Nandan , linux-fbdev@vger.kernel.org, dri-devel , Andrzej Hajda , Thierry Reding , Laurent Pinchart , "Kristian H. Kristensen" , Sam Ravnborg , Daniel Thompson , Lee Jones , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Jani Nikula , linux-arm-msm , Abhinav Kumar , Sean Paul , Kalyan Thota , Krishna Manikandan , Jingoo Han , LKML , freedreno Subject: Re: [v8 0/6] drm: Support basic DPCD backlight in panel-simple and add a new panel ATNA33XC20 Message-ID: References: <1624726268-14869-1-git-send-email-rajeevny@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Patchwork-Hint: comment Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jul 09, 2021 at 06:54:05AM -0700, Doug Anderson wrote: > Hi, > > On Sat, Jun 26, 2021 at 9:52 AM Rajeev Nandan wrote: > > > > This series adds the support for the eDP panel that needs the backlight > > controlling over the DP AUX channel using DPCD registers of the panel > > as per the VESA's standard. > > > > This series also adds support for the Samsung eDP AMOLED panel that > > needs DP AUX to control the backlight, and introduces new delays in the > > @panel_desc.delay to support this panel. > > > > This patch series depends on the following two series: > > - Doug's series [1], exposed the DP AUX channel to the panel-simple. > > - Lyude's series [2], introduced new drm helper functions for DPCD > > backlight. > > > > This series is the logical successor to the series [3]. > > > > Changes in v1: > > - Created dpcd backlight helper with very basic functionality, added > > backlight registration in the ti-sn65dsi86 bridge driver. > > > > Changes in v2: > > - Created a new DisplayPort aux backlight driver and moved the code from > > drm_dp_aux_backlight.c (v1) to the new driver. > > > > Changes in v3: > > - Fixed module compilation (kernel test bot). > > > > Changes in v4: > > - Added basic DPCD backlight support in panel-simple. > > - Added support for a new Samsung panel ATNA33XC20 that needs DPCD > > backlight controlling and has a requirement of delays between enable > > GPIO and regulator. > > > > Changes in v5: > > Addressed review suggestions from Douglas: > > - Created a new API drm_panel_dp_aux_backlight() in drm_panel.c > > - Moved DP AUX backlight functions from panel-simple.c to drm_panel.c > > - panel-simple probe() calls drm_panel_dp_aux_backlight() to create > > backlight when the backlight phandle is not specified in panel DT > > and DP AUX channel is present. > > - Added check for drm_edp_backlight_supported() before registering. > > - Removed the @uses_dpcd_backlight flag from panel_desc as this > > should be auto-detected. > > - Updated comments/descriptions. > > > > Changes in v6: > > - Rebased > > - Updated wanrning messages, fixed word wrapping in comments. > > - Fixed ordering of memory allocation > > > > Changes in v7: > > - Updated the disable_to_power_off and power_to_enable panel delays > > as discovered at (Douglas) > > > > Changes in v8: > > - Now using backlight_is_blank() to get the backlight blank status (Sam Ravnborg) > > - Added a new patch #4 to fix the warnings for eDP panel description (Sam Ravnborg) > > > > [1] https://lore.kernel.org/dri-devel/20210525000159.3384921-1-dianders@chromium.org/ > > [2] https://lore.kernel.org/dri-devel/20210514181504.565252-1-lyude@redhat.com/ > > [3] https://lore.kernel.org/dri-devel/1619416756-3533-1-git-send-email-rajeevny@codeaurora.org/ > > > > Rajeev Nandan (6): > > drm/panel: add basic DP AUX backlight support > > drm/panel-simple: Support DP AUX backlight > > drm/panel-simple: Support for delays between GPIO & regulator > > drm/panel-simple: Update validation warnings for eDP panel description > > dt-bindings: display: simple: Add Samsung ATNA33XC20 > > drm/panel-simple: Add Samsung ATNA33XC20 > > > > .../bindings/display/panel/panel-simple.yaml | 2 + > > drivers/gpu/drm/drm_panel.c | 108 +++++++++++++++++++++ > > drivers/gpu/drm/panel/panel-simple.c | 73 +++++++++++++- > > include/drm/drm_panel.h | 15 ++- > > 4 files changed, 190 insertions(+), 8 deletions(-) > > Pushed to drm-misc-next. > > 4bfe6c8f7c23 drm/panel-simple: Add Samsung ATNA33XC20 > c20dec193584 dt-bindings: display: simple: Add Samsung ATNA33XC20 > 13aceea56fd5 drm/panel-simple: Update validation warnings for eDP > panel description > 18a1488bf1e1 drm/panel-simple: Support for delays between GPIO & regulator > bfd451403d70 drm/panel-simple: Support DP AUX backlight > 10f7b40e4f30 drm/panel: add basic DP AUX backlight support depmod: ERROR: Cycle detected: drm_kms_helper -> drm -> drm_kms_helper Looks to be due to drm_edp_backlight_enable(). -- Ville Syrj?l? Intel