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

[Daniel Vetter]

On Thu, Oct 5, 2017 at 9:18 PM, Mathieu Larouche
<mathieu.larouche at matrox.com> wrote:
> I would like to know if anyone had the time to look at this patch and when I
> could expect to have it accepted.

MODULE_VERSION is kinda considered uncool in drm. For KMS drivers
(which mga200g is one) there's really no reason, for render/gem
drivers we have tons of other ways to communicate the uapi
version/feature support to userspace. There's also the occasional
thread on lkml to just rip it all out as essentially nonsense.

Please explain what you need this for and we can point you in the
right direction.
-Daniel
>
> Thank you,
>
> Mathieu
>
>
> On 18/08/2017 4:24 PM, mathieu.larouche at matrox.com wrote:
>>
>> From: Mathieu Larouche <mathieu.larouche at matrox.com>
>>
>> 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 <mathieu.larouche at matrox.com>
>> ---
>>   drivers/gpu/drm/mgag200/mgag200_drv.c | 3 +++
>>   drivers/gpu/drm/mgag200/mgag200_drv.h | 2 +-
>>   2 files changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c
>> b/drivers/gpu/drm/mgag200/mgag200_drv.c
>> index 9ac0078..ad3f193 100644
>> --- a/drivers/gpu/drm/mgag200/mgag200_drv.c
>> +++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
>> @@ -134,3 +134,6 @@ static void __exit mgag200_exit(void)
>>   MODULE_AUTHOR(DRIVER_AUTHOR);
>>   MODULE_DESCRIPTION(DRIVER_DESC);
>>   MODULE_LICENSE("GPL");
>> +MODULE_VERSION(__stringify(DRIVER_MAJOR) "."
>> +              __stringify(DRIVER_MINOR) "."
>> +              __stringify(DRIVER_PATCHLEVEL));
>> diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.h
>> b/drivers/gpu/drm/mgag200/mgag200_drv.h
>> index c88b6ec..df4833b 100644
>> --- a/drivers/gpu/drm/mgag200/mgag200_drv.h
>> +++ b/drivers/gpu/drm/mgag200/mgag200_drv.h
>> @@ -37,7 +37,7 @@
>>   #define DRIVER_DATE           "20110418"
>>     #define DRIVER_MAJOR                1
>> -#define DRIVER_MINOR           0
>> +#define DRIVER_MINOR           1
>>   #define DRIVER_PATCHLEVEL     0
>>     #define MGAG200FB_CONN_LIMIT 1
>
>
> --
> Mathieu Larouche Ing./Eng.
> Software Designer
> Matrox Graphics Inc.
> Phone : 514 822-6000 x7905
> Email : mathieu.larouche at matrox.com
>
> CONFIDENTIAL & WITHOUT PREJUDICE
>
> This e-mail and any files transmitted with it, is confidential, may be
> protected under NDA and/or privileged and shall be treated as such. The
> e-mail and its attachments are intended only for use of the individual(s) or
> entity(ies) indicated above. Any other person is hereby advised that it
> strictly forbidden to disclose, distribute or reproduce this message. If you
> have received this e-mail in error, please advise me by return e-mail or by
> telephone at 514-822-6000, immediately and destroy the message and its
> contents immediately. Thank You.
>
> CONFIDENTIEL- SANS PRÉJUDICE
>
> Ce courriel et tout document qui y est joint, est confidentiel, peut être
> privilégié et protégé par entente de confidentialité et est à l'usage
> exclusif du destinataire. Toute autre personne est par les présentes avisée
> qu'il lui est strictement interdit de le diffuser, le distribuer ou le
> reproduire. Si vous recevez ce courriel par erreur, veuillez m'en aviser
> immédiatement, par retour de courriel ou par téléphone au (514) 822-6000 et
> détruire ce message et toute copie de celui-ci immédiatement. Merci.
>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch