[PATCH xserver] modesetting: Do not use function local static variables

Adam Jackson ajax at nwnk.net
Fri Sep 2 17:05:46 UTC 2016


On Wed, 2016-08-31 at 09:14 -0700, Keith Packard wrote:
> > Hans de Goede <hdegoede at redhat.com> writes:
> 
> > The modesetting driver may be driving 2 screens (slave and master
> > gpu), which may have different behavior wrt hardware cursor support.
> > 
> > So stop using static variables and instead store the hw-cursor support
> > related data in a per screen struct. While at it actually make it per
> > crtc data as in theory different crtc's could have different hw-cursor
> > support.
> > 
> > Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> 
> The things you find if you look long enough...
> 
> Reviewed-by: Keith Packard <keithp at keithp.com>

remote: I: patch #108118 updated using rev 6c984ac9a737304ab5e26de90c76a79d970b6e33.
remote: I: 1 patch(es) updated to state Accepted.
To ssh://git.freedesktop.org/git/xorg/xserver
   d8c288e..6c984ac  master -> master

- ajax


More information about the xorg-devel mailing list