Re: [PATCH 0/7] gnss: add new GNSS subsystem

From: Marcel Holtmann
Date: Wed Apr 25 2018 - 02:49:59 EST

Hi Pavel,

>> This series adds a new subsystem for GNSS receivers (e.g. GPS
>> receivers).
> Actually... I'd just call it GPS subsystem. Yes, GPS is a bit
> misleading, but so is GNSS. We'd like Loran to use similar interface,
> right? We'd like to QZSS to use similar interface, tooâ

GNSS is the term that is known and used by everybody in the industry to identify GPS, GLONASS, Galileo etc.

> . QZSS is not _global_ positioning system. Still they call it GPS. I'd
> call it GPS too.
> (Alternatively, we could have drivers/position and /dev/pos0...)
> Looking closer... I'm not sure if it makes sense to push different
> protocols (SiRF, NMEA, ...) through one device. Userland should know
> what protocol to expect... Yes, there will be common code between
> /dev/nmea0 and /dev/sirf0â

Device node naming is pointless, use DEVTYPE uevent property for identifying the expected protocol.