2020-05-22 10:34:51

by Hari

Subject: [PATCH] Fix typo in the comment

Continuous Double "the" in a comment. Changed it to single "the"

Signed-off-by: Hari <[email protected]>
---
drivers/net/ethernet/intel/e1000/e1000_hw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/intel/e1000/e1000_hw.c b/drivers/net/ethernet/intel/e1000/e1000_hw.c
index 48428d6a00be..623e516a9630 100644
--- a/drivers/net/ethernet/intel/e1000/e1000_hw.c
+++ b/drivers/net/ethernet/intel/e1000/e1000_hw.c
@@ -3960,7 +3960,7 @@ static s32 e1000_do_read_eeprom(struct e1000_hw *hw, u16 offset, u16 words,
* @hw: Struct containing variables accessed by shared code
*
* Reads the first 64 16 bit words of the EEPROM and sums the values read.
- * If the the sum of the 64 16 bit words is 0xBABA, the EEPROM's checksum is
+ * If the sum of the 64 16 bit words is 0xBABA, the EEPROM's checksum is
* valid.
*/
s32 e1000_validate_eeprom_checksum(struct e1000_hw *hw)
--
2.17.1

2020-05-23 07:15:30

by Jeff Kirsher

Subject: RE: [Intel-wired-lan] [PATCH] Fix typo in the comment

> -----Original Message-----
> From: Intel-wired-lan <[email protected]> On Behalf Of Hari
> Sent: Friday, May 22, 2020 03:30
> To: [email protected]; [email protected]
> Cc: [email protected]; [email protected]; linux-
> [email protected]; Hari <[email protected]>
> Subject: [Intel-wired-lan] [PATCH] Fix typo in the comment
[Kirsher, Jeffrey T]

Please fix the title to "e1000: Fix typo in the comment", other than that your patch looks fine and I will add it to my queue after the title of your patch is fixed.

>
> Continuous Double "the" in a comment. Changed it to single "the"
>
> Signed-off-by: Hari <[email protected]>
> ---
> drivers/net/ethernet/intel/e1000/e1000_hw.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/intel/e1000/e1000_hw.c
> b/drivers/net/ethernet/intel/e1000/e1000_hw.c
> index 48428d6a00be..623e516a9630 100644
> --- a/drivers/net/ethernet/intel/e1000/e1000_hw.c
> +++ b/drivers/net/ethernet/intel/e1000/e1000_hw.c
> @@ -3960,7 +3960,7 @@ static s32 e1000_do_read_eeprom(struct e1000_hw
> *hw, u16 offset, u16 words,
> * @hw: Struct containing variables accessed by shared code
> *
> * Reads the first 64 16 bit words of the EEPROM and sums the values read.
> - * If the the sum of the 64 16 bit words is 0xBABA, the EEPROM's checksum is
> + * If the sum of the 64 16 bit words is 0xBABA, the EEPROM's checksum is
> * valid.
> */
> s32 e1000_validate_eeprom_checksum(struct e1000_hw *hw)
> --
> 2.17.1
>
> _______________________________________________
> Intel-wired-lan mailing list
> [email protected]
> https://lists.osuosl.org/mailman/listinfo/intel-wired-lan

2020-05-28 22:56:46

by Brown, Aaron F

Subject: RE: [PATCH] Fix typo in the comment

> From: Hari <[email protected]>
> Sent: Friday, May 22, 2020 3:30 AM
> To: [email protected]; [email protected]
> Cc: Hari <[email protected]>; [email protected]; linux-
> [email protected]; [email protected]
> Subject: [PATCH] Fix typo in the comment
>
> Continuous Double "the" in a comment. Changed it to single "the"
>
> Signed-off-by: Hari <[email protected]>
> ---
> drivers/net/ethernet/intel/e1000/e1000_hw.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Tested-by: Aaron Brown <[email protected]>