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

[schowdhu]

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@xxxxxxxxxxxxxx>
> > ---
> >   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@xxxxxxxxxxxxxx>
> > +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.