On Mon, Sep 30, 2024 at 04:01:38PM +0300, Andy Shevchenko wrote: > On Mon, Sep 30, 2024 at 01:08:42PM +0530, Raag Jadav wrote: > > Now that we have device wedged event in place, add wedge_recovery sysfs > > attribute which will expose recovery methods supported by the DRM device. > > This is useful for userspace consumers in cases where the device supports > > multiple recovery methods which can be used as fallbacks. > > > > $ cat /sys/class/drm/card<N>/wedge_recovery > > rebind > > bus-reset > > reboot > > ... > > > +static ssize_t wedge_recovery_show(struct device *device, > > + struct device_attribute *attr, char *buf) > > Looking at the below line it seems you are fine with 100 limit, so, why two > lines above if they perfectly fit 100? Just trying to avoid another bikeshed about conventions ;) Raag