From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750939AbbEYPcd (ORCPT ); Mon, 25 May 2015 11:32:33 -0400 Received: from mail.windriver.com ([147.11.1.11]:54494 "EHLO mail.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750727AbbEYPcc (ORCPT ); Mon, 25 May 2015 11:32:32 -0400 Message-ID: <5563406A.3010202@windriver.com> Date: Mon, 25 May 2015 11:31:54 -0400 From: Paul Gortmaker User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: tolga ceylan , Greg Kroah-Hartman , Cristina Opriceana , Haneen Mohammed , Greg Donald , Aya Mahfouz , Benoit Taine , , Subject: Re: [PATCH 1/1] drivers: staging: rtl8192u: ieee80211: ieee80211_softmac.c: auth parse error code byte order fix References: <1431489344-1206-1-git-send-email-tolga.ceylan@gmail.com> <55613D28.90609@gmail.com> In-Reply-To: <55613D28.90609@gmail.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [128.224.56.57] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 15-05-23 10:53 PM, tolga ceylan wrote: > On 05/12/2015 08:55 PM, Tolga Ceylan wrote: >> Error code returned from auth_parse() should in cpu byte order. >> >> Signed-off-by: Tolga Ceylan >> --- >> drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c >> index d2e8b12..0cf0e89 100644 >> --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c >> +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c >> @@ -1558,7 +1558,7 @@ static inline u16 auth_parse(struct sk_buff *skb, u8 **challenge, int *chlen) >> } >> } >> >> - return cpu_to_le16(a->status); >> + return le16_to_cpu(a->status); >> >> } >> >> > > I haven't gotten a response for this patch. Just checking back. In the future, you can help speed up the review process by helping the reviewer/maintainer. How? In your commit log. Describe the end user symptoms and how the problem was found, why it happens and what is the best fix for the problem (assuming it isn't trivially obvious). So, instead of just translating the C change into words for a commit log like you had above, you could have instead had... When trying to use an RL8192 with auth mode foo and module parameter blah blah, the driver fails to connect, only showing blah blah in the logs. This happens because auth_parse blah blah and returns blah which is incorrectly interpreted by foo_bar() since it is in the wrong endian as blah is in native byte order and ... We fix it by noting the error value should be ... since ... Also, minor nit ; if you are going to put a path fragment in the subject, then don't replace all the / with ": " --- i.e. use [PATCH] staging: rtl8192u/ieee80211/ieee80211_softmac.c: auth parse error code byte order fix or [PATCH] staging/rtl8192u: ieee80211_softmac.c - auth parse error code byte order fix Paul. -- > > Regards, > Tolga Ceylan >