Re: [PATCH v6 7/7] cdx: add device attributes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 1/26/2023 10:25 PM, Randy Dunlap wrote:
Hi--

On 1/26/23 02:46, Nipun Gupta wrote:
diff --git a/Documentation/ABI/testing/sysfs-bus-cdx b/Documentation/ABI/testing/sysfs-bus-cdx
index 8c2425fdb6d9..69bdbc891743 100644
--- a/Documentation/ABI/testing/sysfs-bus-cdx
+++ b/Documentation/ABI/testing/sysfs-bus-cdx
@@ -10,3 +10,47 @@ Description:
                  For example::

                 # echo 1 > /sys/bus/cdx/rescan
+
+What:                /sys/bus/cdx/devices/.../vendor
+Date:                January 2023
+Contact:     nipun.gupta@xxxxxxx
+Description:
+             Vendor ID for this CDX device. Vendor ID is 16 bit
+             identifier which is specific to the device manufacturer.
+             Combination of Vendor ID and Device ID identifies a device.
+
+What:                /sys/bus/cdx/devices/.../device
+Date:                January 2023
+Contact:     nipun.gupta@xxxxxxx
+Description:
+             Device ID for this CDX device. Device ID is a 16 bit
+             identifier to identify a device type within the range
+             of a device manufacturer.
+             Combination of Vendor ID and Device ID identifies a device.
+
+What:                /sys/bus/cdx/devices/.../reset
+Date:                January 2023
+Contact:     nipun.gupta@xxxxxxx
+Description:

Reads better without "would":

+             Writing a non-zero value to this file would reset the CDX> +            device. On resetting the device, the corresponding driver
+             would be notified twice, once before the device is being
+             reset, and again after the reset has been complete.


                 Writing a non-zero value to this file resets the CDX
                 device. On resetting the device, the corresponding driver
                 is notified twice, once before the device is being
                 reset, and again after the reset has been complete.

+
+                For example::
+
+               # echo 1 > /sys/bus/cdx/.../reset
+
+What:                /sys/bus/cdx/devices/.../remove
+Date:                January 2023
+Contact:     tarak.reddy@xxxxxxx
+Description:
+             Writing a non-zero value to this file would remove the
+             corrosponding device from the CDX bus. If the device is
+             to be reconfigured in the Hardware, the device can be
+             removed, so that the device driver does not access the
+             device while it is being reconfigured.

and: (also fix typo of corresponding)

Thanks for the review. Will fix and update these.

Regards
Nipun


                 Writing a non-zero value to this file removes the
                 corresponding device from the CDX bus. If the device is
                 to be reconfigured in the Hardware, the device can be
                 removed, so that the device driver does not access the
                 device while it is being reconfigured.

+
+             For example::
+
+               # echo 1 > /sys/bus/cdx/devices/.../remove

--
~Randy



[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux