On Tue, Apr 23, 2024 at 08:36:36AM +0300, Mika Westerberg wrote: > On Mon, Apr 22, 2024 at 09:36:24AM -0500, Mario Limonciello wrote: > > On 4/21/2024 05:27, Greg KH wrote: > > > On Tue, Apr 02, 2024 at 01:10:20PM +0300, Mika Westerberg wrote: > > > > Hi all, > > > > > > > > This is mostly for the folks developing and testing Thunderbolt/USB4 > > > > driver side of things but may be useful for others too. Not intented to > > > > replace anything we already have for the regular user stack. > > > > > > > > In summary this is a collection of small tools that provide bit more > > > > user friendly access to the sysfs/debugfs/tracefs interfaces made > > > > available by the driver. Mainly useful for debugging issues around the > > > > software connection manager parts of the driver (although some of these > > > > work with the firmware connection manager too). We have been using this > > > > internally for a while. > > > > > > > > The repository can be found here: > > > > > > > > https://github.com/intel/tbtools > > > > > > > > > > > > > Very cool, thanks for the link! > > > > > > greg k-h > > > > Yes, thanks for sharing! Is it worth adding a link somewhere in kernel > > documentation to make it more discoverable? > > I can't think of any place where this could belong though since this is > not meant for ordinary users (so I guess admin-guide is not good place > for it either). Well maybe as part of the Kconfig, something like below? Does that make sense? diff --git a/drivers/thunderbolt/Kconfig b/drivers/thunderbolt/Kconfig index 279de1ba4487..0abdb69ee9f4 100644 --- a/drivers/thunderbolt/Kconfig +++ b/drivers/thunderbolt/Kconfig @@ -22,7 +22,11 @@ config USB4_DEBUGFS_WRITE bool "Enable write by debugfs to configuration spaces (DANGEROUS)" help Enables writing to device configuration registers through - debugfs interface. + debugfs interface. You can use tools such as Thunderbolt/USB4 + debugging tools to access these registers. For more + information see: + + https://github.com/intel/tbtools Only enable this if you know what you are doing! Never enable this for production systems or distro kernels.
