Return-path: Received: from coyote.holtmann.net ([212.227.132.17]:57920 "EHLO mail.holtmann.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751405AbeBUI1B (ORCPT ); Wed, 21 Feb 2018 03:27:01 -0500 Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 11.2 \(3445.5.20\)) Subject: Re: [RFC PATCH 0/2] net: rfkill: gpio: Fix and support SerDev From: Marcel Holtmann In-Reply-To: Date: Wed, 21 Feb 2018 09:26:59 +0100 Cc: Hans de Goede , Jeremy Cline , Carlo Caione , Carlo Caione , Johannes Berg , linux-wireless@vger.kernel.org, =?utf-8?Q?Fr=C3=A9d=C3=A9ric_Danis?= , Sebastian Reichel , "Rafael J. Wysocki" , Linux Upstreaming Team Message-Id: <0F67E310-008B-4E82-91C8-6316F5FB3DF4@holtmann.org> (sfid-20180221_092706_024487_53B4F1E9) References: <20180220134618.12972-1-carlo@caione.org> <31f0157f-4778-2f9c-977f-8bdbba6f52a4@redhat.com> To: Martin Blumenstingl Sender: linux-wireless-owner@vger.kernel.org List-ID: Hi Martin, >>>> Is this for devices with a RTL8723BS chip? If so then they >>>> still will not work after this since there also no longer is >>>> a /dev/ttyS4 created for the UART for the bluetooth, instead >>>> you probably want: >>>> >>>> >>>> https://github.com/jwrdegoede/linux-sunxi/commit/c383dac5ea00738ac01d704d820aa548494fcd41 >>>> >>>> Which also puts the /dev/ttyS4 back in place. >>> >>> >>> Yeah, this problem came up while working on the RTL8723BS chip but the >>> driver is also broken for the other two GPS devices supported by this >>> driver. >>> Thank you for the patch BTW. >>> >>>> Regards, >>>> >>>> Hans >>>> >>>> p.s. >>>> >>>> My college Jeremy Cline in the Cc is looking into getting proper >>>> bluetooth support in place for the rtl8723bs using serdev binding >>>> and having everything in the kernel, as we now already do for bcm >>>> uart bluetooth modules. >>> >>> >>> Wasn't also Martin (+CC) working on this? See >>> https://www.spinics.net/lists/linux-bluetooth/msg73594.html > thank you for CC'ing me Carlo! > >> Ah I did not know that, cool. Jeremy, this is probably a good starting >> point :) And you should probably coordinate with Martin on this. > the status on this is: > - Marcel wrote a tool to parse the config blob: [0] > - the first patch from my series called "serdev: implement parity > configuration" is now obsolete because an improved version from Ulrich > Hecht has been merged: [1] (this requires a trivial change to the > "serdev_device_set_parity" call in patch #9 of my series) > - I still have Realtek serdev support on my TODO-list but with low priority > > there was a discussion what has to be done to drop the "RFC" prefix > from my series: [3] > the quick summary (from my point of view): > - if possible we should get rid of the config blob (don't use it at > all or generate it in-memory - I couldn't make either of these work so > far but I've not spent much time on it either) so I have no idea what the config blobs are actually doing. However I am afraid they are needed since they are changing setting that should have been in the ROM mask, but they aren’t. What would be interesting to find the vendor command to read out the memory area and match it to the config file. > - create a "library" for the H5 protocol (similar to the H4 protocol) > so the Realtek code doesn't have to be part of hci_h5.c I would prefer to have that from the beginning, but I would accept incremental patches once Realtek support is merged. > - add ACPI support (and not just device-tree support) That should be as simple as just adding the PNPIDs to the driver. > - testing with existing Realtek USB devices is needed > > I have successfully tested v2 of my series on two Amlogic boards I > have which both come with a RTL8723BS SDIO wifi/UART Bluetooth combo > chip. > that said, I only looked at Bluetooth support (I didn't test wifi or > btcoex support) and I don't have any "Realtek USB Bluetooth" dongles > to check that I didn't break support for the existing devices > > @Jeremy: I definitely won't be able to update my patches for the v4.17 > cycle (and I'm not sure how much time I can dedicate to this for the > v4.18 cycle). > it would be great if you could keep me CC'ed on your patches so I can > learn and test them on the boards I have Regards Marcel