From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (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 B5BCF49DBB6 for ; Fri, 11 Sep 2026 15:49:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789141750; cv=none; b=SgpLd5NHTFr2HEr171cWijl+ks9R5MFaPab/tfAl770DNBgg3MsW1hkks5J5mr3cFq8K4gO3kG16+tskaGPfqlVB4rwcdIOuChJhIsdpXUq5AifQoQinIK7eQGOb6VFqXeSY6DInIHL8VsyPUzo83ySdsyE7dOSMfAffG9w+xbs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789141750; c=relaxed/simple; bh=oOTqrBNMzSEm0t+rInGLPDLTHfvrXPo+CQaWGaCeo5c=; h=From:Subject:To:Cc:In-Reply-To:References:Content-Type:Date: Message-Id; b=Ey7PmW5zw08QkVky2QCKHu5nKblpdyJQK5accpPVyOIY5egkiAhTNkwAbi1USPAVpF3klIcQRMTrOIG4ZNMPrqVi7hlhdTUCA0Lnu0s7O4AVv2R4PKeA1hPANRtZGKNs2rfIkYv3gppgrpq2piDXdd1Tp/jJqKp0DWGRWMWWeOk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=Bzox/uuO; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Bzox/uuO" Received: by smtp.kernel.org (Postfix) with ESMTPSA id DC8FD1F00893; Fri, 11 Sep 2026 15:49:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1789141749; bh=baLZca9m/jVhaS+WTiyTOtwD2+JJChaOsn8UfHM6UuI=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=Bzox/uuO/qApBgZ74WOd2twhbzder3sxYgJfJUqMG4esgt1Av3hVVp2K20PaREAWH djpjcJIqNIWhWSRJ8HIFiv8SphKvr4p1WCfWrL2fXVnXHExhM6xRmx/h//8Y8kBnew E+7VZ6raclQzSamDUur4fBWDd/5ud5pe8B0t3YCohTroV2twvLAXCSyxwNXBJe3WXz mFne69DbX1UVExko3PVr1A0xJmuXuRkIjV1JZErmXka3NsMuTzNroWo7QSHAazSqjN YGSfErEyzf2ev382xqrubZEoxWiteIo1iKLT9uwOuJl5mdixPWaxuXqGR1saq/iN/G f8gsYTiNQM34g== From: sashiko-bot@kernel.org Subject: Re: [PATCH 12/17] phy: qcom: edp: Add Nord bias and clock buffer enable callback Reply-To: sashiko-reviews@lists.linux.dev To: "Nabige Aala" Cc: conor+dt@kernel.org, robh@kernel.org, olteanv@gmail.com, linux-phy@lists.infradead.org, dri-devel@lists.freedesktop.org, neil.armstrong@linaro.org, vkoul@kernel.org, devicetree@vger.kernel.org In-Reply-To: <20260911-nord-drv-dt-v1-12-e25c073cf1b6@oss.qualcomm.com> References: <20260911-nord-drv-dt-v1-0-e25c073cf1b6@oss.qualcomm.com> <20260911-nord-drv-dt-v1-12-e25c073cf1b6@oss.qualcomm.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Fri, 11 Sep 2026 15:49:08 +0000 Message-Id: <20260911154908.DC8FD1F00893@smtp.kernel.org> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Thank you for your contribution! Sashiko AI review found 1 potential issue(= s) to consider: - [Medium] Redundant function added due to an incorrect commit message claim -- commit 3bc884119c8e394e11daa5276d830b8881379d66 Author: Ritesh Kumar phy: qcom: edp: Add Nord bias and clock buffer enable callback This commit introduces a new callback for the Nord platform to enable the internal bias generator and clock buffer. It writes 0x1f to the BIAS_EN_CLKBUFLR_EN register. > diff --git a/drivers/phy/qualcomm/phy-qcom-edp.c b/drivers/phy/qualcomm/p= hy-qcom-edp.c > index 6cac4b55b66f6..ec4220cffe693 100644 > --- a/drivers/phy/qualcomm/phy-qcom-edp.c > +++ b/drivers/phy/qualcomm/phy-qcom-edp.c > @@ -1637,6 +1637,14 @@ static int qcom_edp_phy_com_resetsm_cntrl_nord(con= st struct qcom_edp *edp) > } > =20 > =20 > +static int qcom_edp_com_bias_en_clkbuflr_nord(const struct qcom_edp *edp) > +{ > + writel(0x1f, edp->pll + DP_QSERDES_V8_COM_BIAS_EN_CLKBUFLR_EN); > + > + return 0; > +} > + > + [Severity: Medium] The commit message states that setting BIAS_EN_CLKBUFLR_EN to 0x1f "differs= =20 from the value used on existing SoCs". Is this description accurate? The existing qcom_edp_com_bias_en_clkbuflr_v8() function already writes exa= ctly=20 0x1f to the same DP_QSERDES_V8_COM_BIAS_EN_CLKBUFLR_EN register. Does this new qcom_edp_com_bias_en_clkbuflr_nord() function introduce redun= dant=20 code? Could the existing v8 variant be reused to avoid duplication, or shou= ld=20 the commit message be updated to explain why a separate identical callback = is=20 required? --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20260911-nord-drv-d= t-v1-0-e25c073cf1b6@oss.qualcomm.com?part=3D12