[Nouveau] [PATCH v4 1/2] drm: Wrap the check for atomic_commit implementation
Daniel Vetter
daniel at ffwll.ch
Tue Dec 27 09:50:16 UTC 2016
On Fri, Dec 23, 2016 at 02:14:00PM -0800, Dhinakaran Pandiyan wrote:
> This check is useful for drivers that do not have DRIVER_ATOMIC set but
> have atomic modesetting internally implemented. Wrap the check into a
> function since this is used in many places and as a bonus, the function
> name helps to document what the check is for.
>
> v2:
> Change return type to bool (Ville)
> Move the function drm_atomic.h (Daniel)
> Fixed comment marker for documentation
>
> v3:
> Included drmP.h to dereference struct drm_device
I've went with a slightly different solution. The proper one is to extract
drm_dev.h (since drmP.h is horribly messy), but that's a lot more work.
-Daniel
>
> Suggested-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
> Cc: Ben Skeggs <bskeggs at redhat.com>
> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
> ---
> drivers/gpu/drm/drm_fb_helper.c | 6 +++---
> drivers/gpu/drm/nouveau/nouveau_connector.c | 5 +++--
> drivers/gpu/drm/nouveau/nouveau_display.c | 6 +++---
> drivers/gpu/drm/nouveau/nouveau_fbcon.c | 3 ++-
> include/drm/drm_atomic.h | 13 +++++++++++++
> 5 files changed, 24 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
> index 145d55f..730342c 100644
> --- a/drivers/gpu/drm/drm_fb_helper.c
> +++ b/drivers/gpu/drm/drm_fb_helper.c
> @@ -405,7 +405,7 @@ static int restore_fbdev_mode(struct drm_fb_helper *fb_helper)
>
> drm_warn_on_modeset_not_all_locked(dev);
>
> - if (dev->mode_config.funcs->atomic_commit)
> + if (drm_drv_uses_atomic_modeset(dev))
> return restore_fbdev_mode_atomic(fb_helper);
>
> drm_for_each_plane(plane, dev) {
> @@ -1444,7 +1444,7 @@ int drm_fb_helper_pan_display(struct fb_var_screeninfo *var,
> return -EBUSY;
> }
>
> - if (dev->mode_config.funcs->atomic_commit) {
> + if (drm_drv_uses_atomic_modeset(dev)) {
> ret = pan_display_atomic(var, info);
> goto unlock;
> }
> @@ -2060,7 +2060,7 @@ static int drm_pick_crtcs(struct drm_fb_helper *fb_helper,
> * NULL we fallback to the default drm_atomic_helper_best_encoder()
> * helper.
> */
> - if (fb_helper->dev->mode_config.funcs->atomic_commit &&
> + if (drm_drv_uses_atomic_modeset(fb_helper->dev) &&
> !connector_funcs->best_encoder)
> encoder = drm_atomic_helper_best_encoder(connector);
> else
> diff --git a/drivers/gpu/drm/nouveau/nouveau_connector.c b/drivers/gpu/drm/nouveau/nouveau_connector.c
> index 947c200..966d20a 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_connector.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_connector.c
> @@ -33,6 +33,7 @@
> #include <drm/drm_atomic_helper.h>
> #include <drm/drm_edid.h>
> #include <drm/drm_crtc_helper.h>
> +#include <drm/drm_atomic.h>
>
> #include "nouveau_reg.h"
> #include "nouveau_drv.h"
> @@ -769,7 +770,7 @@ nouveau_connector_set_property(struct drm_connector *connector,
> struct drm_encoder *encoder = to_drm_encoder(nv_encoder);
> int ret;
>
> - if (connector->dev->mode_config.funcs->atomic_commit)
> + if (drm_drv_uses_atomic_modeset(connector->dev))
> return drm_atomic_helper_connector_set_property(connector, property, value);
>
> ret = connector->funcs->atomic_set_property(&nv_connector->base,
> @@ -1074,7 +1075,7 @@ nouveau_connector_helper_funcs = {
> static int
> nouveau_connector_dpms(struct drm_connector *connector, int mode)
> {
> - if (connector->dev->mode_config.funcs->atomic_commit)
> + if (drm_drv_uses_atomic_modeset(connector->dev))
> return drm_atomic_helper_connector_dpms(connector, mode);
> return drm_helper_connector_dpms(connector, mode);
> }
> diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c b/drivers/gpu/drm/nouveau/nouveau_display.c
> index c5cf888..add353e 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_display.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_display.c
> @@ -162,7 +162,7 @@ nouveau_display_vblstamp(struct drm_device *dev, unsigned int pipe,
> list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
> if (nouveau_crtc(crtc)->index == pipe) {
> struct drm_display_mode *mode;
> - if (dev->mode_config.funcs->atomic_commit)
> + if (drm_drv_uses_atomic_modeset(dev))
> mode = &crtc->state->adjusted_mode;
> else
> mode = &crtc->hwmode;
> @@ -738,7 +738,7 @@ nouveau_display_suspend(struct drm_device *dev, bool runtime)
> struct nouveau_display *disp = nouveau_display(dev);
> struct drm_crtc *crtc;
>
> - if (dev->mode_config.funcs->atomic_commit) {
> + if (drm_drv_uses_atomic_modeset(dev)) {
> if (!runtime) {
> disp->suspend = nouveau_atomic_suspend(dev);
> if (IS_ERR(disp->suspend)) {
> @@ -784,7 +784,7 @@ nouveau_display_resume(struct drm_device *dev, bool runtime)
> struct drm_crtc *crtc;
> int ret;
>
> - if (dev->mode_config.funcs->atomic_commit) {
> + if (drm_drv_uses_atomic_modeset(dev)) {
> nouveau_display_init(dev);
> if (disp->suspend) {
> drm_atomic_helper_resume(dev, disp->suspend);
> diff --git a/drivers/gpu/drm/nouveau/nouveau_fbcon.c b/drivers/gpu/drm/nouveau/nouveau_fbcon.c
> index 5600f6c..9de6abb 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_fbcon.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_fbcon.c
> @@ -41,6 +41,7 @@
> #include <drm/drm_crtc.h>
> #include <drm/drm_crtc_helper.h>
> #include <drm/drm_fb_helper.h>
> +#include <drm/drm_atomic.h>
>
> #include "nouveau_drv.h"
> #include "nouveau_gem.h"
> @@ -524,7 +525,7 @@ nouveau_fbcon_init(struct drm_device *dev)
> preferred_bpp = 32;
>
> /* disable all the possible outputs/crtcs before entering KMS mode */
> - if (!dev->mode_config.funcs->atomic_commit)
> + if (!drm_drv_uses_atomic_modeset(dev))
> drm_helper_disable_unused_functions(dev);
>
> ret = drm_fb_helper_initial_config(&fbcon->helper, preferred_bpp);
> diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h
> index 8cc7ca2..c894dde 100644
> --- a/include/drm/drm_atomic.h
> +++ b/include/drm/drm_atomic.h
> @@ -29,6 +29,7 @@
> #define DRM_ATOMIC_H_
>
> #include <drm/drm_crtc.h>
> +#include <drm/drmP.h>
>
> /**
> * struct drm_crtc_commit - track modeset commits on a CRTC
> @@ -419,5 +420,17 @@ drm_atomic_crtc_needs_modeset(const struct drm_crtc_state *state)
> state->connectors_changed;
> }
>
> +/**
> + * drm_drv_uses_atomic_modeset - check if the driver implements
> + * atomic_commit()
> + * @dev: DRM device
> + *
> + * This check is useful if drivers do not have DRIVER_ATOMIC set but
> + * have atomic modesetting internally implemented.
> + */
> +static inline bool drm_drv_uses_atomic_modeset(struct drm_device *dev)
> +{
> + return dev->mode_config.funcs->atomic_commit != NULL;
> +}
>
> #endif /* DRM_ATOMIC_H_ */
> --
> 2.7.4
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
More information about the Nouveau
mailing list