Re: [PATCH 4/4] GenWQE: Increase driver version number

From: Frank Haverkamp
Date: Thu Jun 05 2014 - 05:23:43 EST


Hi Greg,

Am Mittwoch, den 04.06.2014, 08:54 -0700 schrieb Greg KH:
> On Wed, Jun 04, 2014 at 10:57:53AM -0300, Kleber Sacilotto de Souza wrote:
> > Increase genwqe driver version number.
> >
> > Signed-off-by: Kleber Sacilotto de Souza <klebers@xxxxxxxxxxxxxxxxxx>
> > ---
> > drivers/misc/genwqe/genwqe_driver.h | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/misc/genwqe/genwqe_driver.h b/drivers/misc/genwqe/genwqe_driver.h
> > index cd52631..a506e9a 100644
> > --- a/drivers/misc/genwqe/genwqe_driver.h
> > +++ b/drivers/misc/genwqe/genwqe_driver.h
> > @@ -36,7 +36,7 @@
> > #include <asm/byteorder.h>
> > #include <linux/genwqe/genwqe_card.h>
> >
> > -#define DRV_VERS_STRING "2.0.15"
> > +#define DRV_VERS_STRING "2.0.21"
>
> Why is this even needed? Can't you go off of the kernel version number
> now? Who needs / wants this number?

I am aware that if just considering the mainline kernels, we could use
the kernel version itself for the purpose of identifying which code we
are running.

But in our lab we are running multiple back-ported versions of this
driver on different Linux distributions using different kernel versions.
Our user-space software needs to know if the driver has or has not
bug-fixes or features. For this purpose, we are using this extra number.

> thanks,
>
> greg k-h
>

Regards

Frank

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/