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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6C146CCA47C for ; Thu, 2 Jun 2022 20:25:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234929AbiFBUZF (ORCPT ); Thu, 2 Jun 2022 16:25:05 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56656 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239186AbiFBUY6 (ORCPT ); Thu, 2 Jun 2022 16:24:58 -0400 Received: from alexa-out-sd-02.qualcomm.com (alexa-out-sd-02.qualcomm.com [199.106.114.39]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 47D76B874 for ; Thu, 2 Jun 2022 13:24:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; i=@quicinc.com; q=dns/txt; s=qcdkim; t=1654201494; x=1685737494; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=5iVKzHg2aC6sarFPY5Jcp0A0AL32g8HzGVyqht6Aoos=; b=gRf7scRhnXvsAogdHZFuGxfgQ/zHp4pCDDBgc49pdo0lic7C7plAiuz2 3c1UHMiOBuwpq1CgpWc6SNwFwynJ+LwhV0KqdV7Wz6TjsvCnxnGQzqXyl 0l8H6hAJbO4p0Q/FjJWzyP5vSTRefDU9JdEKHQ2radZIgpMVf4fbOmVAX 8=; Received: from unknown (HELO ironmsg03-sd.qualcomm.com) ([10.53.140.143]) by alexa-out-sd-02.qualcomm.com with ESMTP; 02 Jun 2022 13:24:53 -0700 X-QCInternal: smtphost Received: from nasanex01c.na.qualcomm.com ([10.47.97.222]) by ironmsg03-sd.qualcomm.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Jun 2022 13:24:53 -0700 Received: from nalasex01a.na.qualcomm.com (10.47.209.196) by nasanex01c.na.qualcomm.com (10.47.97.222) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.22; Thu, 2 Jun 2022 13:24:53 -0700 Received: from [10.38.242.41] (10.80.80.8) by nalasex01a.na.qualcomm.com (10.47.209.196) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.22; Thu, 2 Jun 2022 13:24:51 -0700 Message-ID: Date: Thu, 2 Jun 2022 13:24:48 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.6.2 Subject: Re: [PATCH v3 0/7] drm/msm: clean up the hw catalog init Content-Language: en-US To: Dmitry Baryshkov , Rob Clark , Sean Paul CC: Stephen Boyd , David Airlie , Daniel Vetter , Bjorn Andersson , , , References: <20220602202207.1754351-1-dmitry.baryshkov@linaro.org> From: Abhinav Kumar In-Reply-To: <20220602202207.1754351-1-dmitry.baryshkov@linaro.org> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: nasanex01a.na.qualcomm.com (10.52.223.231) To nalasex01a.na.qualcomm.com (10.47.209.196) Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On 6/2/2022 1:22 PM, Dmitry Baryshkov wrote: > Replace superfluous cfg_init functions, which just assign a static > config to the struct dpu_mdss_cfg, with static instances of struct > dpu_mdss_cfg. > > Changes since v2: > - Add DPU_MIXER_COMBINED_ALPHA to sc7180's mixer features mask (noted > by Abhinav). > > Changes since v1: > - Turn catalog->perf and catalog->dma_cfg to be pointers, otherwise > clang complains that they are not constant. > I see you have dropped the R-bs from all the changes in V2. Can you please add them back? > Dmitry Baryshkov (7): > drm/msm/dpu: use feature bit for LM combined alpha check > drm/msm/dpu: move VBIF_XINL_QOS_LVL_REMAP size to hw_catalog > drm/msm/dpu: remove hwversion field from data structures > drm/msm/dpu: change catalog->perf to be a const pointer > drm/msm/dpu: change catalog->dma_cfg to be a const pointer > drm/msm/dpu: constify struct dpu_mdss_cfg > drm/msm/dpu: make dpu hardware catalog static const > > drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 24 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.h | 4 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 4 +- > .../drm/msm/disp/dpu1/dpu_encoder_phys_wb.c | 12 +- > .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 502 ++++++++---------- > .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 20 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c | 1 - > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_dsc.c | 5 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_dsc.h | 2 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_dspp.c | 1 - > .../gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.c | 5 +- > .../gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.h | 2 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c | 1 - > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c | 7 +- > .../gpu/drm/msm/disp/dpu1/dpu_hw_merge3d.c | 1 - > .../gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.c | 1 - > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c | 5 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h | 4 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c | 1 - > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h | 2 - > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_vbif.c | 5 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_wb.c | 1 - > drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 6 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 2 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 20 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 2 +- > drivers/gpu/drm/msm/disp/dpu1/dpu_rm.h | 2 +- > 27 files changed, 283 insertions(+), 359 deletions(-) >