public inbox for llvm@lists.linux.dev
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Janne Grunau <j@jannau.net>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev
Subject: [asahilinux:asahi-wip 29/31] drivers/gpu/drm/apple/dcp.c:101: warning: Function parameter or struct member 'crtc' not described in 'dcp_crtc_send_page_flip_event'
Date: Mon, 13 May 2024 16:29:05 +0800	[thread overview]
Message-ID: <202405131641.E3WM7KFH-lkp@intel.com> (raw)

tree:   https://github.com/AsahiLinux/linux asahi-wip
head:   65718aefab870e7d3ab6c1863cb9dc2773712312
commit: 64bf00b0969fcf8f27cb21e4db4a0741b47cfdda [29/31] drm: apple: Override drm_vblank's page flip event handling [HACK]
config: arm64-allmodconfig (https://download.01.org/0day-ci/archive/20240513/202405131641.E3WM7KFH-lkp@intel.com/config)
compiler: clang version 19.0.0git (https://github.com/llvm/llvm-project b910bebc300dafb30569cecc3017b446ea8eafa0)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240513/202405131641.E3WM7KFH-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/202405131641.E3WM7KFH-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/gpu/drm/apple/dcp.c:101: warning: Function parameter or struct member 'crtc' not described in 'dcp_crtc_send_page_flip_event'
>> drivers/gpu/drm/apple/dcp.c:101: warning: Function parameter or struct member 'e' not described in 'dcp_crtc_send_page_flip_event'
>> drivers/gpu/drm/apple/dcp.c:101: warning: Function parameter or struct member 'now' not described in 'dcp_crtc_send_page_flip_event'
>> drivers/gpu/drm/apple/dcp.c:101: warning: Function parameter or struct member 'start' not described in 'dcp_crtc_send_page_flip_event'


vim +101 drivers/gpu/drm/apple/dcp.c

    82	
    83	/**
    84	 * dcp_crtc_send_page_flip_event - helper to send vblank event after pageflip
    85	 *
    86	 * Compensate for unknown slack between page flip and arrival of the
    87	 * swap_complete callback. Minimal observed duration on DCP with HDMI output
    88	 * was around 2.3 ms. If the fb swap was submitted closer to the expected
    89	 * swap_complete it gets a penalty of one frame duration. This is on the border
    90	 * of unreasonable considering that Apple advertises support for 240 Hz (frame
    91	 * duration of 4.167 ms).
    92	 * It is unreasonable considering kwin's kms commit scheduling. Kwin commits
    93	 * 1.5 ms + the mode's vblank time before the expected next page flip
    94	 * completion. This results in presenting at half the display's rate for HDMI
    95	 * outputs.
    96	 * This might be a difference between dcp and dcpext.
    97	 */
    98	static void dcp_crtc_send_page_flip_event(struct apple_crtc *crtc,
    99						  struct drm_pending_vblank_event *e,
   100						  ktime_t now, ktime_t start)
 > 101	{
   102		struct drm_device *dev = crtc->base.dev;
   103		u64 seq;
   104		unsigned int pipe = drm_crtc_index(&crtc->base);
   105		ktime_t flip;
   106	
   107		seq = 0;
   108		if (start != KTIME_MIN) {
   109			s64 delta = ktime_us_delta(now, start);
   110			if (delta <= 500)
   111				flip = now;
   112			else if (delta >= 2500)
   113				flip = ktime_sub_us(now, 1000);
   114			else
   115				flip = ktime_sub_us(now, (delta - 500) / 2);
   116		} else {
   117			flip = now;
   118		}
   119		e->pipe = pipe;
   120		send_vblank_event(dev, e, seq, flip);
   121	}
   122	

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

                 reply	other threads:[~2024-05-13  8:29 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=202405131641.E3WM7KFH-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=j@jannau.net \
    --cc=llvm@lists.linux.dev \
    --cc=oe-kbuild-all@lists.linux.dev \
    /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