* [Intel-wired-lan] [PATCH v2] fm10k: remove duplication of the driver summary
@ 2016-02-25 23:58 Jacob Keller
2016-02-26 0:57 ` Allan, Bruce W
0 siblings, 1 reply; 3+ messages in thread
From: Jacob Keller @ 2016-02-25 23:58 UTC (permalink / raw)
To: intel-wired-lan
Use DRV_SUMMARY, similar to DRV_VERSION so that we don't have duplicate
copies of the summary left in the driver.
Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
---
This is a direct replacement of what is already on the queue. This
version drops the summary change and only removes the duplication.
drivers/net/ethernet/intel/fm10k/fm10k_main.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/intel/fm10k/fm10k_main.c b/drivers/net/ethernet/intel/fm10k/fm10k_main.c
index 3a81c1451624..7832f9cdf743 100644
--- a/drivers/net/ethernet/intel/fm10k/fm10k_main.c
+++ b/drivers/net/ethernet/intel/fm10k/fm10k_main.c
@@ -29,15 +29,15 @@
#include "fm10k.h"
#define DRV_VERSION "0.19.3-k"
+#define DRV_SUMMARY "Intel(R) Ethernet Switch Host Interface Driver"
const char fm10k_driver_version[] = DRV_VERSION;
char fm10k_driver_name[] = "fm10k";
-static const char fm10k_driver_string[] =
- "Intel(R) Ethernet Switch Host Interface Driver";
+static const char fm10k_driver_string[] = DRV_SUMMARY;
static const char fm10k_copyright[] =
"Copyright (c) 2013 Intel Corporation.";
MODULE_AUTHOR("Intel Corporation, <linux.nics@intel.com>");
-MODULE_DESCRIPTION("Intel(R) Ethernet Switch Host Interface Driver");
+MODULE_DESCRIPTION(DRV_SUMMARY);
MODULE_LICENSE("GPL");
MODULE_VERSION(DRV_VERSION);
--
2.7.1.429.g45cd78e
^ permalink raw reply related [flat|nested] 3+ messages in thread* [Intel-wired-lan] [PATCH v2] fm10k: remove duplication of the driver summary
2016-02-25 23:58 [Intel-wired-lan] [PATCH v2] fm10k: remove duplication of the driver summary Jacob Keller
@ 2016-02-26 0:57 ` Allan, Bruce W
2016-02-26 1:05 ` Keller, Jacob E
0 siblings, 1 reply; 3+ messages in thread
From: Allan, Bruce W @ 2016-02-26 0:57 UTC (permalink / raw)
To: intel-wired-lan
> -----Original Message-----
> From: Intel-wired-lan [mailto:intel-wired-lan-bounces at lists.osuosl.org] On
> Behalf Of Jacob Keller
> Sent: Thursday, February 25, 2016 3:59 PM
> To: Intel Wired LAN
> Subject: [Intel-wired-lan] [PATCH v2] fm10k: remove duplication of the
> driver summary
>
> Use DRV_SUMMARY, similar to DRV_VERSION so that we don't have
> duplicate
> copies of the summary left in the driver.
>
> Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
> ---
> This is a direct replacement of what is already on the queue. This
> version drops the summary change and only removes the duplication.
>
> drivers/net/ethernet/intel/fm10k/fm10k_main.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/fm10k/fm10k_main.c
> b/drivers/net/ethernet/intel/fm10k/fm10k_main.c
> index 3a81c1451624..7832f9cdf743 100644
> --- a/drivers/net/ethernet/intel/fm10k/fm10k_main.c
> +++ b/drivers/net/ethernet/intel/fm10k/fm10k_main.c
> @@ -29,15 +29,15 @@
> #include "fm10k.h"
>
> #define DRV_VERSION "0.19.3-k"
> +#define DRV_SUMMARY "Intel(R) Ethernet Switch Host Interface
> Driver"
> const char fm10k_driver_version[] = DRV_VERSION;
> char fm10k_driver_name[] = "fm10k";
> -static const char fm10k_driver_string[] =
> - "Intel(R) Ethernet Switch Host Interface Driver";
> +static const char fm10k_driver_string[] = DRV_SUMMARY;
> static const char fm10k_copyright[] =
> "Copyright (c) 2013 Intel Corporation.";
>
> MODULE_AUTHOR("Intel Corporation, <linux.nics@intel.com>");
> -MODULE_DESCRIPTION("Intel(R) Ethernet Switch Host Interface Driver");
> +MODULE_DESCRIPTION(DRV_SUMMARY);
> MODULE_LICENSE("GPL");
> MODULE_VERSION(DRV_VERSION);
Perhaps consider updating the copyright at the same time you are modifying this file?!?
I suggested a change to "Copyright(c) 2013 - 2016 Intel Corporation."
^ permalink raw reply [flat|nested] 3+ messages in thread* [Intel-wired-lan] [PATCH v2] fm10k: remove duplication of the driver summary
2016-02-26 0:57 ` Allan, Bruce W
@ 2016-02-26 1:05 ` Keller, Jacob E
0 siblings, 0 replies; 3+ messages in thread
From: Keller, Jacob E @ 2016-02-26 1:05 UTC (permalink / raw)
To: intel-wired-lan
On Fri, 2016-02-26 at 00:57 +0000, Allan, Bruce W wrote:
> >
> > -----Original Message-----
> > From: Intel-wired-lan [mailto:intel-wired-lan-bounces at lists.osuosl.
> > org] On
> > Behalf Of Jacob Keller
> > Sent: Thursday, February 25, 2016 3:59 PM
> > To: Intel Wired LAN
> > Subject: [Intel-wired-lan] [PATCH v2] fm10k: remove duplication of
> > the
> > driver summary
> >
> > Use DRV_SUMMARY, similar to DRV_VERSION so that we don't have
> > duplicate
> > copies of the summary left in the driver.
> >
> > Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
> > ---
> > This is a direct replacement of what is already on the queue. This
> > version drops the summary change and only removes the duplication.
> >
> > ?drivers/net/ethernet/intel/fm10k/fm10k_main.c | 6 +++---
> > ?1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/net/ethernet/intel/fm10k/fm10k_main.c
> > b/drivers/net/ethernet/intel/fm10k/fm10k_main.c
> > index 3a81c1451624..7832f9cdf743 100644
> > --- a/drivers/net/ethernet/intel/fm10k/fm10k_main.c
> > +++ b/drivers/net/ethernet/intel/fm10k/fm10k_main.c
> > @@ -29,15 +29,15 @@
> > ?#include "fm10k.h"
> >
> > ?#define DRV_VERSION "0.19.3-k"
> > +#define DRV_SUMMARY "Intel(R) Ethernet Switch Host
> > Interface
> > Driver"
> > ?const char fm10k_driver_version[] = DRV_VERSION;
> > ?char fm10k_driver_name[] = "fm10k";
> > -static const char fm10k_driver_string[] =
> > - "Intel(R) Ethernet Switch Host Interface Driver";
> > +static const char fm10k_driver_string[] = DRV_SUMMARY;
> > ?static const char fm10k_copyright[] =
> > ? "Copyright (c) 2013 Intel Corporation.";
> >
> > ?MODULE_AUTHOR("Intel Corporation, <linux.nics@intel.com>");
> > -MODULE_DESCRIPTION("Intel(R) Ethernet Switch Host Interface
> > Driver");
> > +MODULE_DESCRIPTION(DRV_SUMMARY);
> > ?MODULE_LICENSE("GPL");
> > ?MODULE_VERSION(DRV_VERSION);
> Perhaps consider updating the copyright at the same time you are
> modifying this file?!?
>
> I suggested a change to "Copyright(c) 2013 - 2016 Intel Corporation."
>
Sure.
Thanks,
Jake
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-02-26 1:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-25 23:58 [Intel-wired-lan] [PATCH v2] fm10k: remove duplication of the driver summary Jacob Keller
2016-02-26 0:57 ` Allan, Bruce W
2016-02-26 1:05 ` Keller, Jacob E
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.