[PATCH xf86-video-savage] Add check for max[HV]Value to ValidMode hook

[Stefan Dirsch]

On Tue, Jun 26, 2018 at 04:13:01PM +0100, Emil Velikov wrote:
> On 25 June 2018 at 14:55, Stefan Dirsch <sndirsch at suse.de> wrote:
> > xorg-server 1.20 removed this check, so implement this in the driver
> > itself.
> >
> Quick look shows that Adam, updated most drivers although a few are remaining.
> Namely:
> s3verge
> tseng
> vga
> savage - addressed with this patch.
> 
> > Signed-off-by: Stefan Dirsch <sndirsch at suse.de>
> Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
> 
> I'll push this shortly.

Thanks a lot!

> Although if you'd like to address the other
> drivers, that'll be appreciated.

Looks like building the vga driver is no longer appreciated. ;-)

./configure 
configure: error: 
              **************************************************************
              This driver is unmaintained and will not receive further
              patches.  Please use the xf86-video-vesa driver instead.
              If you have a need for this driver that cannot be met by the
              vesa driver, please file a bug on http://bugs.freedesktop.org.
              **************************************************************

I've sent patches for s3virge and tseng driver to the list. BTW,
(open)SUSE no longer ship prehistoric drivers since some time ...

CU,
Stefan

Public Key available
------------------------------------------------------
Stefan Dirsch (Res. & Dev.)   SUSE LINUX GmbH
Tel: 0911-740 53 0            Maxfeldstraße 5
FAX: 0911-740 53 479          D-90409 Nürnberg
http://www.suse.de            Germany 
---------------------------------------------------------------
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham
Norton, HRB 21284 (AG Nürnberg)
---------------------------------------------------------------