Re: mgag200: Wrong looking code in mga_probe_vram

2017-10-19 Thread Mathieu Larouche
On 11/10/2017 5:27 AM, Michal Srb wrote: Hi, While reading the mgag200 driver I noticed something that looks wrong. In mgag200_main.c mga_probe_vram function: orig1 = ioread8(mem + offset); orig2 = ioread8(mem + offset + 0x100); ... iowrite16(orig1, mem + offset); io

Re: [PATCH 1/1] drm/mgag200: Expose version string

2017-10-06 Thread Mathieu Larouche
I would like to know if anyone had the time to look at this patch and when I could expect to have it accepted. Thank you, Mathieu On 18/08/2017 4:24 PM, mathieu.larou...@matrox.com wrote: From: Mathieu Larouche OEMs started distributing this module out of the box so adding the version in

[PATCH 1/1] drm/mgag200: Expose version string

2017-08-21 Thread mathieu . larouche
From: Mathieu Larouche OEMs started distributing this module out of the box so adding the version in modinfo output for this module may be a good idea. Signed-off-by: Mathieu Larouche --- drivers/gpu/drm/mgag200/mgag200_drv.c | 3 +++ drivers/gpu/drm/mgag200/mgag200_drv.h | 2 +- 2 files

Re: [PATCH 00/14] mgag200 fixes

2017-08-01 Thread Mathieu Larouche
dering if it shouldn't be kept disabled for them. -- Mathieu Larouche Ing./Eng. Software Designer Matrox Graphics Inc. Phone : 514 822-6000 x7905 Email : mathieu.larou...@matrox.com ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH 1/1] drm/mgag200: Fix to always set HiPri for G200e4 V2

2017-06-14 Thread mathieu . larouche
From: Mathieu Larouche - Changed the HiPri value for G200e4 to always be 0. - Added Bandwith limitation to block resolution above 1920x1200x60Hz Signed-off-by: Mathieu Larouche --- drivers/gpu/drm/mgag200/mgag200_mode.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff

[PATCH 1/1] drm/mgag200: Fix to always set HiPri for G200e4

2017-06-13 Thread mathieu . larouche
From: Mathieu Larouche - Changed the HiPri value for G200e4 to always be 0. - Added Bandwith limitation to block resolution above 1920x1200x60Hz --- drivers/gpu/drm/mgag200/mgag200_mode.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/mgag200

drm/mgag200: don't use uninitialized variables in mga_g200se_set_plls()

2015-10-20 Thread Mathieu Larouche
On 20/10/2015 11:00 AM, Jan Beulich wrote: On 20.10.15 at 16:47, wrote: >> On 19/10/2015 6:29 AM, Jan Beulich wrote: >>> --- 4.3-rc6/drivers/gpu/drm/mgag200/mgag200_mode.c >>> +++ 4.3-rc6-mgag200-uninit/drivers/gpu/drm/mgag200/mgag200_mode.c >>> @@ -194,7 +194,7 @@ static int mga_g200se_set_

drm/mgag200: don't use uninitialized variables in mga_g200se_set_plls()

2015-10-20 Thread Mathieu Larouche
new rev of G200e"). In any event the > compiler is right in warning that testm/testn are possibly uninitalized > at this point, i.e. some change is needed no matter what. > > Signed-off-by: Jan Beulich > Cc: Mathieu Larouche > --- > drivers/gpu/drm/mgag200/mgag200_mode.

Re: [PATCH 1/3] drm/mgag200: Index 24 in extended CRTC registers is 24 in hex, not decimal.

2013-04-05 Thread Mathieu Larouche
On 05/04/2013 10:51 AM, Christopher Harvey wrote: This change properly enables the "requester" in G200ER cards that is responsible for getting pixels out of memory and clocking them out to the screen. Signed-off-by: Christopher Harvey Signed-off-by: Mathieu Larouche --- drive

[PATCH 1/3] drm/mgag200: Index 24 in extended CRTC registers is 24 in hex, not decimal.

2013-04-05 Thread Mathieu Larouche
On 05/04/2013 10:51 AM, Christopher Harvey wrote: > This change properly enables the "requester" in G200ER cards that is > responsible for getting pixels out of memory and clocking them out to > the screen. > > Signed-off-by: Christopher Harvey Signed