From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.6 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2FE55C04AAF for ; Tue, 21 May 2019 19:46:04 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 05F9321479 for ; Tue, 21 May 2019 19:46:04 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="fw5gIjLJ" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727471AbfEUTqD (ORCPT ); Tue, 21 May 2019 15:46:03 -0400 Received: from mail-pl1-f195.google.com ([209.85.214.195]:40780 "EHLO mail-pl1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727319AbfEUTqC (ORCPT ); Tue, 21 May 2019 15:46:02 -0400 Received: by mail-pl1-f195.google.com with SMTP id g69so8923923plb.7 for ; Tue, 21 May 2019 12:46:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=lyXh2Bb2d9vjZwPG8os8xjTMOsIMYKTtLXK80D2Apf0=; b=fw5gIjLJvlJAYHrD854Ke0JnpyHVVy0N7xYc+nEjGEzsrBwkCNj0Lb+R9q8ifyazXp ItsR12w7AYH+xnlejHn1aQKMTvP87aY4g1ti6nafELgXkX5sFecNO14vrpyi5b0htUrV fOxfYHpf9MLWky9BOxyMVZ92S1I3xE91lCt7A= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=lyXh2Bb2d9vjZwPG8os8xjTMOsIMYKTtLXK80D2Apf0=; b=C1eBa234G7zLacaD8gfg9FOuoI46tnrtauicrWM2jq1hfh2mw3NU5wD1ZfBQ/ZFbHc PQRSdrnxQ4qAxhREyNtxL9aWt1X8G2DHpEhZZir55pZ4wMxodZ96MlqTn5M6rYxHm6Zb lr1WlF46HU3VS4RLcznScmiMNmLOrlQjPC1ldf8tqar+D2eJLMChT0IDXLebvBepY+f1 bDWM1LgQ9L+0l6SJLta4nlhMZbyQTfvZKCVzsP6RvE9uRKYiVT+hxgNbZH1Q4W9RKge6 tZ6rU3WM68TWeWVw4yBofY5CEq8vjgvx09UW2ptRMYvSlH01ulWCy9OuAm9iABp3HXuT z7yA== X-Gm-Message-State: APjAAAVYtFtrOIBYWXPnIssKDQkAyNkYtgQsKGz2g2lk/8b0K1qcsp/3 vNAhQX+yZEJNWkngwtkpnrq6Yw== X-Google-Smtp-Source: APXvYqyStfrCj0r842nGnfm4OalgeT8c2Jdohtw5CB3fLZ1V3cwsjW4Nv7mqrX+HMliweGfKciYahQ== X-Received: by 2002:a17:902:8f8d:: with SMTP id z13mr17922307plo.166.1558467962156; Tue, 21 May 2019 12:46:02 -0700 (PDT) Received: from localhost ([2620:15c:202:1:75a:3f6e:21d:9374]) by smtp.gmail.com with ESMTPSA id u11sm23604007pfh.130.2019.05.21.12.46.01 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 21 May 2019 12:46:01 -0700 (PDT) Date: Tue, 21 May 2019 12:46:00 -0700 From: Matthias Kaehlcke To: Marcel Holtmann , Johan Hedberg Cc: linux-bluetooth@vger.kernel.org, linux-kernel@vger.kernel.org, Balakrishna Godavarthi , Harish Bandi , Rocky Liao Subject: Re: [PATCH v3 2/2] Bluetooth: hci_qca: wcn3990: Drop baudrate change vendor event Message-ID: <20190521194600.GI40515@google.com> References: <20190429232131.183049-1-mka@chromium.org> <20190429232131.183049-2-mka@chromium.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20190429232131.183049-2-mka@chromium.org> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Apr 29, 2019 at 04:21:31PM -0700, Matthias Kaehlcke wrote: > Firmware download to the WCN3990 often fails with a 'TLV response size > mismatch' error: > > [ 133.064659] Bluetooth: hci0: setting up wcn3990 > [ 133.489150] Bluetooth: hci0: QCA controller version 0x02140201 > [ 133.495245] Bluetooth: hci0: QCA Downloading qca/crbtfw21.tlv > [ 133.507214] Bluetooth: hci0: QCA TLV response size mismatch > [ 133.513265] Bluetooth: hci0: QCA Failed to download patch (-84) > > This is caused by a vendor event that corresponds to an earlier command > to change the baudrate. The event is not processed in the context of the > baudrate change and is later interpreted as response to the firmware > download command (which is also a vendor command), but the driver detects > that the event doesn't have the expected amount of associated data. > > More details: > > For the WCN3990 the vendor command for a baudrate change isn't sent as > synchronous HCI command, because the controller sends the corresponding > vendor event with the new baudrate. The event is received and decoded > after the baudrate change of the host port. > > Identify the 'unused' event when it is received and don't add it to > the queue of RX frames. > > Signed-off-by: Matthias Kaehlcke > --- Harish Bandi privately reported to me that he sees frame reassembly errors with this patch and WCN3998. He didn't provide any more details yet, so at this point we only know that there appears to be some difference between WCN3990 and WCN3998 wrt firmware download. For now let's limit this fix to WCN3990. We can revisit WCN3998 later if it is confirmed that it has the same problem.