* [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 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.