linux-kernel - Re: [PATCH] gnss: tell what GNSS means

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <ZT-BCS_tGfbeYQyL@hovoldconsulting.com>
Date:   Mon, 30 Oct 2023 11:10:17 +0100
From:   Johan Hovold <johan@...nel.org>
To:     Pavel Machek <pavel@....cz>
Cc:     Randy Dunlap <rdunlap@...radead.org>, linux-kernel@...r.kernel.org,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Subject: Re: [PATCH] gnss: tell what GNSS means

On Sun, Oct 29, 2023 at 08:15:41AM +0100, Pavel Machek wrote:
> On Mon 2023-10-16 16:38:28, Johan Hovold wrote:
> > On Sun, Sep 24, 2023 at 10:43:46PM -0700, Randy Dunlap wrote:
> > > Tell users what GNSS means in the Kconfig prompt.
> >  
> > >  menuconfig GNSS
> > > -	tristate "GNSS receiver support"
> > > +	tristate "GNSS (Global Navigation Satellite System) receiver support"
> > 
> > No, I don't like the way this clutters menuconfig. The above would make
> > this one of the longest entries and for very little gain.
> 
> Yeah, because everyone knows what GNSS means.

It's an established term (e.g. see [1]), and as I mentioned in the
context you dropped we don't spell out what NVMe or SPI stands for
either.

> Except that most people know GNSS as GPS. You misnamed the subsystem,
> so it now needs explanation :-(.

No, naming it GPS would have been too specific as these receivers are
not just for GPS.

Either way, this is about not cluttering up menuconfig and making that
harder to navigate for no good reason.

Johan

[1] https://www.u-blox.com/en/positioning-chips-and-modules

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)