| Message ID | 20210227095858.604463-1-gregkh@linuxfoundation.org |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | e1000e: use proper #include guard name in hw.h | expand |
On Sat, 2021-02-27 at 10:58 +0100, Greg Kroah-Hartman wrote: > The include guard for the e1000e and e1000 hw.h files are the same, > so > add the proper "E" term to the hw.h file for the e1000e driver. There's a patch in process that addresses this issue [1]. > This resolves some static analyzer warnings, like the one found by > the > "lgtm.com" tool. > > Cc: Jesse Brandeburg <jesse.brandeburg@intel.com> > Cc: Tony Nguyen <anthony.l.nguyen@intel.com> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: intel-wired-lan@lists.osuosl.org > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> [1] https://patchwork.ozlabs.org/project/intel-wired- lan/patch/20210222040005.20126-1-tseewald@gmail.com/ Thanks, Tony
On Tue, Mar 02, 2021 at 01:37:59AM +0000, Nguyen, Anthony L wrote: > On Sat, 2021-02-27 at 10:58 +0100, Greg Kroah-Hartman wrote: > > The include guard for the e1000e and e1000 hw.h files are the same, > > so > > add the proper "E" term to the hw.h file for the e1000e driver. > > There's a patch in process that addresses this issue [1]. Thanks, hopefully it gets fixed somehow :) greg k-h
diff --git a/drivers/net/ethernet/intel/e1000e/hw.h b/drivers/net/ethernet/intel/e1000e/hw.h index 69a2329ea463..f7954cadd979 100644 --- a/drivers/net/ethernet/intel/e1000e/hw.h +++ b/drivers/net/ethernet/intel/e1000e/hw.h @@ -1,8 +1,8 @@ /* SPDX-License-Identifier: GPL-2.0 */ /* Copyright(c) 1999 - 2018 Intel Corporation. */ -#ifndef _E1000_HW_H_ -#define _E1000_HW_H_ +#ifndef _E1000E_HW_H_ +#define _E1000E_HW_H_ #include "regs.h" #include "defines.h"
The include guard for the e1000e and e1000 hw.h files are the same, so add the proper "E" term to the hw.h file for the e1000e driver. This resolves some static analyzer warnings, like the one found by the "lgtm.com" tool. Cc: Jesse Brandeburg <jesse.brandeburg@intel.com> Cc: Tony Nguyen <anthony.l.nguyen@intel.com> Cc: "David S. Miller" <davem@davemloft.net> Cc: Jakub Kicinski <kuba@kernel.org> Cc: intel-wired-lan@lists.osuosl.org Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> --- drivers/net/ethernet/intel/e1000e/hw.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)