2010-03-17 19:45:13

by John W. Linville

Subject: [PATCH] mac80211: correct typos in "unavailable upon resume" warning

Signed-off-by: John W. Linville <[email protected]>
---
net/mac80211/util.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/mac80211/util.c b/net/mac80211/util.c
index c453226..a9d9098 100644
--- a/net/mac80211/util.c
+++ b/net/mac80211/util.c
@@ -1097,8 +1097,8 @@ int ieee80211_reconfig(struct ieee80211_local *local)
*/
res = drv_start(local);
if (res) {
- WARN(local->suspended, "Harware became unavailable "
- "upon resume. This is could be a software issue"
+ WARN(local->suspended, "Hardware became unavailable "
+ "upon resume. This is could be a software issue "
"prior to suspend or a hardware issue\n");
return res;
}
--
1.6.2.5

2010-03-20 13:00:11

by John W. Linville

Subject: Re: [PATCH] mac80211: correct typos in "unavailable upon resume" warning

On Sat, Mar 20, 2010 at 10:09:38AM +0000, Dave wrote:
> John W. Linville wrote:
> > Signed-off-by: John W. Linville <[email protected]>
> > ---
> > net/mac80211/util.c | 4 ++--
> > 1 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/net/mac80211/util.c b/net/mac80211/util.c
> > index c453226..a9d9098 100644
> > --- a/net/mac80211/util.c
> > +++ b/net/mac80211/util.c
> > @@ -1097,8 +1097,8 @@ int ieee80211_reconfig(struct ieee80211_local *local)
> > */
> > res = drv_start(local);
> > if (res) {
> > - WARN(local->suspended, "Harware became unavailable "
> > - "upon resume. This is could be a software issue"
> > + WARN(local->suspended, "Hardware became unavailable "
> > + "upon resume. This is could be a software issue "
>
> ^^
>
> As long as you're addressing that line, you probably want to lose the
> 'is'. Or the 'could be'.

Yes, thanks. Obviously I suck as a proof-reader...

John
--
John W. Linville Someday the world will need a hero, and you
[email protected] might be all we have. Be ready.

2010-03-20 10:09:49

by Dave Kilroy

Subject: Re: [PATCH] mac80211: correct typos in "unavailable upon resume" warning

John W. Linville wrote:
> Signed-off-by: John W. Linville <[email protected]>
> ---
> net/mac80211/util.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/net/mac80211/util.c b/net/mac80211/util.c
> index c453226..a9d9098 100644
> --- a/net/mac80211/util.c
> +++ b/net/mac80211/util.c
> @@ -1097,8 +1097,8 @@ int ieee80211_reconfig(struct ieee80211_local *local)
> */
> res = drv_start(local);
> if (res) {
> - WARN(local->suspended, "Harware became unavailable "
> - "upon resume. This is could be a software issue"
> + WARN(local->suspended, "Hardware became unavailable "
> + "upon resume. This is could be a software issue "

^^

As long as you're addressing that line, you probably want to lose the
'is'. Or the 'could be'.

> "prior to suspend or a hardware issue\n");
> return res;
> }