From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f45.google.com (mail-ej1-f45.google.com [209.85.218.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D5B7E41C302 for ; Mon, 6 Jul 2026 12:34:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783341284; cv=none; b=o/YePtsMnOqG7BbIe8QRojF+aURm8QbBbd9BFm3+m0tuBR/Oq44WhMTKUmStzQXikAkQvKhtgAZC1rmUvSYanj8nSWanR8m7XGsq1NVUduUav/zpjbSCO8Le6MCqdpZsnqRA8kbath4uqrHwnPs+T3/ltQ4sVBUfwqe3U9nsEX8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783341284; c=relaxed/simple; bh=DCZU1avBLdQ3mFCHVCs/pikkDIEkcElag6W6e5QxR+g=; h=Mime-Version:Content-Type:Date:Message-Id:Subject:From:To:Cc: References:In-Reply-To; b=HJWdflG5d49EAQmfS+EjaZ0Naspd6AnB2p8zp2+9Ol9ysOqgnQsRX0xEAklpHbXRof98DeJK3djV6sRCW4j4qGgm/OAfFP5AaDZOiu/Pvi7XkU9CVyujm3Qjwn+ZlYKeZ1O2Cp/aWJUAokkPZiQXvBz6GPQhj2RVPXA8DtF02B4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=T82y0DUd; arc=none smtp.client-ip=209.85.218.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="T82y0DUd" Received: by mail-ej1-f45.google.com with SMTP id a640c23a62f3a-c12a1a3cdb9so278834966b.0 for ; Mon, 06 Jul 2026 05:34:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783341281; x=1783946081; darn=vger.kernel.org; h=in-reply-to:references:cc:to:from:subject:message-id:date :content-type:content-transfer-encoding:mime-version:from:to:cc :subject:date:message-id:reply-to:content-type; bh=x9DuFiLJFQhXF47fb9omkU+r2KlD4K7BS2Tl2g+RiHE=; b=T82y0DUd5BU/de3ixAdDGi7hIRBwgnFy0Y0P6k22URkDSIV0pFtzFhKRirJIWXR35O jGSmSvtbuZp6q84whdCjFB3AjmVViuQXHkugrEB62PgtlmbQrQ8aGhiH6P5QJGb6fCaP f5IDk5Ft1EfJARURoxQ6Uo+OiDs2LnFhpni8F1JQ/M5Nzh4yD55hNw/wNlft4gnRPe9j cp6d+3IPhg8upsWd9eHoSbwx1h3NQi4EqZ3l7ZJ14c1F7f7DrzFTlXV5+C6QfUmSaYo2 ChwQgE/rkAT7kHFSavlwgyzl7KE4kDz7rBYgWK/32liOsxCQoXKOpVhP1B5qP0aVmVCJ 3iKQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783341281; x=1783946081; h=in-reply-to:references:cc:to:from:subject:message-id:date :content-type:content-transfer-encoding:mime-version:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=x9DuFiLJFQhXF47fb9omkU+r2KlD4K7BS2Tl2g+RiHE=; b=J81ypffli9SiBmlJB4wVv8DiEXi+1YxEY7dxYpTEpewj2QjX8ovnjFoF8+5UjZGY0d oySrzsTjoGusiDXGlkGkHiEo0xSVOSZCwC27e/KwKzt5LuHLr1V8WzcRkp4FwPwNq1qm M21c7xmSbT68MC4/j5SQv4VYyoaLK516mOP8inL57Mt7inq7ZiucYsF5NZUmTrMWykHZ EC5BSG4zmmFuOQdcA8LBtG58KF8NPKH/ehBcpGfHvBQxKLphGLYImUQEDn4aBsUdSLr9 iMwHMZqvK1hS70dYfiG6+hWDshah+og6UoUef0MkSwKI+xYxSSh9KZcoRtyfU3H9/Pq9 RwhA== X-Forwarded-Encrypted: i=1; AHgh+Rouk6uJoT0lXkdBwIUgaa9EnlJIm8FUX6otRl261roVdNNCd9qoMH/Jac4wlb+QRDBCcO+yK+78LbEqxVg=@vger.kernel.org X-Gm-Message-State: AOJu0Yy9wr0Qtf6hPQo9+Ox0z5LLDP8suCHFGJEFMYBFKUiRxU+b0Lzy MphSc8XPjAIJTApYWo6ciAj3BS3HWzvBwioH5YBYNqTzIPRjHfcKpyE= X-Gm-Gg: AfdE7clCuajdTyu+OyVCzYD4CP78lW1WdBWh68DGwSZAUvjchDaccB0jMzLZPdyYv1t ywx42vMFh+QRsPNX4mAvvbt1NOIKIJB46Z1S4J1BwuicusClX/Ey0nawnQfpckmVjG345Kp+pfQ 5NN/fD6uNiRIsaBpUqjr2Y6Afjzf7NuyS9q2qUSFE4v26n9//aKrT2coYRTXSdCE/YO8xZLQHMT 0yyDDv256beL/JkeshgSfmhw6P6lE0rjrN16gmqoxU3hpTLYjRWdI7m0+dcGeKr19v70vi57PWc S0iVjpvme24caf2gJehJK3i6NS6Utn4ZdGeLQtSqVpnmCN5wzyYEJ4jK2XDUCkm8vYFFb+evjf2 tXQ9KkIXrzCMs4vPGDYEzKJqeUNI616iwxA7OQadBFpuCv8SioZPc79mSPm6mbXoyfB018xbmpT nQcTgBEdaprS1fpFHPq2y6jnMlnXnUYBnozBKiXkRGCcOzQ0WobUaJzy1mrcI= X-Received: by 2002:a17:907:a709:b0:c12:caad:6efa with SMTP id a640c23a62f3a-c15a676bf23mr21887566b.4.1783341281063; Mon, 06 Jul 2026 05:34:41 -0700 (PDT) Received: from localhost ([186.158.238.108]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-69a19d7a375sm3926102a12.15.2026.07.06.05.34.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 06 Jul 2026 05:34:40 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Mon, 06 Jul 2026 09:34:34 -0300 Message-Id: Subject: Re: [PATCH] drm/panel: novatek-nt37801: Use mipi_dsi_*_multi() functions From: =?utf-8?q?Nicol=C3=A1s_Antinori?= To: "Doug Anderson" Cc: "Krzysztof Kozlowski" , "Brigham Campbell" , "David Airlie" , "Jessica Zhang" , "Maarten Lankhorst" , "Maxime Ripard" , "Neil Armstrong" , "Shuah Khan" , "Simona Vetter" , "Thomas Zimmermann" , , , X-Mailer: aerc 0.20.0 References: <20260701125624.199786-1-nico.antinori.7@gmail.com> In-Reply-To: On Sat Jul 4, 2026 at 12:59 PM -03, Doug Anderson wrote: > Hi, > > On Wed, Jul 1, 2026 at 5:56=E2=80=AFAM Nicol=C3=A1s Antinori > wrote: >> >> Replace mipi_dsi_* functions with their non-deprecated mipi_dsi_*_multi >> counterparts. This change reduces error-checking boilerplate and improve= s >> readability. >> >> Signed-off-by: Nicol=C3=A1s Antinori >> --- >> Note: I Compile-tested this patch since I don't own the panel to test it= . >> >> drivers/gpu/drm/panel/panel-novatek-nt37801.c | 16 ++++++---------- >> 1 file changed, 6 insertions(+), 10 deletions(-) >> >> diff --git a/drivers/gpu/drm/panel/panel-novatek-nt37801.c b/drivers/gpu= /drm/panel/panel-novatek-nt37801.c >> index d6a37d7e0cc6..daa356bbc182 100644 >> --- a/drivers/gpu/drm/panel/panel-novatek-nt37801.c >> +++ b/drivers/gpu/drm/panel/panel-novatek-nt37801.c >> @@ -128,7 +128,8 @@ static int novatek_nt37801_off(struct novatek_nt3780= 1 *ctx) >> static int novatek_nt37801_prepare(struct drm_panel *panel) >> { >> struct novatek_nt37801 *ctx =3D to_novatek_nt37801(panel); >> - struct device *dev =3D &ctx->dsi->dev; >> + struct mipi_dsi_device *dsi =3D ctx->dsi; >> + struct mipi_dsi_multi_context dsi_ctx =3D { .dsi =3D dsi }; >> struct drm_dsc_picture_parameter_set pps; >> int ret; >> >> @@ -145,15 +146,10 @@ static int novatek_nt37801_prepare(struct drm_pane= l *panel) >> >> drm_dsc_pps_payload_pack(&pps, &ctx->dsc); >> >> - ret =3D mipi_dsi_picture_parameter_set(ctx->dsi, &pps); >> - if (ret < 0) { >> - dev_err(panel->dev, "failed to transmit PPS: %d\n", ret)= ; >> - goto err; >> - } >> - >> - ret =3D mipi_dsi_compression_mode(ctx->dsi, true); >> - if (ret < 0) { >> - dev_err(dev, "failed to enable compression mode: %d\n", = ret); >> + mipi_dsi_picture_parameter_set_multi(&dsi_ctx, &pps); >> + mipi_dsi_compression_mode_multi(&dsi_ctx, true); >> + if (dsi_ctx.accum_err) { >> + ret =3D dsi_ctx.accum_err; >> goto err; >> } > > Sure, this looks pretty good. > > Reviewed-by: Douglas Anderson > > For bonus points, I think you could change novatek_nt37801_on() to > return "void" and take a "struct mipi_dsi_multi_context". Since > drm_dsc_pps_payload_pack() is harmless, the function can now just be: > > novatek_nt37801_on(&dsi_ctx); > drm_dsc_pps_payload_pack(&pps, &ctx->dsc); > mipi_dsi_picture_parameter_set_multi(&dsi_ctx, &pps); > mipi_dsi_compression_mode_multi(&dsi_ctx, true); > if (dsi_ctx.accum_err) { > ... > } Will send a v2 with those changes. Thank you for the suggestion!