On Tue, Feb 06, 2024 at 02:47:04PM +0000, Greg KH wrote: > On Tue, Feb 06, 2024 at 02:56:23PM +0200, Niklas Neronin wrote: > > This patch introduces the USB Billboard Driver. Its purpose is to display, > > via debugfs, basic information about connected Billboard devices. > > Very cool, I was wondering if/when someone was going to write a kernel > driver for this type of hardware. > > But why debugfs? Normally that is locked down for root-access-only by > the system (rightfully so), why is this information restricted? > > And why is this a kernel driver at all? Why can't you just do this in > userspace and add support to 'lsusb' for it? I'm to blame for that. I wanted a way to see the billboard information when something goes wrong with the alt mode entry in an environment where I don't necessarily have tools like lsusb - I think I need to include usbtools package to my Buildroot to get that app. I also proposed debugfs, because for me this would be purely for debugging purposes. Later I was hoping to use this information in the Type-C drivers to help in situations where the alt mode entry fails and UCSI does not give any information about the partner (which unfortunately is the reality on several platforms). This is really just a proposal - perhaps we should have started with RFC first. But I think Niklas has done a great job in any case. thanks, -- heikki
