On Thu, Jun 6, 2024 at 3:19 PM Andreas Kemnade <andreas@xxxxxxxxxxxx> wrote: > > Hi Luiz, > > On Thu, 6 Jun 2024 16:04:10 -0400 > Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> wrote: > > > Hi Andreas, > > > > On Thu, Jun 6, 2024 at 2:30 PM Andreas Kemnade <andreas@xxxxxxxxxxxx> wrote: > > > > > > Some of these chips have GNSS support. In some vendor kernels > > > a driver on top of misc/ti-st can be found providing a /dev/tigps > > > device which speaks the secretive Air Independent Interface (AI2) protocol. I think you may have sent me a file to test, but I can't find the e-mail. Can you tell me what tool you used to test it? I can get gnss0 to enumerate, so I am close. [ 20.759857] hci-ti serial0-0: using DT '/ocp@68000000/serial@4806c000/bluetooth-gnss' for 'enable' GPIO lookup [ 20.770263] of_get_named_gpiod_flags: parsed 'enable-gpios' property of node '/ocp@68000000/serial@4806c000/bluetooth-gnss[0]' - status (0) [ 29.221588] gnss: GNSS driver registered with major 244 adam > > > > > > To be more compatible with userspace send out NMEA by default but > > > allow a more raw mode by using a module parameter. > > > > > > This was tested on the Epson Moverio BT-200. > > > > > > Who will take this series (1-3)? GNSS with ack from Bluetooth? > > > > > > Changes since V3: > > > - Finally remove the period from 1/4 subject > > > - include things directly for get_unaligned_le16() to fix 0-day issues > > > > > > Changes since V2: > > > - Optimize waits > > > - Fix some packet analysis / checksum computation issue > > > - Adding a proposal for removing those waits as RFC > > > - Minor spell corrections and improved descriptions > > > > > > Changes since V1: > > > - Set up things for NMEA output > > > - Powerup/down at open()/close() > > > - split out logic between drivers/bluetooth and drivers/gnss > > > - leave out drivers/misc/ti-st driver removal to avoid > > > filling up mailboxes during the iterations, this series is > > > still a proof that it is not needed, will take the brush after > > > this series is accepted. > > > > > > > > > Andreas Kemnade (4): > > > gnss: Add AI2 protocol used by some TI combo chips > > > Bluetooth: ti-st: Add GNSS subdevice for TI Wilink chips > > > > The bluetooth one looks relatively simple so I could take that one and > > push to bluetooth-next if there are no dependencies on the other > > changes. > > > There is: > > include/linux/ti_wilink_st.h | 8 + > > We have compile time deps here. Patch 3 compile time depends on patch 2. If we > cannot take everything in for 6.11, you might opt to take the bluetooth part. > That would work. > > Regards, > Andreas
