Return-path: Received: from mms1.broadcom.com ([216.31.210.17]:2703 "EHLO mms1.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754355Ab2KSSsi (ORCPT ); Mon, 19 Nov 2012 13:48:38 -0500 Message-ID: <50AA7ED9.6030803@broadcom.com> (sfid-20121119_194841_094180_794E6477) Date: Mon, 19 Nov 2012 19:47:53 +0100 From: "Arend van Spriel" MIME-Version: 1.0 To: "Seth Forshee" cc: linux-wireless@vger.kernel.org, "John W. Linville" , "Franky (Zhenhui) Lin" , "Brett Rudley" , "Roland Vossen" , "Kan Yan" , brcm80211-dev-list@broadcom.com, "Daniel Wagner" Subject: Re: [PATCH v2 08/22] brcmsmac: Reduce number of entries in tx DMA rings References: <1352988492-21340-1-git-send-email-seth.forshee@canonical.com> <1352988492-21340-9-git-send-email-seth.forshee@canonical.com> In-Reply-To: <1352988492-21340-9-git-send-email-seth.forshee@canonical.com> Content-Type: text/plain; charset=iso-8859-1; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 11/15/2012 03:07 PM, Seth Forshee wrote: > Currently up to 256 frames can be queued for each DMA ring. This is > excessive, and now that we have better flow control we can get by with > less. Experimentation has shown 64 to work well. Reviewed-by: Pieter-Paul Giesberts Reviewed-by: Arend van Spriel > Signed-off-by: Seth Forshee > --- > drivers/net/wireless/brcm80211/brcmsmac/main.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-)