[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <874b566f-8e19-4af8-bb6b-e0e5eff7b5f5@linux.dev>
Date: Mon, 7 Apr 2025 22:54:41 +0530
From: Aradhya Bhatia <aradhya.bhatia@...ux.dev>
To: Tomi Valkeinen <tomi.valkeinen@...asonboard.com>,
Jyri Sarha <jyri.sarha@....fi>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
Vinod Koul <vkoul@...nel.org>, Kishon Vijay Abraham I <kishon@...nel.org>,
Andrzej Hajda <andrzej.hajda@...el.com>,
Neil Armstrong <neil.armstrong@...aro.org>, Robert Foss <rfoss@...nel.org>,
Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
Jonas Karlman <jonas@...boo.se>, Jernej Skrabec <jernej.skrabec@...il.com>
Cc: dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
linux-phy@...ts.infradead.org, Francesco Dolcini <francesco@...cini.it>,
Devarsh Thakkar <devarsht@...com>
Subject: Re: [PATCH v2 05/18] phy: cdns-dphy: Remove leftover code
Hi Tomi,
Thank you for the patches!
On 02/04/25 19:00, Tomi Valkeinen wrote:
> The code in cdns-dphy has probably been part of a DSI driver in the
> past. Remove DSI defines and variables which are not used or do not
> actually do anything. Also rename cdns_dsi_get_dphy_pll_cfg() to
> cdns_get_dphy_pll_cfg(), i.e. drop the "dsi", as it's not relevant here.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@...asonboard.com>
> ---
> drivers/phy/cadence/cdns-dphy.c | 19 ++++---------------
> 1 file changed, 4 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/phy/cadence/cdns-dphy.c b/drivers/phy/cadence/cdns-dphy.c
> index f79ec4fab409..7f8b70ec10c5 100644
> --- a/drivers/phy/cadence/cdns-dphy.c
> +++ b/drivers/phy/cadence/cdns-dphy.c
> @@ -55,14 +55,6 @@
> #define DPHY_PSM_CFG_FROM_REG BIT(0)
> #define DPHY_PSM_CLK_DIV(x) ((x) << 1)
>
> -#define DSI_HBP_FRAME_OVERHEAD 12
> -#define DSI_HSA_FRAME_OVERHEAD 14
> -#define DSI_HFP_FRAME_OVERHEAD 6
> -#define DSI_HSS_VSS_VSE_FRAME_OVERHEAD 4
> -#define DSI_BLANKING_FRAME_OVERHEAD 6
> -#define DSI_NULL_FRAME_OVERHEAD 6
> -#define DSI_EOT_PKT_SIZE 4
> -
> #define DPHY_TX_J721E_WIZ_PLL_CTRL 0xF04
> #define DPHY_TX_J721E_WIZ_STATUS 0xF08
> #define DPHY_TX_J721E_WIZ_RST_CTRL 0xF0C
> @@ -117,10 +109,9 @@ static const unsigned int tx_bands[] = {
> 870, 950, 1000, 1200, 1400, 1600, 1800, 2000, 2200, 2500
> };
>
> -static int cdns_dsi_get_dphy_pll_cfg(struct cdns_dphy *dphy,
> - struct cdns_dphy_cfg *cfg,
> - struct phy_configure_opts_mipi_dphy *opts,
> - unsigned int *dsi_hfp_ext)
> +static int cdns_get_dphy_pll_cfg(struct cdns_dphy *dphy,
nit: cdns_dphy_get_pll_cfg seems more appropriate. We even have a
cdns_dphy_set_pll_cfg.
> + struct cdns_dphy_cfg *cfg,
> + struct phy_configure_opts_mipi_dphy *opts)
> {
> unsigned long pll_ref_hz = clk_get_rate(dphy->pll_ref_clk);
> u64 dlane_bps;
> @@ -289,15 +280,13 @@ static int cdns_dphy_config_from_opts(struct phy *phy,
> struct cdns_dphy_cfg *cfg)
> {
> struct cdns_dphy *dphy = phy_get_drvdata(phy);
> - unsigned int dsi_hfp_ext = 0;
> int ret;
>
> ret = phy_mipi_dphy_config_validate(opts);
> if (ret)
> return ret;
>
> - ret = cdns_dsi_get_dphy_pll_cfg(dphy, cfg,
> - opts, &dsi_hfp_ext);
> + ret = cdns_get_dphy_pll_cfg(dphy, cfg, opts);
> if (ret)
> return ret;
>
>
Apart from the small comment above,
Reviewed-by: Aradhya Bhatia <aradhya.bhatia@...ux.dev>
--
Regards
Aradhya
Powered by blists - more mailing lists