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=-0.9 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS autolearn=unavailable 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 51EE8C04A6B for ; Wed, 8 May 2019 20:58:30 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 2BC8E2173B for ; Wed, 8 May 2019 20:58:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727911AbfEHU6Y convert rfc822-to-8bit (ORCPT ); Wed, 8 May 2019 16:58:24 -0400 Received: from coyote.holtmann.net ([212.227.132.17]:35652 "EHLO mail.holtmann.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727097AbfEHU6Y (ORCPT ); Wed, 8 May 2019 16:58:24 -0400 Received: from marcel-macpro.fritz.box (p4FF9FD9B.dip0.t-ipconnect.de [79.249.253.155]) by mail.holtmann.org (Postfix) with ESMTPSA id 34B19CEFAA; Wed, 8 May 2019 23:06:37 +0200 (CEST) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.8\)) Subject: Re: [PATCH 00/14] Add support for FM radio in hcill and kill TI_ST From: Marcel Holtmann In-Reply-To: Date: Wed, 8 May 2019 22:58:21 +0200 Cc: Sebastian Reichel , Hans Verkuil , Mauro Carvalho Chehab , Tony Lindgren , Rob Herring , Greg Kroah-Hartman , Pavel Machek , "open list:BLUETOOTH DRIVERS" , linux-media@vger.kernel.org, Linux-OMAP , Linux Kernel Mailing List Content-Transfer-Encoding: 8BIT Message-Id: References: <20181221011752.25627-1-sre@kernel.org> <4f47f7f2-3abb-856c-4db5-675caf8057c7@xs4all.nl> <20190319133154.7tbfafy7pguzw2tk@earth.universe> To: Adam Ford X-Mailer: Apple Mail (2.3445.104.8) Sender: linux-bluetooth-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-bluetooth@vger.kernel.org Hi Adam, >>>>> This moves all remaining users of the legacy TI_ST driver to hcill (patches >>>>> 1-3). Then patches 4-7 convert wl128x-radio driver to a standard platform >>>>> device driver with support for multiple instances. Patch 7 will result in >>>>> (userless) TI_ST driver no longer supporting radio at runtime. Patch 8-11 do >>>>> some cleanups in the wl128x-radio driver. Finally patch 12 removes the TI_ST >>>>> specific parts from wl128x-radio and adds the required infrastructure to use it >>>>> with the serdev hcill driver instead. The remaining patches 13 and 14 remove >>>>> the old TI_ST code. >>>>> >>>>> The new code has been tested on the Motorola Droid 4. For testing the audio >>>>> should be configured to route Ext to Speaker or Headphone. Then you need to >>>>> plug headphone, since its cable is used as antenna. For testing there is a >>>>> 'radio' utility packages in Debian. When you start the utility you need to >>>>> specify a frequency, since initial get_frequency returns an error: >>>> >>>> What is the status of this series? >>>> >>>> Based on some of the replies (from Adam Ford in particular) it appears that >>>> this isn't ready to be merged, so is a v2 planned? >>> >>> Yes, a v2 is planned, but I'm super busy at the moment. I don't >>> expect to send something for this merge window. Neither LogicPD >>> nor IGEP use FM radio, so I can just remove FM support from the >>> TI_ST framework. Converting those platforms to hci_ll can be done >>> in a different patchset. >>> >>> If that was the only issue there would be a v2 already. But Marcel >>> Holtmann suggested to pass the custom packet data through the BT >>> subsystem, which is non-trivial (at least for me) :) >> >> I am running some tests today on the wl1283-st on the Logic PD Torpedo >> board. Tony had suggested a few options, so I'm going to try those. >> Looking at those today. If/when you have a V2, please CC me on it. If >> it's been posted, can you send me a link? I would really like to see >> the st-kim driver go away so I'd like to resolve the issues with the >> torpedo board. > > I have run a bunch of tests on the 5.1 kernel. I am able to get the > firmware to load now and the hci0 goes up. I was able to establish a > BLE connection to a TI Sensor Tag and read and write data to it with > good success on the wl1283. > > Unfortunately, when I tried to do some more extensive testing over > classic Bluetooth, I got an error that repeats itself at seemingly > random intervals: > Bluetooth: hci0: Frame reassembly failed (-84) > > I can still scan and pair, but these Frame reassembly failed errors > appear to come and go. there are only 3 places in h4_recv_buf that return EILSEQ. Just add an extra printk to these to figure out which one it is. Maybe it is just extra packet types that we need to handle. If it is not the packet type one, print what packet we have that is causing this. Regards Marcel