2012-10-23 06:22:55

by Justin P. Mattock

[permalink] [raw]
Subject: [PATCH 1/2]staging: "ehci-w90x900" Fix typos.

From: "Justin P. Mattock" <[email protected]>

Signed-off-by: Justin P. Mattock <[email protected]>

---

The below patch fixes a typo in staging: "ehci-w90x900"

drivers/usb/host/ehci-w90x900.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/host/ehci-w90x900.c b/drivers/usb/host/ehci-w90x900.c
index ec59808..a9c28a0 100644
--- a/drivers/usb/host/ehci-w90x900.c
+++ b/drivers/usb/host/ehci-w90x900.c
@@ -13,7 +13,7 @@

#include <linux/platform_device.h>

-/*ebable phy0 and phy1 for w90p910*/
+/*enable phy0 and phy1 for w90p910*/
#define ENPHY (0x01<<8)
#define PHY0_CTR (0xA4)
#define PHY1_CTR (0xA8)
--
1.7.10.4


2012-10-23 06:23:00

by Justin P. Mattock

[permalink] [raw]
Subject: [PATCH 2/2]staging: "winbond" Fix typos.

From: "Justin P. Mattock" <[email protected]>

Signed-off-by: Justin P. Mattock <[email protected]>

---

The below patch fixes a typo in staging: "winbond"

drivers/staging/winbond/mds.c | 2 +-
drivers/staging/winbond/wbhal.h | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/winbond/mds.c b/drivers/staging/winbond/mds.c
index 1b8b8ac..43990e8 100644
--- a/drivers/staging/winbond/mds.c
+++ b/drivers/staging/winbond/mds.c
@@ -315,7 +315,7 @@ static void Mds_HeaderCopy(struct wbsoft_priv *adapter, struct wb35_descriptor *

pT00->T00_tx_packet_id = pDes->Descriptor_ID; /* Set packet ID */
pT00->T00_header_length = 24; /* Set header length */
- pT01->T01_retry_abort_ebable = 1; /* 921013 931130.5.h */
+ pT01->T01_retry_abort_enable = 1; /* 921013 931130.5.h */

/* Key ID setup */
pT01->T01_wep_id = 0;
diff --git a/drivers/staging/winbond/wbhal.h b/drivers/staging/winbond/wbhal.h
index 39e84a0..289ee54 100644
--- a/drivers/staging/winbond/wbhal.h
+++ b/drivers/staging/winbond/wbhal.h
@@ -226,11 +226,11 @@ struct T01_descriptor {
u32 T01_add_challenge_text:1;
u32 T01_inhibit_crc:1;
u32 T01_loop_back_wep_mode:1;
- u32 T01_retry_abort_ebable:1;
+ u32 T01_retry_abort_enable:1;
};
#else
struct {
- u32 T01_retry_abort_ebable:1;
+ u32 T01_retry_abort_enable:1;
u32 T01_loop_back_wep_mode:1;
u32 T01_inhibit_crc:1;
u32 T01_add_challenge_text:1;
--
1.7.10.4

2012-10-23 14:13:06

by Alan Stern

[permalink] [raw]
Subject: Re: [PATCH 1/2]staging: "ehci-w90x900" Fix typos.

On Mon, 22 Oct 2012, Justin P. Mattock wrote:

> From: "Justin P. Mattock" <[email protected]>
>
> Signed-off-by: Justin P. Mattock <[email protected]>
>
> ---
>
> The below patch fixes a typo in staging: "ehci-w90x900"

What does ehci-w90x900 have to do with staging?

> drivers/usb/host/ehci-w90x900.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/usb/host/ehci-w90x900.c b/drivers/usb/host/ehci-w90x900.c
> index ec59808..a9c28a0 100644
> --- a/drivers/usb/host/ehci-w90x900.c
> +++ b/drivers/usb/host/ehci-w90x900.c
> @@ -13,7 +13,7 @@
>
> #include <linux/platform_device.h>
>
> -/*ebable phy0 and phy1 for w90p910*/
> +/*enable phy0 and phy1 for w90p910*/

Why bother to correct the spelling without also correcting the missing
whitespace?

> #define ENPHY (0x01<<8)
> #define PHY0_CTR (0xA4)
> #define PHY1_CTR (0xA8)

Alan Stern