Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Sender: hemantk=codeaurora.org@mg.codeaurora.org
DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org F037CC433C9
Subject: Re: [PATCH v9 3/4] docs: Add documentation for userspace client
 interface
To:     Jakub Kicinski <kuba@kernel.org>,
        Jeffrey Hugo <jhugo@codeaurora.org>
Cc:     manivannan.sadhasivam@linaro.org, gregkh@linuxfoundation.org,
        linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
        bbhatt@codeaurora.org, loic.poulain@linaro.org,
        netdev@vger.kernel.org
References: <1603495075-11462-1-git-send-email-hemantk@codeaurora.org>
 <1603495075-11462-4-git-send-email-hemantk@codeaurora.org>
 <20201025144627.65b2324e@kicinski-fedora-PC1C0HJN.hsd1.ca.comcast.net>
 <e92a5a5b-ac62-a6d8-b6b4-b65587e64255@codeaurora.org>
 <20201026155617.350c45ab@kicinski-fedora-PC1C0HJN.hsd1.ca.comcast.net>
From:   Hemant Kumar <hemantk@codeaurora.org>
Message-ID: <b7d2f4d9-6755-c099-07f7-f503b97c9f09@codeaurora.org>
Date:   Mon, 26 Oct 2020 16:22:07 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
MIME-Version: 1.0
In-Reply-To: <20201026155617.350c45ab@kicinski-fedora-PC1C0HJN.hsd1.ca.comcast.net>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Jakub,

On 10/26/20 3:56 PM, Jakub Kicinski wrote:
> On Mon, 26 Oct 2020 07:38:46 -0600 Jeffrey Hugo wrote:
>> On 10/25/2020 3:46 PM, Jakub Kicinski wrote:
>>> On Fri, 23 Oct 2020 16:17:54 -0700 Hemant Kumar wrote:
>>>> +UCI driver enables userspace clients to communicate to external MHI devices
>>>> +like modem and WLAN. UCI driver probe creates standard character device file
>>>> +nodes for userspace clients to perform open, read, write, poll and release file
>>>> +operations.
>>>
>>> What's the user space that talks to this?
>>
>> Multiple.
>>
>> Each channel has a different purpose.  There it is expected that a
>> different userspace application would be using it.
>>
>> Hemant implemented the loopback channel, which is a simple channel that
>> just sends you back anything you send it.  Typically this is consumed by
>> a test application.
>>
>> Diag is a typical channel to be consumed by userspace.  This is consumed
>> by various applications that talk to the remote device for diagnostic
>> information (logs and such).
>>
>> Sahara is another common channel that is usually used for the multistage
>> firmware loading process.
> 
> Thanks for the info, are there any open source tests based on the
> loopback channel (perhaps even in tree?)
> 
> Since that's the only channel enabled in this set its the only one
> we can comment on.
> 
i am not aware of any open source tests based on loopback channel. My 
testing includes multiple sessions of echo, cat etc using adb to confirm 
what is sent is received back. Loic is using UCI driver for his use case 
too.

Loic, in case you have any use case (which is part of open source) which 
can use UCI driver, pls share that info ?

I think as soon as UCI becomes part of the tree, more and more channels 
would get added to the driver having open source code for that from 
other folks in community (Loic would be one of them i guess).

Thanks,
Hemant
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project