From patchwork Wed May 23 19:31:01 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jeykumar Sankaran X-Patchwork-Id: 10422197 X-Patchwork-Delegate: agross@codeaurora.org Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 2C57C6032C for ; Wed, 23 May 2018 19:31:40 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 19075291B5 for ; Wed, 23 May 2018 19:31:40 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 0D299291BA; Wed, 23 May 2018 19:31:40 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI, T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 2574C291B5 for ; Wed, 23 May 2018 19:31:37 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934421AbeEWTbe (ORCPT ); Wed, 23 May 2018 15:31:34 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:42010 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934589AbeEWTb1 (ORCPT ); Wed, 23 May 2018 15:31:27 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 7CA7F60F6E; Wed, 23 May 2018 19:31:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1527103886; bh=XVX0OGCLrdbvuXVftgYH0kSORqPGoREPP3Ww5KJExrY=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=E1hqzni5KYnMvsHOEM1gWMCE6rzPKmSTIrFQS2Dd9rJ3yyX8x3iEWWiUVbPaJbDKD SrGBrRQ4ROrrBf3fxnYqrTGMUWcEXykJLPLC6XatOedkm4E0i71r3Iva14cD8/8IOn yWlX6hScPa32eajqGRWdPd09mtOPuKJrehTN6zls= Received: from jeykumar-linux.qualcomm.com (i-global254.qualcomm.com [199.106.103.254]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: jsanka@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 67BA160618; Wed, 23 May 2018 19:31:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1527103875; bh=XVX0OGCLrdbvuXVftgYH0kSORqPGoREPP3Ww5KJExrY=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=jCz4WCW45ISDOlKXdr2ZpyKQN+TLpSGaZtHZppP/uQqXqeOFXT5IC9kJlQoTzNlZx qM6sAfz0xD1UZogGTYhUSliscM8BzFb+ONc4+JdD8I9AHYjjW2eoBYaq1BmIuWlDQk rLZIEMEYOU31SZhnMJM/7shsnZntmTTQRARM0kdE= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 67BA160618 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=jsanka@codeaurora.org From: Jeykumar Sankaran To: dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, linux-arm-msm@vger.kernel.org Cc: Jeykumar Sankaran , seanpaul@chromium.org, robdclark@gmail.com, hoegsberg@google.com Subject: [DPU PATCH 6/7] drm/msm: remove msm_prop files Date: Wed, 23 May 2018 12:31:01 -0700 Message-Id: <1527103862-13934-7-git-send-email-jsanka@codeaurora.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1527103862-13934-1-git-send-email-jsanka@codeaurora.org> References: <1527103862-13934-1-git-send-email-jsanka@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Remove hand rolled msm property caching to handle DPU custom properties. This change also cleans up all its dependencies to cache and restore respective drm states. Signed-off-by: Jeykumar Sankaran Reviewed-by: Sean Paul --- drivers/gpu/drm/msm/Makefile | 1 - drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 2 - drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 239 +-------- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 16 - drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h | 2 - drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 123 +---- drivers/gpu/drm/msm/disp/dpu1/dpu_plane.h | 12 - drivers/gpu/drm/msm/msm_drv.h | 16 +- drivers/gpu/drm/msm/msm_prop.c | 688 -------------------------- drivers/gpu/drm/msm/msm_prop.h | 438 ---------------- 10 files changed, 8 insertions(+), 1529 deletions(-) delete mode 100644 drivers/gpu/drm/msm/msm_prop.c delete mode 100644 drivers/gpu/drm/msm/msm_prop.h diff --git a/drivers/gpu/drm/msm/Makefile b/drivers/gpu/drm/msm/Makefile index d289503..5331188 100644 --- a/drivers/gpu/drm/msm/Makefile +++ b/drivers/gpu/drm/msm/Makefile @@ -76,7 +76,6 @@ msm-y := \ dpu_io_util.o \ dpu_dbg_evtlog.o \ dpu_power_handle.o \ - msm_prop.o \ msm_atomic.o \ msm_debugfs.o \ msm_drv.o \ diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c index c4820de..e4b82d5 100644 --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c @@ -19,8 +19,6 @@ #include #include -#include "msm_prop.h" - #include "dpu_kms.h" #include "dpu_trace.h" #include "dpu_crtc.h" diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c index b0a3a30..43d9985 100644 --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c @@ -579,10 +579,6 @@ static void dpu_crtc_destroy(struct drm_crtc *crtc) if (!crtc) return; - if (dpu_crtc->blob_info) - drm_property_blob_put(dpu_crtc->blob_info); - msm_property_destroy(&dpu_crtc->property_info); - _dpu_crtc_deinit_events(dpu_crtc); drm_crtc_cleanup(crtc); @@ -1390,9 +1386,7 @@ static void dpu_crtc_destroy_state(struct drm_crtc *crtc, __drm_atomic_helper_crtc_destroy_state(state); - /* destroy value helper */ - msm_property_destroy_state(&dpu_crtc->property_info, cstate, - &cstate->property_state); + kfree(cstate); } static int _dpu_crtc_wait_for_frame_done(struct drm_crtc *crtc) @@ -1641,17 +1635,12 @@ static struct drm_crtc_state *dpu_crtc_duplicate_state(struct drm_crtc *crtc) dpu_crtc = to_dpu_crtc(crtc); old_cstate = to_dpu_crtc_state(crtc->state); - cstate = msm_property_alloc_state(&dpu_crtc->property_info); + cstate = kmemdup(old_cstate, sizeof(*old_cstate), GFP_KERNEL); if (!cstate) { DPU_ERROR("failed to allocate state\n"); return NULL; } - /* duplicate value helper */ - msm_property_duplicate_state(&dpu_crtc->property_info, - old_cstate, cstate, - &cstate->property_state, cstate->property_values); - /* duplicate base helper */ __drm_atomic_helper_crtc_duplicate_state(crtc, &cstate->base); @@ -1687,17 +1676,12 @@ static void dpu_crtc_reset(struct drm_crtc *crtc) } dpu_crtc = to_dpu_crtc(crtc); - cstate = msm_property_alloc_state(&dpu_crtc->property_info); + cstate = kzalloc(sizeof(*cstate), GFP_KERNEL); if (!cstate) { DPU_ERROR("failed to allocate state\n"); return; } - /* reset value helper */ - msm_property_reset_state(&dpu_crtc->property_info, cstate, - &cstate->property_state, - cstate->property_values); - _dpu_crtc_rp_reset(&cstate->rp, &dpu_crtc->rp_lock, &dpu_crtc->rp_head); @@ -2194,212 +2178,6 @@ void dpu_crtc_cancel_pending_flip(struct drm_crtc *crtc, struct drm_file *file) _dpu_crtc_complete_flip(crtc, file); } -/** - * dpu_crtc_install_properties - install all drm properties for crtc - * @crtc: Pointer to drm crtc structure - */ -static void dpu_crtc_install_properties(struct drm_crtc *crtc, - struct dpu_mdss_cfg *catalog) -{ - struct dpu_crtc *dpu_crtc; - struct drm_device *dev; - struct dpu_kms_info *info; - struct dpu_kms *dpu_kms; - - DPU_DEBUG("\n"); - - if (!crtc || !catalog) { - DPU_ERROR("invalid crtc or catalog\n"); - return; - } - - dpu_crtc = to_dpu_crtc(crtc); - dev = crtc->dev; - dpu_kms = _dpu_crtc_get_kms(crtc); - - if (!dpu_kms) { - DPU_ERROR("invalid argument\n"); - return; - } - - info = kzalloc(sizeof(struct dpu_kms_info), GFP_KERNEL); - if (!info) { - DPU_ERROR("failed to allocate info memory\n"); - return; - } - - msm_property_install_blob(&dpu_crtc->property_info, "capabilities", - DRM_MODE_PROP_IMMUTABLE, CRTC_PROP_INFO); - - dpu_kms_info_reset(info); - - dpu_kms_info_add_keyint(info, "hw_version", catalog->hwversion); - dpu_kms_info_add_keyint(info, "max_linewidth", - catalog->caps->max_mixer_width); - dpu_kms_info_add_keyint(info, "max_blendstages", - catalog->caps->max_mixer_blendstages); - if (catalog->caps->qseed_type == DPU_SSPP_SCALER_QSEED2) - dpu_kms_info_add_keystr(info, "qseed_type", "qseed2"); - if (catalog->caps->qseed_type == DPU_SSPP_SCALER_QSEED3) - dpu_kms_info_add_keystr(info, "qseed_type", "qseed3"); - - if (dpu_is_custom_client()) { - if (catalog->caps->smart_dma_rev == DPU_SSPP_SMART_DMA_V1) - dpu_kms_info_add_keystr(info, - "smart_dma_rev", "smart_dma_v1"); - if (catalog->caps->smart_dma_rev == DPU_SSPP_SMART_DMA_V2) - dpu_kms_info_add_keystr(info, - "smart_dma_rev", "smart_dma_v2"); - } - - dpu_kms_info_add_keyint(info, "has_src_split", - catalog->caps->has_src_split); - if (catalog->perf.max_bw_low) - dpu_kms_info_add_keyint(info, "max_bandwidth_low", - catalog->perf.max_bw_low * 1000LL); - if (catalog->perf.max_bw_high) - dpu_kms_info_add_keyint(info, "max_bandwidth_high", - catalog->perf.max_bw_high * 1000LL); - if (catalog->perf.min_core_ib) - dpu_kms_info_add_keyint(info, "min_core_ib", - catalog->perf.min_core_ib * 1000LL); - if (catalog->perf.min_llcc_ib) - dpu_kms_info_add_keyint(info, "min_llcc_ib", - catalog->perf.min_llcc_ib * 1000LL); - if (catalog->perf.min_dram_ib) - dpu_kms_info_add_keyint(info, "min_dram_ib", - catalog->perf.min_dram_ib * 1000LL); - if (dpu_kms->perf.max_core_clk_rate) - dpu_kms_info_add_keyint(info, "max_mdp_clk", - dpu_kms->perf.max_core_clk_rate); - dpu_kms_info_add_keystr(info, "core_ib_ff", - catalog->perf.core_ib_ff); - dpu_kms_info_add_keystr(info, "core_clk_ff", - catalog->perf.core_clk_ff); - dpu_kms_info_add_keystr(info, "comp_ratio_rt", - catalog->perf.comp_ratio_rt); - dpu_kms_info_add_keystr(info, "comp_ratio_nrt", - catalog->perf.comp_ratio_nrt); - dpu_kms_info_add_keyint(info, "dest_scale_prefill_lines", - catalog->perf.dest_scale_prefill_lines); - dpu_kms_info_add_keyint(info, "undersized_prefill_lines", - catalog->perf.undersized_prefill_lines); - dpu_kms_info_add_keyint(info, "macrotile_prefill_lines", - catalog->perf.macrotile_prefill_lines); - dpu_kms_info_add_keyint(info, "yuv_nv12_prefill_lines", - catalog->perf.yuv_nv12_prefill_lines); - dpu_kms_info_add_keyint(info, "linear_prefill_lines", - catalog->perf.linear_prefill_lines); - dpu_kms_info_add_keyint(info, "downscaling_prefill_lines", - catalog->perf.downscaling_prefill_lines); - dpu_kms_info_add_keyint(info, "xtra_prefill_lines", - catalog->perf.xtra_prefill_lines); - dpu_kms_info_add_keyint(info, "amortizable_threshold", - catalog->perf.amortizable_threshold); - dpu_kms_info_add_keyint(info, "min_prefill_lines", - catalog->perf.min_prefill_lines); - - msm_property_set_blob(&dpu_crtc->property_info, &dpu_crtc->blob_info, - info->data, DPU_KMS_INFO_DATALEN(info), CRTC_PROP_INFO); - - kfree(info); -} - -/** - * dpu_crtc_atomic_set_property - atomically set a crtc drm property - * @crtc: Pointer to drm crtc structure - * @state: Pointer to drm crtc state structure - * @property: Pointer to targeted drm property - * @val: Updated property value - * @Returns: Zero on success - */ -static int dpu_crtc_atomic_set_property(struct drm_crtc *crtc, - struct drm_crtc_state *state, - struct drm_property *property, - uint64_t val) -{ - struct dpu_crtc *dpu_crtc; - struct dpu_crtc_state *cstate; - int ret = -EINVAL; - - if (!crtc || !state || !property) { - DPU_ERROR("invalid argument(s)\n"); - } else { - dpu_crtc = to_dpu_crtc(crtc); - cstate = to_dpu_crtc_state(state); - ret = msm_property_atomic_set(&dpu_crtc->property_info, - &cstate->property_state, property, val); - if (ret) - DRM_ERROR("failed to set the property\n"); - - DPU_DEBUG("crtc%d %s[%d] <= 0x%llx ret=%d\n", crtc->base.id, - property->name, property->base.id, val, ret); - } - - return ret; -} - -/** - * dpu_crtc_set_property - set a crtc drm property - * @crtc: Pointer to drm crtc structure - * @property: Pointer to targeted drm property - * @val: Updated property value - * @Returns: Zero on success - */ -static int dpu_crtc_set_property(struct drm_crtc *crtc, - struct drm_property *property, uint64_t val) -{ - DPU_DEBUG("\n"); - - return dpu_crtc_atomic_set_property(crtc, crtc->state, property, val); -} - -/** - * dpu_crtc_atomic_get_property - retrieve a crtc drm property - * @crtc: Pointer to drm crtc structure - * @state: Pointer to drm crtc state structure - * @property: Pointer to targeted drm property - * @val: Pointer to variable for receiving property value - * @Returns: Zero on success - */ -static int dpu_crtc_atomic_get_property(struct drm_crtc *crtc, - const struct drm_crtc_state *state, - struct drm_property *property, - uint64_t *val) -{ - struct dpu_crtc *dpu_crtc; - struct dpu_crtc_state *cstate; - struct drm_encoder *encoder; - int i, ret = -EINVAL; - bool is_cmd = true; - - if (!crtc || !state) { - DPU_ERROR("invalid argument(s)\n"); - } else { - dpu_crtc = to_dpu_crtc(crtc); - cstate = to_dpu_crtc_state(state); - - /** - * set the cmd flag only when all the encoders attached - * to the crtc are in cmd mode. Consider all other cases - * as video mode. - */ - drm_for_each_encoder(encoder, crtc->dev) { - if (encoder->crtc == crtc) - is_cmd = dpu_encoder_check_mode(encoder, - MSM_DISPLAY_CAP_CMD_MODE); - } - - i = msm_property_index(&dpu_crtc->property_info, property); - ret = msm_property_atomic_get(&dpu_crtc->property_info, - &cstate->property_state, - property, val); - if (ret) - DRM_ERROR("get property failed\n"); - } - return ret; -} - #ifdef CONFIG_DEBUG_FS static int _dpu_debugfs_status_show(struct seq_file *s, void *data) { @@ -2759,9 +2537,6 @@ static void dpu_crtc_early_unregister(struct drm_crtc *crtc) .set_config = drm_atomic_helper_set_config, .destroy = dpu_crtc_destroy, .page_flip = drm_atomic_helper_page_flip, - .set_property = dpu_crtc_set_property, - .atomic_set_property = dpu_crtc_atomic_set_property, - .atomic_get_property = dpu_crtc_atomic_get_property, .reset = dpu_crtc_reset, .atomic_duplicate_state = dpu_crtc_duplicate_state, .atomic_destroy_state = dpu_crtc_destroy_state, @@ -2924,14 +2699,6 @@ struct drm_crtc *dpu_crtc_init(struct drm_device *dev, struct drm_plane *plane) return ERR_PTR(rc); } - /* create CRTC properties */ - msm_property_init(&dpu_crtc->property_info, &crtc->base, dev, - priv->crtc_property, dpu_crtc->property_data, - CRTC_PROP_COUNT, CRTC_PROP_BLOBCOUNT, - sizeof(struct dpu_crtc_state)); - - dpu_crtc_install_properties(crtc, kms->catalog); - DPU_DEBUG("%s: successfully initialized crtc\n", dpu_crtc->name); return crtc; } diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h index 6628eb3..f752101 100644 --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h @@ -22,7 +22,6 @@ #include #include #include -#include "msm_prop.h" #include "dpu_kms.h" #include "dpu_core_perf.h" #include "dpu_hw_blk.h" @@ -200,10 +199,6 @@ struct dpu_crtc { struct drm_pending_vblank_event *event; u32 vsync_count; - struct msm_property_info property_info; - struct msm_property_data property_data[CRTC_PROP_COUNT]; - struct drm_property_blob *blob_info; - struct dpu_hw_stage_cfg stage_cfg; struct dentry *debugfs_root; @@ -324,8 +319,6 @@ struct dpu_crtc_state { bool is_ppsplit; struct dpu_rect lm_bounds[CRTC_DUAL_MIXERS]; - struct msm_property_state property_state; - struct msm_property_value property_values[CRTC_PROP_COUNT]; uint64_t input_fence_timeout_ns; struct dpu_core_perf_params new_perf; @@ -336,15 +329,6 @@ struct dpu_crtc_state { container_of(x, struct dpu_crtc_state, base) /** - * dpu_crtc_get_property - query integer value of crtc property - * @S: Pointer to crtc state - * @X: Property index, from enum msm_mdp_crtc_property - * Returns: Integer value of requested property - */ -#define dpu_crtc_get_property(S, X) \ - ((S) && ((X) < CRTC_PROP_COUNT) ? ((S)->property_values[(X)].value) : 0) - -/** * dpu_crtc_get_mixer_width - get the mixer width * Mixer width will be same as panel width(/2 for split) */ diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h index d853ad9..0339dfd 100644 --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h @@ -20,8 +20,6 @@ #define __DPU_ENCODER_H__ #include - -#include "msm_prop.h" #include "dpu_hw_mdss.h" #define DPU_ENCODER_FRAME_EVENT_DONE BIT(0) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c index 28735c8..faf5903 100644 --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c @@ -20,9 +20,8 @@ #include #include -#include "msm_prop.h" -#include "msm_drv.h" +#include "msm_drv.h" #include "dpu_kms.h" #include "dpu_formats.h" #include "dpu_hw_sspp.h" @@ -119,9 +118,6 @@ struct dpu_plane { const struct dpu_sspp_sub_blks *pipe_sblk; char pipe_name[DPU_NAME_SIZE]; - struct msm_property_info property_info; - struct msm_property_data property_data[PLANE_PROP_COUNT]; - struct drm_property_blob *blob_info; /* debugfs related stuff */ struct dentry *debugfs_root; @@ -1401,7 +1397,6 @@ static int dpu_plane_sspp_atomic_update(struct drm_plane *plane, } _dpu_plane_set_qos_remap(plane); - return 0; } @@ -1486,89 +1481,6 @@ void dpu_plane_restore(struct drm_plane *plane) dpu_plane_atomic_update(plane, plane->state); } -/* helper to install properties which are common to planes and crtcs */ -static void _dpu_plane_install_properties(struct drm_plane *plane, - struct dpu_mdss_cfg *catalog, u32 master_plane_id) -{ - struct dpu_plane *pdpu = to_dpu_plane(plane); - int zpos_max = 255; - int zpos_def = 0; - - if (!plane || !pdpu) { - DPU_ERROR("invalid plane\n"); - return; - } else if (!pdpu->pipe_hw || !pdpu->pipe_sblk) { - DPU_ERROR("invalid plane, pipe_hw %d pipe_sblk %d\n", - pdpu->pipe_hw != 0, pdpu->pipe_sblk != 0); - return; - } else if (!catalog) { - DPU_ERROR("invalid catalog\n"); - return; - } - - pdpu->catalog = catalog; - - if (dpu_is_custom_client()) { - if (catalog->mixer_count && - catalog->mixer[0].sblk->maxblendstages) { - zpos_max = catalog->mixer[0].sblk->maxblendstages - 1; - if (zpos_max > DPU_STAGE_MAX - DPU_STAGE_0 - 1) - zpos_max = DPU_STAGE_MAX - DPU_STAGE_0 - 1; - } - } else if (plane->type != DRM_PLANE_TYPE_PRIMARY) { - /* reserve zpos == 0 for primary planes */ - zpos_def = drm_plane_index(plane) + 1; - } -} - -static int dpu_plane_atomic_set_property(struct drm_plane *plane, - struct drm_plane_state *state, struct drm_property *property, - uint64_t val) -{ - struct dpu_plane *pdpu = plane ? to_dpu_plane(plane) : NULL; - struct dpu_plane_state *pstate; - int ret = -EINVAL; - - DPU_DEBUG_PLANE(pdpu, "\n"); - - if (!plane) { - DPU_ERROR("invalid plane\n"); - } else if (!state) { - DPU_ERROR_PLANE(pdpu, "invalid state\n"); - } else { - pstate = to_dpu_plane_state(state); - ret = msm_property_atomic_set(&pdpu->property_info, - &pstate->property_state, property, val); - } - - DPU_DEBUG_PLANE(pdpu, "%s[%d] <= 0x%llx ret=%d\n", - property->name, property->base.id, val, ret); - - return ret; -} - -static int dpu_plane_atomic_get_property(struct drm_plane *plane, - const struct drm_plane_state *state, - struct drm_property *property, uint64_t *val) -{ - struct dpu_plane *pdpu = plane ? to_dpu_plane(plane) : NULL; - struct dpu_plane_state *pstate; - int ret = -EINVAL; - - if (!plane) { - DPU_ERROR("invalid plane\n"); - } else if (!state) { - DPU_ERROR("invalid state\n"); - } else { - DPU_DEBUG_PLANE(pdpu, "\n"); - pstate = to_dpu_plane_state(state); - ret = msm_property_atomic_get(&pdpu->property_info, - &pstate->property_state, property, val); - } - - return ret; -} - static void dpu_plane_destroy(struct drm_plane *plane) { struct dpu_plane *pdpu = plane ? to_dpu_plane(plane) : NULL; @@ -1578,9 +1490,6 @@ static void dpu_plane_destroy(struct drm_plane *plane) if (pdpu) { _dpu_plane_set_qos_ctrl(plane, false, DPU_PLANE_QOS_PANIC_CTRL); - if (pdpu->blob_info) - drm_property_blob_put(pdpu->blob_info); - msm_property_destroy(&pdpu->property_info); mutex_destroy(&pdpu->lock); drm_plane_helper_disable(plane); @@ -1598,7 +1507,6 @@ static void dpu_plane_destroy(struct drm_plane *plane) static void dpu_plane_destroy_state(struct drm_plane *plane, struct drm_plane_state *state) { - struct dpu_plane *pdpu; struct dpu_plane_state *pstate; if (!plane || !state) { @@ -1607,18 +1515,13 @@ static void dpu_plane_destroy_state(struct drm_plane *plane, return; } - pdpu = to_dpu_plane(plane); pstate = to_dpu_plane_state(state); - DPU_DEBUG_PLANE(pdpu, "\n"); - /* remove ref count for frame buffers */ if (state->fb) drm_framebuffer_put(state->fb); - /* destroy value helper */ - msm_property_destroy_state(&pdpu->property_info, pstate, - &pstate->property_state); + kfree(pstate); } static struct drm_plane_state * @@ -1638,7 +1541,7 @@ static void dpu_plane_destroy_state(struct drm_plane *plane, old_state = to_dpu_plane_state(plane->state); pdpu = to_dpu_plane(plane); - pstate = msm_property_alloc_state(&pdpu->property_info); + pstate = kmemdup(old_state, sizeof(*old_state), GFP_KERNEL); if (!pstate) { DPU_ERROR_PLANE(pdpu, "failed to allocate state\n"); return NULL; @@ -1646,10 +1549,6 @@ static void dpu_plane_destroy_state(struct drm_plane *plane, DPU_DEBUG_PLANE(pdpu, "\n"); - /* duplicate value helper */ - msm_property_duplicate_state(&pdpu->property_info, old_state, pstate, - &pstate->property_state, pstate->property_values); - pstate->pending = false; __drm_atomic_helper_plane_duplicate_state(plane, &pstate->base); @@ -1676,17 +1575,12 @@ static void dpu_plane_reset(struct drm_plane *plane) plane->state = 0; } - pstate = msm_property_alloc_state(&pdpu->property_info); + pstate = kzalloc(sizeof(*pstate), GFP_KERNEL); if (!pstate) { DPU_ERROR_PLANE(pdpu, "failed to allocate state\n"); return; } - /* reset value helper */ - msm_property_reset_state(&pdpu->property_info, pstate, - &pstate->property_state, - pstate->property_values); - pstate->base.plane = plane; plane->state = &pstate->base; @@ -1919,8 +1813,6 @@ static void dpu_plane_early_unregister(struct drm_plane *plane) .update_plane = drm_atomic_helper_update_plane, .disable_plane = drm_atomic_helper_disable_plane, .destroy = dpu_plane_destroy, - .atomic_set_property = dpu_plane_atomic_set_property, - .atomic_get_property = dpu_plane_atomic_get_property, .reset = dpu_plane_reset, .atomic_duplicate_state = dpu_plane_duplicate_state, .atomic_destroy_state = dpu_plane_destroy_state, @@ -2064,13 +1956,6 @@ struct drm_plane *dpu_plane_init(struct drm_device *dev, /* success! finalize initialization */ drm_plane_helper_add(plane, &dpu_plane_helper_funcs); - msm_property_init(&pdpu->property_info, &plane->base, dev, - priv->plane_property, pdpu->property_data, - PLANE_PROP_COUNT, PLANE_PROP_BLOBCOUNT, - sizeof(struct dpu_plane_state)); - - _dpu_plane_install_properties(plane, kms->catalog, master_plane_id); - /* save user friendly pipe name for later */ snprintf(pdpu->pipe_name, DPU_NAME_SIZE, "plane%u", plane->base.id); diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.h index a06c8b4..f6fe6dd 100644 --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.h +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.h @@ -21,7 +21,6 @@ #include -#include "msm_prop.h" #include "dpu_kms.h" #include "dpu_hw_mdss.h" #include "dpu_hw_sspp.h" @@ -44,8 +43,6 @@ */ struct dpu_plane_state { struct drm_plane_state base; - struct msm_property_state property_state; - struct msm_property_value property_values[PLANE_PROP_COUNT]; struct msm_gem_address_space *aspace; void *input_fence; enum dpu_stage stage; @@ -74,15 +71,6 @@ struct dpu_multirect_plane_states { container_of(x, struct dpu_plane_state, base) /** - * dpu_plane_get_property - Query integer value of plane property - * @S: Pointer to plane state - * @X: Property index, from enum msm_mdp_plane_property - * Returns: Integer value of requested property - */ -#define dpu_plane_get_property(S, X) ((S) && ((X) < PLANE_PROP_COUNT) ? \ - ((S)->property_values[(X)].value) : 0) - -/** * dpu_plane_pipe - return sspp identifier for the given plane * @plane: Pointer to DRM plane object * Returns: sspp identifier of the given plane diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h index 8e80b4b..edb395d 100644 --- a/drivers/gpu/drm/msm/msm_drv.h +++ b/drivers/gpu/drm/msm/msm_drv.h @@ -77,26 +77,13 @@ struct msm_file_private { }; enum msm_mdp_plane_property { - /* # of blob properties */ - PLANE_PROP_BLOBCOUNT, - /* range properties */ - PLANE_PROP_ZPOS = PLANE_PROP_BLOBCOUNT, + PLANE_PROP_ZPOS, /* total # of properties */ PLANE_PROP_COUNT }; -enum msm_mdp_crtc_property { - CRTC_PROP_INFO, - - /* # of blob properties */ - CRTC_PROP_BLOBCOUNT, - - /* total # of properties */ - CRTC_PROP_COUNT -}; - struct msm_vblank_ctrl { struct kthread_work work; struct list_head event_list; @@ -364,7 +351,6 @@ struct msm_drm_private { /* Properties */ struct drm_property *plane_property[PLANE_PROP_COUNT]; - struct drm_property *crtc_property[CRTC_PROP_COUNT]; /* Color processing properties for the crtc */ struct drm_property **cp_property; diff --git a/drivers/gpu/drm/msm/msm_prop.c b/drivers/gpu/drm/msm/msm_prop.c deleted file mode 100644 index 8f27cde..0000000 --- a/drivers/gpu/drm/msm/msm_prop.c +++ /dev/null @@ -1,688 +0,0 @@ -/* Copyright (c) 2016-2018, The Linux Foundation. All rights reserved. - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License version 2 and - * only version 2 as published by the Free Software Foundation. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - */ - -#include "msm_prop.h" - -void msm_property_init(struct msm_property_info *info, - struct drm_mode_object *base, - struct drm_device *dev, - struct drm_property **property_array, - struct msm_property_data *property_data, - uint32_t property_count, - uint32_t blob_count, - uint32_t state_size) -{ - /* prevent access if any of these are NULL */ - if (!base || !dev || !property_array || !property_data) { - property_count = 0; - blob_count = 0; - - DRM_ERROR("invalid arguments, forcing zero properties\n"); - return; - } - - /* can't have more blob properties than total properties */ - if (blob_count > property_count) { - blob_count = property_count; - - DBG("Capping number of blob properties to %d", blob_count); - } - - if (!info) { - DRM_ERROR("info pointer is NULL\n"); - } else { - info->base = base; - info->dev = dev; - info->property_array = property_array; - info->property_data = property_data; - info->property_count = property_count; - info->blob_count = blob_count; - info->install_request = 0; - info->install_count = 0; - info->recent_idx = 0; - info->is_active = false; - info->state_size = state_size; - info->state_cache_size = 0; - mutex_init(&info->property_lock); - - memset(property_data, - 0, - sizeof(struct msm_property_data) * - property_count); - } -} - -void msm_property_destroy(struct msm_property_info *info) -{ - if (!info) - return; - - /* free state cache */ - while (info->state_cache_size > 0) - kfree(info->state_cache[--(info->state_cache_size)]); - - mutex_destroy(&info->property_lock); -} - -int msm_property_pop_dirty(struct msm_property_info *info, - struct msm_property_state *property_state) -{ - struct list_head *item; - int rc = 0; - - if (!info || !property_state || !property_state->values) { - DRM_ERROR("invalid argument(s)\n"); - return -EINVAL; - } - - mutex_lock(&info->property_lock); - if (list_empty(&property_state->dirty_list)) { - rc = -EAGAIN; - } else { - item = property_state->dirty_list.next; - list_del_init(item); - rc = container_of(item, struct msm_property_value, dirty_node) - - property_state->values; - DRM_DEBUG_KMS("property %d dirty\n", rc); - } - mutex_unlock(&info->property_lock); - - return rc; -} - -/** - * _msm_property_set_dirty_no_lock - flag given property as being dirty - * This function doesn't mutex protect the - * dirty linked list. - * @info: Pointer to property info container struct - * @property_state: Pointer to property state container struct - * @property_idx: Property index - */ -static void _msm_property_set_dirty_no_lock( - struct msm_property_info *info, - struct msm_property_state *property_state, - uint32_t property_idx) -{ - if (!info || !property_state || !property_state->values || - property_idx >= info->property_count) { - DRM_ERROR("invalid argument(s), idx %u\n", property_idx); - return; - } - - /* avoid re-inserting if already dirty */ - if (!list_empty(&property_state->values[property_idx].dirty_node)) { - DRM_DEBUG_KMS("property %u already dirty\n", property_idx); - return; - } - - list_add_tail(&property_state->values[property_idx].dirty_node, - &property_state->dirty_list); -} - -/** - * _msm_property_install_integer - install standard drm range property - * @info: Pointer to property info container struct - * @name: Property name - * @flags: Other property type flags, e.g. DRM_MODE_PROP_IMMUTABLE - * @min: Min property value - * @max: Max property value - * @init: Default Property value - * @property_idx: Property index - * @force_dirty: Whether or not to filter 'dirty' status on unchanged values - */ -static void _msm_property_install_integer(struct msm_property_info *info, - const char *name, int flags, uint64_t min, uint64_t max, - uint64_t init, uint32_t property_idx, bool force_dirty) -{ - struct drm_property **prop; - - if (!info) - return; - - ++info->install_request; - - if (!name || (property_idx >= info->property_count)) { - DRM_ERROR("invalid argument(s), %s\n", name ? name : "null"); - } else { - prop = &info->property_array[property_idx]; - /* - * Properties need to be attached to each drm object that - * uses them, but only need to be created once - */ - if (*prop == 0) { - *prop = drm_property_create_range(info->dev, - flags, name, min, max); - if (*prop == 0) - DRM_ERROR("create %s property failed\n", name); - } - - /* save init value for later */ - info->property_data[property_idx].default_value = init; - info->property_data[property_idx].force_dirty = force_dirty; - - /* always attach property, if created */ - if (*prop) { - drm_object_attach_property(info->base, *prop, init); - ++info->install_count; - } - } -} - -void msm_property_install_range(struct msm_property_info *info, - const char *name, int flags, uint64_t min, uint64_t max, - uint64_t init, uint32_t property_idx) -{ - _msm_property_install_integer(info, name, flags, - min, max, init, property_idx, false); -} - -void msm_property_install_volatile_range(struct msm_property_info *info, - const char *name, int flags, uint64_t min, uint64_t max, - uint64_t init, uint32_t property_idx) -{ - _msm_property_install_integer(info, name, flags, - min, max, init, property_idx, true); -} - -void msm_property_install_rotation(struct msm_property_info *info, - struct drm_plane *plane, unsigned int rotation, - unsigned int supported_rotations, uint32_t property_idx) -{ - if (!info || !plane) - return; - - ++info->install_request; - - if (property_idx >= info->property_count) { - DRM_ERROR("invalid property index %d\n", property_idx); - return; - } - - if (drm_plane_create_rotation_property(plane, - rotation, supported_rotations)) { - DRM_ERROR("create rotation property failed\n"); - return; - } - - if (plane->rotation_property) { - info->property_array[property_idx] = plane->rotation_property; - - /* save init value for later */ - info->property_data[property_idx].default_value = rotation; - info->property_data[property_idx].force_dirty = false; - - ++info->install_count; - } -} - -void msm_property_install_enum(struct msm_property_info *info, - const char *name, int flags, int is_bitmask, - const struct drm_prop_enum_list *values, int num_values, - uint32_t property_idx) -{ - struct drm_property **prop; - - if (!info) - return; - - ++info->install_request; - - if (!name || !values || !num_values || - (property_idx >= info->property_count)) { - DRM_ERROR("invalid argument(s), %s\n", name ? name : "null"); - } else { - prop = &info->property_array[property_idx]; - /* - * Properties need to be attached to each drm object that - * uses them, but only need to be created once - */ - if (*prop == 0) { - /* 'bitmask' is a special type of 'enum' */ - if (is_bitmask) - *prop = drm_property_create_bitmask(info->dev, - DRM_MODE_PROP_BITMASK | flags, - name, values, num_values, -1); - else - *prop = drm_property_create_enum(info->dev, - DRM_MODE_PROP_ENUM | flags, - name, values, num_values); - if (*prop == 0) - DRM_ERROR("create %s property failed\n", name); - } - - /* save init value for later */ - info->property_data[property_idx].default_value = 0; - info->property_data[property_idx].force_dirty = false; - - /* select first defined value for enums */ - if (!is_bitmask) - info->property_data[property_idx].default_value = - values->type; - - /* always attach property, if created */ - if (*prop) { - drm_object_attach_property(info->base, *prop, - info->property_data - [property_idx].default_value); - ++info->install_count; - } - } -} - -void msm_property_install_blob(struct msm_property_info *info, - const char *name, int flags, uint32_t property_idx) -{ - struct drm_property **prop; - - if (!info) - return; - - ++info->install_request; - - if (!name || (property_idx >= info->blob_count)) { - DRM_ERROR("invalid argument(s), %s\n", name ? name : "null"); - } else { - prop = &info->property_array[property_idx]; - /* - * Properties need to be attached to each drm object that - * uses them, but only need to be created once - */ - if (*prop == 0) { - /* use 'create' for blob property place holder */ - *prop = drm_property_create(info->dev, - DRM_MODE_PROP_BLOB | flags, name, 0); - if (*prop == 0) - DRM_ERROR("create %s property failed\n", name); - } - - /* save init value for later */ - info->property_data[property_idx].default_value = 0; - info->property_data[property_idx].force_dirty = true; - - /* always attach property, if created */ - if (*prop) { - drm_object_attach_property(info->base, *prop, -1); - ++info->install_count; - } - } -} - -int msm_property_install_get_status(struct msm_property_info *info) -{ - int rc = -ENOMEM; - - if (info && (info->install_request == info->install_count)) - rc = 0; - - return rc; -} - -int msm_property_index(struct msm_property_info *info, - struct drm_property *property) -{ - uint32_t count; - int32_t idx; - int rc = -EINVAL; - - if (!info || !property) { - DRM_ERROR("invalid argument(s)\n"); - } else { - /* - * Linear search, but start from last found index. This will - * help if any single property is accessed multiple times in a - * row. Ideally, we could keep a list of properties sorted in - * the order of most recent access, but that may be overkill - * for now. - */ - mutex_lock(&info->property_lock); - idx = info->recent_idx; - count = info->property_count; - while (count) { - --count; - - /* stop searching on match */ - if (info->property_array[idx] == property) { - info->recent_idx = idx; - rc = idx; - break; - } - - /* move to next valid index */ - if (--idx < 0) - idx = info->property_count - 1; - } - mutex_unlock(&info->property_lock); - } - - return rc; -} - -int msm_property_set_dirty(struct msm_property_info *info, - struct msm_property_state *property_state, - int property_idx) -{ - if (!info || !property_state || !property_state->values) { - DRM_ERROR("invalid argument(s)\n"); - return -EINVAL; - } - mutex_lock(&info->property_lock); - _msm_property_set_dirty_no_lock(info, property_state, property_idx); - mutex_unlock(&info->property_lock); - return 0; -} - -int msm_property_atomic_set(struct msm_property_info *info, - struct msm_property_state *property_state, - struct drm_property *property, uint64_t val) -{ - struct drm_property_blob *blob; - int property_idx, rc = -EINVAL; - - property_idx = msm_property_index(info, property); - if (!info || !property_state || - (property_idx == -EINVAL) || !property_state->values) { - DRM_DEBUG("invalid argument(s)\n"); - } else { - /* extra handling for incoming properties */ - mutex_lock(&info->property_lock); - if ((property->flags & DRM_MODE_PROP_BLOB) && - (property_idx < info->blob_count)) { - /* DRM lookup also takes a reference */ - blob = drm_property_lookup_blob(info->dev, - (uint32_t)val); - if (!blob) { - DRM_ERROR("blob not found\n"); - val = 0; - } else { - DBG("Blob %u saved", blob->base.id); - val = blob->base.id; - - /* save blob - need to clear previous ref */ - if (property_state->values[property_idx].blob) - drm_property_blob_put( - property_state->values[ - property_idx].blob); - property_state->values[property_idx].blob = - blob; - } - } - - /* update value and flag as dirty */ - if (property_state->values[property_idx].value != val || - info->property_data[property_idx].force_dirty) { - property_state->values[property_idx].value = val; - _msm_property_set_dirty_no_lock(info, property_state, - property_idx); - - DBG("%s - %lld", property->name, val); - } - mutex_unlock(&info->property_lock); - rc = 0; - } - - return rc; -} - -int msm_property_atomic_get(struct msm_property_info *info, - struct msm_property_state *property_state, - struct drm_property *property, uint64_t *val) -{ - int property_idx, rc = -EINVAL; - - property_idx = msm_property_index(info, property); - if (!info || (property_idx == -EINVAL) || - !property_state->values || !val) { - DRM_DEBUG("Invalid argument(s)\n"); - } else { - mutex_lock(&info->property_lock); - *val = property_state->values[property_idx].value; - mutex_unlock(&info->property_lock); - rc = 0; - } - - return rc; -} - -void *msm_property_alloc_state(struct msm_property_info *info) -{ - void *state = NULL; - - if (!info) { - DRM_ERROR("invalid property info\n"); - return NULL; - } - - mutex_lock(&info->property_lock); - if (info->state_cache_size) - state = info->state_cache[--(info->state_cache_size)]; - mutex_unlock(&info->property_lock); - - if (!state && info->state_size) - state = kmalloc(info->state_size, GFP_KERNEL); - - if (!state) - DRM_ERROR("failed to allocate state\n"); - - return state; -} - -/** - * _msm_property_free_state - helper function for freeing local state objects - * @info: Pointer to property info container struct - * @st: Pointer to state object - */ -static void _msm_property_free_state(struct msm_property_info *info, void *st) -{ - if (!info || !st) - return; - - mutex_lock(&info->property_lock); - if (info->state_cache_size < MSM_PROP_STATE_CACHE_SIZE) - info->state_cache[(info->state_cache_size)++] = st; - else - kfree(st); - mutex_unlock(&info->property_lock); -} - -void msm_property_reset_state(struct msm_property_info *info, void *state, - struct msm_property_state *property_state, - struct msm_property_value *property_values) -{ - uint32_t i; - - if (!info) { - DRM_ERROR("invalid property info\n"); - return; - } - - if (state) - memset(state, 0, info->state_size); - - if (property_state) { - property_state->property_count = info->property_count; - property_state->values = property_values; - INIT_LIST_HEAD(&property_state->dirty_list); - } - - /* - * Assign default property values. This helper is mostly used - * to initialize newly created state objects. - */ - if (property_values) - for (i = 0; i < info->property_count; ++i) { - property_values[i].value = - info->property_data[i].default_value; - property_values[i].blob = NULL; - INIT_LIST_HEAD(&property_values[i].dirty_node); - } -} - -void msm_property_duplicate_state(struct msm_property_info *info, - void *old_state, void *state, - struct msm_property_state *property_state, - struct msm_property_value *property_values) -{ - uint32_t i; - - if (!info || !old_state || !state) { - DRM_ERROR("invalid argument(s)\n"); - return; - } - - memcpy(state, old_state, info->state_size); - - if (!property_state) - return; - - INIT_LIST_HEAD(&property_state->dirty_list); - property_state->values = property_values; - - if (property_state->values) - /* add ref count for blobs and initialize dirty nodes */ - for (i = 0; i < info->property_count; ++i) { - if (property_state->values[i].blob) - drm_property_blob_get( - property_state->values[i].blob); - INIT_LIST_HEAD(&property_state->values[i].dirty_node); - } -} - -void msm_property_destroy_state(struct msm_property_info *info, void *state, - struct msm_property_state *property_state) -{ - uint32_t i; - - if (!info || !state) { - DRM_ERROR("invalid argument(s)\n"); - return; - } - if (property_state && property_state->values) { - /* remove ref count for blobs */ - for (i = 0; i < info->property_count; ++i) - if (property_state->values[i].blob) { - drm_property_blob_put( - property_state->values[i].blob); - property_state->values[i].blob = NULL; - } - } - - _msm_property_free_state(info, state); -} - -void *msm_property_get_blob(struct msm_property_info *info, - struct msm_property_state *property_state, - size_t *byte_len, - uint32_t property_idx) -{ - struct drm_property_blob *blob; - size_t len = 0; - void *rc = 0; - - if (!info || !property_state || !property_state->values || - (property_idx >= info->blob_count)) { - DRM_ERROR("invalid argument(s)\n"); - } else { - blob = property_state->values[property_idx].blob; - if (blob) { - len = blob->length; - rc = &blob->data; - } - } - - if (byte_len) - *byte_len = len; - - return rc; -} - -int msm_property_set_blob(struct msm_property_info *info, - struct drm_property_blob **blob_reference, - void *blob_data, - size_t byte_len, - uint32_t property_idx) -{ - struct drm_property_blob *blob = NULL; - int rc = -EINVAL; - - if (!info || !blob_reference || (property_idx >= info->blob_count)) { - DRM_ERROR("invalid argument(s)\n"); - } else { - /* create blob */ - if (blob_data && byte_len) { - blob = drm_property_create_blob(info->dev, - byte_len, - blob_data); - if (IS_ERR_OR_NULL(blob)) { - rc = PTR_ERR(blob); - DRM_ERROR("failed to create blob, %d\n", rc); - goto exit; - } - } - - /* update drm object */ - rc = drm_object_property_set_value(info->base, - info->property_array[property_idx], - blob ? blob->base.id : 0); - if (rc) { - DRM_ERROR("failed to set blob to property\n"); - if (blob) - drm_property_blob_put(blob); - goto exit; - } - - /* update local reference */ - if (*blob_reference) - drm_property_blob_put(*blob_reference); - *blob_reference = blob; - } - -exit: - return rc; -} - -int msm_property_set_property(struct msm_property_info *info, - struct msm_property_state *property_state, - uint32_t property_idx, - uint64_t val) -{ - int rc = -EINVAL; - - if (!info || (property_idx >= info->property_count) || - property_idx < info->blob_count || - !property_state || !property_state->values) { - DRM_ERROR("invalid argument(s)\n"); - } else { - struct drm_property *drm_prop; - - mutex_lock(&info->property_lock); - - /* update cached value */ - property_state->values[property_idx].value = val; - - /* update the new default value for immutables */ - drm_prop = info->property_array[property_idx]; - if (drm_prop->flags & DRM_MODE_PROP_IMMUTABLE) - info->property_data[property_idx].default_value = val; - - mutex_unlock(&info->property_lock); - - /* update drm object */ - rc = drm_object_property_set_value(info->base, drm_prop, val); - if (rc) - DRM_ERROR("failed set property value, idx %d rc %d\n", - property_idx, rc); - - } - - return rc; -} - diff --git a/drivers/gpu/drm/msm/msm_prop.h b/drivers/gpu/drm/msm/msm_prop.h deleted file mode 100644 index 6785955..0000000 --- a/drivers/gpu/drm/msm/msm_prop.h +++ /dev/null @@ -1,438 +0,0 @@ -/* Copyright (c) 2016-2018, The Linux Foundation. All rights reserved. - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License version 2 and - * only version 2 as published by the Free Software Foundation. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - */ - -#ifndef _MSM_PROP_H_ -#define _MSM_PROP_H_ - -#include -#include "msm_drv.h" - -#define MSM_PROP_STATE_CACHE_SIZE 2 - -/** - * struct msm_property_data - opaque structure for tracking per - * drm-object per property stuff - * @default_value: Default property value for this drm object - * @force_dirty: Always dirty property on incoming sets, rather than checking - * for modified values - */ -struct msm_property_data { - uint64_t default_value; - bool force_dirty; -}; - -/** - * struct msm_property_value - opaque structure for tracking per - * drm-object per property stuff - * @value: Current property value for this drm object - * @blob: Pointer to associated blob data, if available - * @dirty_node: Linked list node to track if property is dirty or not - */ -struct msm_property_value { - uint64_t value; - struct drm_property_blob *blob; - struct list_head dirty_node; -}; - -/** - * struct msm_property_info: Structure for property/state helper functions - * @base: Pointer to base drm object (plane/crtc/etc.) - * @dev: Pointer to drm device object - * @property_array: Pointer to array for storing created property objects - * @property_data: Pointer to array for storing private property data - * @property_count: Total number of properties - * @blob_count: Total number of blob properties, should be <= count - * @install_request: Total number of property 'install' requests - * @install_count: Total number of successful 'install' requests - * @recent_idx: Index of property most recently accessed by set/get - * @is_active: Whether or not drm component properties are 'active' - * @state_cache: Cache of local states, to prevent alloc/free thrashing - * @state_size: Size of local state structures - * @state_cache_size: Number of state structures currently stored in state_cache - * @property_lock: Mutex to protect local variables - */ -struct msm_property_info { - struct drm_mode_object *base; - struct drm_device *dev; - - struct drm_property **property_array; - struct msm_property_data *property_data; - uint32_t property_count; - uint32_t blob_count; - uint32_t install_request; - uint32_t install_count; - - int32_t recent_idx; - - bool is_active; - - void *state_cache[MSM_PROP_STATE_CACHE_SIZE]; - uint32_t state_size; - int32_t state_cache_size; - struct mutex property_lock; -}; - -/** - * struct msm_property_state - Structure for local property state information - * @property_count: Total number of properties - * @values: Pointer to array of msm_property_value objects - * @dirty_list: List of all properties that have been 'atomic_set' but not - * yet cleared with 'msm_property_pop_dirty' - */ -struct msm_property_state { - uint32_t property_count; - struct msm_property_value *values; - struct list_head dirty_list; -}; - -/** - * msm_property_index_to_drm_property - get drm property struct from prop index - * @info: Pointer to property info container struct - * @property_idx: Property index - * Returns: drm_property pointer associated with property index - */ -static inline -struct drm_property *msm_property_index_to_drm_property( - struct msm_property_info *info, uint32_t property_idx) -{ - if (!info || property_idx >= info->property_count) - return NULL; - - return info->property_array[property_idx]; -} - -/** - * msm_property_get_default - query default value of a property - * @info: Pointer to property info container struct - * @property_idx: Property index - * Returns: Default value for specified property - */ -static inline -uint64_t msm_property_get_default(struct msm_property_info *info, - uint32_t property_idx) -{ - uint64_t rc = 0; - - if (!info) - return 0; - - mutex_lock(&info->property_lock); - if (property_idx < info->property_count) - rc = info->property_data[property_idx].default_value; - mutex_unlock(&info->property_lock); - - return rc; -} - -/** - * msm_property_set_is_active - set overall 'active' status for all properties - * @info: Pointer to property info container struct - * @is_active: New 'is active' status - */ -static inline -void msm_property_set_is_active(struct msm_property_info *info, bool is_active) -{ - if (info) { - mutex_lock(&info->property_lock); - info->is_active = is_active; - mutex_unlock(&info->property_lock); - } -} - -/** - * msm_property_get_is_active - query property 'is active' status - * @info: Pointer to property info container struct - * Returns: Current 'is active's status - */ -static inline -bool msm_property_get_is_active(struct msm_property_info *info) -{ - bool rc = false; - - if (info) { - mutex_lock(&info->property_lock); - rc = info->is_active; - mutex_unlock(&info->property_lock); - } - - return rc; -} - -/** - * msm_property_pop_dirty - determine next dirty property and clear - * its dirty flag - * @info: Pointer to property info container struct - * @property_state: Pointer to property state container struct - * Returns: Valid msm property index on success, - * -EAGAIN if no dirty properties are available - * Property indicies returned from this function are similar - * to those returned by the msm_property_index function. - */ -int msm_property_pop_dirty(struct msm_property_info *info, - struct msm_property_state *property_state); - -/** - * msm_property_init - initialize property info structure - * @info: Pointer to property info container struct - * @base: Pointer to base drm object (plane/crtc/etc.) - * @dev: Pointer to drm device object - * @property_array: Pointer to array for storing created property objects - * @property_data: Pointer to array for storing private property data - * @property_count: Total number of properties - * @blob_count: Total number of blob properties, should be <= count - * @state_size: Size of local state object - */ -void msm_property_init(struct msm_property_info *info, - struct drm_mode_object *base, - struct drm_device *dev, - struct drm_property **property_array, - struct msm_property_data *property_data, - uint32_t property_count, - uint32_t blob_count, - uint32_t state_size); - -/** - * msm_property_destroy - destroy helper info structure - * - * @info: Pointer to property info container struct - */ -void msm_property_destroy(struct msm_property_info *info); - -/** - * msm_property_install_range - install standard drm range property - * @info: Pointer to property info container struct - * @name: Property name - * @flags: Other property type flags, e.g. DRM_MODE_PROP_IMMUTABLE - * @min: Min property value - * @max: Max property value - * @init: Default Property value - * @property_idx: Property index - */ -void msm_property_install_range(struct msm_property_info *info, - const char *name, - int flags, - uint64_t min, - uint64_t max, - uint64_t init, - uint32_t property_idx); - -/** - * msm_property_install_volatile_range - install drm range property - * This function is similar to msm_property_install_range, but assumes - * that the property is meant for holding user pointers or descriptors - * that may reference volatile data without having an updated value. - * @info: Pointer to property info container struct - * @name: Property name - * @flags: Other property type flags, e.g. DRM_MODE_PROP_IMMUTABLE - * @min: Min property value - * @max: Max property value - * @init: Default Property value - * @property_idx: Property index - */ -void msm_property_install_volatile_range(struct msm_property_info *info, - const char *name, - int flags, - uint64_t min, - uint64_t max, - uint64_t init, - uint32_t property_idx); - -/** - * msm_property_install_rotation - install standard drm rotation property - * @info: Pointer to property info container struct - * @plane: Pointer to drm_plane which this property is installed on - * @rotation: The default value for this new property - * @supported_rotations: Bitmask of supported rotation values (see - * drm_plane_create_rotation_property for more details) - * @property_idx: Property index - */ -void msm_property_install_rotation(struct msm_property_info *info, - struct drm_plane *plane, - unsigned int rotation, - unsigned int supported_rotations, - uint32_t property_idx); - -/** - * msm_property_install_enum - install standard drm enum/bitmask property - * @info: Pointer to property info container struct - * @name: Property name - * @flags: Other property type flags, e.g. DRM_MODE_PROP_IMMUTABLE - * @is_bitmask: Set to non-zero to create a bitmask property, rather than an - * enumeration one - * @values: Array of allowable enumeration/bitmask values - * @num_values: Size of values array - * @property_idx: Property index - */ -void msm_property_install_enum(struct msm_property_info *info, - const char *name, - int flags, - int is_bitmask, - const struct drm_prop_enum_list *values, - int num_values, - uint32_t property_idx); - -/** - * msm_property_install_blob - install standard drm blob property - * @info: Pointer to property info container struct - * @name: Property name - * @flags: Extra flags for property creation - * @property_idx: Property index - */ -void msm_property_install_blob(struct msm_property_info *info, - const char *name, - int flags, - uint32_t property_idx); - -/** - * msm_property_install_get_status - query overal status of property additions - * @info: Pointer to property info container struct - * Returns: Zero if previous property install calls were all successful - */ -int msm_property_install_get_status(struct msm_property_info *info); - -/** - * msm_property_index - determine property index from drm_property ptr - * @info: Pointer to property info container struct - * @property: Incoming property pointer - * Returns: Valid property index, or -EINVAL on error - */ -int msm_property_index(struct msm_property_info *info, - struct drm_property *property); - -/** - * msm_property_set_dirty - forcibly flag a property as dirty - * @info: Pointer to property info container struct - * @property_state: Pointer to property state container struct - * @property_idx: Property index - * Returns: Zero on success - */ -int msm_property_set_dirty(struct msm_property_info *info, - struct msm_property_state *property_state, - int property_idx); - -/** - * msm_property_atomic_set - helper function for atomic property set callback - * @info: Pointer to property info container struct - * @property_state: Pointer to local state structure - * @property: Incoming property pointer - * @val: Incoming property value - * Returns: Zero on success - */ -int msm_property_atomic_set(struct msm_property_info *info, - struct msm_property_state *property_state, - struct drm_property *property, - uint64_t val); - -/** - * msm_property_atomic_get - helper function for atomic property get callback - * @info: Pointer to property info container struct - * @property_state: Pointer to local state structure - * @property: Incoming property pointer - * @val: Pointer to variable for receiving property value - * Returns: Zero on success - */ -int msm_property_atomic_get(struct msm_property_info *info, - struct msm_property_state *property_state, - struct drm_property *property, - uint64_t *val); - -/** - * msm_property_alloc_state - helper function for allocating local state objects - * @info: Pointer to property info container struct - */ -void *msm_property_alloc_state(struct msm_property_info *info); - -/** - * msm_property_reset_state - helper function for state reset callback - * @info: Pointer to property info container struct - * @state: Pointer to local state structure - * @property_state: Pointer to property state container struct - * @property_values: Pointer to property values cache array - */ -void msm_property_reset_state(struct msm_property_info *info, void *state, - struct msm_property_state *property_state, - struct msm_property_value *property_values); - -/** - * msm_property_duplicate_state - helper function for duplicate state cb - * @info: Pointer to property info container struct - * @old_state: Pointer to original state structure - * @state: Pointer to newly created state structure - * @property_state: Pointer to destination property state container struct - * @property_values: Pointer to property values cache array - */ -void msm_property_duplicate_state(struct msm_property_info *info, - void *old_state, - void *state, - struct msm_property_state *property_state, - struct msm_property_value *property_values); - -/** - * msm_property_destroy_state - helper function for destroy state cb - * @info: Pointer to property info container struct - * @state: Pointer to local state structure - * @property_state: Pointer to property state container struct - */ -void msm_property_destroy_state(struct msm_property_info *info, - void *state, - struct msm_property_state *property_state); - -/** - * msm_property_get_blob - obtain cached data pointer for drm blob property - * @info: Pointer to property info container struct - * @property_state: Pointer to property state container struct - * @byte_len: Optional pointer to variable for accepting blob size - * @property_idx: Property index - * Returns: Pointer to blob data - */ -void *msm_property_get_blob(struct msm_property_info *info, - struct msm_property_state *property_state, - size_t *byte_len, - uint32_t property_idx); - -/** - * msm_property_set_blob - update blob property on a drm object - * This function updates the blob property value of the given drm object. Its - * intended use is to update blob properties that have been created with the - * DRM_MODE_PROP_IMMUTABLE flag set. - * @info: Pointer to property info container struct - * @blob_reference: Reference to a pointer that holds the created data blob - * @blob_data: Pointer to blob data - * @byte_len: Length of blob data, in bytes - * @property_idx: Property index - * Returns: Zero on success - */ -int msm_property_set_blob(struct msm_property_info *info, - struct drm_property_blob **blob_reference, - void *blob_data, - size_t byte_len, - uint32_t property_idx); - -/** - * msm_property_set_property - update property on a drm object - * This function updates the property value of the given drm object. Its - * intended use is to update properties that have been created with the - * DRM_MODE_PROP_IMMUTABLE flag set. - * Note: This function cannot be called on a blob. - * @info: Pointer to property info container struct - * @property_state: Pointer to property state container struct - * @property_idx: Property index - * @val: value of the property to set - * Returns: Zero on success - */ -int msm_property_set_property(struct msm_property_info *info, - struct msm_property_state *property_state, - uint32_t property_idx, - uint64_t val); - -#endif /* _MSM_PROP_H_ */ -