The prefix should be 'brcmsmac'. On 1/22/2019 4:21 PM, 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.
I could argue that if above is true it would be better to make the debugfs function return void, but I won't ;-p
In start_creating() the parent dentry is indeed checked for IS_ERR() so... Acked-by: Arend van Spriel
Cc: Arend van Spriel <[email protected]> Cc: Franky Lin <[email protected]> Cc: Hante Meuleman <[email protected]> Cc: Chi-Hsien Lin <[email protected]> Cc: Wright Feng <[email protected]> Cc: Kalle Valo <[email protected]> Cc: [email protected] Cc: [email protected] Cc: [email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]> --- .../broadcom/brcm80211/brcmsmac/debug.c | 26 +++---------------- .../broadcom/brcm80211/brcmsmac/debug.h | 2 +- 2 files changed, 5 insertions(+), 23 deletions(-)
