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 X-Spam-Level: X-Spam-Status: No, score=-6.6 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 17A88C33CAF for ; Mon, 13 Jan 2020 19:02:47 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D830E214AF for ; Mon, 13 Jan 2020 19:02:46 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="cXihKozM" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726878AbgAMTCq (ORCPT ); Mon, 13 Jan 2020 14:02:46 -0500 Received: from mail-wm1-f68.google.com ([209.85.128.68]:36294 "EHLO mail-wm1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726435AbgAMTCq (ORCPT ); Mon, 13 Jan 2020 14:02:46 -0500 Received: by mail-wm1-f68.google.com with SMTP id p17so10949067wma.1 for ; Mon, 13 Jan 2020 11:02:44 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=g5bN8wH7FTCKMaY/O9Oej61HYM/NEwy3Ac5fo9Lh1CM=; b=cXihKozMLIK8aRkwyi6CUSBoYteUAU/J+qqx2TR55NghW/eABcq2WEso8oD5IqdhhL eFEGmuYYH39qJED57r6lsswXLGelKAqIsAx27U2EThlh8URYtwWfwlDas//1vDH4bIum 1TOnoMuDDn8pkMddd09d117aHDVRmBwewvnRJd6nKZH0xhzsE348Fem7TuiS793ACX48 it0UvTYfUUNDkloSqqD3K1hQoQNf4x/omD8TaTOwwkuiPqNizObDjwTdO5g1Mz+VMzFN dtnu5SO8V6aZlfy53MYp4iib/tw9nX5XITsOiHbb+vRDE2ubtXM9AHZruSa66ld0E7o/ lENw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=g5bN8wH7FTCKMaY/O9Oej61HYM/NEwy3Ac5fo9Lh1CM=; b=ed/2NXUrCBL8hfbfxpwEsoKzRQh0bcGpfAm9LEFpm3JN4pPwHDL5RJ+XP0LHNhtlmU zMzc8V/GnL2/RYDzat4oqdxkbHTkJ7ndwXDJR4pX1EPWvCT+OzW+25sVCwNd8kz5C9Ea u1CSglTCgMcE29oMOc3KSgX0U8hSjgJe4eootyTOVQYcUrIcKfwYJaAvocqLA9d/TvkK XZi1gpx4HgyExODRT7BAhUQPtS2Lmy3MRwN11DpS3x72xKi6sHuEezR1QFDZ7bXozw2N wSBRgMLAlYZfoJQBHt8os24z0h17ArP29jHxyl8TGC6NHKKGTyaSxwnXmomgMBYSFIoW MGug== X-Gm-Message-State: APjAAAVFdKkDKmSd/FGzh2VCRvaM7V+3qgKCIiQ1TEzpbx+h/wFt4qZL NGOg4zXdI7xiV0AYTl9tzAgHnpDYA31ztMbi9zY= X-Google-Smtp-Source: APXvYqypf0BaRB3wlCCMXdO+KwGjdkP3TWave4BGBfjMXMdIt0UaOb1bcTDdhs32wKkz3vzR8VjhOSXO3HKDj563ELc= X-Received: by 2002:a1c:f30e:: with SMTP id q14mr22026498wmq.65.1578942164075; Mon, 13 Jan 2020 11:02:44 -0800 (PST) MIME-Version: 1.0 References: <20200110092127.27847-1-tzimmermann@suse.de> <20200110092127.27847-18-tzimmermann@suse.de> In-Reply-To: <20200110092127.27847-18-tzimmermann@suse.de> From: Alex Deucher Date: Mon, 13 Jan 2020 14:02:32 -0500 Message-ID: Subject: Re: [PATCH 17/23] drm/radeon: Convert to CRTC VBLANK callbacks To: Thomas Zimmermann Cc: Dave Airlie , Daniel Vetter , "Deucher, Alexander" , Christian Koenig , Chunming Zhou , Maarten Lankhorst , Patrik Jakobsson , Rob Clark , Sean Paul , Benjamin Gaignard , Vincent Abriou , yannick.fertre@st.com, philippe.cornu@st.com, mcoquelin.stm32@gmail.com, alexandre.torgue@st.com, Eric Anholt , rodrigosiqueiramelo@gmail.com, hamohammed.sa@gmail.com, VMware Graphics , Thomas Hellstrom , Ben Skeggs , "Wentland, Harry" , "Leo (Sunpeng) Li" , Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , linux-arm-msm , Intel Graphics Development , amd-gfx list , Maling list - DRI developers , nouveau , freedreno Content-Type: text/plain; charset="UTF-8" Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Fri, Jan 10, 2020 at 4:22 AM Thomas Zimmermann wrote: > > VBLANK callbacks in struct drm_driver are deprecated in favor of > their equivalents in struct drm_crtc_funcs. Convert radeon over. > > Signed-off-by: Thomas Zimmermann Reviewed-by: Alex Deucher > --- > drivers/gpu/drm/radeon/radeon_display.c | 12 ++++++++-- > drivers/gpu/drm/radeon/radeon_drv.c | 7 ------ > drivers/gpu/drm/radeon/radeon_kms.c | 29 ++++++++++++++----------- > 3 files changed, 26 insertions(+), 22 deletions(-) > > diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c > index 7187158b9963..9116975b6eb9 100644 > --- a/drivers/gpu/drm/radeon/radeon_display.c > +++ b/drivers/gpu/drm/radeon/radeon_display.c > @@ -45,6 +45,10 @@ > #include "atom.h" > #include "radeon.h" > > +u32 radeon_get_vblank_counter_kms(struct drm_crtc *crtc); > +int radeon_enable_vblank_kms(struct drm_crtc *crtc); > +void radeon_disable_vblank_kms(struct drm_crtc *crtc); > + > static void avivo_crtc_load_lut(struct drm_crtc *crtc) > { > struct radeon_crtc *radeon_crtc = to_radeon_crtc(crtc); > @@ -458,7 +462,7 @@ static void radeon_flip_work_func(struct work_struct *__work) > (DRM_SCANOUTPOS_VALID | DRM_SCANOUTPOS_IN_VBLANK) && > (!ASIC_IS_AVIVO(rdev) || > ((int) (work->target_vblank - > - dev->driver->get_vblank_counter(dev, work->crtc_id)) > 0))) > + crtc->funcs->get_vblank_counter(crtc)) > 0))) > usleep_range(1000, 2000); > > /* We borrow the event spin lock for protecting flip_status */ > @@ -574,7 +578,7 @@ static int radeon_crtc_page_flip_target(struct drm_crtc *crtc, > } > work->base = base; > work->target_vblank = target - (uint32_t)drm_crtc_vblank_count(crtc) + > - dev->driver->get_vblank_counter(dev, work->crtc_id); > + crtc->funcs->get_vblank_counter(crtc); > > /* We borrow the event spin lock for protecting flip_work */ > spin_lock_irqsave(&crtc->dev->event_lock, flags); > @@ -666,6 +670,10 @@ static const struct drm_crtc_funcs radeon_crtc_funcs = { > .set_config = radeon_crtc_set_config, > .destroy = radeon_crtc_destroy, > .page_flip_target = radeon_crtc_page_flip_target, > + .get_vblank_counter = radeon_get_vblank_counter_kms, > + .enable_vblank = radeon_enable_vblank_kms, > + .disable_vblank = radeon_disable_vblank_kms, > + .get_vblank_timestamp = drm_crtc_calc_vbltimestamp_from_scanoutpos, > }; > > static void radeon_crtc_init(struct drm_device *dev, int index) > diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c > index 1f597f166bff..49ce2e7d5f9e 100644 > --- a/drivers/gpu/drm/radeon/radeon_drv.c > +++ b/drivers/gpu/drm/radeon/radeon_drv.c > @@ -119,9 +119,6 @@ void radeon_driver_postclose_kms(struct drm_device *dev, > int radeon_suspend_kms(struct drm_device *dev, bool suspend, > bool fbcon, bool freeze); > int radeon_resume_kms(struct drm_device *dev, bool resume, bool fbcon); > -u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe); > -int radeon_enable_vblank_kms(struct drm_device *dev, unsigned int pipe); > -void radeon_disable_vblank_kms(struct drm_device *dev, unsigned int pipe); > void radeon_driver_irq_preinstall_kms(struct drm_device *dev); > int radeon_driver_irq_postinstall_kms(struct drm_device *dev); > void radeon_driver_irq_uninstall_kms(struct drm_device *dev); > @@ -571,10 +568,6 @@ static struct drm_driver kms_driver = { > .postclose = radeon_driver_postclose_kms, > .lastclose = radeon_driver_lastclose_kms, > .unload = radeon_driver_unload_kms, > - .get_vblank_counter = radeon_get_vblank_counter_kms, > - .enable_vblank = radeon_enable_vblank_kms, > - .disable_vblank = radeon_disable_vblank_kms, > - .get_vblank_timestamp = drm_calc_vbltimestamp_from_scanoutpos, > .irq_preinstall = radeon_driver_irq_preinstall_kms, > .irq_postinstall = radeon_driver_irq_postinstall_kms, > .irq_uninstall = radeon_driver_irq_uninstall_kms, > diff --git a/drivers/gpu/drm/radeon/radeon_kms.c b/drivers/gpu/drm/radeon/radeon_kms.c > index d24f23a81656..cab891f86dc0 100644 > --- a/drivers/gpu/drm/radeon/radeon_kms.c > +++ b/drivers/gpu/drm/radeon/radeon_kms.c > @@ -739,14 +739,15 @@ void radeon_driver_postclose_kms(struct drm_device *dev, > /** > * radeon_get_vblank_counter_kms - get frame count > * > - * @dev: drm dev pointer > - * @pipe: crtc to get the frame count from > + * @crtc: crtc to get the frame count from > * > * Gets the frame count on the requested crtc (all asics). > * Returns frame count on success, -EINVAL on failure. > */ > -u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe) > +u32 radeon_get_vblank_counter_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > int vpos, hpos, stat; > u32 count; > struct radeon_device *rdev = dev->dev_private; > @@ -808,25 +809,26 @@ u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe) > /** > * radeon_enable_vblank_kms - enable vblank interrupt > * > - * @dev: drm dev pointer > * @crtc: crtc to enable vblank interrupt for > * > * Enable the interrupt on the requested crtc (all asics). > * Returns 0 on success, -EINVAL on failure. > */ > -int radeon_enable_vblank_kms(struct drm_device *dev, int crtc) > +int radeon_enable_vblank_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > struct radeon_device *rdev = dev->dev_private; > unsigned long irqflags; > int r; > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %d\n", pipe); > return -EINVAL; > } > > spin_lock_irqsave(&rdev->irq.lock, irqflags); > - rdev->irq.crtc_vblank_int[crtc] = true; > + rdev->irq.crtc_vblank_int[pipe] = true; > r = radeon_irq_set(rdev); > spin_unlock_irqrestore(&rdev->irq.lock, irqflags); > return r; > @@ -835,23 +837,24 @@ int radeon_enable_vblank_kms(struct drm_device *dev, int crtc) > /** > * radeon_disable_vblank_kms - disable vblank interrupt > * > - * @dev: drm dev pointer > * @crtc: crtc to disable vblank interrupt for > * > * Disable the interrupt on the requested crtc (all asics). > */ > -void radeon_disable_vblank_kms(struct drm_device *dev, int crtc) > +void radeon_disable_vblank_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > struct radeon_device *rdev = dev->dev_private; > unsigned long irqflags; > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %d\n", pipe); > return; > } > > spin_lock_irqsave(&rdev->irq.lock, irqflags); > - rdev->irq.crtc_vblank_int[crtc] = false; > + rdev->irq.crtc_vblank_int[pipe] = false; > radeon_irq_set(rdev); > spin_unlock_irqrestore(&rdev->irq.lock, irqflags); > } > -- > 2.24.1 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Deucher Subject: Re: [PATCH 17/23] drm/radeon: Convert to CRTC VBLANK callbacks Date: Mon, 13 Jan 2020 14:02:32 -0500 Message-ID: References: <20200110092127.27847-1-tzimmermann@suse.de> <20200110092127.27847-18-tzimmermann@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20200110092127.27847-18-tzimmermann@suse.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Thomas Zimmermann Cc: hamohammed.sa@gmail.com, Dave Airlie , nouveau , Maling list - DRI developers , amd-gfx list , alexandre.torgue@st.com, Thomas Hellstrom , Sean Paul , VMware Graphics , Ben Skeggs , mcoquelin.stm32@gmail.com, "Leo (Sunpeng) Li" , linux-arm-msm , Intel Graphics Development , Rodrigo Vivi , Vincent Abriou , rodrigosiqueiramelo@gmail.com, philippe.cornu@st.com, yannick.fertre@st.com, "Deucher, Alexander" , freedreno , Christian Koenig List-Id: nouveau.vger.kernel.org On Fri, Jan 10, 2020 at 4:22 AM Thomas Zimmermann wrote: > > VBLANK callbacks in struct drm_driver are deprecated in favor of > their equivalents in struct drm_crtc_funcs. Convert radeon over. > > Signed-off-by: Thomas Zimmermann Reviewed-by: Alex Deucher > --- > drivers/gpu/drm/radeon/radeon_display.c | 12 ++++++++-- > drivers/gpu/drm/radeon/radeon_drv.c | 7 ------ > drivers/gpu/drm/radeon/radeon_kms.c | 29 ++++++++++++++----------- > 3 files changed, 26 insertions(+), 22 deletions(-) > > diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c > index 7187158b9963..9116975b6eb9 100644 > --- a/drivers/gpu/drm/radeon/radeon_display.c > +++ b/drivers/gpu/drm/radeon/radeon_display.c > @@ -45,6 +45,10 @@ > #include "atom.h" > #include "radeon.h" > > +u32 radeon_get_vblank_counter_kms(struct drm_crtc *crtc); > +int radeon_enable_vblank_kms(struct drm_crtc *crtc); > +void radeon_disable_vblank_kms(struct drm_crtc *crtc); > + > static void avivo_crtc_load_lut(struct drm_crtc *crtc) > { > struct radeon_crtc *radeon_crtc = to_radeon_crtc(crtc); > @@ -458,7 +462,7 @@ static void radeon_flip_work_func(struct work_struct *__work) > (DRM_SCANOUTPOS_VALID | DRM_SCANOUTPOS_IN_VBLANK) && > (!ASIC_IS_AVIVO(rdev) || > ((int) (work->target_vblank - > - dev->driver->get_vblank_counter(dev, work->crtc_id)) > 0))) > + crtc->funcs->get_vblank_counter(crtc)) > 0))) > usleep_range(1000, 2000); > > /* We borrow the event spin lock for protecting flip_status */ > @@ -574,7 +578,7 @@ static int radeon_crtc_page_flip_target(struct drm_crtc *crtc, > } > work->base = base; > work->target_vblank = target - (uint32_t)drm_crtc_vblank_count(crtc) + > - dev->driver->get_vblank_counter(dev, work->crtc_id); > + crtc->funcs->get_vblank_counter(crtc); > > /* We borrow the event spin lock for protecting flip_work */ > spin_lock_irqsave(&crtc->dev->event_lock, flags); > @@ -666,6 +670,10 @@ static const struct drm_crtc_funcs radeon_crtc_funcs = { > .set_config = radeon_crtc_set_config, > .destroy = radeon_crtc_destroy, > .page_flip_target = radeon_crtc_page_flip_target, > + .get_vblank_counter = radeon_get_vblank_counter_kms, > + .enable_vblank = radeon_enable_vblank_kms, > + .disable_vblank = radeon_disable_vblank_kms, > + .get_vblank_timestamp = drm_crtc_calc_vbltimestamp_from_scanoutpos, > }; > > static void radeon_crtc_init(struct drm_device *dev, int index) > diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c > index 1f597f166bff..49ce2e7d5f9e 100644 > --- a/drivers/gpu/drm/radeon/radeon_drv.c > +++ b/drivers/gpu/drm/radeon/radeon_drv.c > @@ -119,9 +119,6 @@ void radeon_driver_postclose_kms(struct drm_device *dev, > int radeon_suspend_kms(struct drm_device *dev, bool suspend, > bool fbcon, bool freeze); > int radeon_resume_kms(struct drm_device *dev, bool resume, bool fbcon); > -u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe); > -int radeon_enable_vblank_kms(struct drm_device *dev, unsigned int pipe); > -void radeon_disable_vblank_kms(struct drm_device *dev, unsigned int pipe); > void radeon_driver_irq_preinstall_kms(struct drm_device *dev); > int radeon_driver_irq_postinstall_kms(struct drm_device *dev); > void radeon_driver_irq_uninstall_kms(struct drm_device *dev); > @@ -571,10 +568,6 @@ static struct drm_driver kms_driver = { > .postclose = radeon_driver_postclose_kms, > .lastclose = radeon_driver_lastclose_kms, > .unload = radeon_driver_unload_kms, > - .get_vblank_counter = radeon_get_vblank_counter_kms, > - .enable_vblank = radeon_enable_vblank_kms, > - .disable_vblank = radeon_disable_vblank_kms, > - .get_vblank_timestamp = drm_calc_vbltimestamp_from_scanoutpos, > .irq_preinstall = radeon_driver_irq_preinstall_kms, > .irq_postinstall = radeon_driver_irq_postinstall_kms, > .irq_uninstall = radeon_driver_irq_uninstall_kms, > diff --git a/drivers/gpu/drm/radeon/radeon_kms.c b/drivers/gpu/drm/radeon/radeon_kms.c > index d24f23a81656..cab891f86dc0 100644 > --- a/drivers/gpu/drm/radeon/radeon_kms.c > +++ b/drivers/gpu/drm/radeon/radeon_kms.c > @@ -739,14 +739,15 @@ void radeon_driver_postclose_kms(struct drm_device *dev, > /** > * radeon_get_vblank_counter_kms - get frame count > * > - * @dev: drm dev pointer > - * @pipe: crtc to get the frame count from > + * @crtc: crtc to get the frame count from > * > * Gets the frame count on the requested crtc (all asics). > * Returns frame count on success, -EINVAL on failure. > */ > -u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe) > +u32 radeon_get_vblank_counter_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > int vpos, hpos, stat; > u32 count; > struct radeon_device *rdev = dev->dev_private; > @@ -808,25 +809,26 @@ u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe) > /** > * radeon_enable_vblank_kms - enable vblank interrupt > * > - * @dev: drm dev pointer > * @crtc: crtc to enable vblank interrupt for > * > * Enable the interrupt on the requested crtc (all asics). > * Returns 0 on success, -EINVAL on failure. > */ > -int radeon_enable_vblank_kms(struct drm_device *dev, int crtc) > +int radeon_enable_vblank_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > struct radeon_device *rdev = dev->dev_private; > unsigned long irqflags; > int r; > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %d\n", pipe); > return -EINVAL; > } > > spin_lock_irqsave(&rdev->irq.lock, irqflags); > - rdev->irq.crtc_vblank_int[crtc] = true; > + rdev->irq.crtc_vblank_int[pipe] = true; > r = radeon_irq_set(rdev); > spin_unlock_irqrestore(&rdev->irq.lock, irqflags); > return r; > @@ -835,23 +837,24 @@ int radeon_enable_vblank_kms(struct drm_device *dev, int crtc) > /** > * radeon_disable_vblank_kms - disable vblank interrupt > * > - * @dev: drm dev pointer > * @crtc: crtc to disable vblank interrupt for > * > * Disable the interrupt on the requested crtc (all asics). > */ > -void radeon_disable_vblank_kms(struct drm_device *dev, int crtc) > +void radeon_disable_vblank_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > struct radeon_device *rdev = dev->dev_private; > unsigned long irqflags; > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %d\n", pipe); > return; > } > > spin_lock_irqsave(&rdev->irq.lock, irqflags); > - rdev->irq.crtc_vblank_int[crtc] = false; > + rdev->irq.crtc_vblank_int[pipe] = false; > radeon_irq_set(rdev); > spin_unlock_irqrestore(&rdev->irq.lock, irqflags); > } > -- > 2.24.1 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx 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 X-Spam-Level: X-Spam-Status: No, score=-6.3 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 09657C33CB0 for ; Mon, 13 Jan 2020 19:02:48 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id D2B82214AF for ; Mon, 13 Jan 2020 19:02:47 +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="cXihKozM" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D2B82214AF Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id F1A3D6E150; Mon, 13 Jan 2020 19:02:46 +0000 (UTC) Received: from mail-wm1-x344.google.com (mail-wm1-x344.google.com [IPv6:2a00:1450:4864:20::344]) by gabe.freedesktop.org (Postfix) with ESMTPS id 6F3C86E139; Mon, 13 Jan 2020 19:02:45 +0000 (UTC) Received: by mail-wm1-x344.google.com with SMTP id t14so10920561wmi.5; Mon, 13 Jan 2020 11:02:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=g5bN8wH7FTCKMaY/O9Oej61HYM/NEwy3Ac5fo9Lh1CM=; b=cXihKozMLIK8aRkwyi6CUSBoYteUAU/J+qqx2TR55NghW/eABcq2WEso8oD5IqdhhL eFEGmuYYH39qJED57r6lsswXLGelKAqIsAx27U2EThlh8URYtwWfwlDas//1vDH4bIum 1TOnoMuDDn8pkMddd09d117aHDVRmBwewvnRJd6nKZH0xhzsE348Fem7TuiS793ACX48 it0UvTYfUUNDkloSqqD3K1hQoQNf4x/omD8TaTOwwkuiPqNizObDjwTdO5g1Mz+VMzFN dtnu5SO8V6aZlfy53MYp4iib/tw9nX5XITsOiHbb+vRDE2ubtXM9AHZruSa66ld0E7o/ lENw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=g5bN8wH7FTCKMaY/O9Oej61HYM/NEwy3Ac5fo9Lh1CM=; b=aS7Kq/WxJC5+em10sRzuR5ANA8RGjQ/XwQdYEmIeWrceu0b/KciZqMhWnMM1i4vqFF sK7F2ziiL7qwO51VZQtYlZpMYefQPwYbM6UODG+W5dPzrrVgWWbcmRvfwfxkDmTxRuyl yej2taj1nS000qu+hoElsyg55HxgtyA19sR2spa1b9qET7oY/9cUmGfzCk9z4GJNhT27 mX0x1psxyFjDCz0gy2g8OGP0QjDFMe78hL18hMtaPTUxSjCjhi0RF5QKWSyRIeo5pN8c ru6FrqhheVibwe/HqdSqOeWTqlle0mOix5iRcV8quJZhLvrLJG2V2bHFlAzQP7Gxvlid q7ZA== X-Gm-Message-State: APjAAAX+R/8bUJtaraZ1+/V3DZvpKT5A6BEWTP8oLORpHjzkujcqMD2m SNUm4tBlPzzy78qew/55JKTteopPXUXhblZB8mw= X-Google-Smtp-Source: APXvYqypf0BaRB3wlCCMXdO+KwGjdkP3TWave4BGBfjMXMdIt0UaOb1bcTDdhs32wKkz3vzR8VjhOSXO3HKDj563ELc= X-Received: by 2002:a1c:f30e:: with SMTP id q14mr22026498wmq.65.1578942164075; Mon, 13 Jan 2020 11:02:44 -0800 (PST) MIME-Version: 1.0 References: <20200110092127.27847-1-tzimmermann@suse.de> <20200110092127.27847-18-tzimmermann@suse.de> In-Reply-To: <20200110092127.27847-18-tzimmermann@suse.de> From: Alex Deucher Date: Mon, 13 Jan 2020 14:02:32 -0500 Message-ID: Subject: Re: [PATCH 17/23] drm/radeon: Convert to CRTC VBLANK callbacks To: Thomas Zimmermann 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: , Cc: hamohammed.sa@gmail.com, Dave Airlie , nouveau , Maling list - DRI developers , amd-gfx list , alexandre.torgue@st.com, Thomas Hellstrom , Sean Paul , VMware Graphics , Ben Skeggs , mcoquelin.stm32@gmail.com, "Leo \(Sunpeng\) Li" , linux-arm-msm , Intel Graphics Development , Rodrigo Vivi , Vincent Abriou , rodrigosiqueiramelo@gmail.com, philippe.cornu@st.com, yannick.fertre@st.com, "Deucher, Alexander" , freedreno , Christian Koenig Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On Fri, Jan 10, 2020 at 4:22 AM Thomas Zimmermann wrote: > > VBLANK callbacks in struct drm_driver are deprecated in favor of > their equivalents in struct drm_crtc_funcs. Convert radeon over. > > Signed-off-by: Thomas Zimmermann Reviewed-by: Alex Deucher > --- > drivers/gpu/drm/radeon/radeon_display.c | 12 ++++++++-- > drivers/gpu/drm/radeon/radeon_drv.c | 7 ------ > drivers/gpu/drm/radeon/radeon_kms.c | 29 ++++++++++++++----------- > 3 files changed, 26 insertions(+), 22 deletions(-) > > diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c > index 7187158b9963..9116975b6eb9 100644 > --- a/drivers/gpu/drm/radeon/radeon_display.c > +++ b/drivers/gpu/drm/radeon/radeon_display.c > @@ -45,6 +45,10 @@ > #include "atom.h" > #include "radeon.h" > > +u32 radeon_get_vblank_counter_kms(struct drm_crtc *crtc); > +int radeon_enable_vblank_kms(struct drm_crtc *crtc); > +void radeon_disable_vblank_kms(struct drm_crtc *crtc); > + > static void avivo_crtc_load_lut(struct drm_crtc *crtc) > { > struct radeon_crtc *radeon_crtc = to_radeon_crtc(crtc); > @@ -458,7 +462,7 @@ static void radeon_flip_work_func(struct work_struct *__work) > (DRM_SCANOUTPOS_VALID | DRM_SCANOUTPOS_IN_VBLANK) && > (!ASIC_IS_AVIVO(rdev) || > ((int) (work->target_vblank - > - dev->driver->get_vblank_counter(dev, work->crtc_id)) > 0))) > + crtc->funcs->get_vblank_counter(crtc)) > 0))) > usleep_range(1000, 2000); > > /* We borrow the event spin lock for protecting flip_status */ > @@ -574,7 +578,7 @@ static int radeon_crtc_page_flip_target(struct drm_crtc *crtc, > } > work->base = base; > work->target_vblank = target - (uint32_t)drm_crtc_vblank_count(crtc) + > - dev->driver->get_vblank_counter(dev, work->crtc_id); > + crtc->funcs->get_vblank_counter(crtc); > > /* We borrow the event spin lock for protecting flip_work */ > spin_lock_irqsave(&crtc->dev->event_lock, flags); > @@ -666,6 +670,10 @@ static const struct drm_crtc_funcs radeon_crtc_funcs = { > .set_config = radeon_crtc_set_config, > .destroy = radeon_crtc_destroy, > .page_flip_target = radeon_crtc_page_flip_target, > + .get_vblank_counter = radeon_get_vblank_counter_kms, > + .enable_vblank = radeon_enable_vblank_kms, > + .disable_vblank = radeon_disable_vblank_kms, > + .get_vblank_timestamp = drm_crtc_calc_vbltimestamp_from_scanoutpos, > }; > > static void radeon_crtc_init(struct drm_device *dev, int index) > diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c > index 1f597f166bff..49ce2e7d5f9e 100644 > --- a/drivers/gpu/drm/radeon/radeon_drv.c > +++ b/drivers/gpu/drm/radeon/radeon_drv.c > @@ -119,9 +119,6 @@ void radeon_driver_postclose_kms(struct drm_device *dev, > int radeon_suspend_kms(struct drm_device *dev, bool suspend, > bool fbcon, bool freeze); > int radeon_resume_kms(struct drm_device *dev, bool resume, bool fbcon); > -u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe); > -int radeon_enable_vblank_kms(struct drm_device *dev, unsigned int pipe); > -void radeon_disable_vblank_kms(struct drm_device *dev, unsigned int pipe); > void radeon_driver_irq_preinstall_kms(struct drm_device *dev); > int radeon_driver_irq_postinstall_kms(struct drm_device *dev); > void radeon_driver_irq_uninstall_kms(struct drm_device *dev); > @@ -571,10 +568,6 @@ static struct drm_driver kms_driver = { > .postclose = radeon_driver_postclose_kms, > .lastclose = radeon_driver_lastclose_kms, > .unload = radeon_driver_unload_kms, > - .get_vblank_counter = radeon_get_vblank_counter_kms, > - .enable_vblank = radeon_enable_vblank_kms, > - .disable_vblank = radeon_disable_vblank_kms, > - .get_vblank_timestamp = drm_calc_vbltimestamp_from_scanoutpos, > .irq_preinstall = radeon_driver_irq_preinstall_kms, > .irq_postinstall = radeon_driver_irq_postinstall_kms, > .irq_uninstall = radeon_driver_irq_uninstall_kms, > diff --git a/drivers/gpu/drm/radeon/radeon_kms.c b/drivers/gpu/drm/radeon/radeon_kms.c > index d24f23a81656..cab891f86dc0 100644 > --- a/drivers/gpu/drm/radeon/radeon_kms.c > +++ b/drivers/gpu/drm/radeon/radeon_kms.c > @@ -739,14 +739,15 @@ void radeon_driver_postclose_kms(struct drm_device *dev, > /** > * radeon_get_vblank_counter_kms - get frame count > * > - * @dev: drm dev pointer > - * @pipe: crtc to get the frame count from > + * @crtc: crtc to get the frame count from > * > * Gets the frame count on the requested crtc (all asics). > * Returns frame count on success, -EINVAL on failure. > */ > -u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe) > +u32 radeon_get_vblank_counter_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > int vpos, hpos, stat; > u32 count; > struct radeon_device *rdev = dev->dev_private; > @@ -808,25 +809,26 @@ u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe) > /** > * radeon_enable_vblank_kms - enable vblank interrupt > * > - * @dev: drm dev pointer > * @crtc: crtc to enable vblank interrupt for > * > * Enable the interrupt on the requested crtc (all asics). > * Returns 0 on success, -EINVAL on failure. > */ > -int radeon_enable_vblank_kms(struct drm_device *dev, int crtc) > +int radeon_enable_vblank_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > struct radeon_device *rdev = dev->dev_private; > unsigned long irqflags; > int r; > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %d\n", pipe); > return -EINVAL; > } > > spin_lock_irqsave(&rdev->irq.lock, irqflags); > - rdev->irq.crtc_vblank_int[crtc] = true; > + rdev->irq.crtc_vblank_int[pipe] = true; > r = radeon_irq_set(rdev); > spin_unlock_irqrestore(&rdev->irq.lock, irqflags); > return r; > @@ -835,23 +837,24 @@ int radeon_enable_vblank_kms(struct drm_device *dev, int crtc) > /** > * radeon_disable_vblank_kms - disable vblank interrupt > * > - * @dev: drm dev pointer > * @crtc: crtc to disable vblank interrupt for > * > * Disable the interrupt on the requested crtc (all asics). > */ > -void radeon_disable_vblank_kms(struct drm_device *dev, int crtc) > +void radeon_disable_vblank_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > struct radeon_device *rdev = dev->dev_private; > unsigned long irqflags; > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %d\n", pipe); > return; > } > > spin_lock_irqsave(&rdev->irq.lock, irqflags); > - rdev->irq.crtc_vblank_int[crtc] = false; > + rdev->irq.crtc_vblank_int[pipe] = false; > radeon_irq_set(rdev); > spin_unlock_irqrestore(&rdev->irq.lock, irqflags); > } > -- > 2.24.1 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel 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 X-Spam-Level: X-Spam-Status: No, score=-6.3 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 43096C33CAE for ; Mon, 13 Jan 2020 19:02:47 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 1772B2187F for ; Mon, 13 Jan 2020 19:02:47 +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="cXihKozM" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1772B2187F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=intel-gfx-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 7B2B86E14B; Mon, 13 Jan 2020 19:02:46 +0000 (UTC) Received: from mail-wm1-x344.google.com (mail-wm1-x344.google.com [IPv6:2a00:1450:4864:20::344]) by gabe.freedesktop.org (Postfix) with ESMTPS id 6F3C86E139; Mon, 13 Jan 2020 19:02:45 +0000 (UTC) Received: by mail-wm1-x344.google.com with SMTP id t14so10920561wmi.5; Mon, 13 Jan 2020 11:02:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=g5bN8wH7FTCKMaY/O9Oej61HYM/NEwy3Ac5fo9Lh1CM=; b=cXihKozMLIK8aRkwyi6CUSBoYteUAU/J+qqx2TR55NghW/eABcq2WEso8oD5IqdhhL eFEGmuYYH39qJED57r6lsswXLGelKAqIsAx27U2EThlh8URYtwWfwlDas//1vDH4bIum 1TOnoMuDDn8pkMddd09d117aHDVRmBwewvnRJd6nKZH0xhzsE348Fem7TuiS793ACX48 it0UvTYfUUNDkloSqqD3K1hQoQNf4x/omD8TaTOwwkuiPqNizObDjwTdO5g1Mz+VMzFN dtnu5SO8V6aZlfy53MYp4iib/tw9nX5XITsOiHbb+vRDE2ubtXM9AHZruSa66ld0E7o/ lENw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=g5bN8wH7FTCKMaY/O9Oej61HYM/NEwy3Ac5fo9Lh1CM=; b=aS7Kq/WxJC5+em10sRzuR5ANA8RGjQ/XwQdYEmIeWrceu0b/KciZqMhWnMM1i4vqFF sK7F2ziiL7qwO51VZQtYlZpMYefQPwYbM6UODG+W5dPzrrVgWWbcmRvfwfxkDmTxRuyl yej2taj1nS000qu+hoElsyg55HxgtyA19sR2spa1b9qET7oY/9cUmGfzCk9z4GJNhT27 mX0x1psxyFjDCz0gy2g8OGP0QjDFMe78hL18hMtaPTUxSjCjhi0RF5QKWSyRIeo5pN8c ru6FrqhheVibwe/HqdSqOeWTqlle0mOix5iRcV8quJZhLvrLJG2V2bHFlAzQP7Gxvlid q7ZA== X-Gm-Message-State: APjAAAX+R/8bUJtaraZ1+/V3DZvpKT5A6BEWTP8oLORpHjzkujcqMD2m SNUm4tBlPzzy78qew/55JKTteopPXUXhblZB8mw= X-Google-Smtp-Source: APXvYqypf0BaRB3wlCCMXdO+KwGjdkP3TWave4BGBfjMXMdIt0UaOb1bcTDdhs32wKkz3vzR8VjhOSXO3HKDj563ELc= X-Received: by 2002:a1c:f30e:: with SMTP id q14mr22026498wmq.65.1578942164075; Mon, 13 Jan 2020 11:02:44 -0800 (PST) MIME-Version: 1.0 References: <20200110092127.27847-1-tzimmermann@suse.de> <20200110092127.27847-18-tzimmermann@suse.de> In-Reply-To: <20200110092127.27847-18-tzimmermann@suse.de> From: Alex Deucher Date: Mon, 13 Jan 2020 14:02:32 -0500 Message-ID: To: Thomas Zimmermann Subject: Re: [Intel-gfx] [PATCH 17/23] drm/radeon: Convert to CRTC VBLANK callbacks X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: hamohammed.sa@gmail.com, Dave Airlie , nouveau , Maling list - DRI developers , Eric Anholt , amd-gfx list , Benjamin Gaignard , alexandre.torgue@st.com, Chunming Zhou , Thomas Hellstrom , VMware Graphics , Ben Skeggs , "Wentland, Harry" , mcoquelin.stm32@gmail.com, "Leo \(Sunpeng\) Li" , linux-arm-msm , Intel Graphics Development , Vincent Abriou , rodrigosiqueiramelo@gmail.com, philippe.cornu@st.com, yannick.fertre@st.com, "Deucher, Alexander" , freedreno , Christian Koenig Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" On Fri, Jan 10, 2020 at 4:22 AM Thomas Zimmermann wrote: > > VBLANK callbacks in struct drm_driver are deprecated in favor of > their equivalents in struct drm_crtc_funcs. Convert radeon over. > > Signed-off-by: Thomas Zimmermann Reviewed-by: Alex Deucher > --- > drivers/gpu/drm/radeon/radeon_display.c | 12 ++++++++-- > drivers/gpu/drm/radeon/radeon_drv.c | 7 ------ > drivers/gpu/drm/radeon/radeon_kms.c | 29 ++++++++++++++----------- > 3 files changed, 26 insertions(+), 22 deletions(-) > > diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c > index 7187158b9963..9116975b6eb9 100644 > --- a/drivers/gpu/drm/radeon/radeon_display.c > +++ b/drivers/gpu/drm/radeon/radeon_display.c > @@ -45,6 +45,10 @@ > #include "atom.h" > #include "radeon.h" > > +u32 radeon_get_vblank_counter_kms(struct drm_crtc *crtc); > +int radeon_enable_vblank_kms(struct drm_crtc *crtc); > +void radeon_disable_vblank_kms(struct drm_crtc *crtc); > + > static void avivo_crtc_load_lut(struct drm_crtc *crtc) > { > struct radeon_crtc *radeon_crtc = to_radeon_crtc(crtc); > @@ -458,7 +462,7 @@ static void radeon_flip_work_func(struct work_struct *__work) > (DRM_SCANOUTPOS_VALID | DRM_SCANOUTPOS_IN_VBLANK) && > (!ASIC_IS_AVIVO(rdev) || > ((int) (work->target_vblank - > - dev->driver->get_vblank_counter(dev, work->crtc_id)) > 0))) > + crtc->funcs->get_vblank_counter(crtc)) > 0))) > usleep_range(1000, 2000); > > /* We borrow the event spin lock for protecting flip_status */ > @@ -574,7 +578,7 @@ static int radeon_crtc_page_flip_target(struct drm_crtc *crtc, > } > work->base = base; > work->target_vblank = target - (uint32_t)drm_crtc_vblank_count(crtc) + > - dev->driver->get_vblank_counter(dev, work->crtc_id); > + crtc->funcs->get_vblank_counter(crtc); > > /* We borrow the event spin lock for protecting flip_work */ > spin_lock_irqsave(&crtc->dev->event_lock, flags); > @@ -666,6 +670,10 @@ static const struct drm_crtc_funcs radeon_crtc_funcs = { > .set_config = radeon_crtc_set_config, > .destroy = radeon_crtc_destroy, > .page_flip_target = radeon_crtc_page_flip_target, > + .get_vblank_counter = radeon_get_vblank_counter_kms, > + .enable_vblank = radeon_enable_vblank_kms, > + .disable_vblank = radeon_disable_vblank_kms, > + .get_vblank_timestamp = drm_crtc_calc_vbltimestamp_from_scanoutpos, > }; > > static void radeon_crtc_init(struct drm_device *dev, int index) > diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c > index 1f597f166bff..49ce2e7d5f9e 100644 > --- a/drivers/gpu/drm/radeon/radeon_drv.c > +++ b/drivers/gpu/drm/radeon/radeon_drv.c > @@ -119,9 +119,6 @@ void radeon_driver_postclose_kms(struct drm_device *dev, > int radeon_suspend_kms(struct drm_device *dev, bool suspend, > bool fbcon, bool freeze); > int radeon_resume_kms(struct drm_device *dev, bool resume, bool fbcon); > -u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe); > -int radeon_enable_vblank_kms(struct drm_device *dev, unsigned int pipe); > -void radeon_disable_vblank_kms(struct drm_device *dev, unsigned int pipe); > void radeon_driver_irq_preinstall_kms(struct drm_device *dev); > int radeon_driver_irq_postinstall_kms(struct drm_device *dev); > void radeon_driver_irq_uninstall_kms(struct drm_device *dev); > @@ -571,10 +568,6 @@ static struct drm_driver kms_driver = { > .postclose = radeon_driver_postclose_kms, > .lastclose = radeon_driver_lastclose_kms, > .unload = radeon_driver_unload_kms, > - .get_vblank_counter = radeon_get_vblank_counter_kms, > - .enable_vblank = radeon_enable_vblank_kms, > - .disable_vblank = radeon_disable_vblank_kms, > - .get_vblank_timestamp = drm_calc_vbltimestamp_from_scanoutpos, > .irq_preinstall = radeon_driver_irq_preinstall_kms, > .irq_postinstall = radeon_driver_irq_postinstall_kms, > .irq_uninstall = radeon_driver_irq_uninstall_kms, > diff --git a/drivers/gpu/drm/radeon/radeon_kms.c b/drivers/gpu/drm/radeon/radeon_kms.c > index d24f23a81656..cab891f86dc0 100644 > --- a/drivers/gpu/drm/radeon/radeon_kms.c > +++ b/drivers/gpu/drm/radeon/radeon_kms.c > @@ -739,14 +739,15 @@ void radeon_driver_postclose_kms(struct drm_device *dev, > /** > * radeon_get_vblank_counter_kms - get frame count > * > - * @dev: drm dev pointer > - * @pipe: crtc to get the frame count from > + * @crtc: crtc to get the frame count from > * > * Gets the frame count on the requested crtc (all asics). > * Returns frame count on success, -EINVAL on failure. > */ > -u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe) > +u32 radeon_get_vblank_counter_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > int vpos, hpos, stat; > u32 count; > struct radeon_device *rdev = dev->dev_private; > @@ -808,25 +809,26 @@ u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe) > /** > * radeon_enable_vblank_kms - enable vblank interrupt > * > - * @dev: drm dev pointer > * @crtc: crtc to enable vblank interrupt for > * > * Enable the interrupt on the requested crtc (all asics). > * Returns 0 on success, -EINVAL on failure. > */ > -int radeon_enable_vblank_kms(struct drm_device *dev, int crtc) > +int radeon_enable_vblank_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > struct radeon_device *rdev = dev->dev_private; > unsigned long irqflags; > int r; > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %d\n", pipe); > return -EINVAL; > } > > spin_lock_irqsave(&rdev->irq.lock, irqflags); > - rdev->irq.crtc_vblank_int[crtc] = true; > + rdev->irq.crtc_vblank_int[pipe] = true; > r = radeon_irq_set(rdev); > spin_unlock_irqrestore(&rdev->irq.lock, irqflags); > return r; > @@ -835,23 +837,24 @@ int radeon_enable_vblank_kms(struct drm_device *dev, int crtc) > /** > * radeon_disable_vblank_kms - disable vblank interrupt > * > - * @dev: drm dev pointer > * @crtc: crtc to disable vblank interrupt for > * > * Disable the interrupt on the requested crtc (all asics). > */ > -void radeon_disable_vblank_kms(struct drm_device *dev, int crtc) > +void radeon_disable_vblank_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > struct radeon_device *rdev = dev->dev_private; > unsigned long irqflags; > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %d\n", pipe); > return; > } > > spin_lock_irqsave(&rdev->irq.lock, irqflags); > - rdev->irq.crtc_vblank_int[crtc] = false; > + rdev->irq.crtc_vblank_int[pipe] = false; > radeon_irq_set(rdev); > spin_unlock_irqrestore(&rdev->irq.lock, irqflags); > } > -- > 2.24.1 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx 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 X-Spam-Level: X-Spam-Status: No, score=-6.3 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 77615C33CA9 for ; Mon, 13 Jan 2020 19:02:46 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 4A96D214AF for ; Mon, 13 Jan 2020 19:02:46 +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="cXihKozM" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 4A96D214AF Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=amd-gfx-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 019276E139; Mon, 13 Jan 2020 19:02:46 +0000 (UTC) Received: from mail-wm1-x344.google.com (mail-wm1-x344.google.com [IPv6:2a00:1450:4864:20::344]) by gabe.freedesktop.org (Postfix) with ESMTPS id 6F3C86E139; Mon, 13 Jan 2020 19:02:45 +0000 (UTC) Received: by mail-wm1-x344.google.com with SMTP id t14so10920561wmi.5; Mon, 13 Jan 2020 11:02:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=g5bN8wH7FTCKMaY/O9Oej61HYM/NEwy3Ac5fo9Lh1CM=; b=cXihKozMLIK8aRkwyi6CUSBoYteUAU/J+qqx2TR55NghW/eABcq2WEso8oD5IqdhhL eFEGmuYYH39qJED57r6lsswXLGelKAqIsAx27U2EThlh8URYtwWfwlDas//1vDH4bIum 1TOnoMuDDn8pkMddd09d117aHDVRmBwewvnRJd6nKZH0xhzsE348Fem7TuiS793ACX48 it0UvTYfUUNDkloSqqD3K1hQoQNf4x/omD8TaTOwwkuiPqNizObDjwTdO5g1Mz+VMzFN dtnu5SO8V6aZlfy53MYp4iib/tw9nX5XITsOiHbb+vRDE2ubtXM9AHZruSa66ld0E7o/ lENw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=g5bN8wH7FTCKMaY/O9Oej61HYM/NEwy3Ac5fo9Lh1CM=; b=aS7Kq/WxJC5+em10sRzuR5ANA8RGjQ/XwQdYEmIeWrceu0b/KciZqMhWnMM1i4vqFF sK7F2ziiL7qwO51VZQtYlZpMYefQPwYbM6UODG+W5dPzrrVgWWbcmRvfwfxkDmTxRuyl yej2taj1nS000qu+hoElsyg55HxgtyA19sR2spa1b9qET7oY/9cUmGfzCk9z4GJNhT27 mX0x1psxyFjDCz0gy2g8OGP0QjDFMe78hL18hMtaPTUxSjCjhi0RF5QKWSyRIeo5pN8c ru6FrqhheVibwe/HqdSqOeWTqlle0mOix5iRcV8quJZhLvrLJG2V2bHFlAzQP7Gxvlid q7ZA== X-Gm-Message-State: APjAAAX+R/8bUJtaraZ1+/V3DZvpKT5A6BEWTP8oLORpHjzkujcqMD2m SNUm4tBlPzzy78qew/55JKTteopPXUXhblZB8mw= X-Google-Smtp-Source: APXvYqypf0BaRB3wlCCMXdO+KwGjdkP3TWave4BGBfjMXMdIt0UaOb1bcTDdhs32wKkz3vzR8VjhOSXO3HKDj563ELc= X-Received: by 2002:a1c:f30e:: with SMTP id q14mr22026498wmq.65.1578942164075; Mon, 13 Jan 2020 11:02:44 -0800 (PST) MIME-Version: 1.0 References: <20200110092127.27847-1-tzimmermann@suse.de> <20200110092127.27847-18-tzimmermann@suse.de> In-Reply-To: <20200110092127.27847-18-tzimmermann@suse.de> From: Alex Deucher Date: Mon, 13 Jan 2020 14:02:32 -0500 Message-ID: Subject: Re: [PATCH 17/23] drm/radeon: Convert to CRTC VBLANK callbacks To: Thomas Zimmermann X-BeenThere: amd-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: hamohammed.sa@gmail.com, Dave Airlie , nouveau , Joonas Lahtinen , Maling list - DRI developers , Eric Anholt , amd-gfx list , Benjamin Gaignard , alexandre.torgue@st.com, Chunming Zhou , Thomas Hellstrom , Sean Paul , Patrik Jakobsson , VMware Graphics , Ben Skeggs , "Wentland, Harry" , mcoquelin.stm32@gmail.com, "Leo \(Sunpeng\) Li" , linux-arm-msm , Intel Graphics Development , Maarten Lankhorst , Jani Nikula , Rodrigo Vivi , Vincent Abriou , rodrigosiqueiramelo@gmail.com, philippe.cornu@st.com, yannick.fertre@st.com, Rob Clark , Daniel Vetter , "Deucher, Alexander" , freedreno , Christian Koenig Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: amd-gfx-bounces@lists.freedesktop.org Sender: "amd-gfx" On Fri, Jan 10, 2020 at 4:22 AM Thomas Zimmermann wrote: > > VBLANK callbacks in struct drm_driver are deprecated in favor of > their equivalents in struct drm_crtc_funcs. Convert radeon over. > > Signed-off-by: Thomas Zimmermann Reviewed-by: Alex Deucher > --- > drivers/gpu/drm/radeon/radeon_display.c | 12 ++++++++-- > drivers/gpu/drm/radeon/radeon_drv.c | 7 ------ > drivers/gpu/drm/radeon/radeon_kms.c | 29 ++++++++++++++----------- > 3 files changed, 26 insertions(+), 22 deletions(-) > > diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c > index 7187158b9963..9116975b6eb9 100644 > --- a/drivers/gpu/drm/radeon/radeon_display.c > +++ b/drivers/gpu/drm/radeon/radeon_display.c > @@ -45,6 +45,10 @@ > #include "atom.h" > #include "radeon.h" > > +u32 radeon_get_vblank_counter_kms(struct drm_crtc *crtc); > +int radeon_enable_vblank_kms(struct drm_crtc *crtc); > +void radeon_disable_vblank_kms(struct drm_crtc *crtc); > + > static void avivo_crtc_load_lut(struct drm_crtc *crtc) > { > struct radeon_crtc *radeon_crtc = to_radeon_crtc(crtc); > @@ -458,7 +462,7 @@ static void radeon_flip_work_func(struct work_struct *__work) > (DRM_SCANOUTPOS_VALID | DRM_SCANOUTPOS_IN_VBLANK) && > (!ASIC_IS_AVIVO(rdev) || > ((int) (work->target_vblank - > - dev->driver->get_vblank_counter(dev, work->crtc_id)) > 0))) > + crtc->funcs->get_vblank_counter(crtc)) > 0))) > usleep_range(1000, 2000); > > /* We borrow the event spin lock for protecting flip_status */ > @@ -574,7 +578,7 @@ static int radeon_crtc_page_flip_target(struct drm_crtc *crtc, > } > work->base = base; > work->target_vblank = target - (uint32_t)drm_crtc_vblank_count(crtc) + > - dev->driver->get_vblank_counter(dev, work->crtc_id); > + crtc->funcs->get_vblank_counter(crtc); > > /* We borrow the event spin lock for protecting flip_work */ > spin_lock_irqsave(&crtc->dev->event_lock, flags); > @@ -666,6 +670,10 @@ static const struct drm_crtc_funcs radeon_crtc_funcs = { > .set_config = radeon_crtc_set_config, > .destroy = radeon_crtc_destroy, > .page_flip_target = radeon_crtc_page_flip_target, > + .get_vblank_counter = radeon_get_vblank_counter_kms, > + .enable_vblank = radeon_enable_vblank_kms, > + .disable_vblank = radeon_disable_vblank_kms, > + .get_vblank_timestamp = drm_crtc_calc_vbltimestamp_from_scanoutpos, > }; > > static void radeon_crtc_init(struct drm_device *dev, int index) > diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c > index 1f597f166bff..49ce2e7d5f9e 100644 > --- a/drivers/gpu/drm/radeon/radeon_drv.c > +++ b/drivers/gpu/drm/radeon/radeon_drv.c > @@ -119,9 +119,6 @@ void radeon_driver_postclose_kms(struct drm_device *dev, > int radeon_suspend_kms(struct drm_device *dev, bool suspend, > bool fbcon, bool freeze); > int radeon_resume_kms(struct drm_device *dev, bool resume, bool fbcon); > -u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe); > -int radeon_enable_vblank_kms(struct drm_device *dev, unsigned int pipe); > -void radeon_disable_vblank_kms(struct drm_device *dev, unsigned int pipe); > void radeon_driver_irq_preinstall_kms(struct drm_device *dev); > int radeon_driver_irq_postinstall_kms(struct drm_device *dev); > void radeon_driver_irq_uninstall_kms(struct drm_device *dev); > @@ -571,10 +568,6 @@ static struct drm_driver kms_driver = { > .postclose = radeon_driver_postclose_kms, > .lastclose = radeon_driver_lastclose_kms, > .unload = radeon_driver_unload_kms, > - .get_vblank_counter = radeon_get_vblank_counter_kms, > - .enable_vblank = radeon_enable_vblank_kms, > - .disable_vblank = radeon_disable_vblank_kms, > - .get_vblank_timestamp = drm_calc_vbltimestamp_from_scanoutpos, > .irq_preinstall = radeon_driver_irq_preinstall_kms, > .irq_postinstall = radeon_driver_irq_postinstall_kms, > .irq_uninstall = radeon_driver_irq_uninstall_kms, > diff --git a/drivers/gpu/drm/radeon/radeon_kms.c b/drivers/gpu/drm/radeon/radeon_kms.c > index d24f23a81656..cab891f86dc0 100644 > --- a/drivers/gpu/drm/radeon/radeon_kms.c > +++ b/drivers/gpu/drm/radeon/radeon_kms.c > @@ -739,14 +739,15 @@ void radeon_driver_postclose_kms(struct drm_device *dev, > /** > * radeon_get_vblank_counter_kms - get frame count > * > - * @dev: drm dev pointer > - * @pipe: crtc to get the frame count from > + * @crtc: crtc to get the frame count from > * > * Gets the frame count on the requested crtc (all asics). > * Returns frame count on success, -EINVAL on failure. > */ > -u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe) > +u32 radeon_get_vblank_counter_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > int vpos, hpos, stat; > u32 count; > struct radeon_device *rdev = dev->dev_private; > @@ -808,25 +809,26 @@ u32 radeon_get_vblank_counter_kms(struct drm_device *dev, unsigned int pipe) > /** > * radeon_enable_vblank_kms - enable vblank interrupt > * > - * @dev: drm dev pointer > * @crtc: crtc to enable vblank interrupt for > * > * Enable the interrupt on the requested crtc (all asics). > * Returns 0 on success, -EINVAL on failure. > */ > -int radeon_enable_vblank_kms(struct drm_device *dev, int crtc) > +int radeon_enable_vblank_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > struct radeon_device *rdev = dev->dev_private; > unsigned long irqflags; > int r; > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %d\n", pipe); > return -EINVAL; > } > > spin_lock_irqsave(&rdev->irq.lock, irqflags); > - rdev->irq.crtc_vblank_int[crtc] = true; > + rdev->irq.crtc_vblank_int[pipe] = true; > r = radeon_irq_set(rdev); > spin_unlock_irqrestore(&rdev->irq.lock, irqflags); > return r; > @@ -835,23 +837,24 @@ int radeon_enable_vblank_kms(struct drm_device *dev, int crtc) > /** > * radeon_disable_vblank_kms - disable vblank interrupt > * > - * @dev: drm dev pointer > * @crtc: crtc to disable vblank interrupt for > * > * Disable the interrupt on the requested crtc (all asics). > */ > -void radeon_disable_vblank_kms(struct drm_device *dev, int crtc) > +void radeon_disable_vblank_kms(struct drm_crtc *crtc) > { > + struct drm_device *dev = crtc->dev; > + unsigned int pipe = crtc->index; > struct radeon_device *rdev = dev->dev_private; > unsigned long irqflags; > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %d\n", pipe); > return; > } > > spin_lock_irqsave(&rdev->irq.lock, irqflags); > - rdev->irq.crtc_vblank_int[crtc] = false; > + rdev->irq.crtc_vblank_int[pipe] = false; > radeon_irq_set(rdev); > spin_unlock_irqrestore(&rdev->irq.lock, irqflags); > } > -- > 2.24.1 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx