From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E9004273D66 for ; Wed, 17 Dec 2025 02:46:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1765939570; cv=none; b=Bi7ZSUTRdhbLwIuN++XCnApwckLBA1eAjfLS7T7/iurQs6X6oyKEFuzEHNBHsMrmIC54x8xOxnvwBMawegH1KtNWLZjlM5NmEulodbgq7eByI7BNK2ZZVs2syM5pg2ecWtlFNp2w07k73o1Kk/j7SBhNXqIxFn/EeJ8TYwYR6hg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1765939570; c=relaxed/simple; bh=+taVl9d0DmNYlbBhHvM5GjECJ3vUFg7jyaoe51/b/yE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=BZ8+DC0I5PTNv7rp6DP0ASCVOJR1G7W+ZDCbwrYMC7l2nfldBAdtc7NthxeQMUK04MK3ahofK+jOEdkqWC3/Qns+/qRknppMU0nUQh6sSoY3iCkqSrfTBJoGRBo9pNmDxHxle2Y7Nik4VldzeUpwyonYQfa/S1lj7PYoan9XCE8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=kkP9Y3yC; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=hpSA3xKh; arc=none smtp.client-ip=205.220.168.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="kkP9Y3yC"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="hpSA3xKh" Received: from pps.filterd (m0279863.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 5BGMQmkt723777 for ; Wed, 17 Dec 2025 02:46:04 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to; s=qcppdkim1; bh=JCJBPzyRlhs0ukky8nnWNxDG T3jIQ2bJUvRtWHrXAZA=; b=kkP9Y3yCoFi7+UDQPn6FBzxoy7INU9c/w6Tsuzjf OAcaisEwIdxxWr+wUSL07+/BCTYYlYNb3uAiFZ2TCjZlNVNPOEDJ5UOs/Y24ulKN Jj1LxAPNDoF82HY4H8VcQTl1Uz6r5pGeFModVu7iYYEBd50xbLBWhd7u25A/asN4 5EHKl/5ZDqH9LFjo1diyEn/awSDui1/VDScTcgma+oTs1T9Jq+fvs+1jOlLbTUtz QahlZurh9Tj0K3s/GyxrSZuK8ezTd3qYtonCUyNQqx0YPbrSUckft+Oc5VGZ+vtl K3bJw/k3vz1FXHIZP16CAK4soXqLSLVUopwUlSGvgPTLww== Received: from mail-qv1-f71.google.com (mail-qv1-f71.google.com [209.85.219.71]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4b3fh18nj0-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Wed, 17 Dec 2025 02:46:04 +0000 (GMT) Received: by mail-qv1-f71.google.com with SMTP id 6a1803df08f44-88a360b8086so73548036d6.3 for ; Tue, 16 Dec 2025 18:46:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1765939563; x=1766544363; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=JCJBPzyRlhs0ukky8nnWNxDGT3jIQ2bJUvRtWHrXAZA=; b=hpSA3xKhgTZyB0UiO1vKRFKe5t6stUBu4HYzkfAE7KoLudtMB19PEw4wyjWGbzJif3 Wgko/Uhy2nMomIkjKbb3WVlQ1W1Diz+8YYQVSS6+wkDCYnOpQF6DFhnOc5ctbmhU/lhA N4OhRtPqgRTKWtcf2gTEUmyqpXujip4tvxkt2LPQPtt2OHxA1R0Bklv8i+YiKMtBedy9 YmqMZKk6neJZByr7wnEZOoAfAIaI7puASsD75v4eYsjZXlT2TNslySpwVgkswlzeWdqf RxmwId/f2OFt8VkmiYDIzCZP13JtfI0Q039WO58HAF2ZXY9BmK23JmHTWbT6E47hdurN YgNw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1765939563; x=1766544363; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=JCJBPzyRlhs0ukky8nnWNxDGT3jIQ2bJUvRtWHrXAZA=; b=k8NhRbBqK/bgjBlmwD3xRLmE00BKVGFvo5LsUyzQCEm7Zd2Ob+aQNdciX/iiVUe69o gkOJ+IotY1OEB1fMVGfHCmutiT2elU6TFJ97n15+ycpuby5D7bRmjbDSKeGMqpYIsrji OTeT6QPrrJzGzvSptCmEOUKCalnccyDt8p8udCwokM5WpNwJLyu3zJiZ/IGKrKI3ShLc alq8sAmMMDyWWVw3ORMVt6AGL3oCaWIAY09bnFwQ8ESZW/5Fv7yDOfzgCaCE8P1T7BhS 2qr2/N+EolD0NI75fssj+Mejr9BdsR1/U8yyfBxyaAYne4vf/9NU4fjj1PhCvhfc1pXb 9DGA== X-Forwarded-Encrypted: i=1; AJvYcCUdjflOYg5Wty0OJW7jAFSrdBpfNIAGE1gYAJ3/cci8vhu6SMOqoWyc/5QpEFMqiixJsiOgEdZUmKFBxGk=@vger.kernel.org X-Gm-Message-State: AOJu0YzDobEvU2Miilzzi3jKGCnBV12BzcSKXBLuH1UQIqZOWWGQU2mU i2pg/gSNHyXgtfk46Cqm13Ji3ewWC0xH4qQ4dJ9NM6tDTbfSwTnrHZ7OUekav7PIzi+2AXz81fG dVxqLwhve2Y6Cer7nHUqxAQKqBNSRCu6zScW+ZBkavgkQVh0zc70B0sIfowvWvnpuUSs= X-Gm-Gg: AY/fxX4quCNJ/qwdCmkWUxLQb5fLvKvkzfHJHvIzVGQ/pdJpFSgJ3sryb3XfPxHjOVU QxLCW+I0mR8Aat+2FEnm3+zZkMxU1lD8P6hjsmx9+t/v8NmHznw+xX9mrcjvVR23XQZT8d9tork SrfF/BsH+Ksmn/wt4Fuh62ZJ7zgmKQ3uE1V6xU3v4fc4D4oIXun+a3y2KDIHulkH/CS83xQKXb7 wykd2lMNBloTWMH38Q+91LCxx9dSykWWHtRMjsSfJagIcy+IItimnIsHqsWcLEXq95yBDhX3XOC yT+bmxz/J8IIOHDLYXIJ3+ls52U4W8dMcCZFpphEc2ccxTIApz18wv+qlLgZxYkNjNyP7expBYY IaGtUf5+KV/SugL0FPVkiA8BaXXPSUlLPz7SNFlprU0mKuXO7G//246uSdRfg8qbFok8qb1Q= X-Received: by 2002:a05:6214:5b87:b0:888:82c3:dfc0 with SMTP id 6a1803df08f44-88882c3e67cmr227694806d6.55.1765939563340; Tue, 16 Dec 2025 18:46:03 -0800 (PST) X-Google-Smtp-Source: AGHT+IGx1RaDoBtJw9fMmc0Xmr4th/lJOfvqaul2PsNrWDcut+ChvzvkgYHcb05lGbK5/M73moqytA== X-Received: by 2002:a05:6214:5b87:b0:888:82c3:dfc0 with SMTP id 6a1803df08f44-88882c3e67cmr227694546d6.55.1765939562816; Tue, 16 Dec 2025 18:46:02 -0800 (PST) Received: from yuanjiey.ap.qualcomm.com (Global_NAT1_IAD_FW.qualcomm.com. [129.46.232.65]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-8899ea36210sm85899146d6.27.2025.12.16.18.45.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 16 Dec 2025 18:46:01 -0800 (PST) Date: Wed, 17 Dec 2025 10:45:50 +0800 From: yuanjiey To: Dmitry Baryshkov Cc: robin.clark@oss.qualcomm.com, lumag@kernel.org, abhinav.kumar@linux.dev, jesszhan0024@gmail.com, sean@poorly.run, marijn.suijten@somainline.org, airlied@gmail.com, simona@ffwll.ch, maarten.lankhorst@linux.intel.com, mripard@kernel.org, tzimmermann@suse.de, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, neil.armstrong@linaro.org, yongxing.mou@oss.qualcomm.com, konrad.dybcio@oss.qualcomm.com, linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, tingwei.zhang@oss.qualcomm.com, aiqun.yu@oss.qualcomm.com Subject: Re: [PATCH v3 10/11] drm/msm/dpu: Refactor SSPP to compatible DPU 13.0.0 Message-ID: References: <20251215083854.577-1-yuanjie.yang@oss.qualcomm.com> <20251215083854.577-11-yuanjie.yang@oss.qualcomm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Proofpoint-GUID: X8ZnT9-tAbQdJk7wtucVFS_xDCudJnv7 X-Authority-Analysis: v=2.4 cv=CYoFJbrl c=1 sm=1 tr=0 ts=6942196c cx=c_pps a=UgVkIMxJMSkC9lv97toC5g==:117 a=C3Dk8TwHQYyIj7nOf9RCJw==:17 a=kj9zAlcOel0A:10 a=wP3pNCr1ah4A:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=EUspDBNiAAAA:8 a=tNEIORBzTMdH9SYpPiUA:9 a=CjuIK1q_8ugA:10 a=1HOtulTD9v-eNWfpl4qZ:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMjE3MDAyMSBTYWx0ZWRfX7LKHKTnlF6EW IwEf3LsDt/Psij1lZSFZHlXstPakxaBBT17tIapumzxSP2flP+MxD3hSDNOCpjF7gXzQOwi/Opc NdSZcA16hGZ0jvwhR5EgyuoRE2GOgYUlc/9ccU4B2F0GTYc/bUtYPwp2hCmQtfX3tolGJlEYJYG rX4C9ZIUsMNSLmf1siLP43QrIOjvz6kEOEa/QfpKkGkz8e0OJfMEWfCejZVU9sRVx2OwTsmpEDn s5cvz9dW4udLAXSJxinFNa9m4fgYHbEC3cJcdcrMx54vyyDLbOaBgxKrOALiNiUDuxr09IlM/n2 wqPITGLL2/t0pseDk17RtfUFtPy+yZfSbbpvpCRkMcE2y82zIkezWHn5UTThC6Ff6JUUVSGEQ8q hatuiodak0zYpmm1FRRZ29XCfSDavg== X-Proofpoint-ORIG-GUID: X8ZnT9-tAbQdJk7wtucVFS_xDCudJnv7 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.100.49 definitions=2025-12-16_03,2025-12-16_05,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 adultscore=0 malwarescore=0 phishscore=0 lowpriorityscore=0 spamscore=0 priorityscore=1501 bulkscore=0 clxscore=1015 impostorscore=0 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2510240001 definitions=main-2512170021 On Tue, Dec 16, 2025 at 04:22:32PM +0200, Dmitry Baryshkov wrote: > On Tue, Dec 16, 2025 at 02:56:31PM +0800, yuanjiey wrote: > > On Mon, Dec 15, 2025 at 10:08:22PM +0200, Dmitry Baryshkov wrote: > > > On Mon, Dec 15, 2025 at 04:38:53PM +0800, yuanjie yang wrote: > > > > From: Yuanjie Yang > > > > > > > > DPU version 13.0.0 introduces structural changes including > > > > register additions, removals, and relocations. > > > > > > > > Refactor SSPP-related code to be compatible with DPU 13.0.0 > > > > modifications. > > > > > > > > Co-developed-by: Yongxing Mou > > > > Signed-off-by: Yongxing Mou > > > > Signed-off-by: Yuanjie Yang > > > > --- > > > > > > We've fixed the order of the interrupts patch. Now you are adding SSPP > > > customization for 13.x _after_ adding the first 13.x support. Is that > > > supposed to work? > > > > Yes, will reorganize order. > > And after comparing with v2, I'm really surprised. It was better before > and then you changed the order of the patches. Why? You were asked to > split it, but not to move it to the end. I make the mistake. Sure, I will keep the v2 patch order in next patch. > > > > > > > > .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 15 +- > > > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c | 155 ++++++++++-------- > > > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h | 52 ++++++ > > > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.c | 18 ++ > > > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_util.h | 3 + > > > > drivers/gpu/drm/msm/disp/dpu1/dpu_hw_wb.c | 17 +- > > > > 6 files changed, 191 insertions(+), 69 deletions(-) > > > > > > > > diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h > > > > switch (ctx->ubwc->ubwc_enc_version) { > > > > case UBWC_1_0: > > > > fast_clear = fmt->alpha_enable ? BIT(31) : 0; > > > > - DPU_REG_WRITE(c, SSPP_UBWC_STATIC_CTRL, > > > > - fast_clear | (ctx->ubwc->ubwc_swizzle & 0x1) | > > > > - BIT(8) | > > > > - (ctx->ubwc->highest_bank_bit << 4)); > > > > + DPU_REG_WRITE(c, ubwc_ctrl_off, > > > > + fast_clear | (ctx->ubwc->ubwc_swizzle & 0x1) | > > > > + BIT(8) | > > > > + (ctx->ubwc->highest_bank_bit << 4)); > > > > > > I have asked to drop unrelated changes. You didn't. Why? You are > > > changing whitespaces for no reason. It's just a noise which hides the > > > actual change here. > > > > here ubwc reg layout change in DPU 13. > > > > ubwc_ctrl_off > > veriosn < 13 > > reg: SSPP_UBWC_STATIC_CTRL > > verison >= 13 > > reg: SSPP_REC_UBWC_STATIC_CTRL > > > > So I do some fix. > > What does it have to do with the whitespaces? Fix _one_ line. get it, will drop unrelated whitespaces. > > > > > > break; > > > > case UBWC_2_0: > > > > fast_clear = fmt->alpha_enable ? BIT(31) : 0; > > > > - DPU_REG_WRITE(c, SSPP_UBWC_STATIC_CTRL, > > > > - fast_clear | (ctx->ubwc->ubwc_swizzle) | > > > > - (ctx->ubwc->highest_bank_bit << 4)); > > > > + DPU_REG_WRITE(c, ubwc_ctrl_off, > > > > + fast_clear | (ctx->ubwc->ubwc_swizzle) | > > > > + (ctx->ubwc->highest_bank_bit << 4)); > > > > break; > > > > case UBWC_3_0: > > > > - DPU_REG_WRITE(c, SSPP_UBWC_STATIC_CTRL, > > > > - BIT(30) | (ctx->ubwc->ubwc_swizzle) | > > > > - (ctx->ubwc->highest_bank_bit << 4)); > > > > + DPU_REG_WRITE(c, ubwc_ctrl_off, > > > > + BIT(30) | (ctx->ubwc->ubwc_swizzle) | > > > > + (ctx->ubwc->highest_bank_bit << 4)); > > > > break; > > > > case UBWC_4_0: > > > > - DPU_REG_WRITE(c, SSPP_UBWC_STATIC_CTRL, > > > > - MSM_FORMAT_IS_YUV(fmt) ? 0 : BIT(30)); > > > > + DPU_REG_WRITE(c, ubwc_ctrl_off, > > > > + MSM_FORMAT_IS_YUV(fmt) ? 0 : BIT(30)); > > > > break; > > > > } > > > > } > > > > @@ -313,19 +337,18 @@ static void dpu_hw_sspp_setup_format(struct dpu_sw_pipe *pipe, > > > > > > > > /* update scaler opmode, if appropriate */ > > > > if (test_bit(DPU_SSPP_CSC, &ctx->cap->features)) > > > > - _sspp_setup_opmode(ctx, VIG_OP_CSC_EN | VIG_OP_CSC_SRC_DATAFMT, > > > > - MSM_FORMAT_IS_YUV(fmt)); > > > > + dpu_hw_sspp_setup_opmode(ctx, VIG_OP_CSC_EN | VIG_OP_CSC_SRC_DATAFMT, > > > > + MSM_FORMAT_IS_YUV(fmt)); > > > > else if (test_bit(DPU_SSPP_CSC_10BIT, &ctx->cap->features)) > > > > - _sspp_setup_csc10_opmode(ctx, > > > > - VIG_CSC_10_EN | VIG_CSC_10_SRC_DATAFMT, > > > > - MSM_FORMAT_IS_YUV(fmt)); > > > > + dpu_hw_sspp_setup_csc10_opmode(ctx, > > > > + VIG_CSC_10_EN | VIG_CSC_10_SRC_DATAFMT, > > > > + MSM_FORMAT_IS_YUV(fmt)); > > > > > > Again, useless whitespace changes. > > checkpatch.pl says here is alignment issuse, so I do this fix. > > The issue was present before your patch. If you want to fix it, fix it > in the separate patch or ignore it. get it, will drop unrelated whitespaces. > > > > > > > > > > DPU_REG_WRITE(c, format_off, src_format); > > > > DPU_REG_WRITE(c, unpack_pat_off, unpack); > > > > DPU_REG_WRITE(c, op_mode_off, opmode); > > > > - > > > > > > Why? > > > > yes, will drop "-" diff. > > > > > > /* clear previous UBWC error */ > > > > - DPU_REG_WRITE(c, SSPP_UBWC_ERROR_STATUS, BIT(31)); > > > > + DPU_REG_WRITE(c, ubwc_err_off, BIT(31)); > > > > } > > > > > > > > static void dpu_hw_sspp_setup_pe_config(struct dpu_hw_sspp *ctx, > > > > @@ -385,9 +408,9 @@ static void dpu_hw_sspp_setup_pe_config(struct dpu_hw_sspp *ctx, > > > > tot_req_pixels[3]); > > > > } > > > > > > > > -static void _dpu_hw_sspp_setup_scaler3(struct dpu_hw_sspp *ctx, > > > > - struct dpu_hw_scaler3_cfg *scaler3_cfg, > > > > - const struct msm_format *format) > > > > +void dpu_hw_sspp_setup_scaler3(struct dpu_hw_sspp *ctx, > > > > + struct dpu_hw_scaler3_cfg *scaler3_cfg, > > > > + const struct msm_format *format) > > > > > > And here... > > checkpatch.pl says here is alignment issuse, so I do this fix. > > And I'm asking you to don't do it. Don't clutter the patch with > unrelated changes (and whitespace / alignment changes are generally > unrelated). > > -- > With best wishes > Dmitry Thanks, Yuanjie