Return-path: Received: from mail-pz0-f66.google.com ([209.85.210.66]:37904 "EHLO mail-pz0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750887Ab1AAHnJ (ORCPT ); Sat, 1 Jan 2011 02:43:09 -0500 Message-ID: <4D1EDB22.2020308@gmail.com> Date: Fri, 31 Dec 2010 23:43:30 -0800 From: "Justin P. Mattock" MIME-Version: 1.0 To: Finn Thain CC: trivial@kernel.org, linux-m68k@lists.linux-m68k.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, ivtv-devel@ivtvdriver.org, linux-media@vger.kernel.org, linux-wireless@vger.kernel.org, linux-scsi@vger.kernel.org, spi-devel-general@lists.sourceforge.net, devel@driverdev.osuosl.org, linux-usb@vger.kernel.org Subject: Re: [PATCH 03/15]drivers:staging:rtl8187se:r8180_hw.h Typo change diable to disable. References: <1293750484-1161-1-git-send-email-justinmattock@gmail.com> <1293750484-1161-2-git-send-email-justinmattock@gmail.com> <1293750484-1161-3-git-send-email-justinmattock@gmail.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 12/31/2010 10:48 PM, Finn Thain wrote: > > On Thu, 30 Dec 2010, Justin P. Mattock wrote: > >> The below patch fixes a typo "diable" to "disable". Please let me know if this >> is correct or not. >> >> Signed-off-by: Justin P. Mattock >> >> --- >> drivers/staging/rtl8187se/r8180_hw.h | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/drivers/staging/rtl8187se/r8180_hw.h b/drivers/staging/rtl8187se/r8180_hw.h >> index 3fca144..2911d40 100644 >> --- a/drivers/staging/rtl8187se/r8180_hw.h >> +++ b/drivers/staging/rtl8187se/r8180_hw.h >> @@ -554,7 +554,7 @@ >> /* by amy for power save */ >> /* by amy for antenna */ >> #define EEPROM_SW_REVD_OFFSET 0x3f >> -/* BIT[8-9] is for SW Antenna Diversity. Only the value EEPROM_SW_AD_ENABLE means enable, other values are diable. */ >> +/* BIT[8-9] is for SW Antenna Diversity. Only the value EEPROM_SW_AD_ENABLE means enable, other values are disabled. */ > > I think, "other values disable" was what you meant? > > Finn > >> #define EEPROM_SW_AD_MASK 0x0300 >> #define EEPROM_SW_AD_ENABLE 0x0100 >> >> > no! I changed it to disabled to make it proper.. Justin P. Mattock