Re: [PATCH v4 0/4] Bluetooth/gnss: GNSS support for TiWi chips

From: Andreas Kemnade
Date: Thu Jun 06 2024 - 16:19:57 EST


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.
> >
> > 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