From patchwork Thu Sep 13 03:49:49 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rob Clark X-Patchwork-Id: 1450951 Return-Path: X-Original-To: patchwork-dri-devel@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by patchwork2.kernel.org (Postfix) with ESMTP id A3078DF24C for ; Thu, 13 Sep 2012 09:35:46 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 8EC1C9EB46 for ; Thu, 13 Sep 2012 02:35:46 -0700 (PDT) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-ob0-f177.google.com (mail-ob0-f177.google.com [209.85.214.177]) by gabe.freedesktop.org (Postfix) with ESMTP id 628C39EB74 for ; Wed, 12 Sep 2012 20:50:46 -0700 (PDT) Received: by mail-ob0-f177.google.com with SMTP id ta17so3999383obb.36 for ; Wed, 12 Sep 2012 20:50:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:from:to:cc:subject:date:message-id:x-mailer:in-reply-to :references; bh=qAbO/bUoam5BWCgNf4DnvdiPyVWyWy4C/Q5s1lh7SWU=; b=T4xgMnwEClsdNNn9z25NqGlKfrBdzUDEqjpPmsWDaCprtwLq3QCYTA+BegzIlursXp OlL8TZM+az4iGolDpD1Mz+r/Non9a6nKRtoY/P7br+6Vc4HeXmRvvEO0z/tAtdSej6dj AEAvnNNEpqdv94l4GtYnakF5r5bZ67n58E3KSz05QZR4zmNL/Tu6ri1Ll6zcrcjiw3Hc 74XsMrQ6Vmr99yGa6coiB+eO7pg6RAWuD6Ft+F1UaWEWawvExH08TEKsYe1iI6PLe2Ri JvXrIag4ZYSp6G2ByfWumVwRDsNHT0tGGnsQnNdhDyCs0lV2hC+Pzecg3ERE+sKi05bb bd1Q== Received: by 10.182.78.228 with SMTP id e4mr459621obx.77.1347508246092; Wed, 12 Sep 2012 20:50:46 -0700 (PDT) Received: from localhost (ppp-70-129-131-42.dsl.rcsntx.swbell.net. [70.129.131.42]) by mx.google.com with ESMTPS id kc5sm22528093obb.21.2012.09.12.20.50.45 (version=TLSv1/SSLv3 cipher=OTHER); Wed, 12 Sep 2012 20:50:45 -0700 (PDT) From: Rob Clark To: dri-devel@lists.freedesktop.org Subject: [RFC 03/11] drm: add DRM_MODE_PROP_DYNAMIC property flag Date: Wed, 12 Sep 2012 22:49:49 -0500 Message-Id: <1347508195-14939-4-git-send-email-rob.clark@linaro.org> X-Mailer: git-send-email 1.7.9.5 In-Reply-To: <1347508195-14939-1-git-send-email-rob.clark@linaro.org> References: <1347508195-14939-1-git-send-email-rob.clark@linaro.org> Cc: daniel.vetter@ffwll.ch, Rob Clark , patches@linaro.org X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: dri-devel-bounces+patchwork-dri-devel=patchwork.kernel.org@lists.freedesktop.org Errors-To: dri-devel-bounces+patchwork-dri-devel=patchwork.kernel.org@lists.freedesktop.org From: Rob Clark This indicates to userspace that the property is something that can be set dynamically without requiring a "test" step to check if the hw is capable. This allows a userspace compositor, such as weston, to avoid an extra ioctl to check whether it needs to fall-back to GPU to composite some surface prior to submission of GPU render commands. --- include/drm/drm_mode.h | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/include/drm/drm_mode.h b/include/drm/drm_mode.h index cee4c53..15689d4 100644 --- a/include/drm/drm_mode.h +++ b/include/drm/drm_mode.h @@ -232,6 +232,15 @@ struct drm_mode_get_connector { #define DRM_MODE_PROP_BLOB (1<<4) #define DRM_MODE_PROP_BITMASK (1<<5) /* bitmask of enumerated types */ #define DRM_MODE_PROP_OBJECT (1<<6) /* drm mode object */ +/* Properties that are not dynamic cannot safely be changed without a + * atomic-modeset / atomic-pageflip test step. But if userspace is + * only changing dynamic properties, it is guaranteed that the change + * will not exceed hw limits, so no test step is required. + * + * Note that fb_id properties are a bit ambiguous.. they of course can + * be changed dynamically, assuming the pixel format does not change. + */ +#define DRM_MODE_PROP_DYNAMIC (1<<24) struct drm_mode_property_enum { __u64 value;