From: Adam Ford <aford173@gmail.com>
To: Andreas Kemnade <andreas@kemnade.info>
Cc: "Tony Lindgren" <tony@atomide.com>,
marcel@holtmann.org, johan.hedberg@gmail.com,
luiz.dentz@gmail.com, johan@kernel.org, arnd@arndb.de,
gregkh@linuxfoundation.org, linux-bluetooth@vger.kernel.org,
linux-kernel@vger.kernel.org, tomi.valkeinen@ideasonboard.com,
"Péter Ujfalusi" <peter.ujfalusi@gmail.com>,
robh@kernel.org
Subject: Re: [RFC PATCH 0/3] bluetooth/gnss: GNSS support for TiWi chips
Date: Fri, 8 Dec 2023 08:39:21 -0600 [thread overview]
Message-ID: <CAHCN7x+veM=izOVGEOvKpTFca53C0VhrLkscJqdcBX1riOOWbA@mail.gmail.com> (raw)
In-Reply-To: <20231127215108.6e985819@aktux>
On Mon, Nov 27, 2023 at 2:51 PM Andreas Kemnade <andreas@kemnade.info> wrote:
>
> Hi,
>
> On Mon, 27 Nov 2023 15:54:24 +0200
> Tony Lindgren <tony@atomide.com> wrote:
>
> [...]
> > > - Output at /dev/gnssX:
> > > AI2 vs. NMEA
> > > The chip can be configured into sending AI2-encapsulated NMEA,
> > > or proving data in a binary format.
> > > Some research has to be done yet for the details.
> > > A pile of logs is waiting for further analysis...
> > >
> > > Arguments for/against NMEA:
> > > + Userspace is prepared to handle it
> > > + Power management can be easily done by the kernel
> > > - Less functionality can be used.
> >
> > I'd go with NMEA format as the default setting :)
> >
> yes, that would also be my preference.
>
> > > Arguments for/against AI2:
> > > + Full functionality can be accessed from userspace (incl. A-GPS,
> > > maybe raw satellite data)
> > > - Userspace has to behave to have proper power management
> > > - No freely (not even as in beer) tool available to fully use AI2,
> > > so there will be only a real advantage after long "French Cafe"
> > > sessions.
> >
> > Seems AI2 could be optionally enabled as needed with some writes
> > to /dev/gnss0 to change the mode?
>
> Hmm, we have
> /sys/class/gnss/gnss0/type to get the mode, maybe we add some file
> to change the mode? Or having it hidden behing a module parameter
> and implement something better accessible if any need arrives?
>
> If we want NMEA output, probably some init commands will be sent at
> open(), but userspace doing something with AI2 probably does not want
> the device touched, so it should probably be already be set before open().
Is there another revision coming or should I try to test this one? I
am not very familiar with the GNSS part of the module, but it sounds
like there was some consensus as to which direction we should go. I
should have a little time this weekend.
thanks
adam
>
> Regards,
> Andreas
>
next prev parent reply other threads:[~2023-12-08 14:39 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-26 19:18 [RFC PATCH 0/3] bluetooth/gnss: GNSS support for TiWi chips Andreas Kemnade
2023-11-26 19:18 ` [RFC PATCH 1/3] gnss: Add AI2 protocol used by some TI combo chips Andreas Kemnade
2023-11-26 19:18 ` [RFC PATCH 2/3] bluetooth: ti-st: add GNSS support for TI Wilink chips Andreas Kemnade
2023-11-26 19:18 ` [RFC PATCH 3/3] drivers: misc: ti-st: begin to deorbit Andreas Kemnade
2023-11-27 8:25 ` Greg KH
2023-11-27 13:50 ` Tony Lindgren
2023-12-10 21:50 ` Andreas Kemnade
2023-11-27 13:54 ` [RFC PATCH 0/3] bluetooth/gnss: GNSS support for TiWi chips Tony Lindgren
2023-11-27 20:51 ` Andreas Kemnade
2023-12-08 14:39 ` Adam Ford [this message]
2023-12-08 17:47 ` Andreas Kemnade
2023-12-08 16:25 ` Johan Hovold
2023-12-08 22:13 ` Andreas Kemnade
2023-11-27 14:03 ` Adam Ford
2023-11-27 19:51 ` Andreas Kemnade
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAHCN7x+veM=izOVGEOvKpTFca53C0VhrLkscJqdcBX1riOOWbA@mail.gmail.com' \
--to=aford173@gmail.com \
--cc=andreas@kemnade.info \
--cc=arnd@arndb.de \
--cc=gregkh@linuxfoundation.org \
--cc=johan.hedberg@gmail.com \
--cc=johan@kernel.org \
--cc=linux-bluetooth@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=luiz.dentz@gmail.com \
--cc=marcel@holtmann.org \
--cc=peter.ujfalusi@gmail.com \
--cc=robh@kernel.org \
--cc=tomi.valkeinen@ideasonboard.com \
--cc=tony@atomide.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).