All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Thomas Zimmermann <tzimmermann@suse.de>
Cc: "Luck, Tony" <tony.luck@intel.com>,
	"jfalempe@redhat.com" <jfalempe@redhat.com>,
	"airlied@redhat.com" <airlied@redhat.com>,
	"sam@ravnborg.org" <sam@ravnborg.org>,
	"emil.l.velikov@gmail.com" <emil.l.velikov@gmail.com>,
	"maarten.lankhorst@linux.intel.com"
	<maarten.lankhorst@linux.intel.com>,
	"mripard@kernel.org" <mripard@kernel.org>,
	"airlied@gmail.com" <airlied@gmail.com>,
	"daniel@ffwll.ch" <daniel@ffwll.ch>,
	"dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH v5 0/7] drm/mgag200: Implement VBLANK support
Date: Fri, 4 Oct 2024 13:01:06 +0300	[thread overview]
Message-ID: <Zv-84gdD85CqVeh5@intel.com> (raw)
In-Reply-To: <ad02af39-b9b5-4b04-878b-78b3eb7885a5@suse.de>

On Fri, Oct 04, 2024 at 11:17:02AM +0200, Thomas Zimmermann wrote:
> Hi
> 
> Am 02.10.24 um 18:15 schrieb Luck, Tony:
> >> Thanks for the bug report. Can you provide the output of 'sudo lspci
> >> -vvv' for the graphics device?
> > Thomas,
> >
> > Sure. Here's the output (run on the v6.11.0 kernel)
> 
> Thanks. It doesn't look much different from other systems. IRQ is also 
> assigned.
> 
> Attached is a patch that fixes a possible off-by-one error in the 
> register settings. This would affect the bug you're reporting. If 
> possible, please apply the patch to your 6.12-rc1, test and report the 
> result.

Didn't one of these weird variants have some bug where the
CRTC startadd was not working? Is this one of those?
That to me sounds like maybe linecomp has internally been
tied to be always active somehow. Perhaps that would
also prevent it from generating the interrupt...

Anyways, sounds like someone should just double check whether
the status bit ever get asserted or not. If yes, then the
problem must be with interrupt delivery, otherwise the
problem is that the internal interrupt is never even
generated. In the latter case you could try using the
vsync interrupt instead.

-- 
Ville Syrjälä
Intel

  reply	other threads:[~2024-10-04 10:01 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-18 10:44 [PATCH v5 0/7] drm/mgag200: Implement VBLANK support Thomas Zimmermann
2024-07-18 10:44 ` [PATCH v5 1/7] drm/mgag200: Use hexadecimal register indeces Thomas Zimmermann
2024-07-18 10:44 ` [PATCH v5 2/7] drm/mgag200: Align register field names with documentation Thomas Zimmermann
2024-07-18 10:44 ` [PATCH v5 3/7] drm/mgag200: Use adjusted mode values for CRTCs Thomas Zimmermann
2024-07-18 10:44 ` [PATCH v5 4/7] drm/mgag200: Add dedicated variables for blanking fields Thomas Zimmermann
2024-07-18 10:44 ` [PATCH v5 5/7] drm/mgag200: Add dedicted variable for <linecomp> field Thomas Zimmermann
2024-07-18 10:44 ` [PATCH v5 6/7] drm/mgag200: Add vblank support Thomas Zimmermann
2024-07-18 11:34   ` Jocelyn Falempe
2024-07-18 11:55     ` Thomas Zimmermann
2024-07-18 10:44 ` [PATCH v5 7/7] drm/mgag200: Implement struct drm_crtc_funcs.get_vblank_timestamp Thomas Zimmermann
2024-10-01 22:41 ` [PATCH v5 0/7] drm/mgag200: Implement VBLANK support Tony Luck
2024-10-02  7:06   ` Thomas Zimmermann
2024-10-02 16:15     ` Luck, Tony
2024-10-04  9:17       ` Thomas Zimmermann
2024-10-04 10:01         ` Ville Syrjälä [this message]
2024-10-04 11:03           ` Thomas Zimmermann
2024-10-04 11:19             ` Ville Syrjälä
2024-10-04 11:47               ` Thomas Zimmermann
2024-10-07 13:37           ` Thomas Zimmermann
2024-10-08 18:45             ` Ville Syrjälä
2024-10-04 16:58         ` Luck, Tony
2024-10-10  9:21           ` Thomas Zimmermann
2024-10-10 16:07             ` Luck, Tony
2024-10-10 18:12               ` Luck, Tony
2024-10-11  7:03                 ` Thomas Zimmermann
2024-10-11 16:36                   ` Luck, Tony
2024-10-11 16:44                     ` Luck, Tony
2024-10-14 12:39                       ` Thomas Zimmermann
2024-10-14 17:14                         ` Luck, Tony
2024-10-15  7:13                           ` Thomas Zimmermann
2024-10-25 17:32                             ` Luck, Tony
2024-10-11  7:09   ` Jocelyn Falempe

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=Zv-84gdD85CqVeh5@intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=airlied@gmail.com \
    --cc=airlied@redhat.com \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=emil.l.velikov@gmail.com \
    --cc=jfalempe@redhat.com \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=mripard@kernel.org \
    --cc=sam@ravnborg.org \
    --cc=tony.luck@intel.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.