Re: [PATCH v3 10/10] drm/bridge: cdns: Convert to phy framework

From: kbuild test robot <lkp@intel.com>
To: Maxime Ripard <maxime.ripard@bootlin.com>
Cc: kbuild-all@01.org, Kishon Vijay Abraham I <kishon@ti.com>,
	Boris Brezillon <boris.brezillon@bootlin.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	linux-media@vger.kernel.org,
	Archit Taneja <architt@codeaurora.org>,
	Andrzej Hajda <a.hajda@samsung.com>, Chen-Yu Tsai <wens@csie.org>,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
	linux-arm-kernel@lists.infradead.org,
	Krzysztof Witos <kwitos@cadence.com>,
	Rafal Ciepiela <rafalc@cadence.com>,
	Maxime Ripard <maxime.ripard@bootlin.com>
Subject: Re: [PATCH v3 10/10] drm/bridge: cdns: Convert to phy framework
Date: Sun, 9 Dec 2018 12:37:14 +0800	[thread overview]
Message-ID: <201812091226.0pgdtDzO%fengguang.wu@intel.com> (raw)
In-Reply-To: <64bda0631842bf10ca140cd562b71feea2f98ff2.1544190837.git-series.maxime.ripard@bootlin.com>

[-- Attachment #1: Type: text/plain, Size: 2711 bytes --]

Hi Maxime,

I love your patch! Yet something to improve:

[auto build test ERROR on phy/next]
[cannot apply to v4.20-rc5]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Maxime-Ripard/phy-Add-MIPI-D-PHY-mode/20181208-034527
base:   https://git.kernel.org/pub/scm/linux/kernel/git/kishon/linux-phy.git next
config: i386-randconfig-s0-12051035 (attached as .config)
compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   drivers/gpu/drm/bridge/cdns-dsi.o: In function `cdns_dsi_check_conf':
>> drivers/gpu/drm/bridge/cdns-dsi.c:612: undefined reference to `phy_mipi_dphy_get_default_config'

vim +612 drivers/gpu/drm/bridge/cdns-dsi.c

   596	
   597	static int cdns_dsi_check_conf(struct cdns_dsi *dsi,
   598				       const struct drm_display_mode *mode,
   599				       struct cdns_dsi_cfg *dsi_cfg,
   600				       bool mode_valid_check)
   601	{
   602		struct cdns_dsi_output *output = &dsi->output;
   603		struct phy_configure_opts_mipi_dphy *phy_cfg = &output->phy_opts.mipi_dphy;
   604		unsigned long dsi_hss_hsa_hse_hbp;
   605		unsigned int nlanes = output->dev->lanes;
   606		int ret;
   607	
   608		ret = cdns_dsi_mode2cfg(dsi, mode, dsi_cfg, mode_valid_check);
   609		if (ret)
   610			return ret;
   611	
 > 612		phy_mipi_dphy_get_default_config(mode->crtc_clock * 1000,
   613						 mipi_dsi_pixel_format_to_bpp(output->dev->format),
   614						 nlanes, phy_cfg);
   615	
   616		ret = cdns_dsi_adjust_phy_config(dsi, dsi_cfg, phy_cfg, mode, mode_valid_check);
   617		if (ret)
   618			return ret;
   619	
   620		ret = phy_validate(dsi->dphy, PHY_MODE_MIPI_DPHY, 0, &output->phy_opts);
   621		if (ret)
   622			return ret;
   623	
   624		dsi_hss_hsa_hse_hbp = dsi_cfg->hbp + DSI_HBP_FRAME_OVERHEAD;
   625		if (output->dev->mode_flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE)
   626			dsi_hss_hsa_hse_hbp += dsi_cfg->hsa + DSI_HSA_FRAME_OVERHEAD;
   627	
   628		/*
   629		 * Make sure DPI(HFP) > DSI(HSS+HSA+HSE+HBP) to guarantee that the FIFO
   630		 * is empty before we start a receiving a new line on the DPI
   631		 * interface.
   632		 */
   633		if ((u64)phy_cfg->hs_clk_rate * mode_to_dpi_hfp(mode) * nlanes <
   634		    (u64)dsi_hss_hsa_hse_hbp *
   635		    (mode_valid_check ? mode->clock : mode->crtc_clock) * 1000)
   636			return -EINVAL;
   637	
   638		return 0;
   639	}
   640	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 30580 bytes --]