* [PATCH 1/1] drm/mgag200: Expose version string
@ 2017-08-18 20:24 mathieu.larouche
2017-10-05 19:18 ` Mathieu Larouche
0 siblings, 1 reply; 3+ messages in thread
From: mathieu.larouche @ 2017-08-18 20:24 UTC (permalink / raw)
To: dri-devel; +Cc: Mathieu Larouche, smahmoud
From: Mathieu Larouche <mathieu.larouche@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@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
--
1.8.3.1
_______________________________________________
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 1/1] drm/mgag200: Expose version string
2017-08-18 20:24 [PATCH 1/1] drm/mgag200: Expose version string mathieu.larouche
@ 2017-10-05 19:18 ` Mathieu Larouche
2017-10-06 12:07 ` Daniel Vetter
0 siblings, 1 reply; 3+ messages in thread
From: Mathieu Larouche @ 2017-10-05 19:18 UTC (permalink / raw)
To: mathieu.larouche, dri-devel; +Cc: smahmoud
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.larouche@matrox.com wrote:
> From: Mathieu Larouche <mathieu.larouche@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@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@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@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 1/1] drm/mgag200: Expose version string
2017-10-05 19:18 ` Mathieu Larouche
@ 2017-10-06 12:07 ` Daniel Vetter
0 siblings, 0 replies; 3+ messages in thread
From: Daniel Vetter @ 2017-10-06 12:07 UTC (permalink / raw)
To: Mathieu Larouche; +Cc: dri-devel, smahmoud
On Thu, Oct 5, 2017 at 9:18 PM, Mathieu Larouche
<mathieu.larouche@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@matrox.com wrote:
>>
>> From: Mathieu Larouche <mathieu.larouche@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@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@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@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
_______________________________________________
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:[~2017-10-06 12:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-18 20:24 [PATCH 1/1] drm/mgag200: Expose version string mathieu.larouche
2017-10-05 19:18 ` Mathieu Larouche
2017-10-06 12:07 ` Daniel Vetter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).