linux-kernel - Re: [PATCH V0 4/7] usb: common: eud: Added the driver support for Embedded USB Debugger(EUD)

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <edb2e8a9f293bb4d55650cec983c9ced@codeaurora.org>
Date:   Tue, 05 Oct 2021 19:27:50 +0530
From:   schowdhu@...eaurora.org
To:     Randy Dunlap <rdunlap@...radead.org>
Cc:     linux-arm-msm@...r.kernel.org, linux-usb@...r.kernel.org,
        devicetree@...r.kernel.org,
        Bryan O'Donoghue <pure.logic@...us-software.ie>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Greg KH <greg@...ah.com>, linux-kernel@...r.kernel.org,
        ckadabi@...eaurora.org, tsoni@...eaurora.org,
        bryanh@...eaurora.org, psodagud@...eaurora.org,
        satyap@...eaurora.org, pheragu@...eaurora.org,
        Rajendra Nayak <rnayak@...eaurora.org>,
        Sibi Sankar <sibis@...eaurora.org>,
        Sai Prakash Ranjan <saiprakash.ranjan@...eaurora.org>
Subject: Re: [PATCH V0 4/7] usb: common: eud: Added the driver support for
 Embedded USB Debugger(EUD)

On 2021-10-05 01:38, Randy Dunlap wrote:
> On 10/4/21 4:16 AM, Souradeep Chowdhury wrote:
>> Add support for control peripheral of EUD (Embedded USB Debugger) to
>> listen to events such as USB attach/detach, pet EUD to indicate 
>> software
> 
>   I don't quite understand: what pets the EUD? how does it do that?

Pet EUD is an interrupt that the EUD driver receives periodically to
indicate if the software is functional. On getting the interrupt the
EUD driver first does a detach pet and then followed by the an attach
pet.

> 
>> is functional.Reusing the platform device kobj, sysfs entry 'enable' 
>> is
> 
>      functional. Reusing

Ack

> 
>> created to enable or disable EUD.
>> 
>> To enable the eud the following needs to be done
>> echo 1 >/sys/bus/platform/.../enable
>> 
>> To disable eud, following is the command
>> echo 0 >/sys/bus/platform/.../enable
>> 
>> Signed-off-by: Souradeep Chowdhury<schowdhu@...eaurora.org>
>> ---
>>   Documentation/ABI/testing/sysfs-driver-eud |   7 +
>>   drivers/usb/common/Kconfig                 |   9 +
>>   drivers/usb/common/Makefile                |   1 +
>>   drivers/usb/common/qcom_eud.c              | 256 
>> +++++++++++++++++++++++++++++
>>   4 files changed, 273 insertions(+)
>>   create mode 100644 Documentation/ABI/testing/sysfs-driver-eud
>>   create mode 100644 drivers/usb/common/qcom_eud.c
>> 
>> diff --git a/Documentation/ABI/testing/sysfs-driver-eud 
>> b/Documentation/ABI/testing/sysfs-driver-eud
>> new file mode 100644
>> index 0000000..14a02da
>> --- /dev/null
>> +++ b/Documentation/ABI/testing/sysfs-driver-eud
>> @@ -0,0 +1,7 @@
>> +What:		/sys/bus/platform/.../enable
>> +Date:           October 2021
>> +Contact:        Souradeep Chowdhury<schowdhu@...eaurora.org>
>> +Description:
>> +		The Enable/Disable sysfs interface for Embedded
>> +		USB Debugger(EUD).This enables and disables the
> 
> 		    Debugger (EUD). This enables

Ack

> 
>> +		EUD based on a 1 or a 0 value.