On Mon, Mar 29, 2021 at 10:53:38AM +0000, Gustavo Pimentel wrote: > On Mon, Mar 29, 2021 at 11:40:28, Greg Kroah-Hartman > <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > > On Mon, Mar 29, 2021 at 10:25:25AM +0000, Gustavo Pimentel wrote: > > > On Mon, Mar 29, 2021 at 11:8:11, Greg Kroah-Hartman > > > <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > > > > > > On Mon, Mar 29, 2021 at 11:59:40AM +0200, Gustavo Pimentel wrote: > > > > > This patch describes the sysfs interface implemented on the dw-xdata-pcie > > > > > driver. > > > > > > > > > > Signed-off-by: Gustavo Pimentel <gustavo.pimentel@xxxxxxxxxxxx> > > > > > --- > > > > > Documentation/ABI/testing/sysfs-driver-xdata | 46 ++++++++++++++++++++++++++++ > > > > > 1 file changed, 46 insertions(+) > > > > > create mode 100644 Documentation/ABI/testing/sysfs-driver-xdata > > > > > > > > > > diff --git a/Documentation/ABI/testing/sysfs-driver-xdata b/Documentation/ABI/testing/sysfs-driver-xdata > > > > > new file mode 100644 > > > > > index 00000000..66af19a > > > > > --- /dev/null > > > > > +++ b/Documentation/ABI/testing/sysfs-driver-xdata > > > > > @@ -0,0 +1,46 @@ > > > > > +What: /sys/class/misc/drivers/dw-xdata-pcie.<device>/write > > > > > +Date: April 2021 > > > > > +KernelVersion: 5.13 > > > > > +Contact: Gustavo Pimentel <gustavo.pimentel@xxxxxxxxxxxx> > > > > > +Description: Allows the user to enable the PCIe traffic generator which > > > > > + will create write TLPs frames - from the Root Complex to the > > > > > + Endpoint direction. > > > > > + Usage e.g. > > > > > + echo 1 > /sys/class/misc/dw-xdata-pcie.<device>/write > > > > > > > > Again, this does not match the code. Either fix the code (which I > > > > recommend), or change this and the other sysfs descriptions of writing > > > > values here. > > > > > > I've commented about this previously, but I didn't get feedback on it, > > > therefore I assumed that justification was okay. > > > I will change the code to accept only the "1" input on the *_store() > > > > Why not use the built-in function to parse "1/y/Y" that the kernel > > provides for this type of thing? > > I found kstrtobool() just now. I'm adapting the code as we speak. > After testing I will send the v10 as soon as possible. There is no rush, take your time. greg k-h