Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp2223647imm; Sat, 28 Jul 2018 11:53:40 -0700 (PDT) X-Google-Smtp-Source: AAOMgpfFlWyBG5LLxg4ApeonV4dqSsGPN77+Y/chC28OrfXNIh15lhmJCLTtk1+T3v4WVlsmy6ig X-Received: by 2002:a63:4c56:: with SMTP id m22-v6mr10358240pgl.299.1532804020411; Sat, 28 Jul 2018 11:53:40 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1532804020; cv=none; d=google.com; s=arc-20160816; b=Yhfz60EweVJQ3EaRoUtmf2YThyOTygo1I8Z+Lli1vOqG9FPHzyHsjFTQAB044yKaHL rXIK1T0kPqwHByl+4HZU4B3dnKmj1isrUCND84aKqK4TPgVHhMzHp0y+K9Go/D26j4Uq 14BMrj/lEZGJ3K5ndMLhyuHLwezal8D9tz4cPN8BUiI8swAC6knwWnzaFbBGiEkMn3Ru aGALDkeMxrkWWSz5GpEgRxfbT2AfATkY9hvtMHWBCB+XrTYq1wdA/Fx+LRarLzfOfLtR eu8QT5dLtd3nAUEmWhE8lGG7+I+9Hvy11Xx+NCzfcFHkYVFGAWaKFGx6K4Q74+9cXCkh L2Eg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :references:in-reply-to:mime-version:dkim-signature :arc-authentication-results; bh=UWXQIoKzwZ9RzuXhKo7ijE1kSckkD4dJ4gaZdtnRwTo=; b=s1wbSnYd/7MGijNV0GZSP2KTA/WJdDo4dnJId9rn5usrEjqFCa2KTkeHf40NMjCB0o Qd0zRJGm20vguuYnyP8ZKOdhjz4wEpVUHwEAUbAtHaD4nDSSEAgyH9ZR1wC+7dtf3NnS RSA+oxasE6b8MZ5B3y7VyeSUrTmqpyl7shZiw5oMdhEoQNkUnuSzcwZRJlitJBJSgIAb AnRVvO5EABa2u3V8MC6CjvtlntXK81En4VlamVB151wJc6ljmePkG1ywHqkD9UL/Uiur etksFq51X5v8YdT+V+a8K4jI86BxJnBF+KSc0Ydevq5GiwqeoaW0rOCn3jP+qnZjE2/C 0DMA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=lWe6pMs5; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id b2-v6si6049824plm.202.2018.07.28.11.53.24; Sat, 28 Jul 2018 11:53:40 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=lWe6pMs5; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730423AbeG1US0 (ORCPT + 99 others); Sat, 28 Jul 2018 16:18:26 -0400 Received: from mail-lj1-f195.google.com ([209.85.208.195]:40634 "EHLO mail-lj1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730356AbeG1USZ (ORCPT ); Sat, 28 Jul 2018 16:18:25 -0400 Received: by mail-lj1-f195.google.com with SMTP id j19-v6so7136105ljc.7; Sat, 28 Jul 2018 11:51:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=UWXQIoKzwZ9RzuXhKo7ijE1kSckkD4dJ4gaZdtnRwTo=; b=lWe6pMs5o+/AG0UG4cKcYXs15yRpfGGQNAALp49ZXmz47Ozt2zAAIZG4iyFA3oZVYA ca8mveYIOIDrd/wvHP0PsjLxMKVrt4GxH5yBa44X5S2bI57JjpDNoK8BDCi6WXVOm9xw V0qfPwOuiaW1R+X9V9mwnlp0Zepe/yy1D888Az2T0RXSuDzHh3Fg7QrOef2jtq52bRL1 SXCQpJ+x+iBjwLYMKY6TmJc6ElIKTSnoyOKb6CEjycKgPY2QcfKeVixOIkfe80qQ2Zd3 MpTzziYBY61d+AfQ42Gc3W0pl4vyyu623MYOPBg7xxB2x0SdmdwtwHxHtKcvqpwhtECi x+qg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=UWXQIoKzwZ9RzuXhKo7ijE1kSckkD4dJ4gaZdtnRwTo=; b=lhZ/UcunigegvM2UB2d+tRrJp735IVGGhufo77leIU34Yq4e5R4L74G2p0UfB6Ne/h E6BKH0LuaSHYyG5PzcFnDoXVh+7uQaOs4CqzZ2KihCYRkCaYuXEo+XvQTmbnt+UZ4Qf6 IrX0ToGet7XbysyDZwvibKqJqXXjKGvC4IuhxvLST5X0oU7ytjOHCHkes7rXI6GjSpMV NMTT3XDvF/uoLKyX1o+o0Nkqr0dn/rdKeMw8ypLBkp4fCetk0I9mS3rKo60S7Yr4z8EZ DYUis7nQj9S8eHojYJOL9siK77yx4sjzg1LNPDHGaSOtaZ/85UvKY9tOBdN9+O+eXR2S 9TYg== X-Gm-Message-State: AOUpUlGG2XurW0x+yALNqg8fSefuRAWJhe6QslQbCFDzz7ahkmLnzqNR cc48jVJSYXMWseJYnYn5mblSSG+3GKh/xpz2SDE= X-Received: by 2002:a2e:1301:: with SMTP id 1-v6mr7980228ljt.56.1532803859637; Sat, 28 Jul 2018 11:50:59 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:a19:2b14:0:0:0:0:0 with HTTP; Sat, 28 Jul 2018 11:50:58 -0700 (PDT) In-Reply-To: References: <20180728154007.GA28426@jordon-HP-15-Notebook-PC> From: Souptick Joarder Date: Sun, 29 Jul 2018 00:20:58 +0530 Message-ID: Subject: Re: [PATCH] drm/rcar-du: Convert drm_atomic_helper_suspend/resume() To: Vaishali Thakkar Cc: Laurent Pinchart , airlied@linux.ie, Ajit Linux , dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, Linux Kernel Mailing List , Daniel Vetter Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Jul 28, 2018 at 11:20 PM, Vaishali Thakkar wrote: > On Sat, Jul 28, 2018 at 9:10 PM, Souptick Joarder wrote: >> convert drm_atomic_helper_suspend/resume() to use >> drm_mode_config_helper_suspend/resume(). > > Hi Souptick, > > Thanks for your patch. > >> Signed-off-by: Souptick Joarder >> Signed-off-by: Ajit Negi >> --- >> drivers/gpu/drm/rcar-du/rcar_du_drv.c | 21 ++------------------- >> 1 file changed, 2 insertions(+), 19 deletions(-) >> >> diff --git a/drivers/gpu/drm/rcar-du/rcar_du_drv.c b/drivers/gpu/drm/rcar-du/rcar_du_drv.c >> index 02aee6c..288220f 100644 >> --- a/drivers/gpu/drm/rcar-du/rcar_du_drv.c >> +++ b/drivers/gpu/drm/rcar-du/rcar_du_drv.c >> @@ -357,32 +357,15 @@ static void rcar_du_lastclose(struct drm_device *dev) >> static int rcar_du_pm_suspend(struct device *dev) >> { >> struct rcar_du_device *rcdu = dev_get_drvdata(dev); >> - struct drm_atomic_state *state; >> >> - drm_kms_helper_poll_disable(rcdu->ddev); >> - drm_fbdev_cma_set_suspend_unlocked(rcdu->fbdev, true); >> - >> - state = drm_atomic_helper_suspend(rcdu->ddev); >> - if (IS_ERR(state)) { >> - drm_fbdev_cma_set_suspend_unlocked(rcdu->fbdev, false); > > I don't think we can use drm_mode_config_helper_(suspend/resume) > API here as this file uses CMA functions. drm_fbdev_cma_set_suspend_unlocked() is wrapper function which invokes drm_fb_helper_set_suspend_unlocked(). Where the new API drm_mode_config_helper_suspend/resume() directly invokes drm_fb_helper_set_suspend_unlocked(). So it is safe to replace exiting code with API drm_mode_config_helper_suspend/resume(). > And from git grep it seems that > there are very few drivers using it at the moment, so not sure if introducing > new API functions similar to drm_mode_config will make sense or not. > https://www.kernel.org/doc/html/latest/gpu/todo.html It was picked up from TODO list after discussing with Daniel. > Thanks. > >> - drm_kms_helper_poll_enable(rcdu->ddev); >> - return PTR_ERR(state); >> - } >> - >> - rcdu->suspend_state = state; >> - >> - return 0; >> + return drm_mode_config_helper_suspend(rcdu->ddev); >> } >> >> static int rcar_du_pm_resume(struct device *dev) >> { >> struct rcar_du_device *rcdu = dev_get_drvdata(dev); >> >> - drm_atomic_helper_resume(rcdu->ddev, rcdu->suspend_state); >> - drm_fbdev_cma_set_suspend_unlocked(rcdu->fbdev, false); >> - drm_kms_helper_poll_enable(rcdu->ddev); >> - >> - return 0; >> + return drm_mode_config_helper_resume(rcdu->ddev); >> } >> #endif >> >> -- >> 1.9.1 >>