Re: [PATCH 2/3] HID: valve-index: Reboot headset on system power transitions

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Fri, Sep 11, 2026 at 11:16:52AM +0200, Michal Pecio wrote:
> On Fri, 11 Sep 2026 07:54:12 +0200, Greg Kroah-Hartman wrote:
> > > +What:		/sys/bus/hid/devices/<bus>:<vid>:<pid>.<n>/reboot
> > > +Date:		October 2026
> > > +Contact:	linux-input@vger.kernel.org
> > > +Description:
> > > +		Writing a boolean true value reboots the Valve Index headset to
> > > +		recover its EDID service. Writing a boolean false value has no
> > > +		effect. This file is write-only.  
> > 
> > Shouldn't this just be a debugfs file?  Making it a sysfs file seems
> > "risky" as it's not a normal operation.
> 
> Not sure what's "risky" about it, it's hopefully a root-only thing?
> 
> And it's the only supported way for userspace to recover buggy devices
> from certain failure condition, it's been suggested that this could be
> run by udev scripts. Isn't it more "risky" to ask userspace to mount
> and tinker with debugfs as a matter of routine?

You are creating an "odd" user/kernel api that is used for debugging,
that's not what sysfs is for.  sysfs is to show attributes that a device
has NOT to cause the device to go off and do some random thing (yes,
there are exceptions, but generally that's the rule.)

debugfs is "do whatever you want", so that's a better place for this.

thanks,

greg k-h