On Wed, Jun 12, 2019 at 12:07:17PM +0200, Greg Kroah-Hartman wrote: > On Tue, Jun 11, 2019 at 09:10:24PM +0200, Greg Kroah-Hartman wrote: > > When calling debugfs functions, there is no need to ever check the > > return value. The function can work or not, but the code logic should > > never do something different based on this. > > > > Cc: Maya Erez <merez@xxxxxxxxxxxxxx> > > Cc: Kalle Valo <kvalo@xxxxxxxxxxxxxx> > > Cc: "David S. Miller" <davem@xxxxxxxxxxxxx> > > Cc: linux-wireless@xxxxxxxxxxxxxxx > > Cc: wil6210@xxxxxxxxxxxxxxxx > > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > --- > > drivers/net/wireless/ath/wil6210/debugfs.c | 80 ++++++---------------- > > 1 file changed, 22 insertions(+), 58 deletions(-) > > Oops, 0-day finally woke up and shows I messed this patch up. Please > drop it, I will submit a v2 soon. {sigh}, no, that was a different patch that broke things. This pathc is fine, please consider it as-is. I'll go get more coffee... greg k-h