[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <022f6596-3b4c-6c19-f918-8dc1cce509ba@quicinc.com>
Date: Thu, 21 Nov 2024 16:02:40 -0700
From: Jeffrey Hugo <quic_jhugo@...cinc.com>
To: Sergey Ryazanov <ryazanov.s.a@...il.com>
CC: Jerry Meng <jerry.meng.lk@...ctel.com>, <loic.poulain@...aro.org>,
Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>,
<netdev@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<linux-arm-msm@...r.kernel.org>
Subject: Re: [PATCH net-next v2] net: wwan: Add WWAN sahara port type
On 11/21/2024 3:53 PM, Sergey Ryazanov wrote:
> Hi Jeffrey,
>
> On 20.11.2024 23:48, Jeffrey Hugo wrote:
>> On 11/20/2024 1:36 PM, Sergey Ryazanov wrote:
>>> +Manivannan
>>>
>>> Hello Jerry,
>>>
>>> this version looks a way better, still there is one minor thing to
>>> improve. See below.
>>>
>>> Manivannan, Loic, could you advice is it Ok to export that SAHARA
>>> port as is?
>>
>> I'm against this.
>>
>> There is an in-kernel Sahara implementation, which is going to be used
>> by QDU100. If WWAN is going to own the "SAHARA" MHI channel name,
>> then no one else can use it which will conflict with QDU100.
>>
>> I expect the in-kernel implementation can be leveraged for this.
>
> Make sense. Can you share a link to this in-kernel implementation? I've
> searched through the code and found nothing similar. Is it merged or has
> it a different name?
drivers/accel/qaic/sahara.c
-Jeff
Powered by blists - more mailing lists