Message ID | 20200522103024.9697-1-harichandrakanthan@gmail.com |
---|---|
State | Changes Requested |
Headers | show |
Series | Fix typo in the comment | expand |
> -----Original Message----- > From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of Hari > Sent: Friday, May 22, 2020 03:30 > To: davem@davemloft.net; kuba@kernel.org > Cc: netdev@vger.kernel.org; intel-wired-lan@lists.osuosl.org; linux- > kernel@vger.kernel.org; Hari <harichandrakanthan@gmail.com> > 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 <harichandrakanthan@gmail.com> > --- > 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 > Intel-wired-lan@osuosl.org > https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
> From: Hari <harichandrakanthan@gmail.com> > Sent: Friday, May 22, 2020 3:30 AM > To: davem@davemloft.net; kuba@kernel.org > Cc: Hari <harichandrakanthan@gmail.com>; netdev@vger.kernel.org; linux- > kernel@vger.kernel.org; intel-wired-lan@lists.osuosl.org > Subject: [PATCH] Fix typo in the comment > > Continuous Double "the" in a comment. Changed it to single "the" > > Signed-off-by: Hari <harichandrakanthan@gmail.com> > --- > drivers/net/ethernet/intel/e1000/e1000_hw.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Tested-by: Aaron Brown <aaron.f.brown@intel.com>
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)
Continuous Double "the" in a comment. Changed it to single "the" Signed-off-by: Hari <harichandrakanthan@gmail.com> --- drivers/net/ethernet/intel/e1000/e1000_hw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)