public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Sui Jingfeng <suijingfeng@loongson.cn>,
	Maxime Ripard <mripard@kernel.org>,
	Thomas Zimmermann <tzimmermann@suse.de>
Cc: oe-kbuild-all@lists.linux.dev, dri-devel@lists.freedesktop.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 3/8] drm/loongson: Allow attach drm bridge driver by calling lsdc_output_init()
Date: Mon, 6 Nov 2023 01:27:56 +0800	[thread overview]
Message-ID: <202311060159.WME8x93V-lkp@intel.com> (raw)
In-Reply-To: <20231029194607.379459-4-suijingfeng@loongson.cn>

Hi Sui,

kernel test robot noticed the following build warnings:

[auto build test WARNING on drm-misc/drm-misc-next]
[also build test WARNING on linus/master v6.6 next-20231103]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Sui-Jingfeng/drm-loongson-Introduce-a-minimal-support-for-Loongson-VBIOS/20231030-034730
base:   git://anongit.freedesktop.org/drm/drm-misc drm-misc-next
patch link:    https://lore.kernel.org/r/20231029194607.379459-4-suijingfeng%40loongson.cn
patch subject: [PATCH 3/8] drm/loongson: Allow attach drm bridge driver by calling lsdc_output_init()
config: x86_64-randconfig-122-20231102 (https://download.01.org/0day-ci/archive/20231106/202311060159.WME8x93V-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231106/202311060159.WME8x93V-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202311060159.WME8x93V-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
>> drivers/gpu/drm/loongson/lsdc_output.c:555:5: sparse: sparse: symbol 'lsdc_encoder_init' was not declared. Should it be static?
>> drivers/gpu/drm/loongson/lsdc_output.c:578:5: sparse: sparse: symbol 'lsdc_connector_init' was not declared. Should it be static?

vim +/lsdc_encoder_init +555 drivers/gpu/drm/loongson/lsdc_output.c

   554	
 > 555	int lsdc_encoder_init(struct drm_device *ddev,
   556			      struct lsdc_output *output,
   557			      unsigned int pipe)
   558	{
   559		const struct lsdc_output_desc *descp = output->descp;
   560		struct drm_encoder *encoder = &output->encoder;
   561		int ret;
   562	
   563		ret = drm_encoder_init(ddev,
   564				       encoder,
   565				       descp->encoder_funcs,
   566				       descp->encoder_type,
   567				       descp->name);
   568		if (ret)
   569			return ret;
   570	
   571		encoder->possible_crtcs = BIT(pipe);
   572	
   573		drm_encoder_helper_add(encoder, descp->encoder_helper_funcs);
   574	
   575		return 0;
   576	}
   577	
 > 578	int lsdc_connector_init(struct drm_device *ddev,
   579				struct lsdc_output *output,
   580				struct i2c_adapter *ddc,
   581				unsigned int pipe)
   582	{
   583		const struct lsdc_output_desc *descp = output->descp;
   584		struct drm_connector *connector = &output->connector;
   585		int ret;
   586	
   587		ret = drm_connector_init_with_ddc(ddev,
   588						  connector,
   589						  descp->connector_funcs,
   590						  descp->connector_type,
   591						  ddc);
   592		if (ret)
   593			return ret;
   594	
   595		drm_connector_helper_add(connector, descp->connector_helper_funcs);
   596	
   597		drm_connector_attach_encoder(connector, &output->encoder);
   598	
   599		connector->polled = DRM_CONNECTOR_POLL_CONNECT |
   600				    DRM_CONNECTOR_POLL_DISCONNECT;
   601	
   602		connector->interlace_allowed = 0;
   603		connector->doublescan_allowed = 0;
   604	
   605		drm_info(ddev, "DisplayPipe-%u has %s\n", pipe, descp->name);
   606	
   607		return 0;
   608	}
   609	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

  parent reply	other threads:[~2023-11-05 17:28 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-29 19:45 [PATCH 0/8] drm/loongson: Submit a mini VBIOS support and a display bridge driver Sui Jingfeng
2023-10-29 19:46 ` [PATCH 1/8] drm/loongson: Introduce a minimal support for Loongson VBIOS Sui Jingfeng
2023-10-29 22:59   ` Dmitry Baryshkov
2023-10-30  3:20     ` Sui Jingfeng
2023-10-29 19:46 ` [PATCH 2/8] drm/loongson: Introduce a drm bridge driver for it66121 HDMI transmitter Sui Jingfeng
2023-10-29 22:53   ` Dmitry Baryshkov
2023-10-30  9:42     ` Sui Jingfeng
2023-10-30 10:01       ` Dmitry Baryshkov
2023-10-30 13:25         ` Sui Jingfeng
2023-10-30 13:39           ` Maxime Ripard
2023-10-30 14:39             ` Sui Jingfeng
2023-10-30 14:56               ` Maxime Ripard
2023-10-30 13:48           ` Dmitry Baryshkov
2023-10-30 14:25             ` Sui Jingfeng
2023-10-30 10:13       ` Maxime Ripard
2023-10-29 19:46 ` [PATCH 3/8] drm/loongson: Allow attach drm bridge driver by calling lsdc_output_init() Sui Jingfeng
2023-10-29 23:10   ` Dmitry Baryshkov
2023-10-30  4:13     ` Sui Jingfeng
2023-10-30  8:54       ` Sui Jingfeng
2023-10-30  9:16       ` Dmitry Baryshkov
2023-10-30  9:13     ` Sui Jingfeng
2023-10-30  0:12   ` kernel test robot
2023-11-01  2:42   ` kernel test robot
2023-11-04  7:41   ` kernel test robot
2023-11-05 17:27   ` kernel test robot [this message]
2023-10-29 19:46 ` [PATCH 4/8] drm/loongson: Started to attach display bridge driver for LS7A1000 Sui Jingfeng
2023-10-29 23:14   ` Dmitry Baryshkov
2023-10-29 19:46 ` [PATCH 5/8] drm/loongson: Using vbios for the LS7A2000 output initialization Sui Jingfeng
2023-10-30  2:28   ` kernel test robot
2023-11-06 17:46   ` kernel test robot
2023-10-29 19:46 ` [PATCH 6/8] drm/loongson: Clean up the output part of LS7A2000 Sui Jingfeng
2023-10-29 23:17   ` Dmitry Baryshkov
2023-10-29 19:46 ` [PATCH 7/8] drm/loongson: Support to infer DC reversion from CPU's PRID value Sui Jingfeng
2023-10-29 23:19   ` Dmitry Baryshkov
2023-10-30  8:50     ` Sui Jingfeng
2023-10-29 19:46 ` [PATCH 8/8] drm/loongson: Add support for the display subsystem in LS2K2000 Sui Jingfeng

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202311060159.WME8x93V-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mripard@kernel.org \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=suijingfeng@loongson.cn \
    --cc=tzimmermann@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox