* [PATCH] drm/mgag200: Debug-print unique revisions id on G200 SE
@ 2019-12-04 13:34 Thomas Zimmermann
2019-12-05 8:03 ` Gerd Hoffmann
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Zimmermann @ 2019-12-04 13:34 UTC (permalink / raw)
To: airlied, daniel, john.p.donnelly, kraxel, sam
Cc: Thomas Zimmermann, dri-devel
The behavior of MGA G200 SE depends on the rev id. Print the id when
debugging is enabled.
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
---
drivers/gpu/drm/mgag200/mgag200_main.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c b/drivers/gpu/drm/mgag200/mgag200_main.c
index e1bc5b0aa774..aeb7c10ded0d 100644
--- a/drivers/gpu/drm/mgag200/mgag200_main.c
+++ b/drivers/gpu/drm/mgag200/mgag200_main.c
@@ -118,8 +118,11 @@ static int mgag200_device_init(struct drm_device *dev,
return -ENOMEM;
/* stash G200 SE model number for later use */
- if (IS_G200_SE(mdev))
+ if (IS_G200_SE(mdev)) {
mdev->unique_rev_id = RREG32(0x1e24);
+ DRM_DEBUG("G200 SE unique revision id is 0x%x\n",
+ mdev->unique_rev_id);
+ }
ret = mga_vram_init(mdev);
if (ret)
--
2.23.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/mgag200: Debug-print unique revisions id on G200 SE
2019-12-04 13:34 [PATCH] drm/mgag200: Debug-print unique revisions id on G200 SE Thomas Zimmermann
@ 2019-12-05 8:03 ` Gerd Hoffmann
2019-12-05 8:05 ` Thomas Zimmermann
0 siblings, 1 reply; 3+ messages in thread
From: Gerd Hoffmann @ 2019-12-05 8:03 UTC (permalink / raw)
To: Thomas Zimmermann; +Cc: airlied, sam, dri-devel, john.p.donnelly
On Wed, Dec 04, 2019 at 02:34:35PM +0100, Thomas Zimmermann wrote:
> The behavior of MGA G200 SE depends on the rev id. Print the id when
> debugging is enabled.
That is the one where some need a workaround, right?
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> ---
> drivers/gpu/drm/mgag200/mgag200_main.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c b/drivers/gpu/drm/mgag200/mgag200_main.c
> index e1bc5b0aa774..aeb7c10ded0d 100644
> --- a/drivers/gpu/drm/mgag200/mgag200_main.c
> +++ b/drivers/gpu/drm/mgag200/mgag200_main.c
> @@ -118,8 +118,11 @@ static int mgag200_device_init(struct drm_device *dev,
> return -ENOMEM;
>
> /* stash G200 SE model number for later use */
> - if (IS_G200_SE(mdev))
> + if (IS_G200_SE(mdev)) {
> mdev->unique_rev_id = RREG32(0x1e24);
> + DRM_DEBUG("G200 SE unique revision id is 0x%x\n",
> + mdev->unique_rev_id);
Maybe use DRM_INFO here? Temporarily for a few kernel revisions, until
things are settled?
(just a suggestion, your choice).
Acked-by: Gerd Hoffmann <kraxel@redhat.com>
cheers,
Gerd
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/mgag200: Debug-print unique revisions id on G200 SE
2019-12-05 8:03 ` Gerd Hoffmann
@ 2019-12-05 8:05 ` Thomas Zimmermann
0 siblings, 0 replies; 3+ messages in thread
From: Thomas Zimmermann @ 2019-12-05 8:05 UTC (permalink / raw)
To: Gerd Hoffmann; +Cc: airlied, sam, dri-devel, john.p.donnelly
[-- Attachment #1.1.1: Type: text/plain, Size: 1583 bytes --]
Hi
Am 05.12.19 um 09:03 schrieb Gerd Hoffmann:
> On Wed, Dec 04, 2019 at 02:34:35PM +0100, Thomas Zimmermann wrote:
>> The behavior of MGA G200 SE depends on the rev id. Print the id when
>> debugging is enabled.
>
> That is the one where some need a workaround, right?
Yes. Hopefully this ID helps with figuring out the broken ones.
>
>> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
>> ---
>> drivers/gpu/drm/mgag200/mgag200_main.c | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c b/drivers/gpu/drm/mgag200/mgag200_main.c
>> index e1bc5b0aa774..aeb7c10ded0d 100644
>> --- a/drivers/gpu/drm/mgag200/mgag200_main.c
>> +++ b/drivers/gpu/drm/mgag200/mgag200_main.c
>> @@ -118,8 +118,11 @@ static int mgag200_device_init(struct drm_device *dev,
>> return -ENOMEM;
>>
>> /* stash G200 SE model number for later use */
>> - if (IS_G200_SE(mdev))
>> + if (IS_G200_SE(mdev)) {
>> mdev->unique_rev_id = RREG32(0x1e24);
>> + DRM_DEBUG("G200 SE unique revision id is 0x%x\n",
>> + mdev->unique_rev_id);
>
> Maybe use DRM_INFO here? Temporarily for a few kernel revisions, until
> things are settled?
>
> (just a suggestion, your choice).
>
> Acked-by: Gerd Hoffmann <kraxel@redhat.com>
Thanks.
Best regards
Thomas
>
> cheers,
> Gerd
>
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Felix Imendörffer
[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-12-05 8:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-04 13:34 [PATCH] drm/mgag200: Debug-print unique revisions id on G200 SE Thomas Zimmermann
2019-12-05 8:03 ` Gerd Hoffmann
2019-12-05 8:05 ` Thomas Zimmermann
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.