| Message ID | 20150505214851.GA3013@bytefire-computer (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Kalle Valo |
| Headers | show |
On Tue, 2015-05-05 at 22:48 +0100, Okash Khawaja wrote: > The PCI cache line size value, stored in cline variable, was being > compared against decimal values written as hex literals. This patch > changes those literals to correct hex values. This is fine. The grammar might be better as something like: The PCI cache line size value was being compared against decimal values prefixed with 0x. Fix the literals to use the correct hex values. > Please note that this patch is on top of > another code cleanup patch '[PATCH] adm8211: fix checkpatch errors for > indentation and new line around switch-case'. But this information doesn't belong here as it would also be in a git log when someone uses git am to apply it to a tree. It'd be better to send a patch series using "git format-email --cover-letter -2 -o <dir>" and use git send-email for the series. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, May 05, 2015 at 03:01:53PM -0700, Joe Perches wrote: > On Tue, 2015-05-05 at 22:48 +0100, Okash Khawaja wrote: > > The PCI cache line size value, stored in cline variable, was being > > compared against decimal values written as hex literals. This patch > > changes those literals to correct hex values. > > This is fine. > > The grammar might be better as something like: > > The PCI cache line size value was being compared against > decimal values prefixed with 0x. > > Fix the literals to use the correct hex values. > > > Please note that this patch is on top of > > another code cleanup patch '[PATCH] adm8211: fix checkpatch errors for > > indentation and new line around switch-case'. > > But this information doesn't belong here as > it would also be in a git log when someone > uses git am to apply it to a tree. > > It'd be better to send a patch series using > "git format-email --cover-letter -2 -o <dir>" > and use git send-email for the series. > > This is my first time going throug the process of submitting patches, something you may have already noticed. I have created and submitted patch series.Thanks -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/wireless/adm8211.c b/drivers/net/wireless/adm8211.c index 058fb4b..76c908f 100644 --- a/drivers/net/wireless/adm8211.c +++ b/drivers/net/wireless/adm8211.c @@ -1101,10 +1101,10 @@ static void adm8211_hw_init(struct ieee80211_hw *dev) case 0x8: reg |= (0x1 << 14); break; - case 0x16: + case 0x10: reg |= (0x2 << 14); break; - case 0x32: + case 0x20: reg |= (0x3 << 14); break; default:
The PCI cache line size value, stored in cline variable, was being compared against decimal values written as hex literals. This patch changes those literals to correct hex values. Please note that this patch is on top of another code cleanup patch '[PATCH] adm8211: fix checkpatch errors for indentation and new line around switch-case'. Signed-off-by: Okash Khawaja <okash.khawaja@gmail.com> --- drivers/net/wireless/adm8211.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)