[kbingham-rcar:kbingham/drm-misc/next/sn65dsi86/hpd 10/21] drivers/gpu/drm/bridge/parade-ps8640.c:524:9: error: implicit declaration of function 'drm_bridge_chain_pre_enable'; did you mean 'drm_bridge_hpd_enable'?
From: kernel test robot
Date: Wed Jun 22 2022 - 21:28:30 EST
tree: https://git.kernel.org/pub/scm/linux/kernel/git/kbingham/rcar.git kbingham/drm-misc/next/sn65dsi86/hpd
head: 69c67e7d89e9e68e6ffb17deec7112af1d529ac9
commit: b88732c0df106217e663da0458bd46d061ec3ce4 [10/21] drm/bridge: Drop unused drm_bridge_chain functions
config: arc-randconfig-r043-20220622 (https://download.01.org/0day-ci/archive/20220623/202206230936.HWBqaAwV-lkp@xxxxxxxxx/config)
compiler: arceb-elf-gcc (GCC) 11.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/kbingham/rcar.git/commit/?id=b88732c0df106217e663da0458bd46d061ec3ce4
git remote add kbingham-rcar https://git.kernel.org/pub/scm/linux/kernel/git/kbingham/rcar.git
git fetch --no-tags kbingham-rcar kbingham/drm-misc/next/sn65dsi86/hpd
git checkout b88732c0df106217e663da0458bd46d061ec3ce4
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=arc SHELL=/bin/bash drivers/gpu/drm/bridge/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@xxxxxxxxx>
All errors (new ones prefixed by >>):
drivers/gpu/drm/bridge/parade-ps8640.c: In function 'ps8640_bridge_get_edid':
>> drivers/gpu/drm/bridge/parade-ps8640.c:524:9: error: implicit declaration of function 'drm_bridge_chain_pre_enable'; did you mean 'drm_bridge_hpd_enable'? [-Werror=implicit-function-declaration]
524 | drm_bridge_chain_pre_enable(bridge);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
| drm_bridge_hpd_enable
>> drivers/gpu/drm/bridge/parade-ps8640.c:534:17: error: implicit declaration of function 'drm_bridge_chain_post_disable'; did you mean 'drm_atomic_bridge_chain_post_disable'? [-Werror=implicit-function-declaration]
534 | drm_bridge_chain_post_disable(bridge);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| drm_atomic_bridge_chain_post_disable
cc1: some warnings being treated as errors
vim +524 drivers/gpu/drm/bridge/parade-ps8640.c
13afcdd7277eff9 Philip Chen 2021-09-21 504
d82c12ab16b8477 Enric Balletbo i Serra 2020-08-26 505 static struct edid *ps8640_bridge_get_edid(struct drm_bridge *bridge,
d82c12ab16b8477 Enric Balletbo i Serra 2020-08-26 506 struct drm_connector *connector)
d82c12ab16b8477 Enric Balletbo i Serra 2020-08-26 507 {
d82c12ab16b8477 Enric Balletbo i Serra 2020-08-26 508 struct ps8640 *ps_bridge = bridge_to_ps8640(bridge);
826cff3f7ebba46 Philip Chen 2021-10-28 509 bool poweroff = !ps_bridge->pre_enabled;
46f206304db0311 Enric Balletbo i Serra 2020-08-27 510 struct edid *edid;
46f206304db0311 Enric Balletbo i Serra 2020-08-27 511
46f206304db0311 Enric Balletbo i Serra 2020-08-27 512 /*
46f206304db0311 Enric Balletbo i Serra 2020-08-27 513 * When we end calling get_edid() triggered by an ioctl, i.e
46f206304db0311 Enric Balletbo i Serra 2020-08-27 514 *
46f206304db0311 Enric Balletbo i Serra 2020-08-27 515 * drm_mode_getconnector (ioctl)
46f206304db0311 Enric Balletbo i Serra 2020-08-27 516 * -> drm_helper_probe_single_connector_modes
46f206304db0311 Enric Balletbo i Serra 2020-08-27 517 * -> drm_bridge_connector_get_modes
46f206304db0311 Enric Balletbo i Serra 2020-08-27 518 * -> ps8640_bridge_get_edid
46f206304db0311 Enric Balletbo i Serra 2020-08-27 519 *
46f206304db0311 Enric Balletbo i Serra 2020-08-27 520 * We need to make sure that what we need is enabled before reading
46f206304db0311 Enric Balletbo i Serra 2020-08-27 521 * EDID, for this chip, we need to do a full poweron, otherwise it will
46f206304db0311 Enric Balletbo i Serra 2020-08-27 522 * fail.
46f206304db0311 Enric Balletbo i Serra 2020-08-27 523 */
46f206304db0311 Enric Balletbo i Serra 2020-08-27 @524 drm_bridge_chain_pre_enable(bridge);
d82c12ab16b8477 Enric Balletbo i Serra 2020-08-26 525
46f206304db0311 Enric Balletbo i Serra 2020-08-27 526 edid = drm_get_edid(connector,
d82c12ab16b8477 Enric Balletbo i Serra 2020-08-26 527 ps_bridge->page[PAGE0_DP_CNTL]->adapter);
46f206304db0311 Enric Balletbo i Serra 2020-08-27 528
46f206304db0311 Enric Balletbo i Serra 2020-08-27 529 /*
46f206304db0311 Enric Balletbo i Serra 2020-08-27 530 * If we call the get_edid() function without having enabled the chip
46f206304db0311 Enric Balletbo i Serra 2020-08-27 531 * before, return the chip to its original power state.
46f206304db0311 Enric Balletbo i Serra 2020-08-27 532 */
46f206304db0311 Enric Balletbo i Serra 2020-08-27 533 if (poweroff)
46f206304db0311 Enric Balletbo i Serra 2020-08-27 @534 drm_bridge_chain_post_disable(bridge);
46f206304db0311 Enric Balletbo i Serra 2020-08-27 535
46f206304db0311 Enric Balletbo i Serra 2020-08-27 536 return edid;
d82c12ab16b8477 Enric Balletbo i Serra 2020-08-26 537 }
d82c12ab16b8477 Enric Balletbo i Serra 2020-08-26 538
:::::: The code at line 524 was first introduced by commit
:::::: 46f206304db0311b0920479f42accaa7cb472fdc drm/bridge: ps8640: Rework power state handling
:::::: TO: Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx>
:::::: CC: Neil Armstrong <narmstrong@xxxxxxxxxxxx>
--
0-DAY CI Kernel Test Service
https://01.org/lkp