From patchwork Tue Feb 11 07:37:06 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bartosz Markowski X-Patchwork-Id: 3625351 Return-Path: X-Original-To: patchwork-linux-wireless@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id CA6F5BF418 for ; Tue, 11 Feb 2014 07:38:15 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id A392E200D6 for ; Tue, 11 Feb 2014 07:38:10 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 13B5B201FE for ; Tue, 11 Feb 2014 07:38:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752750AbaBKHh4 (ORCPT ); Tue, 11 Feb 2014 02:37:56 -0500 Received: from mail-ea0-f169.google.com ([209.85.215.169]:44500 "EHLO mail-ea0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752740AbaBKHhx (ORCPT ); Tue, 11 Feb 2014 02:37:53 -0500 Received: by mail-ea0-f169.google.com with SMTP id h10so3498585eak.14 for ; Mon, 10 Feb 2014 23:37:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tieto.com; s=google; h=from:to:cc:subject:date:message-id; bh=XBeVzsAXRiyudD8VZBw7Xkm7GRKxcJiQzBtv1lbjW0s=; b=QEaMmM/YKYzhGcrXPhDtfH8TagyCKY1FevqkVbnQRepMEkl5v5ORJR1VmSbSfn93Hb tQGlmJeno4fEQ9dgtq/OmwmoWRiCkhjvrnMMMxpjApJCOIwRdzuFIF+xFUK8bfq974us V0ylJqB3ua+Kf4kHRESycLha7Nmn9WLDrHSIw= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=XBeVzsAXRiyudD8VZBw7Xkm7GRKxcJiQzBtv1lbjW0s=; b=VGgv9re7PzM0LIzC5FA5++CM55UVBlFwF/QsZ5fcbBdbldKvLyuaKDB0JCF+nj2eDk zn+uKc4FLOdUpmibxTS/GTKpGrSq6/EVqDl1qUhAVTFiPmd/KaIVq7Lp2K+Css89e/FK BnpRfwSk70cOzYTIdtuD/IThreBSEXQ9wjlgaKZi8qy7y9AFW4w8sOuUnfUos1ws5cFN tNo0PLkXlwsj4AgzPdQLvYfdtZdpW/ZtLUw3nGLOzMFLih1fQFZygpWyeoZUf4PKGVyv phweV2tfHDtJKjZPDtyC0JLaof6RRQ/JeMz6kTE626xHC2vZ8rE7+JNOicYKqxwZfqgD en2g== X-Gm-Message-State: ALoCoQlJueJM0JXV3Qyqxh9ZJwOqZsEmatscYJKaweXh/TbO1z0KIa47qYSossZMBO4pNswqgzKv/bYgkHXjzXi87jloGiLAsM5AORgDBmOVDmW5uKm7ngg= X-Received: by 10.14.39.3 with SMTP id c3mr42078484eeb.42.1392104271704; Mon, 10 Feb 2014 23:37:51 -0800 (PST) Received: from uw000975.eu.tieto.com. ([91.198.246.10]) by mx.google.com with ESMTPSA id 46sm63859102ees.4.2014.02.10.23.37.50 for (version=TLSv1.1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Mon, 10 Feb 2014 23:37:51 -0800 (PST) From: Bartosz Markowski To: ath10k@lists.infradead.org Cc: linux-wireless@vger.kernel.org, Bartosz Markowski Subject: [PATCH] ath10k: remove excessive rx msdu len check Date: Tue, 11 Feb 2014 08:37:06 +0100 Message-Id: <1392104226-12096-1-git-send-email-bartosz.markowski@tieto.com> X-Mailer: git-send-email 1.7.10 X-DomainID: tieto.com Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org X-Spam-Status: No, score=-7.4 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,RP_MATCHES_RCVD,T_DKIM_INVALID,UNPARSEABLE_RELAY autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP This throw a lot of pointless warnings in case of DFS (radar detection) and PHYERR events from firmware, when firmware may actually insert more data, than we assume. Besides of being noisy this debug does not protect or check anything usefull currently. It was introduced long time ago while debugging aggregations. So just removing it. Signed-off-by: Bartosz Markowski --- drivers/net/wireless/ath/ath10k/htt_rx.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/htt_rx.c b/drivers/net/wireless/ath/ath10k/htt_rx.c index fe8bd1b..f57f5cf 100644 --- a/drivers/net/wireless/ath/ath10k/htt_rx.c +++ b/drivers/net/wireless/ath/ath10k/htt_rx.c @@ -430,12 +430,6 @@ static int ath10k_htt_rx_amsdu_pop(struct ath10k_htt *htt, msdu_chaining = 1; } - if (msdu_len > 0) { - /* This may suggest FW bug? */ - ath10k_warn("htt rx msdu len not consumed (%d)\n", - msdu_len); - } - last_msdu = __le32_to_cpu(rx_desc->msdu_end.info0) & RX_MSDU_END_INFO0_LAST_MSDU;