[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <321ecbc9-7033-1caa-356c-8bac1338ecb3@linux.intel.com>
Date: Wed, 15 Jun 2022 11:49:41 -0700
From: "moises.veleta" <moises.veleta@...ux.intel.com>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc: netdev@...r.kernel.org, kuba@...nel.org, davem@...emloft.net,
johannes@...solutions.net, ryazanov.s.a@...il.com,
loic.poulain@...aro.org, m.chetan.kumar@...el.com,
chandrashekar.devegowda@...el.com, linuxwwan@...el.com,
chiranjeevi.rapolu@...ux.intel.com, haijun.liu@...iatek.com,
ricardo.martinez@...ux.intel.com, dinesh.sharma@...el.com,
ilpo.jarvinen@...ux.intel.com, moises.veleta@...el.com,
Madhusmita Sahu <madhusmita.sahu@...el.com>
Subject: Re: [PATCH net-next 1/1] net: wwan: t7xx: Add AP CLDMA and GNSS port
On 6/15/22 02:02, Andy Shevchenko wrote:
> On Tue, Jun 14, 2022 at 01:57:56PM -0700, Moises Veleta wrote:
>> From: Haijun Liu <haijun.liu@...iatek.com>
>>
>> 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.
> This doesn't explain the sAP -> AP renaming in a several cases.
> If that renaming is needed, it perhaps requires a separate patch or
> at least a good paragraph to explain why.
>
The renaming was done since sAP (small Application Processor) represents
Application Processor succinctly. The word "small" only adds unnecessary
word that would confuse readers if we kept both "sAP" and "AP". I will
add that information as an additional paragraph.
Powered by blists - more mailing lists