netdev - Re: [PATCH net-next v2 1/1] net: wwan: t7xx: Add AP CLDMA and GNSS port

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: <CAHNKnsTbnY5ZQRRgA=6bgN_6=ENO3Ycetiwv0sqqTAjqg2BhiA@mail.gmail.com>
Date:   Wed, 29 Jun 2022 22:36:39 +0300
From:   Sergey Ryazanov <ryazanov.s.a@...il.com>
To:     Moises Veleta <moises.veleta@...ux.intel.com>
Cc:     netdev@...r.kernel.org, Jakub Kicinski <kuba@...nel.org>,
        David Miller <davem@...emloft.net>,
        Johannes Berg <johannes@...solutions.net>,
        Loic Poulain <loic.poulain@...aro.org>,
        M Chetan Kumar <m.chetan.kumar@...el.com>,
        "Devegowda, Chandrashekar" <chandrashekar.devegowda@...el.com>,
        Intel Corporation <linuxwwan@...el.com>,
        chiranjeevi.rapolu@...ux.intel.com,
        Haijun Liu (刘海军) 
        <haijun.liu@...iatek.com>,
        Ricardo Martinez <ricardo.martinez@...ux.intel.com>,
        Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
        "Sharma, Dinesh" <dinesh.sharma@...el.com>,
        Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>,
        "Veleta, Moises" <moises.veleta@...el.com>,
        Madhusmita Sahu <madhusmita.sahu@...el.com>,
        Johan Hovold <johan@...nel.org>
Subject: Re: [PATCH net-next v2 1/1] net: wwan: t7xx: Add AP CLDMA and GNSS port

CC GNSS subsystem maintainer.

Johan, you can find detailed port functionality description in the
previous thread [1].

1. https://lore.kernel.org/netdev/20220614205756.6792-1-moises.veleta@linux.intel.com/

On Wed, Jun 29, 2022 at 10:24 PM Sergey Ryazanov <ryazanov.s.a@...il.com> wrote:
>
> Hello Moises,
>
> On Tue, Jun 28, 2022 at 7:51 PM Moises Veleta
> <moises.veleta@...ux.intel.com> wrote:
>> The t7xx device contains two Cross Layer DMA (CLDMA) interfaces to
>> communicate with AP and Modem processors respectively. So far only
>> MD-CLDMA was being used, this patch enables AP-CLDMA and the GNSS
>> port which requires such channel.
>>
>> GNSS AT control port allows Modem Manager to control GPS for:
>> - Start/Stop GNSS sessions,
>> - Configuration commands to support Assisted GNSS positioning
>> - Crash & reboot (notifications when resetting device (AP) & host)
>> - Settings to Enable/Disable GNSS solution
>> - Geofencing
>
> I am still in doubt, should we export the pure GNSS port via the wwan
> subsystem. Personally, I like Loic's suggestion to export this port
> via the gnss subsystem. This way, the modem interface will be
> consistent with the kernel device model. What drawbacks do you see in
> this approach?

--
Sergey