Re: [PATCH 4/6] dma: pxa_dma: no need to check return value of debugfs_create functions

2019-08-13 Thread Robert Jarzmik
Greg Kroah-Hartman writes: > On Sat, Aug 10, 2019 at 09:27:26PM +0200, Robert Jarzmik wrote: >> Greg Kroah-Hartman writes: >> >> This is not strictly equivalent. >> Imagine that the debugfs_create_dir() fails and returns NULL : > How can that happen? Well in v5.0-rc1 that could happen ...

Re: [PATCH 4/6] dma: pxa_dma: no need to check return value of debugfs_create functions

2019-08-11 Thread Greg Kroah-Hartman
On Sat, Aug 10, 2019 at 09:27:26PM +0200, Robert Jarzmik wrote: > Greg Kroah-Hartman writes: > > Hi Greg, > > > 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

Re: [PATCH 4/6] dma: pxa_dma: no need to check return value of debugfs_create functions

2019-08-10 Thread Robert Jarzmik
Greg Kroah-Hartman writes: Hi Greg, > 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. > > Also, because there is no need to save the file dentry, remove the

[PATCH 4/6] dma: pxa_dma: no need to check return value of debugfs_create functions

2019-06-12 Thread Greg Kroah-Hartman
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. Also, because there is no need to save the file dentry, remove the variable that was saving it as it was never even