Messages in this thread | | | Date | Wed, 3 Oct 2012 12:35:21 +0300 | From | "Kirill A. Shutemov" <> | Subject | Re: [PATCHv2 3/6] staging: rts_pstor: reuse kbasename() |
| |
On Wed, Oct 03, 2012 at 11:43:00AM +0300, Andy Shevchenko wrote: > The custom filename function mostly repeats the kernel's kbasename. This patch > simplifies it. The updated filename() will not check for the '\' in the > filenames. It seems redundant in Linux. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > Cc: YAMANE Toshiaki <yamanetoshi@gmail.com> > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > --- > drivers/staging/rts_pstor/trace.h | 16 +++------------- > 1 file changed, 3 insertions(+), 13 deletions(-) > > diff --git a/drivers/staging/rts_pstor/trace.h b/drivers/staging/rts_pstor/trace.h > index cf60a1b..59c5686 100644 > --- a/drivers/staging/rts_pstor/trace.h > +++ b/drivers/staging/rts_pstor/trace.h > @@ -24,26 +24,16 @@ > #ifndef __REALTEK_RTSX_TRACE_H > #define __REALTEK_RTSX_TRACE_H > > +#include <linux/string.h> > + > #define _MSG_TRACE > > #ifdef _MSG_TRACE > static inline char *filename(char *path) > { > - char *ptr; > - > if (path == NULL) > return NULL; > - > - ptr = path; > - > - while (*ptr != '\0') { > - if ((*ptr == '\\') || (*ptr == '/')) > - path = ptr + 1; > - > - ptr++; > - } > - > - return path; > + return kbasename(path);
Looks like you silentely drop const qualifier here from kbasename() return value.
-- Kirill A. Shutemov
| |